From 128f8dbf2ef8e69b19d74335ac1e25441b9d69fc Mon Sep 17 00:00:00 2001
From: Uku Taht <uku.taht@gmail.com>
Date: Wed, 18 Aug 2021 15:21:47 +0300
Subject: [PATCH] Fix favicons and spelling in top sources

---
 assets/js/dashboard/stats/modals/sources.js      | 7 +++++--
 assets/js/dashboard/stats/sources/source-list.js | 7 +++----
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/assets/js/dashboard/stats/modals/sources.js b/assets/js/dashboard/stats/modals/sources.js
index 2d591ffb0c..c5d7ea6cf0 100644
--- a/assets/js/dashboard/stats/modals/sources.js
+++ b/assets/js/dashboard/stats/modals/sources.js
@@ -8,7 +8,7 @@ import numberFormatter, {durationFormatter} from '../../number-formatter'
 import {parseQuery} from '../../query'
 
 const TITLES = {
-  sources: 'Top sources',
+  sources: 'Top Sources',
   utm_mediums: 'Top UTM mediums',
   utm_sources: 'Top UTM sources',
   utm_campaigns: 'Top UTM campaigns'
@@ -85,7 +85,10 @@ class SourcesModal extends React.Component {
     return (
       <tr className="text-sm dark:text-gray-200" key={source.name}>
         <td className="p-2">
-          <img src={`https://icons.duckduckgo.com/ip3/${source.url}.ico`} referrerPolicy="no-referrer" className="h-4 w-4 mr-2 align-middle inline" />
+          <img
+            src={`/favicon/sources/${encodeURIComponent(source.name)}`}
+            className="h-4 w-4 mr-2 align-middle inline"
+          />
           <Link className="hover:underline" to={{search: query.toString(), pathname: '/' + encodeURIComponent(this.props.site.domain)}}>{ source.name }</Link>
         </td>
         <td className="p-2 w-32 font-medium" align="right">{numberFormatter(source.count)}</td>
diff --git a/assets/js/dashboard/stats/sources/source-list.js b/assets/js/dashboard/stats/sources/source-list.js
index 93824d0515..37ba0c8fbc 100644
--- a/assets/js/dashboard/stats/sources/source-list.js
+++ b/assets/js/dashboard/stats/sources/source-list.js
@@ -59,8 +59,7 @@ class AllSources extends React.Component {
               to={{search: query.toString()}}
             >
               <img
-                src={`/favicon/sources/${referrer.name}`}
-                referrerPolicy="no-referrer"
+                src={`/favicon/sources/${encodeURIComponent(referrer.name)}`}
                 className="inline w-4 h-4 mr-2 -mt-px align-middle"
               />
               { referrer.name }
@@ -173,7 +172,7 @@ class UTMSources extends React.Component {
         >
 
           <span className="flex px-2 py-1.5 dark:text-gray-300 relative z-9 break-all">
-            <Link 
+            <Link
               className="md:truncate block hover:underline"
               to={{search: query.toString()}}
             >
@@ -214,7 +213,7 @@ class UTMSources extends React.Component {
     return (
       <React.Fragment>
         <div className="flex justify-between w-full">
-          <h3 className="font-bold dark:text-gray-100">Top sources</h3>
+          <h3 className="font-bold dark:text-gray-100">Top Sources</h3>
           { this.props.renderTabs() }
         </div>
         { this.state.loading && <div className="mx-auto loading mt-44"><div></div></div> }