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

feat(cli): add vi solidus / keybinding #14908

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tmillr
Copy link
Contributor

@tmillr tmillr commented Jan 24, 2025

Description

  • Add keybinding for / when in vi normal mode which activates the history menu.
  • Make keybinding mode (edit_mode) case-insensitive.

This keybinding exists both in vim and GNU Readline (e.g. bash) when in vi normal mode. The reason this keybinding is getting added here (and not in reedline) is because it triggers the history menu, and should only be defined when the history menu exists. Menus are defined externally to reedline.

User-Facing Changes

Added keybinding for / when in vi normal mode which activates the history menu.

Tests + Formatting

After Submitting

TODO: Update docs

- Add keybinding for `/` when in vi normal mode which activates the history menu.
- Make keybinding `mode` (`edit_mode`) case-insensitive.
@tmillr tmillr force-pushed the feat-vi-solidus-keybinding branch from 460da0d to 8276f84 Compare January 24, 2025 18:28
@tmillr tmillr changed the title feat(cli): add vi solidus keybinding feat(cli): add vi solidus / keybinding Feb 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants