From 99a04ae663e1953e535a9f131cda4b11fcafcbb9 Mon Sep 17 00:00:00 2001
From: Sarah Dayan <sarah.dayan@algolia.com>
Date: Wed, 9 Sep 2020 15:10:39 +0200
Subject: [PATCH] feat: remove left panel when refinements are empty (#149)

---
 src/components/Search.js | 100 ++++++++++++++++++++++-----------------
 1 file changed, 57 insertions(+), 43 deletions(-)

diff --git a/src/components/Search.js b/src/components/Search.js
index aaa6a1e3..5a57da23 100644
--- a/src/components/Search.js
+++ b/src/components/Search.js
@@ -30,6 +30,8 @@ export function Search(props) {
   ];
   const sorts = defaultSort.concat(config.sorts);
 
+  const hasRefinements = Boolean(config.refinements.length);
+
   React.useEffect(() => {
     if (filtersAnchor.current && props.isFiltering) {
       filtersAnchor.current.scrollTop = 0;
@@ -65,46 +67,56 @@ export function Search(props) {
         </header>
 
         <div className="uni-Content">
-          <div
-            data-layout="mobile"
-            className="uni-LeftPanel-Overlay"
-            onClick={() => props.setIsFiltering(false)}
-          />
-          <div className="uni-LeftPanel">
-            <div className="uni-Refinements">
-              <div className="uni-Refinements-scrollable" ref={filtersAnchor}>
-                <header
-                  className="uni-Refinements-heading"
-                  data-layout="mobile"
-                >
-                  <span>Filters</span>
-                  <button
-                    onClick={() => {
-                      props.setIsFiltering(false);
-                    }}
-                    className="uni-Refinements-closeButton"
-                    title="Close filters"
+          {hasRefinements && (
+            <>
+              <div
+                data-layout="mobile"
+                className="uni-LeftPanel-Overlay"
+                onClick={() => props.setIsFiltering(false)}
+              />
+              <div className="uni-LeftPanel">
+                <div className="uni-Refinements">
+                  <div
+                    className="uni-Refinements-scrollable"
+                    ref={filtersAnchor}
+                  >
+                    <header
+                      className="uni-Refinements-heading"
+                      data-layout="mobile"
+                    >
+                      <span>Filters</span>
+                      <button
+                        onClick={() => {
+                          props.setIsFiltering(false);
+                        }}
+                        className="uni-Refinements-closeButton"
+                        title="Close filters"
+                      >
+                        <CloseIcon />
+                      </button>
+                    </header>
+                    {isMobile && <CurrentRefinements />}
+                    <Refinements />
+                  </div>
+                  <footer
+                    className="uni-Refinements-footer"
+                    data-layout="mobile"
                   >
-                    <CloseIcon />
-                  </button>
-                </header>
-                {isMobile && <CurrentRefinements />}
-                <Refinements />
+                    <ResetButton
+                      onClick={() => {
+                        props.setIsFiltering(false);
+                      }}
+                    />
+                    <SeeResultsButton
+                      onClick={() => {
+                        props.setIsFiltering(false);
+                      }}
+                    />
+                  </footer>
+                </div>
               </div>
-              <footer className="uni-Refinements-footer" data-layout="mobile">
-                <ResetButton
-                  onClick={() => {
-                    props.setIsFiltering(false);
-                  }}
-                />
-                <SeeResultsButton
-                  onClick={() => {
-                    props.setIsFiltering(false);
-                  }}
-                />
-              </footer>
-            </div>
-          </div>
+            </>
+          )}
 
           <div className="uni-RightPanel">
             <header className="uni-BodyHeader">
@@ -139,11 +151,13 @@ export function Search(props) {
               </NoResultsHandler>
             </main>
           </div>
-          <FiltersButton
-            onClick={() => {
-              props.setIsFiltering(true);
-            }}
-          />
+          {hasRefinements && (
+            <FiltersButton
+              onClick={() => {
+                props.setIsFiltering(true);
+              }}
+            />
+          )}
         </div>
       </div>
     </InstantSearch>