17cupsofcoffee / tetra

🎮 A simple 2D game framework written in Rust
MIT License
907 stars 63 forks source link

Change KeyModifier to be layout-based again #295

Closed 17cupsofcoffee closed 2 years ago

17cupsofcoffee commented 2 years ago

As reported on #293, the keyboard API changes introduced in 0.6.6 made it very difficult to implement 'native' feeling keyboard shortcuts. Since this is basically the entire reason for KeyModifier to exist, I think it's worth changing the behaviour for that type back to how it was.

cc: @sumibi-yakitori - can you try this branch and let me know if it behaves as expected?

sumibi-yakitori commented 2 years ago

@17cupsofcoffee Thanks. I have confirmed that there is no problem with that behavior. I've only checked MacOS and Ubuntu... 😄