Closed Bug 944662 Opened 11 years ago Closed 10 years ago

Set one of the Bulgarian Input Method as default keyboard for Bulgarian language

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(b2g-v2.2 fixed)

RESOLVED FIXED
2.1 S8 (7Nov)
Tracking Status
b2g-v2.2 --- fixed

People

(Reporter: rudyl, Assigned: rudyl)

References

Details

Attachments

(1 file, 1 obsolete file)

46 bytes, text/x-github-pull-request
janjongboom
: review+
Details | Review
+++ This bug was initially created as a clone of Bug #936840 +++

This is a follow-up to modify shared/resources/keyboard_layouts.json so that one of the Bulgarian IME would be set as the default keyboard when the language is changed to Bulagrian.

Besides, since Bulgarian is a non-latin language (?), I think English layout should be enabled as well.
Attached file Patch V1 (obsolete) —
This patch is set the default keyboard layout used by the system when the user language (OS UI language) is changed.

Hi  Mihail,

Do you think it is ok to set bg-BDS only as the default keyboard layout for Bulgarian?
Thanks.
Attachment #8340313 - Flags: feedback?(chilyashev)
Comment on attachment 8340313 [details] [review]
Patch V1

Hello Rudy,

Yes, I think it's OK since the other layouts will be available.


Thank you.
Comment on attachment 8340313 [details] [review]
Patch V1

Hello again,

I still can't edit the flags of the attachment, so please consider the feedback granted.
Attachment #8340313 - Flags: feedback?(chilyashev) → feedback+
What's the status of this bug? Did you forget to set assignee to yourself?
Flags: needinfo?(rlu)
Yeah, seems this is lost in my tracking.
Will submit an updated patch later.
Assignee: nobody → rlu
Flags: needinfo?(rlu)
Attached file Patch V1
layout mapping config update.

Jan, could you help review this simple change?
Thanks.
Attachment #8340313 - Attachment is obsolete: true
Attachment #8516455 - Flags: review?(janjongboom)
Status: NEW → ASSIGNED
Comment on attachment 8516455 [details] [review]
Patch V1

I'm pretty sure the en & bg lines should be other way around, now en is default layout and bg is secondary.
Attachment #8516455 - Flags: review?(janjongboom) → review-
Comment on attachment 8516455 [details] [review]
Patch V1

I think the order should be irrelevant, since we could not change the order of layouts, it is always the same order as how we preload them.

But anyway, to sync with the other languages, I changed it as you suggests.
Thanks.
Attachment #8516455 - Flags: review- → review?(janjongboom)
Comment on attachment 8516455 [details] [review]
Patch V1

r=me
Attachment #8516455 - Flags: review?(janjongboom) → review+
master,
https://github.com/mozilla-b2g/gaia/commit/726c83231f74d7145d66f25af4b97908a0d8ad6f

--
Thanks.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.1 S8 (7Nov)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: