Skip to content

helix: Apply modification (e.g. switch case) on a single character only in helix mode#38119

Merged
kubkon merged 4 commits intozed-industries:mainfrom
romaninsh:helix-fix-case-without-selection
Sep 18, 2025
Merged

helix: Apply modification (e.g. switch case) on a single character only in helix mode#38119
kubkon merged 4 commits intozed-industries:mainfrom
romaninsh:helix-fix-case-without-selection

Conversation

@romaninsh
Copy link
Contributor

@romaninsh romaninsh commented Sep 13, 2025

Closes #34192

Without selection, only current character would be affected.

Also if #38117 is merged too, then transformations in SelectMode behave correctly too and selection is not collapsed.

Release Notes:

  • helix: Implemented ~, `, Alt-` correctly in normal and select modes

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Sep 13, 2025
@kubkon kubkon self-assigned this Sep 13, 2025
@kubkon kubkon added the area:parity/helix Feedback for Helix parity features label Sep 15, 2025
Copy link
Member

@kubkon kubkon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks! I've added some basic asserts for 'alt-' and '' keystrokes too, and tweaked case of 'Alt-' to lowercase in vim.json` keymap for consistency.

@kubkon kubkon enabled auto-merge (squash) September 18, 2025 08:33
@kubkon kubkon merged commit ed46e2c into zed-industries:main Sep 18, 2025
22 checks passed
@romaninsh romaninsh deleted the helix-fix-case-without-selection branch September 19, 2025 03:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:parity/helix Feedback for Helix parity features cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

helix: Change case behaviour differs from helix

2 participants