From 4bf3c2fd286ee4a92462a418ccfc33172bc74aa3 Mon Sep 17 00:00:00 2001
From: Nolan Lawson <nolan@nolanlawson.com>
Date: Mon, 11 Feb 2019 21:04:00 -0800
Subject: [PATCH] fix: don't show theme picker if not logged in (#967)

Also tweak some of the styles and layout in the theme picker itself
---
 .../settings/instance/ThemeSettings.html         | 16 +++++++++++-----
 src/routes/_pages/settings/general.html          |  6 ++++--
 2 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/src/routes/_components/settings/instance/ThemeSettings.html b/src/routes/_components/settings/instance/ThemeSettings.html
index 015f38b..310b8cb 100644
--- a/src/routes/_components/settings/instance/ThemeSettings.html
+++ b/src/routes/_components/settings/instance/ThemeSettings.html
@@ -2,11 +2,9 @@
   <div class="theme-groups">
     {#each themeGroups as themeGroup}
     <div class="theme-group">
-      {#if themeGroup.dark}
-        <h2>Dark Background</h2>
-      {:else}
-        <h2>Light Background</h2>
-      {/if}
+      <h3>
+        {themeGroup.dark ? 'Dark background' : 'Light background' }
+      </h3>
       {#each themeGroup.themes as theme}
       <div class="theme-picker">
         <input type="radio" id="choice-theme-{theme.name}"
@@ -65,10 +63,18 @@
     border: 2px solid #dadada;
   }
 
+  h3 {
+    font-size: 1.4em;
+  }
+
   @media (max-width: 479px) {
     .theme-groups {
       grid-template-columns: 1fr;
     }
+
+    h3 {
+      margin-top: 0.5em;
+    }
   }
 </style>
 <script>
diff --git a/src/routes/_pages/settings/general.html b/src/routes/_pages/settings/general.html
index 71c7ebf..23a3f1e 100644
--- a/src/routes/_pages/settings/general.html
+++ b/src/routes/_pages/settings/general.html
@@ -63,8 +63,10 @@
     </div>
   </form>
 
-  <h2>{themeTitle}</h2>
-  <ThemeSettings instanceName={$currentInstance} />
+  {#if $isUserLoggedIn }
+    <h2>{themeTitle}</h2>
+    <ThemeSettings instanceName={$currentInstance} />
+  {/if}
 </SettingsLayout>
 <style>
   .ui-settings {