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

Change Meta key to Super #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

lecormi
Copy link

@lecormi lecormi commented Mar 31, 2020

For Linux, the modification makes it possible to simulate the pressing of the "Windows" key which is Super when simulating the pressing of the key key.Code.META. Indeed, knowing that the key has been associated on Windows to the "Windows" key (winuser::VK_LWIN) and on Mac OS to the "Apple" key (event::KeyCode::COMMAND), it seems more coherent to point to the Super key on Linux (key code X11 ``XK_Super_L```).
In any case, it would probably have been better to call this key "Super" and not "Meta" in the code, even if it means creating support for the "Meta" key separately (which is a very rare key, only existing on Sun computer keyboards according to Wikipedia).

@herbert-j-gamer
Copy link

I just debugged this issue and came up with the same solution. "Meta L" ends up being interpreted as "Alt" on my system. It should use "Super L" for the "Windows" key.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants