Skip to content

Add n/p navigation to PR list buffers - #4

Merged
jonathanchu merged 3 commits into
mainfrom
add-n-p-navigation
Feb 19, 2026
Merged

Add n/p navigation to PR list buffers#4
jonathanchu merged 3 commits into
mainfrom
add-n-p-navigation

Conversation

@jonathanchu

Copy link
Copy Markdown
Owner

Description

Adds n/p keybindings for navigating between item rows in all list buffers (PR list, PR status dashboard, PR checks). Navigation is intuitive and will skip headers, separators, and blank lines, only landing on actual item PR rows. This follows the standard Emacs convention for read-only list buffers (e.g. Magit sections).

Closes #3

Checklist

  • My elisp byte-compiles cleanly (M-x byte-compile-file)
  • I've run M-x checkdoc and addressed its feedback
  • I've run package-lint and addressed its feedback
  • I've tested this with a recent version of Emacs

Testing

Verified locally n/p works in PR list buffer and PR status dashboard, skipping the section boundaries.

@jonathanchu

Copy link
Copy Markdown
Owner Author

@chopmo merging this into main.

Navigation keybinding n/p should work in all PR list buffers. Thanks again for the suggestion and let me know what you think about this implementation!

@jonathanchu
jonathanchu merged commit 9fb03c0 into main Feb 19, 2026
2 checks passed
@jonathanchu
jonathanchu deleted the add-n-p-navigation branch February 19, 2026 04:41
@chopmo

chopmo commented Feb 19, 2026

Copy link
Copy Markdown

@jonathanchu Thanks for adding this, works beautifully.

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.

n/p support in PR lists

2 participants