Skip to content

Fix Vi mode cW deleting trailing whitespace - #12790

Closed
linsomniac wants to merge 1 commit into
fish-shell:masterfrom
linsomniac:vim-mode-cw
Closed

Fix Vi mode cW deleting trailing whitespace#12790
linsomniac wants to merge 1 commit into
fish-shell:masterfrom
linsomniac:vim-mode-cw

Conversation

@linsomniac

Copy link
Copy Markdown
Contributor

Discussion in #12788

cW (change-WORD) changed through to the start of the next WORD, deleting the trailing whitespace. 02c0455 (Fix Vi mode cw deleting trailing whitespace, 2026-02-12) fixed this for cw but missed cW, and there is no reason for cw and cW to be inconsistent. Make cW change only to the end of the WORD, like cE and Vim, keeping the trailing whitespace.

Fixes 38e633d (fish_vi_key_bindings: add support for count, 2025-12-16).

TODOs:

  • If addressing an issue, a commit message mentions Fixes issue #<issue-number>
  • Changes to fish usage are reflected in user documentation/manpages.
  • Tests have been added for regressions fixed
  • User-visible changes noted in CHANGELOG.rst

cW (change-WORD) changed through to the start of the next WORD, deleting the
trailing whitespace. 02c0455 (Fix Vi mode cw deleting trailing whitespace,
2026-02-12) fixed this for cw but missed cW, and there is no reason for cw and
cW to be inconsistent. Make cW change only to the end of the WORD, like cE and
Vim, keeping the trailing whitespace.

Fixes 38e633d (fish_vi_key_bindings: add support for count, 2025-12-16).

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
@faho

faho commented May 30, 2026

Copy link
Copy Markdown
Member

Co-Authored-By: Claude Opus 4.8 (1M context) [email protected]

#12630 is leaning towards rejecting AI-contributions, sorry.

@faho faho closed this May 30, 2026
@linsomniac

Copy link
Copy Markdown
Contributor Author

Reporting #12791 so this doesn't get lost.

@krobelus krobelus reopened this May 30, 2026
@krobelus krobelus added this to the fish 4.8 milestone May 30, 2026
@krobelus

Copy link
Copy Markdown
Contributor

thanks for cleaning up my diff

@krobelus krobelus closed this in eb53532 May 30, 2026
@Pixelo789

Copy link
Copy Markdown

Why was this LLM-generated PR merged?

@Pixelo789 Pixelo789 mentioned this pull request May 31, 2026
4 tasks
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