Skip to content

Fix dot-repeat for Vim change operators - #74

Open
zasdaym wants to merge 1 commit into
badranx:masterfrom
zasdaym:fix/repeat-pending-change
Open

zasdaym wants to merge 1 commit into
badranx:masterfrom
zasdaym:fix/repeat-pending-change

Conversation

@zasdaym

@zasdaym zasdaym commented Sep 11, 2026

Copy link
Copy Markdown

Closes #73.

Signed-off-by: Zasda Mikail <zasdaym@gmail.com>
@zasdaym
zasdaym marked this pull request as ready for review September 11, 2026 10:09
@badranx

badranx commented Sep 11, 2026

Copy link
Copy Markdown
Owner

Hey thanks for the PR. I tested the ciw and it works, diw didn't. Another point is that this repo should keep changes to Helix source code minimal and simple. Rebasing on Helix should be a simple task that anyone could do it quickly.

We also need to keep :vim-disable/:vim-enable working (which can be achieved by creating a custom vim command instead of changing Helix's).

For example, we put most logic for vim commands in helix-term/src/commands/vim_patch.rs‎ to reduce changes to Helix master branch.

I should add these contributions info somewhere, sorry.

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.

Dot . does not correctly repeat a ciw change on another word

2 participants