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

Saving posts causes lists to refresh and lose scroll position #558

Closed
surajsharma opened this issue May 13, 2024 · 4 comments
Closed

Saving posts causes lists to refresh and lose scroll position #558

surajsharma opened this issue May 13, 2024 · 4 comments

Comments

@surajsharma
Copy link

Adding an article to saved makes the list scroll in a wierd way

@msfjarvis msfjarvis changed the title bookmarking scroll issue Saving posts causes lists to refresh and lose scroll position May 13, 2024
@msfjarvis
Copy link
Owner

I'm aware of the issue, it's a workaround for the AndroidX Paging library not having support for updating its cached item list without a full network refresh. I've started working on a solution but it'll take some time to get it right.

@msfjarvis
Copy link
Owner

msfjarvis commented Jun 12, 2024

Dropping a quick update on this, I was unable to get AndroidX Paging to do what I was looking for so I've started migrating the app's data layer to Store which natively supports offline-first operations. However, their paging API is currently unstable and due for a massive overhaul very soon so progress here is stalled for a bit until they release the new API.

@surajsharma
Copy link
Author

@msfjarvis thanks for the update. i could port this whole app to React Native and get a much smoother UX but I can't be bothered so I won't. Please take your time here no rush.

@msfjarvis
Copy link
Owner

That's a pretty unnecessary detail to throw into a thank you message but sure...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants