feat: onClick is no longer required on navbar and steplink (#441)

Co-authored-by: Johnathan MEUNIER <jonathan.meunier@axa.fr>
AxaFrance · Jul 22, 2020 · 723d48e · 723d48e
1 parent c6d0ac6
commit 723d48e
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/packages/Form/steps/src/StepLink.js b/packages/Form/steps/src/StepLink.js
@@ -8,7 +8,7 @@ const propTypes = {
   ...Constants.propTypes,
   id: PropTypes.string.isRequired,
   href: PropTypes.string,
-  onClick: PropTypes.func.isRequired,
+  onClick: PropTypes.func,
   number: PropTypes.oneOfType([PropTypes.element, PropTypes.string]).isRequired,
   title: PropTypes.string.isRequired,
   icon: PropTypes.string,

diff --git a/packages/Layout/header/src/NavBar/NavBar.js b/packages/Layout/header/src/NavBar/NavBar.js
@@ -7,7 +7,7 @@ import { setPosition } from './NavBar.helpers';
 const propTypes = {
   ...Constants.propTypes,
   isVisible: PropTypes.bool.isRequired,
-  onClick: PropTypes.func.isRequired,
+  onClick: PropTypes.func,
   id: PropTypes.string,
   toggleMenuId: PropTypes.string,
   children: PropTypes.node,

diff --git a/packages/Layout/header/src/NavBar/NavBarBase.js b/packages/Layout/header/src/NavBar/NavBarBase.js
@@ -6,7 +6,7 @@ import ToggleButton from '../ToggleButton';
 const propTypes = {
   ...Constants.propTypes,
   isVisible: PropTypes.bool.isRequired,
-  onClick: PropTypes.func.isRequired,
+  onClick: PropTypes.func,
   handleKeys: PropTypes.func.isRequired,
   onFocus: PropTypes.func.isRequired,
   onBlur: PropTypes.func.isRequired,