-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
"visible" section controllers #643
Comments
Ah ya, we're only using
Maybe instead we should query Should be easy to unit test too. |
@rnystrom Want to card that up separately? Seems like a nice starter task! |
@rnystrom I took a stab at implementing your fix idea + added a unit test! |
- (NSArray<IGListSectionController<IGListSectionType> *> *)visibleSectionControllers;
in IGListAdapter is not all visible section controllers sometimes.For example, in a list, there is a section controller, with zero cell, but there's a header, footer or other supplementary view. So the section controller is still "visible".
The text was updated successfully, but these errors were encountered: