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

break!: revamp mappings #56

Merged
merged 1 commit into from
Jan 11, 2022
Merged

break!: revamp mappings #56

merged 1 commit into from
Jan 11, 2022

Conversation

fdschmidt93
Copy link
Member

@fdschmidt93 fdschmidt93 commented Jan 11, 2022

Continuing #49 with the slight change of toggling to s to not affect changing to insert mode, that one would bite me too often

@fdschmidt93
Copy link
Member Author

Already apologies to anyone who doesn't like this. Now there is some more logic to default mappings, which are primarily intended for novice and new users to simplify discoverability, as I trust experienced users can easily remap stuff within 5 minutes. If enough users speak up I'm happy to change this again though. I personally don't care so much, but prefer that we now have some logic in place.

In any case, remapping should be very straightforward.

@fdschmidt93 fdschmidt93 merged commit 0c3290b into master Jan 11, 2022
@fdschmidt93 fdschmidt93 deleted the break/revamp-mappings branch January 11, 2022 22:10
@fdschmidt93
Copy link
Member Author

fdschmidt93 commented Jan 11, 2022

Please direct any creds (or blame 😆 ) to @ikerurda

Thanks @ikerurda for getting the ball rolling on this :)

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.

None yet

1 participant