From 7ccb02f6356e59a03a4f2031521023f66fef91c6 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Tue, 11 Jan 2022 10:27:10 -0800
Subject: [PATCH] Use new modal for edits

---
 bookwyrm/templates/book/book.html             |  2 +-
 .../templates/readthrough/readthrough.html    |  2 +-
 .../readthrough/readthrough_list.html         | 25 ++++++++-----------
 ...ough_modal.html => readthrough_modal.html} |  0
 4 files changed, 12 insertions(+), 17 deletions(-)
 rename bookwyrm/templates/readthrough/{add_readthrough_modal.html => readthrough_modal.html} (100%)

diff --git a/bookwyrm/templates/book/book.html b/bookwyrm/templates/book/book.html
index 434881865..e4d13007c 100644
--- a/bookwyrm/templates/book/book.html
+++ b/bookwyrm/templates/book/book.html
@@ -245,7 +245,7 @@
                         </button>
                     </div>
                 </header>
-                {% include "readthrough/add_readthrough_modal.html" with id="add-readthrough" %}
+                {% include "readthrough/readthrough_modal.html" with id="add-readthrough" %}
 
                 {% if not readthroughs.exists %}
                 <p>{% trans "You don't have any reading activity for this book." %}</p>
diff --git a/bookwyrm/templates/readthrough/readthrough.html b/bookwyrm/templates/readthrough/readthrough.html
index efd776834..bed59d29b 100644
--- a/bookwyrm/templates/readthrough/readthrough.html
+++ b/bookwyrm/templates/readthrough/readthrough.html
@@ -10,6 +10,6 @@ Add read dates for "<em>{{ title }}</em>"
 
 {% block content %}
 
-{% include "readthrough/add_readthrough_modal.html" with book=book active=True static=True %}
+{% include "readthrough/readthrough_modal.html" with book=book active=True static=True %}
 
 {% endblock %}
diff --git a/bookwyrm/templates/readthrough/readthrough_list.html b/bookwyrm/templates/readthrough/readthrough_list.html
index 21ce557cc..9f9db9a55 100644
--- a/bookwyrm/templates/readthrough/readthrough_list.html
+++ b/bookwyrm/templates/readthrough/readthrough_list.html
@@ -3,7 +3,7 @@
 {% load tz %}
 {% load utilities %}
 <div class="content">
-    <div id="hide_edit_readthrough_{{ readthrough.id }}" class="box is-shadowless has-background-white-bis">
+    <div class="box is-shadowless has-background-white-bis">
         <div class="columns">
             <div class="column">
                 {% trans "Progress Updates:" %}
@@ -58,7 +58,11 @@
                 <div class="field has-addons">
                     <div class="control">
                         {% trans "Edit read dates" as button_text %}
-                        {% include 'snippets/toggle/toggle_button.html' with class="is-small" text=button_text icon="pencil" controls_text="edit_readthrough" controls_uid=readthrough.id focus="edit_readthrough" %}
+                        <button class="button is-small" type="button" data-modal-open="edit_readthrough_{{ readthrough.id }}">
+                            <span class="icon icon-pencil" title="{{ button_text }}">
+                                <span class="is-sr-only">{{ button_text }}</span>
+                            </span>
+                        </button>
                     </div>
                     <div class="control">
                         {% trans "Delete these read dates" as button_text %}
@@ -74,16 +78,7 @@
     </div>
 </div>
 
-<div class="box is-hidden" id="edit_readthrough_{{ readthrough.id }}" tabindex="0">
-    <h3 class="title is-5">{% trans "Edit read dates" %}</h3>
-    <form name="edit-readthrough" action="/edit-readthrough" method="post">
-        {% include 'readthrough/readthrough_form.html' with readthrough=readthrough %}
-        <div class="field is-grouped">
-            <button class="button is-primary" type="submit">{% trans "Save" %}</button>
-            {% trans "Cancel" as button_text %}
-            {% include 'snippets/toggle/close_button.html' with text=button_text controls_text="edit_readthrough" controls_uid=readthrough.id %}
-        </div>
-    </form>
-</div>
-{% join "delete_readthrough" readthrough.id as modal_id %}
-{% include 'readthrough/delete_readthrough_modal.html' with id=modal_id %}
+{% join "edit_readthrough" readthrough.id as edit_modal_id %}
+{% include "readthrough/readthrough_modal.html" with readthrough=readthrough id=edit_modal_id %}
+{% join "delete_readthrough" readthrough.id as delete_modal_id %}
+{% include 'readthrough/delete_readthrough_modal.html' with id=delete_modal_id %}
diff --git a/bookwyrm/templates/readthrough/add_readthrough_modal.html b/bookwyrm/templates/readthrough/readthrough_modal.html
similarity index 100%
rename from bookwyrm/templates/readthrough/add_readthrough_modal.html
rename to bookwyrm/templates/readthrough/readthrough_modal.html