fix: always close mobile sidemenu on click (#331)

dataesr · Feb 10, 2023 · 13c7b6f · 13c7b6f
1 parent cae5d6f
commit 13c7b6f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/components/interface/Header/ToolItem.js b/src/components/interface/Header/ToolItem.js
@@ -34,7 +34,7 @@ const ToolItem = ({
         </HtmlTag>
       ) : (
         <Link
-          onClick={onClick ? onClickToolItem : undefined}
+          onClick={onClickToolItem}
           as={asLink}
           target={target}
           className={className}

diff --git a/src/components/interface/Header/__tests__/__snapshots__/Header.test.js.snap b/src/components/interface/Header/__tests__/__snapshots__/Header.test.js.snap
@@ -96,6 +96,7 @@ exports[`<Header /> renders correctly 1`] = `
                 <a
                   className="ds-fr--flex fr-link fr-link--md"
                   href="/path"
+                  onClick={[Function]}
                   target="_self"
                 >
                   <span
@@ -111,6 +112,7 @@ exports[`<Header /> renders correctly 1`] = `
                 <a
                   className="ds-fr--flex fr-link fr-link--md"
                   href="/path"
+                  onClick={[Function]}
                   target="_self"
                 >
                   <span