From 368604bd967de241757b3def5363e44c9c3135ca Mon Sep 17 00:00:00 2001
From: Igor Scheller <igor.scheller@igorshp.de>
Date: Fri, 6 Sep 2019 03:45:56 +0200
Subject: [PATCH] Shifts view: Persist hidden filters

---
 resources/assets/js/forms.js           | 35 +++++++++++++++++++++++---
 resources/views/pages/user-shifts.html |  8 +++---
 2 files changed, 35 insertions(+), 8 deletions(-)

diff --git a/resources/assets/js/forms.js b/resources/assets/js/forms.js
index f5818e97..9970b907 100644
--- a/resources/assets/js/forms.js
+++ b/resources/assets/js/forms.js
@@ -16,7 +16,7 @@ global.checkAll = (id, checked) => {
  * Sets the checkboxes according to the given type
  *
  * @param {string} id The elements ID
- * @param {list} shifts_list A list of numbers
+ * @param {list} shiftsList A list of numbers
  */
 global.checkOwnTypes = (id, shiftsList) => {
     $('#' + id + ' input[type="checkbox"]').each(function () {
@@ -144,10 +144,10 @@ $(function () {
         elem.children('input').on('click', function (ev) {
             ev.stopImmediatePropagation();
             if (typeof elem.data('DateTimePicker') === 'undefined') {
-              elem.datetimepicker(opts);
-              elem.data('DateTimePicker').show();
+                elem.datetimepicker(opts);
+                elem.data('DateTimePicker').show();
             } else {
-              elem.data('DateTimePicker').toggle();
+                elem.data('DateTimePicker').toggle();
             }
         });
     });
@@ -173,3 +173,30 @@ $(function () {
         });
     });
 });
+
+/**
+ * Set the filter selects to latest state
+ *
+ * Uses DOMContentLoaded to prevent flickering
+ */
+window.addEventListener('DOMContentLoaded', () => {
+    const filter = document.getElementById('collapseShiftsFilterSelect');
+    if (!filter || localStorage.getItem('collapseShiftsFilterSelect') !== 'hidden') {
+        return;
+    }
+
+    filter.classList.remove('in');
+});
+$(() => {
+    if (typeof (localStorage) === 'undefined') {
+        return;
+    }
+
+    const onChange = (e) => {
+        localStorage.setItem('collapseShiftsFilterSelect', e.type);
+    };
+
+    $('#collapseShiftsFilterSelect')
+        .on('hidden.bs.collapse', onChange)
+        .on('shown.bs.collapse', onChange);
+});
diff --git a/resources/views/pages/user-shifts.html b/resources/views/pages/user-shifts.html
index 9ac501da..d5a98f80 100644
--- a/resources/views/pages/user-shifts.html
+++ b/resources/views/pages/user-shifts.html
@@ -55,12 +55,12 @@
         <div class="col-md-6">
             <button class="btn btn-info btn-sm hidden-print" style="margin-top: 20px; margin-bottom:10px" type="button"
                     data-toggle="collapse"
-                    data-target="#collapseRoomSelect" aria-expanded="false"
-                    aria-controls="collapseRoomSelect"
+                    data-target="#collapseShiftsFilterSelect" aria-expanded="true"
+                    aria-controls="collapseShiftsFilterSelect"
             >
                 collapse/show filters
             </button>
-            <div class="collapse in" id="collapseRoomSelect">
+            <div class="collapse in" id="collapseShiftsFilterSelect">
                 <div class="row">
                     <div class="col-xs-4 col-xxs-12">%room_select%</div>
                     <div class="col-xs-4 col-xxs-12">%type_select%</div>
@@ -79,5 +79,5 @@
 %shifts_table%
 
 <div class="hidden-print">
-%ical_text%
+    %ical_text%
 </div>
\ No newline at end of file