! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
Force LTR for playback controllers
Status:
RESOLVED: FIXED

Comments

Description Khazar 2013-01-31 09:39:12 CET
For RTL locale, the playback buttons are reversed.

In order to force the order to be from Left to right you need to add code like this http://git.gnome.org/browse/ekiga/commit/?id=76c9b23

I think forcing LTR is appropriate since there are no video or DVD devices that are adjusted to people who use Semitic languages such as Arabic, Urdu etc.

The playlist bar may be stay on the left - I see no problem with it.

In case Mr. Ali Abdallah is not currently active in developing Parole, please consult with him first.
Comment 1 Simon Steinbeiss editbugs 2013-02-22 20:49:53 CET
Hey,
currently Sean and I are maintaining Parole. As neither of us speaks/uses a RTL language/locale, this is really hard for us to test/fix. If you can supply a patch that you tested and that works for you, I think we'd consider it.
Thanks!
Comment 2 Khazar 2013-03-07 06:56:07 CET
Created attachment 4950 
Reversed Controllers

$ LANG=he_IL.utf8 parole
Comment 3 Khazar 2013-03-07 07:02:29 CET
(In reply to comment #1)
> Hey,
> currently Sean and I are maintaining Parole. As neither of us speaks/uses a
> RTL language/locale, this is really hard for us to test/fix. If you can
> supply a patch that you tested and that works for you, I think we'd consider
> it.
> Thanks!

To test patch you just need to run this command and see if order of controllers is the same as English, Russian, German et al.

$ LANG=ar_SY.utf8 parole
Comment 4 Simon Steinbeiss editbugs 2013-03-11 11:12:38 CET
Created attachment 4956 
Hebrew locale and LTR controls

Hi again,
I've had some success with forcing LTR to the controllers, my main question is whether the progressbar should also be forced? (see screenshot)
Comment 5 Khazar 2013-03-12 06:13:23 CET
(In reply to comment #4)
> Created attachment 4956 
> Hebrew locale and LTR controls
> 
> Hi again,
> I've had some success with forcing LTR to the controllers, my main question
> is whether the progressbar should also be forced? (see screenshot)

Yes, the controllers at the bottom should be identical to the bottom area of LTR UI.
I do not see what is the change in last screenshot.
Comment 6 Simon Steinbeiss editbugs 2013-03-12 11:19:06 CET
(In reply to comment #5)
> Yes, the controllers at the bottom should be identical to the bottom area of
> LTR UI.
> I do not see what is the change in last screenshot.

If you look closely, previous/play/next are in the correct order now.
In my opinion the show/hide playlist button has to remain on the left, otherwise it kinda loses its meaning.

So the position of the items won't change from what you see on my last screenshot, only their orientation. That is to say it's not impossible to make them all stay in the LTR order, but we'd have to change the packing of the widgets a bit.

Option 1:
[Show/Hide playlist] [Fullscreen] [Volume] [Playtime] [Previous/Play/Next]

Option 2:
[Show/Hide playlist] [Fullscreen] [Volume] [Previous/Play/Next] [Playtime]
Comment 7 Khazar 2013-03-13 01:14:16 CET
I chose option #2
[Show/Hide playlist] [Fullscreen] [Volume] [Previous/Play/Next] [Playtime]

Please consult with Ali.
I prefer he would be the one who decides on this matter.


Off topic:
My personal choice is:
[Show/Hide playlist] [Previous/Play/Next] [Playtime] [Volume] [Fullscreen]

Which means, I think regular (LTR) UI should look like this:
[Fullscreen] [Previous/Play/Next] [Playtime] [Volume] [Show/Hide playlist]
Comment 8 Simon Steinbeiss editbugs 2013-03-13 09:30:08 CET
Putting the fullscreen button left doesn't make any sense in LTR. It's just a matter of widget packing, we can keep the current LTR layout and still get #2

Ali has neither been active nor reachable via email for a long time (just check the gitlogs), so unless you want this bugreport to remain open and unfixed for an uncertain amount of time, I suppose we fix this.
Comment 9 Khazar 2013-03-15 20:08:23 CET
My choice is option #2.

If Ali would think otherwise, at any given point of time, do not feel like you have to consult with me, though I would like my voice to be listened to.
Comment 10 Simon Steinbeiss editbugs 2013-10-07 11:15:44 CEST
We have a fix for this in the bluesabre-gtk3 branch now, which is targeted for the 0.6 release.
Comment 11 Simon Steinbeiss editbugs 2013-10-11 12:01:41 CEST
The fix for this bug (version #2 of the LTR layout was implemented) has just been merged to git master and will be part of the next release.
Comment 12 Simon Steinbeiss editbugs 2013-10-11 12:01:55 CEST
The fix for this bug (version #2 of the LTR layout was implemented) has just been merged to git master and will be part of the next release.
Comment 13 Simon Steinbeiss editbugs 2013-10-11 12:02:13 CEST
Marked as fixed.
Comment 14 Khazar 2014-12-23 18:51:53 CET
Please upload a current screenshot of what RTL Parole looks like.

Bug #9822

Reported by:
Khazar
Reported on: 2013-01-31
Last modified on: 2014-12-23

People

Assignee:
Simon Steinbeiss
CC List:
1 user

Version

Version:
0.3.0.2

Attachments

Reversed Controllers (11.90 KB, image/png)
2013-03-07 06:56 CET , Khazar
no flags
Hebrew locale and LTR controls (112.74 KB, image/png)
2013-03-11 11:12 CET , Simon Steinbeiss
no flags

Additional information