From 9b64b39f4b3a5df213df9a604e1e1ea0bd9d417d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=A6var=20=C3=96fj=C3=B6r=C3=B0=20Magn=C3=BAsson?=
 <saevar@overcast.is>
Date: Thu, 23 Jan 2020 16:18:15 +0000
Subject: [PATCH] Show 'Close Explorer' button on mobile viewports. Fix #5396
 (#5773)

* Rework 'Close Explorer' button css so that it does not rely on u-hidden. Fixes issue #5396
* Update tests due to changed css classes on 'close explorer' button
---
 client/src/components/Explorer/Explorer.scss           |  1 +
 client/src/components/Explorer/ExplorerPanel.js        |  2 +-
 .../Explorer/__snapshots__/ExplorerPanel.test.js.snap  | 10 +++++-----
 3 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/client/src/components/Explorer/Explorer.scss b/client/src/components/Explorer/Explorer.scss
index a589ba191..5413971ae 100644
--- a/client/src/components/Explorer/Explorer.scss
+++ b/client/src/components/Explorer/Explorer.scss
@@ -58,6 +58,7 @@ $menu-footer-height: 50px;
     color: $c-explorer-secondary;
     border-bottom: 1px solid rgba(200, 200, 200, 0.1);
     cursor: pointer;
+    display: none;
 
     &:focus {
         background-color: $c-explorer-bg-active;
diff --git a/client/src/components/Explorer/ExplorerPanel.js b/client/src/components/Explorer/ExplorerPanel.js
index af2d7da12..433c51fdd 100644
--- a/client/src/components/Explorer/ExplorerPanel.js
+++ b/client/src/components/Explorer/ExplorerPanel.js
@@ -146,7 +146,7 @@ class ExplorerPanel extends React.Component {
           onDeactivate: onClose,
         }}
       >
-        <Button className="c-explorer__close u-hidden" onClick={onClose}>
+        <Button className="c-explorer__close" onClick={onClose}>
           {STRINGS.CLOSE_EXPLORER}
         </Button>
         <Transition name={transition} className="c-explorer" component="nav" label={STRINGS.PAGE_EXPLORER}>
diff --git a/client/src/components/Explorer/__snapshots__/ExplorerPanel.test.js.snap b/client/src/components/Explorer/__snapshots__/ExplorerPanel.test.js.snap
index 0be336143..1d0c9a5e1 100644
--- a/client/src/components/Explorer/__snapshots__/ExplorerPanel.test.js.snap
+++ b/client/src/components/Explorer/__snapshots__/ExplorerPanel.test.js.snap
@@ -17,7 +17,7 @@ exports[`ExplorerPanel general rendering #isError 1`] = `
 >
   <Button
     accessibleLabel={null}
-    className="c-explorer__close u-hidden"
+    className="c-explorer__close"
     dialogTrigger={false}
     href="#"
     icon=""
@@ -96,7 +96,7 @@ exports[`ExplorerPanel general rendering #isFetching 1`] = `
 >
   <Button
     accessibleLabel={null}
-    className="c-explorer__close u-hidden"
+    className="c-explorer__close"
     dialogTrigger={false}
     href="#"
     icon=""
@@ -165,7 +165,7 @@ exports[`ExplorerPanel general rendering #items 1`] = `
 >
   <Button
     accessibleLabel={null}
-    className="c-explorer__close u-hidden"
+    className="c-explorer__close"
     dialogTrigger={false}
     href="#"
     icon=""
@@ -259,7 +259,7 @@ exports[`ExplorerPanel general rendering no children 1`] = `
 >
   <Button
     accessibleLabel={null}
-    className="c-explorer__close u-hidden"
+    className="c-explorer__close"
     dialogTrigger={false}
     href="#"
     icon=""
@@ -322,7 +322,7 @@ exports[`ExplorerPanel general rendering renders 1`] = `
 >
   <Button
     accessibleLabel={null}
-    className="c-explorer__close u-hidden"
+    className="c-explorer__close"
     dialogTrigger={false}
     href="#"
     icon=""