Remove old modal, new modal is our best friend now

bookwyrm/templates/author/sync_modal.html

@@ -1,4 +1,4 @@
-{% extends 'components/new_modal.html' %}
+{% extends 'components/modal.html' %}
 {% load i18n %}
 
 {% block modal-title %}

bookwyrm/templates/book/cover_add_modal.html

@@ -1,4 +1,4 @@
-{% extends 'components/new_modal.html' %}
+{% extends 'components/modal.html' %}
 {% load i18n %}
 
 {% block modal-title %}

bookwyrm/templates/book/delete_readthrough_modal.html

@@ -1,4 +1,4 @@
-{% extends 'components/new_modal.html' %}
+{% extends 'components/modal.html' %}
 {% load i18n %}
 
 {% block modal-title %}{% trans "Delete these read dates?" %}{% endblock %}

bookwyrm/templates/book/sync_modal.html

@@ -1,4 +1,4 @@
-{% extends 'components/new_modal.html' %}
+{% extends 'components/modal.html' %}
 {% load i18n %}
 
 {% block modal-title %}

bookwyrm/templates/components/modal.html

@@ -1,40 +1,33 @@
 {% load i18n %}
-<div
+
-    role="dialog"
+<div class="modal {% if active %}is-active{% endif %}" id="{{ id }}">
-    class="modal {% if active %}is-active{% else %}is-hidden{% endif %}"
+    <div class="modal-background" data-modal-close></div>
-    id="{{ controls_text }}_{{ controls_uid }}"
+    <div class="modal-card" role="dialog" aria-modal="true" tabindex="-1">
-    aria-labelledby="modal_card_title_{{ controls_text }}_{{ controls_uid }}"
+        <header class="modal-card-head">
-    aria-modal="true"
+            <h2 class="modal-card-title mb-0">
->
-    {# @todo Implement focus traps to prevent tabbing out of the modal. #}
-    <div class="modal-background"></div>
-    {% trans "Close" as label %}
-    <div class="modal-card">
-        <header class="modal-card-head" tabindex="0" id="modal_title_{{ controls_text }}_{{ controls_uid }}">
-            <h2 class="modal-card-title is-flex-shrink-1" id="modal_card_title_{{ controls_text }}_{{ controls_uid }}">
                 {% block modal-title %}{% endblock %}
             </h2>
-            {% if static %}
+            <button
-                <a href="/" class="delete">{{ label }}</a>
+                type="button"
-            {% else %}
+                class="delete"
-                {% include 'snippets/toggle/toggle_button.html' with label=label class="delete" nonbutton=True %}
+                aria-label="{% trans 'Close' %}"
-            {% endif %}
+                data-modal-close
+            ></button>
         </header>
+
         {% block modal-form-open %}{% endblock %}
-        {% if not no_body %}
         <section class="modal-card-body">
             {% block modal-body %}{% endblock %}
         </section>
-        {% endif %}
         <footer class="modal-card-foot">
             {% block modal-footer %}{% endblock %}
         </footer>
         {% block modal-form-close %}{% endblock %}
     </div>
-    {% if static %}
+    <button
-        <a href="/" class="modal-close is-large">{{ label }}</a>
+        type="button"
-    {% else %}
+        class="modal-close is-large"
-        {% include 'snippets/toggle/toggle_button.html' with label=label class="modal-close is-large" nonbutton=True %}
+        aria-label="{% trans 'Close' %}"
-    {% endif %}
+        data-modal-close
+    ></button>
 </div>
-

bookwyrm/templates/components/new_modal.html

@@ -1,33 +0,0 @@
-{% load i18n %}
-
-<div class="modal {% if active %}is-active{% endif %}" id="{{ id }}">
-    <div class="modal-background" data-modal-close></div>
-    <div class="modal-card" role="dialog" aria-modal="true" tabindex="-1">
-        <header class="modal-card-head">
-            <h2 class="modal-card-title mb-0">
-                {% block modal-title %}{% endblock %}
-            </h2>
-            <button
-                type="button"
-                class="delete"
-                aria-label="{% trans 'Close' %}"
-                data-modal-close
-            ></button>
-        </header>
-
-        {% block modal-form-open %}{% endblock %}
-        <section class="modal-card-body">
-            {% block modal-body %}{% endblock %}
-        </section>
-        <footer class="modal-card-foot">
-            {% block modal-footer %}{% endblock %}
-        </footer>
-        {% block modal-form-close %}{% endblock %}
-    </div>
-    <button
-        type="button"
-        class="modal-close is-large"
-        aria-label="{% trans 'Close' %}"
-        data-modal-close
-    ></button>
-</div>

bookwyrm/templates/groups/delete_group_modal.html

@@ -1,4 +1,4 @@
-{% extends 'components/new_modal.html' %}
+{% extends 'components/modal.html' %}
 {% load i18n %}
 
 {% block modal-title %}{% trans "Delete this group?" %}{% endblock %}

bookwyrm/templates/lists/delete_list_modal.html

@@ -1,4 +1,4 @@
-{% extends 'components/new_modal.html' %}
+{% extends 'components/modal.html' %}
 {% load i18n %}
 
 {% block modal-title %}{% trans "Delete this list?" %}{% endblock %}

bookwyrm/templates/snippets/reading_modals/layout.html

@@ -1,4 +1,4 @@
-{% extends 'components/new_modal.html' %}
+{% extends 'components/modal.html' %}
 {% load i18n %}
 {% load utilities %}
 

bookwyrm/templates/snippets/report_modal.html

@@ -1,4 +1,4 @@
-{% extends 'components/new_modal.html' %}
+{% extends 'components/modal.html' %}
 {% load i18n %}
 {% load humanize %}