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

Move left when exiting insert mode #699

Merged
merged 1 commit into from
Feb 9, 2024
Merged

Conversation

andreistan26
Copy link
Contributor

@andreistan26 andreistan26 commented Jan 4, 2024

As mentioned in #694 using the vi editor mode does not match the behavior of the vim/vi CLI editors.

src/edit_mode/vi/.mod.rs.swp Outdated Show resolved Hide resolved
@fdncred
Copy link
Collaborator

fdncred commented Feb 9, 2024

Does the second checkbox mean this PR is not finished?

@andreistan26
Copy link
Contributor Author

Updated, will stay as it is

@fdncred
Copy link
Collaborator

fdncred commented Feb 9, 2024

@stormasm I don't use vi mode. Any thoughts on this change?

@stormasm
Copy link
Contributor

stormasm commented Feb 9, 2024

@andreistan26 can you please make sure the code you are changing still works fine with all of the latest changes that we have landed in Reedline recently ? In other words merge your code back into the latest main so we can have the CI run again. And test your code to make sure it works as you think it should....

Thank you !

@andreistan26
Copy link
Contributor Author

andreistan26 commented Feb 9, 2024

Rebased and tested with latest main branch.

@stormasm stormasm merged commit 02f551d into nushell:main Feb 9, 2024
6 checks passed
@stormasm
Copy link
Contributor

stormasm commented Feb 9, 2024

@andreistan26 Thank you for checking the code one more time prior to us landing...
have a nice weekend 😄

fdncred added a commit that referenced this pull request Mar 18, 2024
sholderbach pushed a commit that referenced this pull request Mar 19, 2024
Simulating vim's cursor logic by moving the actual cursor breaks the history traversal. Thus revert.

This reverts commit 02f551d.
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.

4 participants