From 944503dde5f5ba252e48f247d60d26bebfa45633 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Sun, 31 Jan 2021 12:36:53 -0800
Subject: [PATCH] Use close button instead of cancel button on edit form

---
 bookwyrm/templates/lists/form.html        |  3 +--
 bookwyrm/templates/lists/list_layout.html | 11 ++++++++---
 bookwyrm/templates/lists/lists.html       |  7 ++++++-
 3 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/bookwyrm/templates/lists/form.html b/bookwyrm/templates/lists/form.html
index af0b8abf..ec265cc1 100644
--- a/bookwyrm/templates/lists/form.html
+++ b/bookwyrm/templates/lists/form.html
@@ -28,7 +28,7 @@
 
             <label class="field">
                 <input type="radio" name="curation" value="open"{% if list.curation == 'open' %} checked{% endif %}> Open
-                <p class="help mb-2">Anyone can add books to this list, but only you can remove them</p>
+                <p class="help mb-2">Anyone can add books to this list</p>
             </label>
         </fieldset>
     </div>
@@ -39,7 +39,6 @@
     </div>
     <div class="control">
         <button type="submit" class="button is-primary">Save</button>
-        {% include 'snippets/toggle/close_button.html' with controls_text='create-list' text="Cancel" %}
     </div>
 </div>
 
diff --git a/bookwyrm/templates/lists/list_layout.html b/bookwyrm/templates/lists/list_layout.html
index 135655ee..5e8d2c12 100644
--- a/bookwyrm/templates/lists/list_layout.html
+++ b/bookwyrm/templates/lists/list_layout.html
@@ -10,13 +10,18 @@
     </div>
     {% if request.user == list.user %}
     <div class="column is-narrow">
-        {% include 'snippets/toggle/open_button.html' with text="Edit list" icon="pencil" controls_text="create-list" %}
+        {% include 'snippets/toggle/open_button.html' with text="Edit list" icon="pencil" controls_text="edit-list" %}
     </div>
     {% endif %}
 </header>
 
-<form name="create-list" method="post" action="{% url 'list' list.id %}" class="box hidden" id="create-list">
-    <h3 class="title">Edit list</h3>
+<form name="edit-list" method="post" action="{% url 'list' list.id %}" class="box hidden" id="edit-list">
+    <header class="columns">
+        <h3 class="title column">Edit list</h3>
+        <div class="column is-narrow">
+            {% include 'snippets/toggle/toggle_button.html' with controls_text="edit-list" label="close" class="delete" nonbutton=True %}
+        </div>
+    </header>
     {% include 'lists/form.html' %}
 </form>
 
diff --git a/bookwyrm/templates/lists/lists.html b/bookwyrm/templates/lists/lists.html
index 1f230b78..97b15641 100644
--- a/bookwyrm/templates/lists/lists.html
+++ b/bookwyrm/templates/lists/lists.html
@@ -17,7 +17,12 @@
     </header>
 
     <form name="create-list" method="post" action="{% url 'lists' %}" class="box hidden" id="create-list">
-        <h3 class="title">Create list</h3>
+        <header class="columns">
+            <h3 class="title column">Create list</h3>
+            <div class="column is-narrow">
+                {% include 'snippets/toggle/toggle_button.html' with controls_text="create-list" label="close" class="delete" nonbutton=True %}
+            </div>
+        </header>
         {% include 'lists/form.html' %}
     </form>