Skip to content
This repository was archived by the owner on Feb 2, 2023. It is now read-only.

Conversation

@nguyenhuy
Copy link
Contributor

@nguyenhuy nguyenhuy commented Feb 23, 2017

After #3017, it's much easier to support native reloads now. Since updates are already broken down to inserts and deletes by _ASHierarchyChangeSet, ASDataController can consume these inserts and deletes as is, and then ASCollectionView/ASTableView forward the original changes to UICollectionView/UITableView. This resolves #469.

Demo:

Copy link
Contributor

@Adlai-Holler Adlai-Holler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!


// TODO if the change set includes solely section reloads that together are equivalent to reloadData (i.e reload the only section),
// do a reloadData here as an optimization.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since reloadData is never animated, but section reloads may be, we probably won't want to build this optimization.

@Adlai-Holler Adlai-Holler merged commit 230bfb5 into facebookarchive:master Feb 23, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

reloadSections(_, withRowAnimation:_) does not reload the section header view

3 participants