From 708112c486c749a933846d8a6c29f952fa2e0e48 Mon Sep 17 00:00:00 2001 From: Shadowfacts Date: Sun, 28 May 2023 12:16:48 -0700 Subject: [PATCH] Don't reconfigure conversation main status unnecessarily --- .../Conversation/ConversationCollectionViewController.swift | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Tusker/Screens/Conversation/ConversationCollectionViewController.swift b/Tusker/Screens/Conversation/ConversationCollectionViewController.swift index 18df951b..5abdbece 100644 --- a/Tusker/Screens/Conversation/ConversationCollectionViewController.swift +++ b/Tusker/Screens/Conversation/ConversationCollectionViewController.swift @@ -147,7 +147,8 @@ class ConversationCollectionViewController: UIViewController, CollectionViewCont Item.status(id: node.status.id, node: node, state: .unknown, prevLink: index > 0, nextLink: true) } 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 self.addFlattenedChildThreadsToSnapshot(tree.descendants, mainStatus: mainStatus, snapshot: &snapshot)