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

Ensure adapter is created along with recyclerView #901

Open
wants to merge 1 commit into
base: edge
Choose a base branch
from

Conversation

vrih
Copy link
Contributor

@vrih vrih commented Oct 20, 2018

When RecyclerView is first opened it doesn't have an adapter attached. This causes an error to be logged: E/RecyclerView: No adapter attached; skipping layout

The resolution seems to be to move most of the logic from finishLoading() to just after view creation. I'm not quite familiar enough with the structure of the async data loads to fully DRY this up so I would appreciate a pointer or 2.

I'm chasing down a bug that causes RecyclerView to not be properly cleaned up visually if the user exits the view before it has finished rendering and this is a candidate for causing it.

@vrih vrih force-pushed the fix/grid-layout-adapter branch 3 times, most recently from c48c632 to 904fc2d Compare October 27, 2018 17:50
@vrih vrih force-pushed the fix/grid-layout-adapter branch from 904fc2d to 610bf4b Compare November 22, 2018 09:49
Fixes an error raised by RecyclerView: github.daneren2005.dsub E/RecyclerView: No adapter attached; skipping layout
@vrih vrih force-pushed the fix/grid-layout-adapter branch from 610bf4b to 888e459 Compare December 9, 2018 11:53
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

Successfully merging this pull request may close these issues.

1 participant