-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Scroll on pagination change in list records #14681
Scroll on pagination change in list records #14681
Conversation
Does this scroll the table all the way to the top? Or just make sure that the pagination UI is still visible Please can you refactor this to not use |
it scrolls up to the table. i changed it to an event instead of using js |
I just tested this, here is what it did: Screen.Recording.2024-12-05.at.08.49.30.movI do not want it to scroll the table up to the top like this, as when you click on a new item I still want to be able to click the next item without scrolling again. I think the only use case I want it to scroll for is when you are on a page with many records and then you switch to a page with very few records, to scroll the user up to the pagination controls, but not the top. If you want to attempt that use case, I would accept a new PR, but closing for now as that is not currently the behaviour of this PR and I'm not sure if it's achievable. |
By default the scroll position is not affected when changing the page. I've added a method to scroll the wire component into the view.