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.

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

[linux] [x11] non-qwerty keyboard layouts broken #147

Closed
coderobe opened this issue Mar 6, 2018 · 2 comments
Closed

[linux] [x11] non-qwerty keyboard layouts broken #147

coderobe opened this issue Mar 6, 2018 · 2 comments
Assignees
Milestone

Comments

@coderobe
Copy link
Contributor

coderobe commented Mar 6, 2018

The text inputs register the wrong chars with non-qwerty keyboard layouts.
The STDOUT output prints the correct chars.

Reproduction Steps:

  1. Change keyboard layout to qwertz (de)
  2. Focus a text input
  3. Press the , key
  4. Hold left shift
  5. Press the same key again

The text input now shows ,< which is correct for qwerty, but wrong for qwertz (de)
STDOUT logs this:

linux/linux_x11keyboard.c 103: OnKeyboardMessage(): keyname: comma
linux/linux_x11keyboard.c 105: OnKeyboardMessage(): keycode: 188, keyscancode: 59, keysym: 44
linux/linux_x11keyboard.c 112: OnKeyboardMessage(): char: ,
linux/linux_x11keyboard.c 103: OnKeyboardMessage(): keyname: comma
linux/linux_x11keyboard.c 105: OnKeyboardMessage(): keycode: 188, keyscancode: 59, keysym: 44
linux/linux_x11keyboard.c 112: OnKeyboardMessage(): char: ,
linux/linux_x11keyboard.c 103: OnKeyboardMessage(): keyname: Shift_L
linux/linux_x11keyboard.c 105: OnKeyboardMessage(): keycode: 16, keyscancode: 50, keysym: 65505
linux/linux_x11keyboard.c 103: OnKeyboardMessage(): keyname: comma
linux/linux_x11keyboard.c 105: OnKeyboardMessage(): keycode: 188, keyscancode: 59, keysym: 44
ime.c 232: LCUIIME_ToText(): hit shift, ch: ,
linux/linux_x11keyboard.c 112: OnKeyboardMessage(): char: ;
linux/linux_x11keyboard.c 103: OnKeyboardMessage(): keyname: comma
linux/linux_x11keyboard.c 105: OnKeyboardMessage(): keycode: 188, keyscancode: 59, keysym: 44
linux/linux_x11keyboard.c 112: OnKeyboardMessage(): char: ;
linux/linux_x11keyboard.c 103: OnKeyboardMessage(): keyname: Shift_L
linux/linux_x11keyboard.c 105: OnKeyboardMessage(): keycode: 16, keyscancode: 50, keysym: 65505

The OnKeyboardMessage(): char log corresponds to the text input ,;, which is the correct output for this keyboard layout. The text input still has the wrong characters.

LCUI version: current git master
OS and version: Linux 4.15.6

@coderobe
Copy link
Contributor Author

as far as i know, this is still unsolved and should not be closed because it's "stale"

@stale stale bot removed the stale label Jun 18, 2018
Repository owner deleted a comment from stale bot Jun 19, 2018
@lc-soft lc-soft added this to the 1.0.0 Beta 4 milestone Jun 19, 2018
@lc-soft lc-soft self-assigned this Jul 17, 2018
@lc-soft
Copy link
Owner

lc-soft commented Jul 19, 2018

Please try to use the branch fix_147_update_ime to test again.

@lc-soft lc-soft closed this as completed Jul 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants