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

Handle starting and continuing the count separately #9887

Merged
merged 1 commit into from Mar 15, 2024

Conversation

the-mikedavis
Copy link
Member

We need to handle number key events differently depending on whether the count has been started or not:

  • If the count has been started, any number will continue the count.
  • If the count hasn't been started yet, a non-zero numeric key can start the count if it isn't bound to a command.

This supports custom bindings for any of the number keys. (Though those bindings cannot use counts.)

Fixes #9871

We need to handle number key events differently depending on whether the
count has been started or not:

* If the count has been started, any number will continue the count.
* If the count hasn't been started yet, a non-zero numeric key can
  start the count if it isn't bound to a command.

This supports custom bindings for any of the number keys. (Though those
bindings cannot use counts.)
@the-mikedavis the-mikedavis added C-bug Category: This is a bug E-easy Call for participation: Experience needed to fix: Easy / not much S-waiting-on-review Status: Awaiting review from a maintainer. labels Mar 15, 2024
@archseer archseer merged commit 9282f1b into master Mar 15, 2024
6 checks passed
@archseer archseer deleted the fix-count-interaction-with-bindings branch March 15, 2024 23:52
Desdaemon pushed a commit to Desdaemon/helix that referenced this pull request Mar 26, 2024
shortc pushed a commit to shortc/helix that referenced this pull request Mar 31, 2024
postsolar pushed a commit to postsolar/helix that referenced this pull request Apr 4, 2024
postsolar pushed a commit to postsolar/helix that referenced this pull request Apr 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug E-easy Call for participation: Experience needed to fix: Easy / not much S-waiting-on-review Status: Awaiting review from a maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Count handling no longer accepts numbers containing zero
3 participants