hopefully fix checkboxes

MushroomObserver · Aug 16, 2024 · 369e610 · 369e610
1 parent 959cdef
commit 369e610
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 8 deletions.
diff --git a/app/javascript/controllers/checkbox-collapse_controller.js b/app/javascript/controllers/checkbox-collapse_controller.js
@@ -7,19 +7,19 @@ export default class extends Controller {
   connect() {
     this.element.dataset.stimulus = "checkbox-collapse-connected";
 
-    this.checked = this.element.dataset.checked
-    if (this.hasFieldsTarget && this.checked) {
-      this.hideShowFields()
+    this.showPref = this.element.dataset.userPref
+    if (this.hasFieldsTarget && this.showPref) {
+      this.fieldsTarget.style.display = "none"
+      this.fieldsTarget.classList.remove("hidden")
     }
   }
 
   // Only show if user prefers
   hideShowFields() {
     if (this.checkboxTarget.checked) {
-      this.fieldsTarget.classList.remove("hidden")
-      this.hideFields()
+      $(this.fieldsTarget).show()
     } else {
-      this.showFields()
+      $(this.fieldsTarget).hide()
     }
   }
 
@@ -31,6 +31,7 @@ export default class extends Controller {
   }
 
   showFields() {
+    this.fieldsTarget.classList.remove("hidden")
     $(this.fieldsTarget).show()
     this.checkCheckbox()
   }

diff --git a/app/views/controllers/observations/form/_details.html.erb b/app/views/controllers/observations/form/_details.html.erb
@@ -11,7 +11,7 @@ t_s = {
 %>
 
 <%= tag.div(class: "panel-body border-top", id: "observation_details",
-            data: { controller: "checkbox-collapse" }) do %>
+            data: { controller: "checkbox-collapse", user_pref: true }) do %>
 
   <%= tag.div(class: "row mt-3") do %>
     <%= tag.div(class: "col-xs-12 col-md-6") do %>

diff --git a/app/views/controllers/observations/form/_specimen_section.html.erb b/app/views/controllers/observations/form/_specimen_section.html.erb
@@ -5,7 +5,7 @@ herbarium_record. Fields hidden unless box checked. %>
 
 <%= tag.div(id: "observation_specimen_section",
             data: { controller: "checkbox-collapse",
-                    checked: ([email protected](&:hide_specimen_stuff?)) }) do %>
+                    user_pref: ([email protected](&:hide_specimen_stuff?)) }) do %>
 
   <%= tag.div(class: "mt-3") do %>
     <%= check_box_with_label(