Skip to content

Mark legacy modifier keys and codes as deprecated#49

Merged
pyfisch merged 1 commit intorust-windowing:mainfrom
madsmtm:deprecated
Mar 23, 2025
Merged

Mark legacy modifier keys and codes as deprecated#49
pyfisch merged 1 commit intorust-windowing:mainfrom
madsmtm:deprecated

Conversation

@madsmtm
Copy link
Copy Markdown
Member

@madsmtm madsmtm commented Dec 1, 2024

This resolves my concern with Meta vs. Super from #19.

@pyfisch
Copy link
Copy Markdown
Contributor

pyfisch commented Dec 22, 2024

Please rebase the PR.

@madsmtm
Copy link
Copy Markdown
Member Author

madsmtm commented Feb 13, 2025

Rebased now.

@madsmtm madsmtm marked this pull request as ready for review February 13, 2025 17:11
@madsmtm
Copy link
Copy Markdown
Member Author

madsmtm commented Mar 7, 2025

Weird, KeyCode::Cut, KeyCode::Find, etc. seems like they're actually emitted by Winit?
TODO: What does Chrome/Firefox emit on Web if you click these keys?

@madsmtm madsmtm changed the title Mark legacy keys and codes as deprecated Mark legacy modifier keys and codes as deprecated Mar 10, 2025
@madsmtm
Copy link
Copy Markdown
Member Author

madsmtm commented Mar 10, 2025

Weird, KeyCode::Cut, KeyCode::Find, etc. seems like they're actually emitted by Winit?
TODO: What does Chrome/Firefox emit on Web if you click these keys?

Looking at Chromium's source code, it's clear that they do emit these key codes. So I've un-marked them as deprecated for now, and made this PR only focus on the deprecated modifier keys (since that will be confusing when upgrading Winit).

@madsmtm madsmtm mentioned this pull request Mar 10, 2025
9 tasks
@pyfisch pyfisch merged commit 4565361 into rust-windowing:main Mar 23, 2025
3 checks passed
@madsmtm madsmtm deleted the deprecated branch March 23, 2025 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants