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

Add new global option to allow change the time limit on modifier only key triggering #1204

Merged
merged 2 commits into from
Dec 14, 2024

Conversation

wengxt
Copy link
Member

@wengxt wengxt commented Dec 10, 2024

This is intended to solve some issue when user have other intentions.

For example, user may press "Shift" to do some text selection with
mouse. However, input method may not be capable (theoratically
possible on certain platform, but it's not implemented and not possible
on all platform) to detect user actually want to use mouse. If application
does not send "reset()", input method may wrongly trigger the action.

Some other cases include:

  • Ctrl and scroll mouse
  • user wants to type Shift+A, but regrets after press shift. So user
    release all key but accidentally triggers "Shift" to switch input
    method.

This is not perfect but should help solve some unintended action.

… key triggering

This is intended to solve some issue when user have other intentions.

For example, user may press "Shift" to do some text selection with
mouse. However, input method may not be capable (theoratically
possible on certain platform, but it's not implemented and not possible
on all platform) to detect user actually want to use mouse. If application
does not send "reset()", input method may wrongly trigger the action.

Some other cases include:
- Ctrl and scroll mouse
- user wants to type Shift+A, but regrets after press shift. So user
  release all key but accidentally triggers "Shift" to switch input
  method.

This is not perfect but should help solve some unintended action.
@wengxt wengxt merged commit 124cad3 into master Dec 14, 2024
4 checks passed
@wengxt wengxt deleted the modifier branch December 14, 2024 00:48
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.

1 participant