Closed
Bug 347977
Opened 18 years ago
Closed 18 years ago
"about:" is translatable, but its direction is LTR
Categories
(Firefox :: General, defect)
Tracking
()
RESOLVED
FIXED
Firefox 2 beta2
People
(Reporter: zwnj, Assigned: steffen.wilberg)
References
(Blocks 1 open bug)
Details
(Keywords: fixed1.8.1, l12y)
Attachments
(2 files, 2 obsolete files)
69.70 KB,
image/png
|
Details | |
1.57 KB,
patch
|
asaf
:
review+
beltzner
:
approval1.8.1+
|
Details | Diff | Splinter Review |
If the page is translatable, then it should be RTL for RTL locales too.
I'm going to attach the screenshot.
Reporter | ||
Comment 1•18 years ago
|
||
Assignee | ||
Comment 2•18 years ago
|
||
Makes the page direction localizable.
Mano, Axel?
Comment 3•18 years ago
|
||
We shoud just use locale.dir from global.dtd (unless you want to take care of the case where this page isn't translated).
Assignee | ||
Updated•18 years ago
|
Attachment #232998 -
Attachment is obsolete: true
Attachment #232998 -
Flags: review?(l10n)
Assignee | ||
Comment 4•18 years ago
|
||
Attachment #233154 -
Flags: review?(bugs.mano)
Assignee | ||
Updated•18 years ago
|
Attachment #233154 -
Attachment is obsolete: true
Attachment #233154 -
Flags: review?(bugs.mano)
Assignee | ||
Comment 5•18 years ago
|
||
Attachment #233155 -
Flags: review?(bugs.mano)
Comment 6•18 years ago
|
||
Comment on attachment 233155 [details] [diff] [review]
<body dir="&locale.dir;">
r=mano
Attachment #233155 -
Flags: review?(bugs.mano) → review+
Assignee | ||
Comment 7•18 years ago
|
||
Fixed on trunk.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•18 years ago
|
Attachment #233155 -
Flags: approval1.8.1?
Comment 8•18 years ago
|
||
Comment on attachment 233155 [details] [diff] [review]
<body dir="&locale.dir;">
a=drivers, please land on MOZILLA_1_8_BRANCH
Attachment #233155 -
Flags: approval1.8.1? → approval1.8.1+
Comment 9•18 years ago
|
||
Yep, looks good to me.
Assignee | ||
Comment 10•18 years ago
|
||
Fixed on branch.
Keywords: fixed1.8.1
Target Milestone: Firefox 2 → Firefox 2 beta2
You need to log in
before you can comment on or make changes to this bug.
Description
•