Skip to content

revert(panel): restore w as multi-file selection toggle#127

Merged
dlyongemallo merged 1 commit intomainfrom
change_default_multiselection_key
Apr 20, 2026
Merged

revert(panel): restore w as multi-file selection toggle#127
dlyongemallo merged 1 commit intomainfrom
change_default_multiselection_key

Conversation

@dlyongemallo
Copy link
Copy Markdown
Owner

@dlyongemallo dlyongemallo commented Apr 20, 2026

Closes #124.

See #66 for context.

Copilot AI review requested due to automatic review settings April 20, 2026 07:56
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Restores the default multi-file selection toggle in Diffview’s file panel back to w, aligning the runtime defaults with the documented defaults and removing guidance that was only necessary when the toggle was bound to <Space> (leader-key conflict).

Changes:

  • Change the default file_panel keymap for actions.toggle_select_entry from <Space> back to w.
  • Update user docs to reflect w as the toggle key.
  • Remove the <Space>/leader-key conflict tip since it no longer applies to the defaults.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
lua/diffview/config.lua Restores w as the default file-panel multi-select toggle key.
doc/diffview_defaults.txt Updates default-config documentation to match the restored w mapping.
doc/diffview.txt Updates keymap reference section and removes now-irrelevant <Space> remap example.
TIPS.md Removes the <Space> leader-key conflict workaround tip (no longer relevant to defaults).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dlyongemallo dlyongemallo merged commit 9e39464 into main Apr 20, 2026
6 checks passed
@dlyongemallo dlyongemallo deleted the change_default_multiselection_key branch April 20, 2026 08:01
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.

replace <space> as default key for multi-selection

2 participants