1081567 - (breton-keyboard) [Keyboard] adding Breton layout

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::Keyboard, defect)

Description

[dattaz]

The C'HWERTY keyboard layout is the bretons keyboard equivalent for the bretons language.

Comment 1

[dattaz]

Attachment: patch for keyboard

Comment 2

[dattaz]

Attachment: good patch

Comment 3

[Théo Chevalier]

Hi Dattaz,

Thanks for the patch.
Could you update the PR to bind the layout to the br locale here please? https://github.com/mozilla-b2g/gaia/blob/master/build/config/keyboard-layouts.json

Also, I don't think we can set autoCorrectLanguage since we don't have Breton dictionary yet.
And autoCorrectPunctuation is mostly specific to French, it's been added in bug 808740 to match French typographic rules, not sure it's relevant for br locale (I don't know about their rules, though.)

CC'ing Breton team members, in case they want to use other layouts for Breton, if they need to. For instance, Basque is using “eu, es, fr, en” layouts. “eu” is the one used by default.

Comment 4

[Michel Nédélec]

Hi!

Could you add the "fr" and "en" layouts ?
The "br" layout would be the one used by default and the "fr" and "en" would be available.

We have an Hunspell dictionary here :
http://hg.mozilla.org/releases/l10n/mozilla-aurora/br/file/default/extensions/spellcheck/hunspell
Maybe it can help ?

Comment 5

[dattaz]

I have made the change

Comment 6

[Théo Chevalier]

(In reply to Michel Nédélec from comment #4)
> Hi!
> 
> Could you add the "fr" and "en" layouts ?
> The "br" layout would be the one used by default and the "fr" and "en" would
> be available.
> 
> We have an Hunspell dictionary here :
> http://hg.mozilla.org/releases/l10n/mozilla-aurora/br/file/default/
> extensions/spellcheck/hunspell
> Maybe it can help ?

Hi, thanks for the feedback, Michel! It will probably help, yes. I filled bug 1081716 for autocorrect in br.

Comment 7

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

Comment on attachment 8503729 [details] [review]
good patch

Stealing the review.

Comment 8

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

Comment on attachment 8503729 [details] [review]
good patch

I'll clear the review first since this patch seems not complete:
 1. The composite key like C'H cannot fit into our current UI, one way to workaround this (not a perfect way) is to add special CSS rules to resize these composite keys.

 2. The uppercase switching for these composite keys are not functional.
    That is when you press [shift] key and then type them, the keyboard would still output lowercase chars.
    You could check 'upperCase' field in el.js to see if that works.
    (I am not sure that could work for composite key, so please help give a try.)

--
Please flag me again for review for the next round.
Thank you!

Comment 10

[BMO Automation]

Firefox OS is not being worked on