-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Rename KeyCode::*Win
to KeyCode::*Meta
#3240
Comments
'Meta' feels foreign for Windows and Linux users and also note that there are a lot more people on Windows than on MacOS. Imho this key won't be used in most games and apps so no need to bother. |
What emacs calls the meta key is the alt key on most keyboards or option on apple keyboards. Same for vim. |
Super is another possibility. I've made apps in Electron and I think they use both.
That's probably true. Although, shortcuts like Super+W might be handy for multi-window applications. Or Super+C to copy text. And things like that. But if these are seen to be unlikely cases, then I suppose we could just stick with the |
Oh didn't know that. Maybe Otherwise, like I said in my other comment, we can just leave this as is if the community doesn't see any merit in it currently. |
how about |
This is the one! 😂 |
The QMK keyboard firmware uses the word
|
Is having an alias an option? The problem is most keyboards have a "Windows" key regardless of your OS. So, simply changing it from one to another only moves the confusion, it doesn't really solve anything. |
While it could be nice (again, this is how Electron handles it I think), it could be an issue when matching the KeyCode. We'd probably have to write something like: match keycode {
KeyCode::LWin | KeyCode::LMeta => { /* Do something */ },
// ...
} So I think an alias might not be the proper solution.
Yeah, that's what I think it comes down to. Most people are either on Windows or using a keyboard with a Windows key. A change like this might not be favored by the majority then, even if it does help agnosticize the OS/hardware terminology. |
Closing for now, as it seems a change to |
I guess we might just leave this discussion for now and make a poll on this topic in the future. I've did a small poll in my blog and people preferred Super to Meta and no one wanted Win. (only 14 people participated) also thought of an idea that there are a lot of gamers on Windows, but the majority of developers are on Linux/MacOS which might be an interesting point to consider in this context. |
Why not just add some documentation to |
This is an easy compromise for the time being and should at least help |
# Objective `KeyCode::*Win` and `KeyCode::*Alt` might be confusing for some Mac users. ## Solution Added some small documentation to clarify the mappings for those developing on a Mac. ## Additional Context Related issue: #3240
What problem does this solve or what need does it fill?
The KeyCode's
LWin
andRWin
can be somewhat confusing/difficult to find for us Mac users. They seem like they should only work for Windows-style keyboards.What solution would you like?
Rename
KeyCode::LWin
andKeyCode::RWin
with the more genericKeyCode::LMeta
andKeyCode::RMeta
.The text was updated successfully, but these errors were encountered: