Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Unexpected software keyboard layout can be chosen #369

Closed
yukawa opened this issue Jul 23, 2016 · 0 comments
Closed

Unexpected software keyboard layout can be chosen #369

yukawa opened this issue Jul 23, 2016 · 0 comments
Labels
bug OpSys-Android

Comments

@yukawa
Copy link
Collaborator

yukawa commented Jul 23, 2016 •
edited
Loading

Environment

  • Mozc 2.17.2354.103 (as of 02d3afb)
  • Nexus 5X
  • Android 6.0.1 MTC19Z

How to setup the environment to reproduce

  1. Flash Android 6.0.1 MTC19Z into Nexus 5X
  2. Complete the setup wizard.
  3. Build and install Mozc for Android 2.17.2354.103
  4. adb shell ime enable org.mozc.android.inputmethod.japanese/.MozcService
  5. adb shell ime set org.mozc.android.inputmethod.japanese/.MozcService

Steps to reproduce

  1. Open the Contact app
  2. Tap the Floating Action Button (FAB) to add a new contact
  3. Focus in Email field.
  4. Long tap あ/a key on the Mozc to show the settings dialog
  5. Select Mozc settings
  6. Change the keyboard layout from 12 keys to QWERTY.
  7. Tap the back button to show the Contact app again.
  8. Focus in Phone field.
  9. Focus in Name field.

Expected behavior

After the step 8, Mozc shows number layout.
After the step 9, Mozc shows QWERTY layout.
mozc_b22676055_mtc19z_expected

Actual behavior

After the step 8, Mozc shows number layout.
After the step 9, Mozc still shows number layout.
mozc_b22676055_actual

Internal Issue ID: 22676055.
@yukawa yukawa closed this as completed Jul 24, 2016
yukawa pushed a commit that referenced this issue Jul 24, 2016
@sumita12
Fix an edge case of keyboard layout state machine
c879c16 
Pure number keyboard doesn't have any IME switching keys, so we should
NOT show it on general text field.

This fixes #369.

BUG=#369
TEST=manually done
REF_BUG=22676055
REF_CL=99455841
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug OpSys-Android
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@yukawa