1222204 - Add Persian (fa) to master and 2.5 builds

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[Théo Chevalier]

Let’s add this locale to master and 2.5. Repos are ready.

I assume it makes sense for Persian to have ar and en keyboards enabled by default, like for Arabic.

Behavior with this patch will be when choosing Persian on FTU:
- Enable ar and en keyboard layouts
- ar will be the default one

Sara, is that ok, or you would like something else?
Note that Arabic keyboard does not have a dictionary yet, so you won’t see Arabic word suggestions when typing.

Also note that this can be changed at any time, and we’ll be able to add a specific layout for Persian later, but we need at least one default layout for now.

Last thing, is the locale name correctly spelled? "فارسی" this is what users will see in Settings.

Comment 1

[s.mansoury]

Ok great news!
Yes, Please use english keyboard(en) enabled by default.
Yes the locale name is correctly spelled: "فارسی" 

Thank you!

Comment 2

[GitHub Autolander Bot]

Attachment: [gaia] TheoChevalier:add-persian > mozilla-b2g:master

Comment 3

[Théo Chevalier]

Thanks, updated the Pull Request to enable en layout.

Stas, if you can give it a look when you have the time :)

Comment 4

[Staś Małolepszy :stas]

Comment on attachment 8686682 [details] [review]
[gaia] TheoChevalier:add-persian > mozilla-b2g:master

Technically this looks good, r=me.  It's a pity we don't have a Farsi keyboard layout.  Could you file a bug about it?

Comment 5

[Théo Chevalier]

Thanks Stas :)

Looks like layout + word list are being worked out in bug 1033185

Comment 6

[Thatoo]

There are already two different layouts of keyboard for Persian language, one of 10 letters length (more suitable for computer-phone) and one of 11 letters length (more suitable for tablet, would be suitable for computer-phone on landscape also if such option was possible, but I don;t know how to implement that).
A word list is prepared also but I have to find out how to upload it to github. I didn't manage to make it a dictionary yet.
see here https://bugzilla.mozilla.org/show_bug.cgi?id=1033185 .

Comment 7

[Carsten Book [:Tomcat]]

https://github.com/mozilla-b2g/gaia/commit/0630ebd286a0f2b5f97b7cc8c1d65f948c8c6fc9

Comment 8

[Théo Chevalier]

Comment on attachment 8686682 [details] [review]
[gaia] TheoChevalier:add-persian > mozilla-b2g:master

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): n/a
[User impact] if declined: No Persian language
[Testing completed]: master builds
[Risk to taking this patch] (and alternatives if risky): very low
[String changes made]: n/a

Persian locale is ready for 2.5, we would like to ship it.

Comment 9

[Delphine Lebédel [:delphine - Use needinfo]]

Hi Mahe: we have planned to test Persian during the 2.5 localization run. Could you please approve this so it can land on 2.5? Testing started today, would be great if we could get to this during the week :) thanks!

Comment 10

[Mahendra Potharaju [:mahe]]

Comment on attachment 8686682 [details] [review]
[gaia] TheoChevalier:add-persian > mozilla-b2g:master

Approved for 2.5.

Thank you.

Comment 11

[Théo Chevalier]

Thanks. Assuming the master PR can be cherry-picked onto 2.5 (files are identical), adding checking-needed.

Comment 12

[Carsten Book [:Tomcat]]

uplifted in https://github.com/mozilla-b2g/gaia/commit/28d63cf3bdc4417f7ad8cab2230f096bf9f6d3b5 to 2.5