diff --git a/Mac/AppAssets.swift b/Mac/AppAssets.swift
index 4ce60d6bc..731880985 100644
--- a/Mac/AppAssets.swift
+++ b/Mac/AppAssets.swift
@@ -237,14 +237,12 @@ struct AppAssets {
}
}()
- static var timelineStar: RSImage! = {
- if #available(macOS 11.0, *) {
- let image = NSImage(systemSymbolName: "star.fill", accessibilityDescription: nil)!
- let coloredImage = image.tinted(with: NSColor(named: "StarColor")!)
- return coloredImage
- } else {
- return RSImage(named: "timelineStar")
- }
+ static var timelineStarSelected: RSImage! = {
+ return RSImage(named: "timelineStar")?.tinted(with: .white)
+ }()
+
+ static var timelineStarUnselected: RSImage! = {
+ return RSImage(named: "timelineStar")?.tinted(with: starColor)
}()
static var todayFeedImage: IconImage = {
@@ -285,8 +283,8 @@ struct AppAssets {
return RSImage(named: "swipeMarkUnstarred")!
}()
- static var swipeMarkUnstarredColor: NSColor = {
- return NSColor(named: NSColor.Name("swipeMarkUnstarredColor"))!
+ static var starColor: NSColor = {
+ return NSColor(named: NSColor.Name("StarColor"))!
}()
static func image(for accountType: AccountType) -> NSImage? {
diff --git a/Mac/Base.lproj/MainWindow.storyboard b/Mac/Base.lproj/MainWindow.storyboard
index d024a5be1..de9890edf 100644
--- a/Mac/Base.lproj/MainWindow.storyboard
+++ b/Mac/Base.lproj/MainWindow.storyboard
@@ -16,7 +16,7 @@
-
+
@@ -298,39 +298,18 @@
-
-
+
+
-
-
-
-
-
-
-
-
-
-
-
-
+
-
+
-
-
+
+
@@ -408,7 +387,7 @@
-
+
@@ -437,22 +416,19 @@
-
-
-
+
+
-
-
+
-
@@ -495,24 +471,25 @@
-
+
-
+
-[[date_medium]]
+[[date_medium]]
[[body]]
diff --git a/Shared/Extensions/ArticleUtilities.swift b/Shared/Extensions/ArticleUtilities.swift
index c5d2421b1..cfc4e34fb 100644
--- a/Shared/Extensions/ArticleUtilities.swift
+++ b/Shared/Extensions/ArticleUtilities.swift
@@ -134,17 +134,22 @@ extension Article {
byline += ", "
}
isFirstAuthor = false
+
+ var authorEmailAddress: String? = nil
+ if let emailAddress = author.emailAddress, !(emailAddress.contains("noreply@") || emailAddress.contains("no-reply@")) {
+ authorEmailAddress = emailAddress
+ }
- if let emailAddress = author.emailAddress, emailAddress.contains(" ") {
+ if let emailAddress = authorEmailAddress, emailAddress.contains(" ") {
byline += emailAddress // probably name plus email address
}
- else if let name = author.name, let emailAddress = author.emailAddress {
+ else if let name = author.name, let emailAddress = authorEmailAddress {
byline += "\(name) <\(emailAddress)>"
}
else if let name = author.name {
byline += name
}
- else if let emailAddress = author.emailAddress {
+ else if let emailAddress = authorEmailAddress {
byline += "<\(emailAddress)>"
}
else if let url = author.url {