Closed
Bug 402240
Opened 17 years ago
Closed 16 years ago
[RTL] Need a better way to control the statusbar direction
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
FIXED
People
(Reporter: tomer, Assigned: tomer)
References
(Blocks 1 open bug, )
Details
(Keywords: intl, rtl)
Attachments
(3 files, 1 obsolete file)
11.21 KB,
image/png
|
Details | |
3.37 KB,
text/plain
|
Details | |
1.08 KB,
patch
|
tomer
:
approval1.9?
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Updated•17 years ago
|
Comment 1•17 years ago
|
||
Moving over a flock of granparadiso RTL bugs to Firefox for triage.
Component: he-IL / Hebrew → General
Keywords: intl
Product: Mozilla Localizations → Firefox
Version: unspecified → Trunk
Comment 2•17 years ago
|
||
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•17 years ago
|
||
Unicode escapes still work, and will remain to work in .properties files.
Assignee | ||
Comment 4•17 years ago
|
||
Any solution for this issue, or we'll have to do the same *workaround* the Persians did?
Comment 5•17 years ago
|
||
I think the workaround is actually the most practical solution here.
Updated•17 years ago
|
QA Contact: hebrew.he → general
Updated•17 years ago
|
Blocks: Persian-Fx3.5
Assignee | ||
Updated•17 years ago
|
Blocks: fx3-l10n-he
Updated•17 years ago
|
Blocks: fx35-l10n-fa
Updated•17 years ago
|
No longer blocks: Persian-Fx3.5
Comment 6•17 years ago
|
||
Mass-assigning the new rtl keyword to RTL-related (see bug 349193).
Keywords: rtl
Assignee | ||
Comment 7•17 years ago
|
||
Assignee | ||
Comment 8•17 years ago
|
||
Some extra whitespaces removed.
Attachment #321418 -
Attachment is obsolete: true
Assignee | ||
Comment 9•17 years ago
|
||
This is very important fix we should get fixed before going final, as it is visually appear on every request.
Assignee | ||
Comment 10•17 years ago
|
||
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
Depends on: 433777
Comment 11•17 years ago
|
||
This has been reported in launchpad as well
https://bugs.edge.launchpad.net/ubuntu/+source/firefox-3.0/+bug/234029
Assignee | ||
Comment 12•16 years ago
|
||
fixed and landed for 'he'.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
Blocks: Persian-Fx3.5
Updated•16 years ago
|
No longer blocks: fx35-l10n-fa
Assignee | ||
Comment 13•14 years ago
|
||
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.
See Also: → https://launchpad.net/bugs/234029
You need to log in
before you can comment on or make changes to this bug.
Description
•