641942 - Make Windows progress bars RTL-friendly

Status: VERIFIED FIXED

(Core :: Widget: Win32, defect)

Description

[Mounir Lamouri (:mounir)]

They are shown with the bar on the right because this is managed by the layout code but the animations are done by the widget code and currently do not take into account RTL.

Comment 1

[(no longer active)]

Can you elaborate on what you mean in comment 0?

Comment 2

[Mounir Lamouri (:mounir)]

<progress value=0.5> in LTR will be shown like:
[*****     ]
In RTL:
[     *****]

But, on Windows (Vista and later), there is animated glow on the progress bars which, for the moment, goes from left to right.

In addition, indeterminate progress bars also go from left to right.

You can try the URL with this build to have a look:
https://ftp.mozilla.org/pub/mozilla.org/firefox/tryserver-builds/[e-mail]-3665ab582961/

Comment 3

[Mounir Lamouri (:mounir)]

Attachment: Patch v1

Comment 4

[(no longer active)]

(In reply to comment #2)
> <progress value=0.5> in LTR will be shown like:
> [*****     ]
> In RTL:
> [     *****]
> 
> But, on Windows (Vista and later), there is animated glow on the progress bars
> which, for the moment, goes from left to right.

I see, makes sense.

Comment 5

[Jim Mathies [:jimm]]

Comment on attachment 519977 [details] [diff] [review]
Patch v1

Review of attachment 519977 [details] [diff] [review]:

r+

Comment 6

[Mounir Lamouri (:mounir)]

Pushed:
http://hg.mozilla.org/mozilla-central/rev/ea4a5a20e081

Comment 7

[Shawn Wilsher :sdwilsh]

Backed out in http://hg.mozilla.org/mozilla-central/rev/dd9ba28d2bd9 to resolve bug 655860.

Comment 8

[Mounir Lamouri (:mounir)]

The regression wasn't caused by these patches. Re-landed:
http://hg.mozilla.org/mozilla-central/rev/e0b0a18a6905

Comment 9

[George Carstoiu]

Mozilla/5.0 (Windows NT 6.1; WOW64; rv:6.0) Gecko/20100101 Firefox/6.0

Checked this issue using an 'ar' Fx 6.0b2 build.

Yes, the animation is moving from right to left as it should be, but what concerns me is that it doesn't have a consistent pattern - sometimes it disappears and reapers a few centimeters away, it doesn't always start from the beginning of the progress bar.

Considering this, I am setting the status of the bug to Reopened.

Comment 10

[Jim Mathies [:jimm]]

(In reply to comment #9)
> Mozilla/5.0 (Windows NT 6.1; WOW64; rv:6.0) Gecko/20100101 Firefox/6.0
> 
> Checked this issue using an 'ar' Fx 6.0b2 build.
> 
> Yes, the animation is moving from right to left as it should be, but what
> concerns me is that it doesn't have a consistent pattern - sometimes it
> disappears and reapers a few centimeters away, it doesn't always start from
> the beginning of the progress bar.
> 
> Considering this, I am setting the status of the bug to Reopened.

Could you file a new bug George and mark it as blocking this one? (Lets keep this one closed since patches landed here.)

Comment 11

[George Carstoiu]

Logged Bug 672782.

Setting the resolution of this issue to Verified Fixed, as RTL animation was introduced.