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

Note that "super" is often emitted as Meta on Linux #68

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

Conversation

madsmtm
Copy link
Contributor

@madsmtm madsmtm commented Mar 10, 2025

From discussion in rust-windowing/winit#4018 (comment). Happy to take suggestions on the wording.

Related to #49.

@madsmtm madsmtm force-pushed the meta-super-reference branch from 6d69a3f to 2baf1fd Compare March 10, 2025 16:35
@pyfisch
Copy link
Owner

pyfisch commented Mar 23, 2025

I read these two Wikipedia articles and I am not sure if the explanation makes things clearer or more confusing.

Since the text already mentions the "Windows Logo" I think it is sufficiently clear to general audience which key is meant. I will however admit that a some Linux fans ignore the existence of Microsoft and may prefer to call the same key "Super", like the key from the MIT Lisp machine in the same position.

@madsmtm
Copy link
Contributor Author

madsmtm commented Mar 23, 2025

Yeah, I'm not too happy about the wording either. @ArturKovacs WDYT?

@ArturKovacs
Copy link

I think that this PR improves the usability of the documentation because it's now easier to search for someone coming from Linux.

@madsmtm madsmtm force-pushed the meta-super-reference branch from 2baf1fd to 84b1a5e Compare March 27, 2025 16:03
@madsmtm
Copy link
Contributor Author

madsmtm commented Mar 27, 2025

Tried to change the wording a bit, to make it clearer that it is not necessarily the "Super" key itself, but moreso XKB terminology.

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

3 participants