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

'dw' deletes first character of the next word #10

Closed
neur0manc opened this issue Feb 8, 2021 · 4 comments
Closed

'dw' deletes first character of the next word #10

neur0manc opened this issue Feb 8, 2021 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@neur0manc
Copy link

Hi,

thanks for this awesome plugin, I love the cursor transformation in different modes and increased visual feedback in visual mode. :-)

I found an oddity that appears to be a bug. I've tried to look into it myself, but zsh-vi-mode.zsh is a bit too much for my zsh scripting skills (although it is very well commented).

Sample string: "apple orange"

When typing dw at the beginning of the word apple, zsh-vi-mode also removes the o orange. This is unexpected, both default vi-mode in zsh and also vim/vi only remove up until the next word (including whitespace), not the beginning of the next word.

de as a workaround doesn't work, because it leaves a whitespace which dw usually would remove as well.

TL;DR

Input

`apple orange"

Action

dw while cursor is on a of apple

Expected Result

orange

Actual Result

range

@jeffreytse jeffreytse self-assigned this Feb 8, 2021
@jeffreytse jeffreytse added the bug Something isn't working label Feb 8, 2021
@jeffreytse
Copy link
Owner

jeffreytse commented Feb 8, 2021

Hi @ls42

Firstly thanks for your detailed description about the bug and I have confirmed this bug. I will figure out this issue asap, please be patient, and welcome to star this project for further updates.

Thanks and regards

jeffreytse added a commit that referenced this issue Feb 8, 2021
This commit fixed the bugs about deleting and changing an inner
word, changing an inner word and so on.
@jeffreytse
Copy link
Owner

Hi @ls42

🎉 Here is a good news! This issue has been addressed and fixed, please update it and have a try.

Thanks and regards

@neur0manc
Copy link
Author

That is amazing, thank you so much! Tested it, working perfectly now. Have a great day 👋

@jeffreytse
Copy link
Owner

jeffreytse commented Feb 8, 2021

Hi @ls42 , 😃 also hope you have a great day! And hope this plugin could help you work better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants