From e34f375cf50f4f8677217013c4a7d6fd94a4419f Mon Sep 17 00:00:00 2001
From: syeopite <syeopite@syeopite.dev>
Date: Sun, 28 Mar 2021 20:13:16 -0700
Subject: [PATCH] Update styling for buttons and dark-theme

---
 assets/css/default.css           | 23 ++++++++++++++++++-----
 src/invidious/views/trending.ecr |  6 +++---
 2 files changed, 21 insertions(+), 8 deletions(-)

diff --git a/assets/css/default.css b/assets/css/default.css
index 1d62bc01..a8e201c4 100644
--- a/assets/css/default.css
+++ b/assets/css/default.css
@@ -197,7 +197,7 @@ img.thumbnail {
   border-top: 0;
   border-left: 0;
   border-right: 0;
-  border-bottom: 1px solid #ccc;
+  border-bottom: 1px solid #3E4446;
   border-radius: 0;
 
   padding: initial 0;
@@ -223,7 +223,7 @@ input[type="search"]::-webkit-search-cancel-button {
 /* attract focus to the searchbar by adding a subtle transition */
 .searchbar .pure-form input[type="search"]:focus {
   margin-bottom: 0px;
-  border-bottom: 2px solid #aaa;
+  border-bottom: 2px solid #3E4446;
 }
 
 .user-field {
@@ -387,13 +387,14 @@ span > select {
   color: rgb(0, 182, 240);
 }
 
-.dark-theme a {
-  color: #a0a0a0;
+.dark-theme a,
+.dark-theme .footer {
+  color: #A8A095;
   text-decoration: none;
 }
 
 body.dark-theme {
-  background-color: rgba(35, 35, 35, 1);
+  background-color: rgba(26, 29, 30, 1);
   color: #f0f0f0;
 }
 
@@ -420,6 +421,18 @@ body.dark-theme {
   color: inherit;
 }
 
+.dark-theme .pure-menu-link:hover, .dark-theme .pure-menu-link:active {
+  background-color: rgb(34, 36, 38)
+}
+
+.dark-theme .pure-menu-selected > .pure-menu-link {
+  color: #EAE7E1;
+}
+
+.dark-theme .pure-menu-link:visited {
+  background-color: transparent;
+}
+
 @media (prefers-color-scheme: dark) {
   .no-theme a:hover,
   .no-theme a:active {
diff --git a/src/invidious/views/trending.ecr b/src/invidious/views/trending.ecr
index 172ecc65..6d5ff87a 100644
--- a/src/invidious/views/trending.ecr
+++ b/src/invidious/views/trending.ecr
@@ -24,9 +24,9 @@
             <% {"Default", "Music", "Gaming", "Movies"}.each do |option| %>
                 <% if trending_type == option %>
                     <li class="pure-menu-item pure-menu-selected">
-                        <b><a href="/feed/trending?type=<%= option %>&region=<%= region %>" class="pure-menu-link">
-                          <%= translate(locale, option) %>
-                        </a></b>
+                        <a href="/feed/trending?type=<%= option %>&region=<%= region %>" class="pure-menu-link">
+                          <b><%= translate(locale, option) %></b>
+                        </a>
                 <% else %>
                     <li class="pure-menu-item">
                         <a href="/feed/trending?type=<%= option %>&region=<%= region %>" class="pure-menu-link">