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

Add more less-compatible keybindings #29

Closed

Conversation

hupfdule
Copy link

The following keybindings were added:

 - sync:           [r], [R], [ctrl+r]
 - down:           [e], [ctrl+e], [j], [J], [ctrl+j]
 - up:             [y], [Y], [ctrl+y], [k], [K], [ctrl+k]
 - page down:      [space], [alt+space], [f], [z]
 - page up:        [b], [alt+v]
 - half page down: [u]
 - half page up:   [d]

[z] differs a bit from its less-functionality as it doesn't
support the leading count (as ov doesn't accept such a count).

@noborus
Copy link
Owner

noborus commented Oct 23, 2020

Thank you for the pull request.
I want to avoid making all default keybindings like less
(It will assign new and frequently used keys to hard-to-press keys).
I'm sorry, but I want to have a config file template for less users.

The following keybindings were added:

 - sync:           [r], [ctrl+r]
 - down:           [e], [ctrl+e], [j], [J], [ctrl+j]
 - up:             [y], [Y], [ctrl+y], [k], [K], [ctrl+k]
 - page down:      [space], [alt+space], [f], [z]
 - page up:        [b], [alt+v], [w]
 - half page down: [u]
 - half page up:   [d]

[z] and [w] differ a bit from their less-functionality as they don't
support the leading count (as ov doesn't accept such a count).
Allow [ctrl+g] and [ctrl+c] to abort user input.
@hupfdule hupfdule force-pushed the add-less-compatible-keybindings branch from 0f840f1 to 5c5b170 Compare October 25, 2020 21:41
@noborus
Copy link
Owner

noborus commented Oct 27, 2020

I don't think it's correct to make the default keybindings like less.
Too many keys are assigned to a function.
I think this is because less was wrong.

I would like to have a config file for less users.

@noborus noborus closed this Apr 22, 2021
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.

2 participants