Add popup button for selecting article themes on Catalina

This commit is contained in:
Maurice C Parker
2021-09-09 05:47:49 -05:00
parent 6c09bf585c
commit c031aa5cbe
2 changed files with 30 additions and 7 deletions

View File

@@ -18,7 +18,9 @@ enum TimelineSourceMode {
class MainWindowController : NSWindowController, NSUserInterfaceValidations {
private var activityManager = ActivityManager()
@IBOutlet weak var articleThemePopUpButton: NSPopUpButton?
private var activityManager = ActivityManager()
private var isShowingExtractedArticle = false
private var articleExtractor: ArticleExtractor? = nil
@@ -62,6 +64,8 @@ class MainWindowController : NSWindowController, NSUserInterfaceValidations {
sharingServicePickerDelegate = SharingServicePickerDelegate(self.window)
updateArticleThemeMenu()
if #available(macOS 11.0, *) {
let toolbar = NSToolbar(identifier: "MainWindowToolbar")
toolbar.allowsUserCustomization = true
@@ -155,11 +159,11 @@ class MainWindowController : NSWindowController, NSUserInterfaceValidations {
}
@objc func articleThemeNamesDidChangeNotification(_ note: Notification) {
buildArticleThemeMenu()
updateArticleThemeMenu()
}
@objc func currentArticleThemeDidChangeNotification(_ note: Notification) {
buildArticleThemeMenu()
updateArticleThemeMenu()
}
private func updateWindowTitleIfNecessary(_ noteObject: Any?) {
@@ -817,7 +821,6 @@ extension MainWindowController: NSToolbarDelegate {
let description = NSLocalizedString("Article Theme", comment: "Article Theme")
articleThemeMenuToolbarItem.toolTip = description
articleThemeMenuToolbarItem.label = description
buildArticleThemeMenu()
return articleThemeMenuToolbarItem
case .search:
@@ -1413,7 +1416,7 @@ private extension MainWindowController {
return menu
}
func buildArticleThemeMenu() {
func updateArticleThemeMenu() {
let articleThemeMenu = NSMenu()
let defaultThemeItem = NSMenuItem()
@@ -1433,6 +1436,7 @@ private extension MainWindowController {
}
articleThemeMenuToolbarItem.menu = articleThemeMenu
articleThemePopUpButton?.menu = articleThemeMenu
}
}