feat: auto scroll when the focused item is out of viewBound#2
Open
thedroiddiv wants to merge 1 commit into
Open
feat: auto scroll when the focused item is out of viewBound#2thedroiddiv wants to merge 1 commit into
thedroiddiv wants to merge 1 commit into
Conversation
Owner
Author
|
I tested the behaviour by adding Log is getting printed even though mouse is not move physically but the list under mouse scrolls. |
thedroiddiv
force-pushed
the
auto-scroll-on-keyboard-input
branch
from
97692b6 to
1f21f1f
Compare
auto scroll when keyboard navigation moves focus to an item that is out of view
thedroiddiv
force-pushed
the
auto-scroll-on-keyboard-input
branch
2 times, most recently
from
c160a55 to
4def629
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR introduces auto scroll feature when being navigated using keyboard and the focused item is out of viewBound.
2025-08-15.16-49-07.mov
Existing issues:
When the list is scrolled automatically, mouse hover event is triggered which in-turn calls onItemHover forcing the focus to move item under pointer.
This might be actually be an issue in the compose API itself, ideally PointerEvent.Move should be triggered when pointer is moved physically through a pointing device rather than on movement of the content below mouse.