Skip to content

Add extra shortcuts for builtins - #755

Open
9999years wants to merge 1 commit into
Canop:mainfrom
9999years:extra-shortcuts
Open

Add extra shortcuts for builtins#755
9999years wants to merge 1 commit into
Canop:mainfrom
9999years:extra-shortcuts

Conversation

@9999years

@9999years 9999years commented Oct 9, 2023

Copy link
Copy Markdown
Contributor
  • input_go_word_left: alt-b
  • input_go_word_right: alt-f
  • line_down: ctrl-j
  • line_up: ctrl-k
  • panel_left_no_open: ctrl-h
  • panel_right_no_open: ctrl-l

These don't conflict with any existing builtin shortcuts and users can override them if they want.

- input_go_word_left: alt-b
- input_go_word_right: alt-f
- line_down: ctrl-j
- line_up: ctrl-k
- panel_left_no_open: ctrl-h
- panel_right_no_open: ctrl-h
@Canop

Canop commented Oct 10, 2023

Copy link
Copy Markdown
Owner

Those look a little too personal, apart maybe the line_down and line_up. Where do the alt-f for input_go_word_right come from ?
Unless there's some support from other users I'm reluctant to add those: if they're just personal, they can be just in your config.

Other users are invited to give their opinion.

@9999years

Copy link
Copy Markdown
Contributor Author

alt-b and alt-f for word-back and word-forward come from Emacs, although they're supported in many terminals as well.

@ghost

ghost commented Nov 28, 2023

Copy link
Copy Markdown

alt-b and alt-f for word-back and word-forward come from Emacs, although they're supported in many terminals as well.

Is it possible to have readline(3)-like binding when in command mode? Helix editor did it.

@Canop

Canop commented Nov 28, 2023

Copy link
Copy Markdown
Owner

The right solution to me seems to be the addition of a commented-out block of settings in the default configuration, similar to what we already have for vim-like shortcuts: https://github.com/Canop/broot/blob/main/resources/default-conf/verbs.hjson#L113

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