Move favicons in the detail view to the right. They’re show when available and there’s no other feed icon or user avatar.

This commit is contained in:
Brent Simmons 2018-01-03 13:14:14 -08:00
parent 4bb5270860
commit e2c36caeee
2 changed files with 7 additions and 5 deletions

View File

@ -173,7 +173,8 @@ class ArticleRenderer {
d["avatars"] = ""
var didAddAvatar = false
if let avatarHTML = avatarImgTag() {
d["avatars"] = avatarHTML
// d["avatars"] = avatarHTML
d["avatars"] = "<td class=\"header rightAlign avatar\">\(avatarHTML)</td>";
didAddAvatar = true
}
@ -187,10 +188,11 @@ class ArticleRenderer {
d["feedlink"] = feedLink
d["feedlink_withfavicon"] = feedLink
d["favicon"] = ""
// d["favicon"] = ""
if !didAddAvatar, let feed = article.feed {
if let favicon = faviconImgTag(forFeed: feed) {
d["favicon"] = favicon
d["avatars"] = "<td class=\"header rightAlign\">\(favicon)</td>";
// d["favicon"] = favicon
}
}

View File

@ -3,8 +3,8 @@
<div class="headerContainer">
<table cellpadding=0 cellspacing=0 border=0 class="headerTable">
<tr>
<td class="header leftAlign">[[favicon]] <b>[[feedlink]]</b><br />[[byline]]</td>
<td class="header rightAlign avatar">[[avatars]]</td>
<td class="header leftAlign"><b>[[feedlink]]</b><br />[[byline]]</td>
[[avatars]]
</tr>
</table>
</div>