Closed Bug 680255 Opened 9 years ago Closed 9 years ago

Add Urdu to the list of RTL languages

Categories

(Firefox :: General, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 9

People

(Reporter: ehsan, Assigned: ehsan)

References

Details

Attachments

(1 file)

No description provided.
Attached patch Patch (v1)Splinter Review
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #554210 - Flags: review?(l10n)
Comment on attachment 554210 [details] [diff] [review]
Patch (v1)

This should be fine, getting an additional review from Simon, though.
Attachment #554210 - Flags: review?(smontagu)
Attachment #554210 - Flags: review?(l10n)
Attachment #554210 - Flags: review+
Any specific reason why we only add Urdu? A more complete list of languages that we might want to include:
"ar", "arc", "dv", "fa", "he", "ks", "ps", "ug", "ur", "yi", "nqo"

Maybe some of them occasionally use different scripts, but I think most of these would be useful at RTL. This way we are ready for users when they want to start localisation work.
(In reply to Friedel Wolff from comment #3)
> Maybe some of them occasionally use different scripts, but I think most of
> these would be useful at RTL. This way we are ready for users when they want
> to start localisation work.

That's why.  We can add any new locales as RTL as soon as somebody decides to work on them, but I don't want to rush to that...
Attachment #554210 - Flags: review?(smontagu) → review+
http://hg.mozilla.org/integration/mozilla-inbound/rev/88e23391bc2c
Target Milestone: --- → Firefox 9
https://hg.mozilla.org/mozilla-central/rev/88e23391bc2c
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Blocks: 958577
Depends on: 996960
You need to log in before you can comment on or make changes to this bug.