From f281096b392e73d553560869fa0044114d2d5595 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Anne=20L=27H=C3=B4te?= <anne.lhote@gmail.com>
Date: Fri, 20 Oct 2023 17:48:00 +0200
Subject: [PATCH] feat(ui): Delete "More filters" button, see #13

---
 client/src/pages/home/filters.jsx | 58 ++++++++++++-------------------
 1 file changed, 22 insertions(+), 36 deletions(-)

diff --git a/client/src/pages/home/filters.jsx b/client/src/pages/home/filters.jsx
index c7fa6211..a861ca8b 100644
--- a/client/src/pages/home/filters.jsx
+++ b/client/src/pages/home/filters.jsx
@@ -30,7 +30,6 @@ export default function Filters({ sendQuery }) {
         dataIdentifiers: identifiers,
         datasources: sources.map((source) => source.key),
         endYear: '2021',
-        moreOptions: false,
         startYear: '2021',
       });
     } else {
@@ -39,7 +38,6 @@ export default function Filters({ sendQuery }) {
         dataIdentifiers: searchParams.getAll('dataIdentifiers'),
         datasources: searchParams.getAll('datasources'),
         endYear: searchParams.get('endYear'),
-        moreOptions: searchParams.get('moreOptions')?.toString() === 'true',
         startYear: searchParams.get('startYear'),
       });
     }
@@ -118,41 +116,29 @@ export default function Filters({ sendQuery }) {
             </Col>
           </Row>
         </Col>
-        {
-          currentSearchParams.moreOptions && (
-            <Col n="3">
-              {
-                currentSearchParams.datasources.includes('bso') && (
-                  <>
-                    BSO Identifiers:
-                    <CheckboxGroup isInline>
-                      {
-                        identifiers.map((identifier) => (
-                          <Checkbox
-                            checked={currentSearchParams.dataIdentifiers.includes(identifier)}
-                            key={identifier}
-                            label={identifier}
-                            onChange={() => onIdentifiersChange(identifier)}
-                            size="sm"
-                          />
-                        ))
-                      }
-                    </CheckboxGroup>
-                  </>
-                )
-              }
-            </Col>
-          )
-        }
+        <Col n="3">
+          {
+            currentSearchParams.datasources.includes('bso') && (
+              <>
+                BSO Identifiers:
+                <CheckboxGroup isInline>
+                  {
+                    identifiers.map((identifier) => (
+                      <Checkbox
+                        checked={currentSearchParams.dataIdentifiers.includes(identifier)}
+                        key={identifier}
+                        label={identifier}
+                        onChange={() => onIdentifiersChange(identifier)}
+                        size="sm"
+                      />
+                    ))
+                  }
+                </CheckboxGroup>
+              </>
+            )
+          }
+        </Col>
         <Col className="text-right">
-          <Button
-            onClick={() => setSearchParams({ ...currentSearchParams, moreOptions: !currentSearchParams.moreOptions })}
-            secondary
-            size="sm"
-            icon="ri-filter-line"
-          >
-            {currentSearchParams.moreOptions ? 'Less filters' : 'More filters'}
-          </Button>
           <Button
             icon="ri-search-line"
             onClick={checkAndSendQuery}