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

Don’t make publication date a link when it would be to same link as title. Fix #412.

This commit is contained in:
Brent Simmons 2018-09-03 12:50:24 -07:00
parent 8373ef06eb
commit 8d37275edb
1 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
NetNewsWire/MainWindow/Detail/ArticleRenderer.swift
View File

@ -198,7 +198,7 @@ class ArticleRenderer {
let longDate = longDateFormatter.string(from: article.logicalDatePublished)
let mediumDate = mediumDateFormatter.string(from: article.logicalDatePublished)
let shortDate = shortDateFormatter.string(from: article.logicalDatePublished)
if let permalink = article.url {
if dateShouldBeLink() || self.title == "", let permalink = article.url {
d["date_long"] = linkWithText(longDate, permalink)
d["date_medium"] = linkWithText(mediumDate, permalink)
d["date_short"] = linkWithText(shortDate, permalink)
@ -215,6 +215,16 @@ class ArticleRenderer {
return d
}
private func dateShouldBeLink() -> Bool {
guard let permalink = article.url else {
return false
}
guard let preferredLink = article.preferredLink else { // Title uses preferredLink
return false
}
return permalink != preferredLink // Make date a link if it’s a different link from the title’s link
}
struct Avatar {
let imageURL: String
let url: String?