make icons look gooder

Bustead · Sep 3, 2023 · 1c3ce80 · 1c3ce80
1 parent 4a4d68f
commit 1c3ce80
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/app/views/links/index.html.erb b/app/views/links/index.html.erb
@@ -23,7 +23,14 @@
                   <td><%= link.links_group %></td>
                   <td><%= link.links_url %></td>
                   <td><%= link.links_display_name %></td>
-                  <td class="text-center"><i class="<%= link.links_icon %>"></i></td>
+                  <td class="text-center">
+                    <img 
+                      src="https://www.google.com/s2/favicons?domain=<%= URI.parse(link.links_url).host.split('.').last(2).join('.') %>" 
+                      alt="favicon" 
+                      class="fa-2x text-light"
+                      onerror="this.alt=''; this.style.display='none'; this.nextElementSibling.style.display='inline-block';">
+                    <i class="<%= link.links_icon %> fa-2x text-light" style="display:none;"></i>
+                </td>
                   <td class="<%= link.links_enabled ? 'text-success' : 'text-danger' %>">
                     <%= link.links_enabled ? 'Enabled' : 'Disabled' %>
                   </td>

diff --git a/app/views/links/show.html.erb b/app/views/links/show.html.erb
@@ -16,7 +16,12 @@
     <div class="list-group-item d-flex justify-content-between align-items-center">
       <span class="fw-bold">Icon:</span>
       <span class="d-flex align-items-center">
-        <i class="<%= @link.links_icon %>"></i>
+        <img 
+          src="https://www.google.com/s2/favicons?domain=<%= URI.parse(link.links_url).host.split('.').last(2).join('.') %>" 
+          alt="favicon" 
+          class="fa-2x text-light"
+          onerror="this.alt=''; this.style.display='none'; this.nextElementSibling.style.display='inline-block';">
+        <i class="<%= link.links_icon %> fa-2x text-light" style="display:none;"></i>
       </span>
     </div>
     <div class="list-group-item d-flex justify-content-between align-items-center">