-
Notifications
You must be signed in to change notification settings - Fork 26
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
use react-kbs for shortcut #1408
Comments
@lpatiny |
@lpatiny that means we will keep the same way of handling the problem by setting a flag in the global state to whether the mouse is over the Displayer or not and check this when we attend to do action. so I think after I took a look at our code, maybe this will make the code nicer but will not change a lot we will still follow the old way and we can make a meeting for further discussion |
|
@hamed-musallam Could you make a PR that shows the error ? For example the the shortcuts '1', '2', ... We would need to test in on swiss and franch keyboards. |
the demo should be at the side of |
react-kbs is used in some other big react projects and we are wondering was is the problem in your case. So the PR should be in this project. |
this is the problem that i have #1408 (comment) |
the other projects that you mention seem not to have the same use case for example 1- press "2" to save the current preferences and if you press again it reapplies the preferences that you saved |
we can add the demo here https://zakodium-oss.github.io/react-kbs/ |
I was wrong in this example, because ! is the same for German and English keyboard layouts but shift+ 2 is not, and the same for the others shift + 3,4 .... etc |
https://github.com/zakodium-oss/react-kbs/releases/tag/v2.1.0 adds support for defining shortcuts with |
Please check how the following library could be used to manage the shortcuts:
https://github.com/zakodium/react-kbs
We should not forget that many NMRium components may be present in the same page.
The text was updated successfully, but these errors were encountered: