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

Allow to cycle through center/top/bot scroll positions #16134

Merged
merged 4 commits into from
Aug 12, 2024

Conversation

SomeoneToIgnore
Copy link
Contributor

@SomeoneToIgnore SomeoneToIgnore commented Aug 12, 2024

On top of editor::ScrollCursorCenter, editor::ScrollCursorTop, editor::ScrollCursorBottom actions, adds an editor::ScrollCursorCenterTopBottom one, that allows using a single keybinding to scroll between positions on the screen.

The implementation matches a corresponding Emacs feature: there's a timeout (1s) that is kept after every switch, to allow continuously changing the positions, center (initial) -> top -> bottom
Scrolling behavior is the same as the existing actions (e.g. editor will ignore scroll to bottom, if there's not enough space above).

After 1s, next position is reset to the initial, center, one.

Release Notes:

  • Added an editor::ScrollCursorCenterTopBottom action for toggling scroll position with a single keybinding

SomeoneToIgnore and others added 3 commits August 13, 2024 00:13
co-authored-by: Alex Kladov <aleksey.kladov@gmail.com>
co-authored-by: Alex Kladov <aleksey.kladov@gmail.com>
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Aug 12, 2024
@SomeoneToIgnore SomeoneToIgnore merged commit 3bebb8b into main Aug 12, 2024
9 checks passed
@SomeoneToIgnore SomeoneToIgnore deleted the kb/center-top-bot branch August 12, 2024 21:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant