From a3778e04a257e9780943cf344670227a1ca57506 Mon Sep 17 00:00:00 2001
From: Brent Simmons <brent@ranchero.com>
Date: Sun, 25 Feb 2018 22:46:25 -0800
Subject: [PATCH] Force favicons to appear at 16x16 in the detail view. Allow
 favicons to appear larger, when they are in fact larger.

---
 Evergreen/MainWindow/Detail/ArticleRenderer.swift | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/Evergreen/MainWindow/Detail/ArticleRenderer.swift b/Evergreen/MainWindow/Detail/ArticleRenderer.swift
index ca73dde0d..ca82038c0 100644
--- a/Evergreen/MainWindow/Detail/ArticleRenderer.swift
+++ b/Evergreen/MainWindow/Detail/ArticleRenderer.swift
@@ -222,7 +222,7 @@ class ArticleRenderer {
 
 		func html(dimension: Int) -> String {
 
-			let imageTag = "<img src=\"\(imageURL)\" width=\"\(dimension)\" height=\"\(dimension)\""
+			let imageTag = "<img src=\"\(imageURL)\" width=\(dimension) height=\(dimension) />"
 			if let url = url {
 				return linkWithText(imageTag, url)
 			}
@@ -238,7 +238,9 @@ class ArticleRenderer {
 
 		if let favicon = appDelegate.faviconDownloader.favicon(for: feed) {
 			if let s = base64String(forImage: favicon) {
-				let imgTag = "<img src=\"data:image/tiff;base64, " + s + "\" height=16 width=16 />"
+				var dimension = min(favicon.size.height, 48) // Assuming square images.
+				dimension = max(dimension, 16) // Some favicons say they’re < 16. Force them larger.
+				let imgTag = "<img src=\"data:image/tiff;base64, " + s + "\" height=\(dimension) width=\(dimension) />"
 				ArticleRenderer.faviconImgTagCache[feed] = imgTag
 				return imgTag
 			}