Skip to content

Handle starting and continuing the count separately - #9887

Merged
archseer merged 1 commit into
masterfrom
fix-count-interaction-with-bindings
Mar 15, 2024
Merged

Handle starting and continuing the count separately#9887
archseer merged 1 commit into
masterfrom
fix-count-interaction-with-bindings

Conversation

@the-mikedavis

Copy link
Copy Markdown
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
@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
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
mtoohey31 pushed a commit to mtoohey31/helix that referenced this pull request Jun 2, 2024
Vulpesx pushed a commit to Vulpesx/helix that referenced this pull request Jun 7, 2024
smortime pushed a commit to smortime/helix that referenced this pull request Jul 10, 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