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

Use <DEL> for scroll-down in mu4e-view-mode like in Gnus #2757

Closed
wants to merge 1 commit into from

Conversation

manphiz
Copy link
Contributor

@manphiz manphiz commented Sep 11, 2024

  • It looks like <DEL> does not work in mu4e-view-mode, while it does in mu4e-raw-view-mode which inherits keybindings from view-mode.
  • "S-SPC" does not seem to work reliably under terminal either.
  • Adding <DEL> make it more consistent with existing paging as in info, Gnus, etc.

* It looks like <DEL> does not work in mu4e-view-mode, while it does in
mu4e-raw-view-mode which inherits keybindings from view-mode.
* "S-SPC" does not seem to work reliably under terminal either.
* Adding <DEL> make it more consistent with existing paging as in info, Gnus,
etc.
@manphiz
Copy link
Contributor Author

manphiz commented Sep 11, 2024

Not sure whether this is acceptable but probably worth a try.

Also I'm aware there exists a binding on <backspace> for #'mu4e-scroll-down but it didn't work for me, probably because I'm mainly under terminal. If possible, I would suggest for <DEL> to be preferred for scroll-down to be consistent with other modes.

@djcb
Copy link
Owner

djcb commented Sep 25, 2024

Thanks!

At least in GUI mode, <DEL> already is used for marking a message for deletion (which seems reasonable), so I think we don't want to change that in the default configuration.

You can change it in your private configuration of course.

@djcb djcb closed this Sep 25, 2024
@manphiz
Copy link
Contributor Author

manphiz commented Sep 25, 2024

Understood. Thanks for clarifying.

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