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

fix: split on > for sequence mappings #40

Merged
merged 2 commits into from
Mar 8, 2023

Conversation

mikesmithgh
Copy link
Contributor

This is a draft PR, I had an issue mapping with ctrl+w>l syntax. I came up with this approach but have not checked for edge cases or side effects. I'll move out of draft after further investigation or you can pick up as well.

map ctrl+w>j kitten pass_keys.py neighboring_window bottom 'ctrl+w>j' nvim
map ctrl+w>k kitten pass_keys.py neighboring_window top    'ctrl+w>k' nvim
map ctrl+w>h kitten pass_keys.py neighboring_window left   'ctrl+w>h' nvim
map ctrl+w>l kitten pass_keys.py neighboring_window right  'ctrl+w>l' nvim

Thanks for the kitten! 😸

@knubie
Copy link
Owner

knubie commented Feb 17, 2023

Awesome, thanks for the contribution @mikesmithgh !

@mikesmithgh mikesmithgh marked this pull request as ready for review March 7, 2023 01:58
@mikesmithgh
Copy link
Contributor Author

Hey @knubie I am moving from draft to ready for review. I have been using it without any issues.

@knubie knubie merged commit 949ab61 into knubie:master Mar 8, 2023
@knubie
Copy link
Owner

knubie commented Mar 8, 2023

Thanks @mikesmithgh ! I assume this closes #5?

@mikesmithgh
Copy link
Contributor Author

@knubie Yes that should close #5. I just commented on that ticket with my configuration 👍

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.

2 participants