Closed Bug 347977 Opened 18 years ago Closed 18 years ago

"about:" is translatable, but its direction is LTR

Categories

(Firefox :: General, defect)

2.0 Branch
defect
Not set
normal

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)

If the page is translatable, then it should be RTL for RTL locales too. I'm going to attach the screenshot.
Makes the page direction localizable. Mano, Axel?
Assignee: nobody → steffen.wilberg
Status: NEW → ASSIGNED
Attachment #232998 - Flags: review?(l10n)
We shoud just use locale.dir from global.dtd (unless you want to take care of the case where this page isn't translated).
Attachment #232998 - Attachment is obsolete: true
Attachment #232998 - Flags: review?(l10n)
Attached patch <body dir="&locale.dir;"> (obsolete) — Splinter Review
Attachment #233154 - Flags: review?(bugs.mano)
Attachment #233154 - Attachment is obsolete: true
Attachment #233154 - Flags: review?(bugs.mano)
Attachment #233155 - Flags: review?(bugs.mano)
Comment on attachment 233155 [details] [diff] [review] <body dir="&locale.dir;"> r=mano
Attachment #233155 - Flags: review?(bugs.mano) → review+
Fixed on trunk.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Attachment #233155 - Flags: approval1.8.1?
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+
Yep, looks good to me.
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.

Attachment

General

Created:
Updated:
Size: