Skip to content

Fix positionFunction created by ScrollDelegate#3000

Closed
quaff wants to merge 1 commit intospring-projects:mainfrom
quaff:patch-14
Closed

Fix positionFunction created by ScrollDelegate#3000
quaff wants to merge 1 commit intospring-projects:mainfrom
quaff:patch-14

Conversation

@quaff
Copy link
Copy Markdown
Contributor

@quaff quaff commented Jun 6, 2023

Closes GH-2999

Before this commit, positionFunction created by ScrollDelegate retains incorrect state when perform backward scrolling.
1. We must use window.positionAt(1) instead of window.positionAt(0) if we want position at the begin of window, we should call getResultWindow() first.
2. The direction always be FORWARD, it should be BACKWARD if the original direction is BACKWARD.

Closes spring-projectsGH-2999
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jun 6, 2023
@mp911de mp911de self-assigned this Jun 12, 2023
@quaff
Copy link
Copy Markdown
Contributor Author

quaff commented Jun 13, 2023

It covered by #3013

@quaff quaff closed this Jun 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: waiting-for-triage An issue we've not yet triaged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Keyset positionFunction does not retain directional state and uses position of one-off

3 participants