Omega Feng [:Omega] [:馮於懋] Reporter
	Description • 10 years ago

Keyboard Settings should follow recommendation, like settings structure, Number IME visibility, etc.

	Omega Feng [:Omega] [:馮於懋] Reporter
	Comment 1 • 10 years ago

Arthur, can you comment on this?

	Arthur Chen [:arthurcc][inactive after 6/30] Assignee
	Comment 2 • 10 years ago

WIP, still need to update the tests.

	Arthur Chen [:arthurcc][inactive after 6/30] Assignee
	Comment 3 • 10 years ago

Comment on attachment 8452828 [details]
link to https://github.com/mozilla-b2g/gaia/pull/21490

The patch merged the enabled layout panel to the keyboard panel based on the new UX spec. 

EJ, could you help review the patch? The patch simply moved the code from one place to another.

Zac, I also updated the UI tests to reflect the flow change.

Thanks!

	Florin Strugariu [:Bebe]
	Comment 4 • 10 years ago

Comment on attachment 8452828 [details]
link to https://github.com/mozilla-b2g/gaia/pull/21490

The UI tests looks OK

	EJ Chen [:eragonj][:小龍哥][Contributor mode on](inactive)
	Comment 5 • 10 years ago

Arthur, I did check the code and they all look nice. But there is a little bit UI inconsistent compared with UX spec, can you check the comment on Github ? THanks !

	Arthur Chen [:arthurcc][inactive after 6/30] Assignee
	Comment 6 • 10 years ago

If what you meant is the style, we will have a separate bug to change the styles of all menu items.

	EJ Chen [:eragonj][:小龍哥][Contributor mode on](inactive)
	Comment 7 • 10 years ago

Comment on attachment 8452828 [details]
link to https://github.com/mozilla-b2g/gaia/pull/21490

Thanks for the patch, Arthur.

r+++

	Arthur Chen [:arthurcc][inactive after 6/30] Assignee
	Comment 8 • 10 years ago

Thank you, EJ!

master: 8712915c5d36f681308afb475ea387af185e3715