FreshRSS
/
NetNewsWire
mirror of https://github.com/Ranchero-Software/NetNewsWire.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Revert to reloading individual rows with a short animation to remove the flicker caused by unread counts changing too fast

This commit is contained in:
Maurice Parker 2021-11-13 10:33:07 -06:00
parent 9d613fe670
commit e2f3d1867d
1 changed files with 13 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
iOS/MasterFeed/MasterFeedViewController.swift
View File

@ -44,8 +44,6 @@ class MasterFeedViewController: UITableViewController, UndoableCommandRunner {
override var canBecomeFirstResponder: Bool {
return true
}
private var reloadCoalescingQueue = CoalescingQueue(name: "Reload Visible", interval: 0.5)
override func viewDidLoad() {
@ -115,7 +113,18 @@ class MasterFeedViewController: UITableViewController, UndoableCommandRunner {
return
}
queueReloadAllVisible()
var node: Node? = nil
if let coordinator = representedObject as? SceneCoordinator, let feed = coordinator.timelineFeed {
node = coordinator.rootNode.descendantNodeRepresentingObject(feed as AnyObject)
} else {
node = coordinator.rootNode.descendantNodeRepresentingObject(representedObject as AnyObject)
}
guard let unreadCountNode = node, let indexPath = coordinator.indexPathFor(unreadCountNode) else { return }
UIView.animate(withDuration: 0.1) {
self.tableView.reloadRows(at: [indexPath], with: .none)
}
restoreSelectionIfNecessary(adjustScroll: false)
}
@objc func faviconDidBecomeAvailable(_ note: Notification) {
@ -687,7 +696,7 @@ class MasterFeedViewController: UITableViewController, UndoableCommandRunner {
present(vc, animated: true)
}
}
}
// MARK: UIContextMenuInteractionDelegate
@ -834,14 +843,6 @@ private extension MasterFeedViewController {
return ""
}
func queueReloadAllVisible() {
reloadCoalescingQueue.add(self, #selector(reloadQueuedAllVisible))
}
@objc func reloadQueuedAllVisible() {
reloadAllVisibleCells()
}
func configureCellsForRepresentedObject(_ representedObject: AnyObject) {
applyToCellsForRepresentedObject(representedObject, configure)
}