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
Added numpad support #2032
Added numpad support #2032
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we want to deal with the numpad keys
I have not yet tested this, but the code looks good. |
Re-requesting check due to a bug introduced by erroneously adding < and > signs within the special key formatting function |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, that's good. I still haven't tested it myself though.
Thank you! I did not get the all the modifiers to work in combination with the keypad on Windows. But I think that's more of a problem with Winit not reporting it correctly, or even not supported by the operating system. |
Follow up of neovide#2032 See neovim docs [0] as well as winit's [1]. Please note that I don't have a numpad with kEqual and kComma keys, I could only test kPoint. [0] https://github.com/neovim/neovim/blob/1bf645918e94e7e8f770592484164f1ee303f24e/runtime/doc/intro.txt#L377-L379 [1] https://docs.rs/winit/0.29.10/winit/keyboard/enum.KeyCode.html#variant.NumpadComma
Follow up of neovide#2032 See neovim docs [0] as well as winit's [1]. [0] https://github.com/neovim/neovim/blob/1bf645918e94e7e8f770592484164f1ee303f24e/runtime/doc/intro.txt#L377-L379 [1] https://docs.rs/winit/0.29.10/winit/keyboard/enum.KeyCode.html#variant.NumpadComma
Follow up of neovide#2032 See neovim docs [0] as well as winit's [1]. [0] https://github.com/neovim/neovim/blob/1bf645918e94e7e8f770592484164f1ee303f24e/runtime/doc/intro.txt#L377-L379 [1] https://docs.rs/winit/0.29.10/winit/keyboard/enum.KeyCode.html#variant.NumpadComma
Follow up of neovide#2032 See neovim docs [0] as well as winit's [1]. [0] https://github.com/neovim/neovim/blob/1bf645918e94e7e8f770592484164f1ee303f24e/runtime/doc/intro.txt#L377-L379 [1] https://docs.rs/winit/0.29.10/winit/keyboard/enum.KeyCode.html#variant.NumpadComma
Follow up of neovide#2032 See neovim docs [0] as well as winit's [1]. [0] https://github.com/neovim/neovim/blob/1bf645918e94e7e8f770592484164f1ee303f24e/runtime/doc/intro.txt#L377-L379 [1] https://docs.rs/winit/0.29.10/winit/keyboard/enum.KeyCode.html#variant.NumpadComma
Follow up of #2032 See neovim docs [0] as well as winit's [1]. [0] https://github.com/neovim/neovim/blob/1bf645918e94e7e8f770592484164f1ee303f24e/runtime/doc/intro.txt#L377-L379 [1] https://docs.rs/winit/0.29.10/winit/keyboard/enum.KeyCode.html#variant.NumpadComma
What kind of change does this PR introduce?
Did this PR introduce a breaking change?
A breaking change includes anything that breaks backwards compatibility either at compile or run time.