Fahim
/
Tusker
forked from shadowfacts/Tusker
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't reconfigure conversation main status unnecessarily

This commit is contained in:
Shadowfacts 2023-05-28 12:16:48 -07:00
parent 5b321fcc78
commit 708112c486
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
Tusker/Screens/Conversation/ConversationCollectionViewController.swift
View File

@ -147,7 +147,8 @@ class ConversationCollectionViewController: UIViewController, CollectionViewCont
Item.status(id: node.status.id, node: node, state: .unknown, prevLink: index > 0, nextLink: true) Item.status(id: node.status.id, node: node, state: .unknown, prevLink: index > 0, nextLink: true)
} }
snapshot.appendItems(parentItems, toSection: .ancestors) snapshot.appendItems(parentItems, toSection: .ancestors)
snapshot.reconfigureItems([mainStatusItem]) // don't need to reconfigure main item, since when the refreshed copy was loaded
// it would have triggered a reconfigure via the status observer
// convert sub-threads into items for section and add to snapshot // convert sub-threads into items for section and add to snapshot
self.addFlattenedChildThreadsToSnapshot(tree.descendants, mainStatus: mainStatus, snapshot: &snapshot) self.addFlattenedChildThreadsToSnapshot(tree.descendants, mainStatus: mainStatus, snapshot: &snapshot)