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

Stop picker before looping #12528

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

janos-r
Copy link
Contributor

@janos-r janos-r commented Jan 14, 2025

PgDown / PgUp - Not being at the edge after looping is confusing. Especially on single-page lists the jumps seem random.

Also aligned the PgDown / Half-PgDown shortcuts with the ones used in the editor.

Overall more intuitive picker experience.

PgDown/PgUp - Not Being at the edge after looping is confusing.
Especially on single-page lists the jumps seam random.

Also alligned the PgDown / HalfPgDown shortcuts with the ones used in
the editor.
@janos-r
Copy link
Contributor Author

janos-r commented Jan 15, 2025

To illustrate, in this recording I first open the current master branch

  • I write .i to get a result with 2 pages, and I press page-down exactly 10x
  • I write an additional g to get a result with just 1 page, and press again page-down exactly 10x

Then I repeat with my proposed change.

I think that the current page-down / page-up is not very intuitive or useful in how it moves on loops, and this PR could be a very nice improvement.

simplescreenrecorder-2025-01-15_10.21.52.mp4

Co-authored-by: Mathew Horner <105067994+mathew-horner@users.noreply.github.com>
@kirawi kirawi added C-enhancement Category: Improvements A-helix-term Area: Helix term improvements labels Jan 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-helix-term Area: Helix term improvements C-enhancement Category: Improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants