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

List that space can be bound through Char(' ') #486

Merged
merged 1 commit into from Sep 20, 2022

Conversation

sholderbach
Copy link
Member

Necessary to complete nushell/nushell#6590

(Comment: this whole module feels intricately linked to the nushell
implementation as we do not implement the string parsing in reedline)

Necessary to complete nushell/nushell#6590

(Comment: this whole module feels intricately linked to the nushell
implementation as we do not implement the string parsing in reedline)
@sholderbach sholderbach added A-ViKeybinding Area: Vi(m) keybinding support chore Various work that should be done to keep the repo in a nice shape labels Sep 20, 2022
@sholderbach sholderbach merged commit 710393a into nushell:main Sep 20, 2022
@sholderbach sholderbach deleted the list-space-key branch September 20, 2022 10:19
sholderbach added a commit to sholderbach/nushell that referenced this pull request Sep 20, 2022
sholderbach added a commit to nushell/nushell that referenced this pull request Sep 20, 2022
* Add "space" key to bind in vi normal mode

Implements #6586

No special logic to prevent you from binding it in other modes!
Needs a separate change to reedline to make it available in the default
listing of `keybindings list`.

* Update reedline to report the available `space`

Pulls in nushell/reedline#486
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-ViKeybinding Area: Vi(m) keybinding support chore Various work that should be done to keep the repo in a nice shape
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant