Persist item user lists form open/closed state across pageloads

Using good ol'-fashioned cookies! The JS sets it, and then Rails reads
it on pageload. That way, there's no flash of content for it to load in
after JS loads.

app/assets/javascripts/items/item_header.js

@@ -1,21 +1,23 @@
-const userListSections = document.querySelectorAll(
+function setFormStateCookie(value) {
-	".item-header .user-lists-section");
+	const thirtyDays = 60 * 60 * 24 * 30;
-for (const section of userListSections) {
+	document.cookie = `DTIItemPageUserListsFormState=open;max-age=${thirtyDays}`;
+}
+
+const headers = document.querySelectorAll(".item-header");
+for (const header of headers) {
 	try {
-		const dialog = section.querySelector("dialog");
+		const form = header.querySelector(".user-lists-form");
-		const opener = section.querySelector(".dialog-opener");
+		const opener = header.querySelector(".user-lists-form-opener");
-		const closer = section.querySelector(".dialog-closer");
-		if (dialog.showModal) { // check browser support
 		opener.addEventListener("click", (event) => {
-				dialog.show();
+			if (form.hasAttribute("hidden")) {
+				form.removeAttribute("hidden");
+				setFormStateCookie("open");
+			} else {
+				form.setAttribute("hidden", "");
+				setFormStateCookie("closed");
+			}
 			event.preventDefault();
 		});
-			document.body.addEventListener("click", (event) => {
-				if (dialog.open && !section.contains(event.target)) {
-					dialog.close();
-				}
-			});
-		}
 	} catch (error) {
 		console.error(`Error applying dialog behavior to item header:`, error);
 	}

app/assets/stylesheets/partials/_item_header.sass

@@ -59,16 +59,16 @@
           background: #E2E8F0
           color: #1A202C
 
-    .user-lists-section
+    .user-lists-info
       grid-area: lists
       font-size: 85%
       text-align: left
 
-      .dialog-opener
+      .user-lists-form-opener
         &::after
           content: " ›"
 
-      dialog
+  .user-lists-form
     background: $background-color
     border: 1px solid $module-border-color
     border-radius: .5em
@@ -77,12 +77,14 @@
     text-align: center
     z-index: 2
     margin-top: .5em
-        box-shadow: 0px 1px 4px #666
+    margin-inline: auto
+    margin-bottom: 1.5em
+    font-size: 85%
 
     h3
       font-size: 150%
       font-weight: bold
-          margin-bottom: .25em
+      margin-bottom: .75em
     
     .closet-hangers-ownership-groups
       +clearfix

app/helpers/items_helper.rb

@@ -118,5 +118,9 @@ module ItemsHelper
     human_name = pet_type.species.name.humanize
     image_tag(src, :alt => human_name, :title => human_name)
   end
+
+  def item_header_user_lists_form_state
+    cookies.fetch("DTIItemPageUserListsFormState", "closed")
+  end
 end
 

app/views/items/_item_header.haml

@@ -31,16 +31,17 @@
         = link_to t('items.show.resources.auction_genie'), auction_genie_url_for(item)
 
     - if user_signed_in?
-      .user-lists-section
+      .user-lists-info
         = link_to t('items.show.closet_hangers.button'),
           user_closet_hangers_path(current_user),
-          class: 'dialog-opener'
+          class: 'user-lists-form-opener'
-        %dialog
+
+  - if user_signed_in?
+    = form_tag update_quantities_user_item_closet_hangers_path(user_id: current_user, item_id: item), method: :put, class: 'user-lists-form', hidden: item_header_user_lists_form_state != "open" do
       %h3
         = t 'items.show.closet_hangers.header_html',
           user_items_link: link_to(t('your_items'),
           user_closet_hangers_path(current_user))
-          = form_tag update_quantities_user_item_closet_hangers_path(:user_id => current_user, :item_id => item), :method => :put do
       .closet-hangers-ownership-groups
         - [true, false].each do |owned|
           - lists = current_user_lists[owned]