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

Support for DiffUtil / Granular Update Notifications #156

Open
jmar777 opened this issue Dec 11, 2018 · 0 comments
Open

Support for DiffUtil / Granular Update Notifications #156

jmar777 opened this issue Dec 11, 2018 · 0 comments

Comments

@jmar777
Copy link

jmar777 commented Dec 11, 2018

Thanks for making this library available to the public - we've been using it for awhile now, and love it!

I went to refactor our backing RecyclerView.Adapter implementation today, with the aim of using DiffUtil to prevent some unnecessary recycles from happening.

Unfortunately, this introduced some rather unexpected behavior. While this view is loading, the cards switch view types to represent loading/loaded states, and additional cards can be introduced at various locations within the dataset. The experience we observed was that the currently viewed position seemed to jump around during this loading process, and we'd typically end up viewing the last card by the time everything settled.

Seeing as we don't have this problem when doing a straight notifyDataSetChanged(), I'm assuming this is the result of some of the more granular notification methods not being supported? Is this the case, and if so, are there any plans to support them?

Thanks again!

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

No branches or pull requests

2 participants