1342286 - Add "he", "ur", "ar" and "fa" locales to maemo-locale for multi-locale builds

Status: RESOLVED FIXED

(Firefox for Android :: General, defect)

Description

[Delphine Lebédel [:delphine - use need info]]

Support for RTL should be in better shape with Firefox 53, so let's add all these RTL locales to multi-locale builds so they can ship as well :)
They've also been testing consistently on single-locale builds.

Comment 1

[Delphine Lebédel [:delphine - use need info]]

Attachment: bug1342286.patch

Comment 2

[Delphine Lebédel [:delphine - use need info]]

Release Note Request (optional, but appreciated)
[Why is this notable]: New Locales
[Affects Firefox for Android]: yes
[Suggested wording]: Locales added: Arabic (ar), Hebrew (he), Persian (fa) and Urdu (ur)
[Links (documentation, blog post, etc)]:

Comment 3

[Francesco Lodolo [:flod]]

Comment on attachment 8840693 [details] [diff] [review]
bug1342286.patch

Review of attachment 8840693 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, thanks.

Comment 4

[Delphine Lebédel [:delphine - use need info]]

Comment on attachment 8840693 [details] [diff] [review]
bug1342286.patch

Thanks flod! 
Request to land on Aurora only. 
Please see relnotes in comment 2. 

Stoked to get these RTL locales finally shipping :D

Comment 5

[Liz Henry (:lizzard Please n-i to RyanVM, jcristau, or pascal)]

Fantastic, should this have a release note or is this at the aurora-only testing stage?

Comment 6

[Liz Henry (:lizzard Please n-i to RyanVM, jcristau, or pascal)]

Comment on attachment 8840693 [details] [diff] [review]
bug1342286.patch

Aurora-only uplift.

Comment 7

[Delphine Lebédel [:delphine - use need info]]

(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #5)
> Fantastic, should this have a release note or is this at the aurora-only
> testing stage?

Yes, release notes are in comment 2 (did I tag that wrong?). Thanks :D

Comment 8

[Francesco Lodolo [:flod]]

(In reply to Delphine Lebédel [:delphine - use Need Info] from comment #7)
> Yes, release notes are in comment 2 (did I tag that wrong?). Thanks :D

Good question. Do we use the relnotes-firefox flag for Fennec too?

Comment 9

[Ryan VanderMeulen [PTO Oct 2-16, NI :jcristau or :pascalc]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/88998d9aefbd