From cc015536faff14908017da89cdcb98e47b4c74e6 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Sun, 27 Feb 2022 10:12:47 -0800
Subject: [PATCH] Adds theme instructions

---
 bookwyrm/forms.py                       |  4 +--
 bookwyrm/templates/settings/themes.html | 41 +++++++++++++------------
 2 files changed, 22 insertions(+), 23 deletions(-)

diff --git a/bookwyrm/forms.py b/bookwyrm/forms.py
index 926aaecf..4e6bab16 100644
--- a/bookwyrm/forms.py
+++ b/bookwyrm/forms.py
@@ -466,9 +466,7 @@ class ThemeForm(CustomForm):
         fields = ["name", "path"]
         widgets = {
             "name": forms.TextInput(attrs={"aria-describedby": "desc_name"}),
-            "path": ClearableFileInputWithWarning(
-                attrs={"aria-describedby": "desc_path"}
-            ),
+            "path": forms.TextInput(attrs={"aria-describedby": "desc_path", "placeholder": _("example-theme.scss")}),
         }
 
 
diff --git a/bookwyrm/templates/settings/themes.html b/bookwyrm/templates/settings/themes.html
index 6e9eb61c..3eac261d 100644
--- a/bookwyrm/templates/settings/themes.html
+++ b/bookwyrm/templates/settings/themes.html
@@ -16,21 +16,24 @@
 {% endif %}
 
 <section class="block">
-    <div class="notification">
-        <p>
-            <strong>{% trans "Default theme:" %}</strong> {{ site.default_theme.name }}
-        </p>
-
-        <p>
-            <a href="{% url 'settings-site' %}#display">
-                {% trans "Set default theme" %}
-            </a>
-        </p>
+    <div class="notification content">
+        <h2 class="title is-5">{% trans "How to add a theme" %}</h2>
+        <ol>
+            <li>
+                {% trans "Copy the theme file into the <code>bookwyrm/static/css/</code> directory on your server from the command line." %}
+            </li>
+            <li>
+                {% trans "Run <code>./bw-dev compilescss</code>." %}
+            </li>
+            <li>
+                {% trans "Add the file name using the form below to make it available in the application interface." %}
+            </li>
+        </ol>
     </div>
 </section>
 
 <section class="block content">
-    <h2 class="title is-4">{% trans "Upload theme" %}</h2>
+    <h2 class="title is-4">{% trans "Add theme" %}</h2>
 
     {% if theme_form.errors %}
     <div class="notification is-danger is-light">
@@ -60,17 +63,17 @@
             </div>
 
             <div class="column">
-                <label class="label" for="id_theme_file">
-                    {% trans "Theme file" %}
+                <label class="label" for="id_path">
+                    {% trans "Theme filename" %}
                 </label>
                 <div class="control">
-                    {{ theme_form.theme_file }}
-                    {% include 'snippets/form_errors.html' with errors_list=theme_form.theme_file.errors id="desc_theme_file" %}
+                    {{ theme_form.path }}
+                    {% include 'snippets/form_errors.html' with errors_list=theme_form.path.errors id="desc_path" %}
                 </div>
             </div>
         </div>
 
-        <button type="submit" class="button">{% trans "Upload theme" %}</button>
+        <button type="submit" class="button">{% trans "Add theme" %}</button>
     </form>
 </section>
 
@@ -92,13 +95,11 @@
             {% for theme in themes %}
             <tr>
                 <td>{{ theme.name }}</td>
-                <td><code>{{ theme.theme_path }}</code></td>
+                <td><code>{{ theme.path }}</code></td>
                 <td>
-                    {% if theme.theme_file %}
                     <form>
-                        <button type="submit" class="button is-danger is-light">{% trans "Delete" %}</button>
+                        <button type="submit" class="button is-danger is-light">{% trans "Remove theme" %}</button>
                     </form>
-                    {% endif %}
                 </td>
             </tr>
             {% endfor %}