Cleanup after submitting modal

bookwyrm/static/js/status_cache.js

@@ -127,12 +127,25 @@ let StatusCache = new class {
         button.querySelectorAll("[data-shelf-dropdown-identifier] button")
             .forEach(item => item.disabled = false);
 
+        next_identifier = next_identifier == 'complete' ? 'read' : next_identifier;
         // disable the current state
         button.querySelector("[data-shelf-dropdown-identifier=" + identifier + "] button").disabled = true;
-        // hide the option that's shown as the main button
+
         let main_button = button.querySelector("[data-shelf-dropdown-identifier=" + next_identifier + "]");
+
+        // hide the option that's shown as the main button
         BookWyrm.addRemoveClass(main_button, "is-hidden", true);
 
+        // just hide the other two menu options, idk what to do with them
+        button.querySelectorAll("[data-extra-options]")
+            .forEach(item => BookWyrm.addRemoveClass(item, "is-hidden", true));
+
+        // close menu
+        let menu = button.querySelector(".dropdown-trigger[aria-expanded=true]");
+
+        if (menu) {
+            menu.click();
+        }
     }
 }();
 

bookwyrm/templates/snippets/shelve_button/shelve_button_dropdown_options.html

@@ -45,17 +45,15 @@
     </li>
 {% endfor %}
 
-
-{% if dropdown %}
 {% if readthrough and active_shelf.shelf.identifier != 'read' %}
-    <li role="menuitem" class="dropdown-item p-0">
+<li role="menuitem" class="dropdown-item p-0" data-extra-options>
     {% trans "Update progress" as button_text %}
     {% include 'snippets/toggle/toggle_button.html' with class=class text=button_text controls_text="progress_update" controls_uid=button_uuid focus="modal_title_progress_update" %}
 </li>
 {% endif %}
 
 {% if active_shelf.shelf %}
-    <li role="menuitem" class="dropdown-item p-0">
+<li role="menuitem" class="dropdown-item p-0" data-extra-options>
     <form  name="shelve" action="/unshelve/" method="post">
         {% csrf_token %}
         <input type="hidden" name="book" value="{{ active_shelf.book.id }}">
@@ -66,7 +64,6 @@
     </form>
 </li>
 {% endif %}
-{% endif %}
 
 {% endwith %}