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

Change to only resort when sort parameters change

This commit is contained in:
Maurice Parker 2020-07-13 16:06:42 -05:00
parent 1a74701052
commit 4bb88363e7
1 changed files with 21 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
Multiplatform/Shared/Timeline/TimelineModel.swift
View File

@ -55,12 +55,19 @@ class TimelineModel: ObservableObject, UndoableCommandRunner {
return _idToArticleDictionary return _idToArticleDictionary
} }
private var sortDirection: Bool { private var sortDirection = AppDefaults.shared.timelineSortDirection {
AppDefaults.shared.timelineSortDirection didSet {
if sortDirection != oldValue {
sortParametersDidChange()
}
}
} }
private var groupByFeed = AppDefaults.shared.timelineGroupByFeed {
private var groupByFeed: Bool { didSet {
AppDefaults.shared.timelineGroupByFeed if groupByFeed != oldValue {
sortParametersDidChange()
}
}
} }
init() { init() {
@ -207,10 +214,8 @@ private extension TimelineModel {
} }
@objc func userDefaultsDidChange(_ note: Notification) { @objc func userDefaultsDidChange(_ note: Notification) {
performBlockAndRestoreSelection { sortDirection = AppDefaults.shared.timelineSortDirection
articles = articles.sortedByDate(sortDirection ? .orderedDescending : .orderedAscending, groupByFeed: groupByFeed) groupByFeed = AppDefaults.shared.timelineGroupByFeed
rebuildTimelineItems()
}
} }
// MARK: Timeline Management // MARK: Timeline Management
@ -233,6 +238,13 @@ private extension TimelineModel {
} }
} }
func sortParametersDidChange() {
performBlockAndRestoreSelection {
articles = articles.sortedByDate(sortDirection ? .orderedDescending : .orderedAscending, groupByFeed: groupByFeed)
rebuildTimelineItems()
}
}
func performBlockAndRestoreSelection(_ block: (() -> Void)) { func performBlockAndRestoreSelection(_ block: (() -> Void)) {
// let savedSelection = selectedArticleIDs() // let savedSelection = selectedArticleIDs()
block() block()