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

Gnus: add gnus-summary-browse-url binding #825

Merged
merged 1 commit into from
Aug 5, 2024

Conversation

ttybitnik
Copy link
Contributor

Function: gnus-summary-browse-url
Default binding: 'w'
Proposed binding: 'gX'
Rationale: This same Vim-like binding already exists in the package, but only for the article-mode/map. This addition enables a more dynamic workflow allowing users to open articles' URLs directly from summary-mode, as ̶G̶o̶d̶ ̶i̶n̶t̶e̶n̶d̶e̶d̶ the function name describes.

For the record, I personally have overridden this and other bindings in this module to match the Gnus defaults in my configuration, some of them create less friction IMO. However, I believe I will eventually get used to the choices made by the collection, coming from a Gnus/Emacs defaults background. The more I learn Vim and consider the collections' changes/choices for Gnus, the more they make sense 😄 (Anyways, I digress).

@condy0919
Copy link
Collaborator

For the record, I personally have overridden this and other bindings in this module to match the Gnus defaults in my configuration, some of them create less friction IMO. However, I believe I will eventually get used to the choices made by the collection, coming from a Gnus/Emacs defaults background. The more I learn Vim and consider the collections' changes/choices for Gnus, the more they make sense 😄 (Anyways, I digress).

The GNUS bindings are modeled after mu4e's. If you thought it's inconvenient, feel free to open a discussion/PR. 😃

@condy0919 condy0919 merged commit 525463e into emacs-evil:master Aug 5, 2024
7 of 12 checks passed
@ttybitnik ttybitnik deleted the feat/gnus-browse branch August 5, 2024 22:38
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