402240 - [RTL] Need a better way to control the statusbar direction

Status: RESOLVED FIXED

Description

[Tomer Cohen :tomer]

Created attachment 287123 [details]
Current broken Minefield LTR status bar (screenshot with caption)

Text in Hebrew Minefield apper wrong in the status bar due to the fact that we want it to be LTR for URLs but RTL for Status text "Waiting for...". 

Previous versions has #statusbar-display{
direction: ltr !important;} in intl.css, but this seem to break the status messages.

Comment 1

[Axel Hecht [:Pike]]

Moving over a flock of granparadiso RTL bugs to Firefox for triage.

Comment 2

[Simon Montagu :smontagu]

I see that the Persian localization [1] solves this by inserting bidi embedding controls in the localized strings, e.g. 

[RLE]TRANSFERRING DATA FROM [LRE]%1$S[PDF]...[PDF]

[1] http://mxr.mozilla.org/l10n/source/fa/netwerk/necko.properties

Axel, is it possible to mix UTF-8 and escapes (\u202a etc.) in properties files rather than using invisible characters, which make life very difficult when editing?

Comment 3

[Axel Hecht [:Pike]]

Unicode escapes still work, and will remain to work in .properties files.

Comment 4

[Tomer Cohen :tomer]

Any solution for this issue, or we'll have to do the same *workaround* the Persians did?

Comment 5

[Simon Montagu :smontagu]

I think the workaround is actually the most practical solution here.

Comment 6

[Away for a while]

Mass-assigning the new rtl keyword to RTL-related (see bug 349193).

Comment 7

[Tomer Cohen :tomer]

Created attachment 321418 [details]
necko.properties with RLE/LRE..PDF

Comment 8

[Tomer Cohen :tomer]

Created attachment 321419 [details]
necko.properties with RLE/LRE..PDF

Some extra whitespaces removed.

Comment 9

[Tomer Cohen :tomer]

Created attachment 321420 [details] [diff] [review]
patch

This is very important fix we should get fixed before going final, as it is visually appear on every request.

Comment 10

[Tomer Cohen :tomer]

I've added /l10n/he/netwerk/necko.properties into the patch of bug 433777 attachment 321801 [details] [diff] [review]. 

ar people - You might wish to do the same in your locale. 
http://mxr.mozilla.org/l10n/source/ar/netwerk/necko.properties

Comment 11

[John Vivirito]

This has been reported in launchpad as well 
https://bugs.edge.launchpad.net/ubuntu/+source/firefox-3.0/+bug/234029

Comment 12

[Tomer Cohen :tomer]

fixed and landed for 'he'. 

Comment 13

[Tomer Cohen :tomer]

As Firefox 4 is around the corner, I'd suggest reopening this bug in order to find a better way to do it, as RLE..PDF is not the expected solution.