-
-
Notifications
You must be signed in to change notification settings - Fork 112
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
bug(windows): LDML keyboards do not appear to work correctly with Alt keys #12298
Comments
It is failing in Windows because the LDML keyboard processor has not been fully implemented ldml_processor::get_key_list() it just returns an empty list. In Windows the Keyman engine needs to register the interest in the preserved keys otherwise it will not be seen in the hook for the Keyman keyboard processor. Implementation detail: For more information
@srl295 Could you implement |
I see three of these that don't have any 'interesting' implementation. i'll check over the docs. km_core_context_item * get_intermediate_context() override;
km_core_keyboard_key * get_key_list() const override;
km_core_keyboard_imx * get_imx_list() const override; |
|
- move some test utils to statics - add a new test action type and `@@key-list` keyword - print warning on unhanded @-commands Fixes: #12298
- ldml::vkeys class updated to keep a set<> of keys - fix the test case to not leak! Fixes: #12298
- just compare the key list to the key2.kmap table - check the keylist for all LdmlTestSource instances - no syntax needed Fixes: #12298
Trying to repro the failing case.
|
This is reproducing the error. As the alt key has not been added a preserved key, see description above. Therefore the ALT key is going through to the application where is is treated and the menu key.
Testing this again I can only reproduce it by having caps lock on - in which case |
Window 10 OS:
alt+e, ctrl+e, shift+e, ctrl+shift+e, ctrl+alt+e, e
https://github.com/user-attachments/assets/7d1f1153-80d0-438d-874e-de32ffd619a8
Originally posted by @dinakaranr in #12281 (comment)
The text was updated successfully, but these errors were encountered: