Fix #5357: Add brave-favicon for NTP (#7859)

- Add brave-favicon for NTP
brave · Aug 10, 2023 · d4acff4 · d4acff4
1 parent ff67224
commit d4acff4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/Sources/Brave/Frontend/Browser/Tab.swift b/Sources/Brave/Frontend/Browser/Tab.swift
@@ -538,7 +538,11 @@ class Tab: NSObject {
   }
 
   var displayFavicon: Favicon? {
-    if let url = url, InternalURL(url)?.isAboutHomeURL == true { return nil }
+    if let url = url, InternalURL(url)?.isAboutHomeURL == true {
+      return Favicon(image: UIImage(sharedNamed: "brave.logo"),
+                     isMonogramImage: false,
+                     backgroundColor: .clear)
+    }
     return favicon
   }
 

diff --git a/Sources/Brave/Frontend/Browser/Tabs/TabTray/Views/TabTrayCell.swift b/Sources/Brave/Frontend/Browser/Tabs/TabTray/Views/TabTrayCell.swift
@@ -56,7 +56,7 @@ class TabCell: UICollectionViewCell {
     }
 
     titleLabel.text = tab.displayTitle
-    favicon.image = Favicon.defaultImage
+    favicon.image = tab.displayFavicon?.image ?? Favicon.defaultImage
 
     if !tab.displayTitle.isEmpty {
       accessibilityLabel = tab.displayTitle