From f9b45dd7b86dfec64678f9cf16b37d565a3c99e4 Mon Sep 17 00:00:00 2001
From: Omar Roth <omarroth@hotmail.com>
Date: Sun, 5 Aug 2018 19:59:45 -0500
Subject: [PATCH] General cleanup to user preferences

---
 src/invidious/views/data_control.ecr         |  2 +-
 src/invidious/views/preferences.ecr          | 12 ++++++------
 src/invidious/views/subscription_manager.ecr | 18 +++++++++++-------
 3 files changed, 18 insertions(+), 14 deletions(-)

diff --git a/src/invidious/views/data_control.ecr b/src/invidious/views/data_control.ecr
index 45d6752c..be896f86 100644
--- a/src/invidious/views/data_control.ecr
+++ b/src/invidious/views/data_control.ecr
@@ -13,7 +13,7 @@
             </div>
 
             <div class="pure-control-group">
-                <label for="import_youtube">Import <a target="_blank" style="color: #0366d6"
+                <label for="import_youtube">Import <a target="_blank"
                         href="https://support.google.com/youtube/answer/6224202?hl=en-GB">YouTube subscriptions</a></label>
                 <input type="file" id="import_youtube" name="import_youtube">
             </div>
diff --git a/src/invidious/views/preferences.ecr b/src/invidious/views/preferences.ecr
index 31efce89..57e40967 100644
--- a/src/invidious/views/preferences.ecr
+++ b/src/invidious/views/preferences.ecr
@@ -104,15 +104,15 @@ function update_value(element) {
 
             <legend>Data preferences</legend>
             <div class="pure-control-group">
-                <label>
-                    <a href="/clear_watch_history">Clear watch history</a>
-                </label>
+                <a href="/clear_watch_history">Clear watch history</a>
             </div>
             
             <div class="pure-control-group">
-                <label>
-                    <a href="/data_control">Import/Export data</a>
-                </label>
+                <a href="/data_control">Import/Export data</a>
+            </div>
+
+            <div class="pure-control-group">
+                <a href="/subscription_manager">Manage subscriptions></a>
             </div>
 
             <div class="pure-controls">
diff --git a/src/invidious/views/subscription_manager.ecr b/src/invidious/views/subscription_manager.ecr
index b510c327..59cee9d7 100644
--- a/src/invidious/views/subscription_manager.ecr
+++ b/src/invidious/views/subscription_manager.ecr
@@ -14,19 +14,23 @@
 </div>
 
 <% subscriptions.each do |channel| %>
-<h3 class="h-box">
+<div class="h-box">
     <div class="pure-g">
         <div class="pure-u-2-5">
-            <a href="/channel/<%= channel.id %>"><%= channel.author %></a>
+            <h3>
+                <a href="/channel/<%= channel.id %>"><%= channel.author %></a>
+            </h3>
         </div>
         <div class="pure-u-2-5"></div>
         <div class="pure-u-1-5" style="text-align: right;">
-            <a href="/subscription_ajax?action_remove_subscriptions=1&c=<%= channel.id %>">unsubscribe</a>
+            <h3>
+                <a href="/subscription_ajax?action_remove_subscriptions=1&c=<%= channel.id %>">unsubscribe</a>
+            </h3>
         </div>
     </div>
-</h3>
 
-<% if subscriptions[-1].author != channel.author %>
-<hr>
-<% end %>
+    <% if subscriptions[-1].author != channel.author %>
+    <hr>
+    <% end %>
+</div>
 <% end %>
\ No newline at end of file