From 9acea28a35d1e9e916112ca04e75c23d138b24bb Mon Sep 17 00:00:00 2001
From: Rodrigo Antinarelli <rodrigoantinarelli@gmail.com>
Date: Wed, 8 Apr 2020 13:37:28 -0300
Subject: [PATCH] fix onClick check

---
 .../src/components/IconButton/IconButton.jsx  | 41 +++++++++----------
 1 file changed, 20 insertions(+), 21 deletions(-)

diff --git a/platform/ui/src/components/IconButton/IconButton.jsx b/platform/ui/src/components/IconButton/IconButton.jsx
index bd01ee349..7dc59f71b 100644
--- a/platform/ui/src/components/IconButton/IconButton.jsx
+++ b/platform/ui/src/components/IconButton/IconButton.jsx
@@ -5,16 +5,6 @@ import classnames from 'classnames';
 const baseClasses =
   'text-center items-center justify-center outline-none transition duration-300 ease-in-out font-bold focus:outline-none';
 
-const defaults = {
-  color: 'default',
-  disabled: false,
-  fullWidth: false,
-  rounded: 'medium',
-  size: 'medium',
-  type: 'button',
-  variant: 'contained',
-};
-
 const roundedClasses = {
   none: '',
   small: 'rounded',
@@ -80,13 +70,13 @@ const fullWidthClasses = {
 
 const IconButton = ({
   children,
-  variant = defaults.variant,
-  color = defaults.color,
-  size = defaults.size,
-  rounded = defaults.rounded,
-  disabled = defaults.disabled,
-  type = defaults.type,
-  fullWidth = defaults.fullWidth,
+  variant,
+  color,
+  size,
+  rounded,
+  disabled,
+  type,
+  fullWidth,
   onClick,
   className,
   ...rest
@@ -95,9 +85,7 @@ const IconButton = ({
 
   const handleOnClick = (e) => {
     buttonElement.current.blur();
-    if (onClick) {
-      onClick(e);
-    }
+    onClick(e);
   };
 
   return (
@@ -123,8 +111,19 @@ const IconButton = ({
   );
 };
 
+IconButton.defaultProps = {
+  onClick: () => {},
+  color: 'default',
+  disabled: false,
+  fullWidth: false,
+  rounded: 'medium',
+  size: 'medium',
+  type: 'button',
+  variant: 'contained',
+};
+
 IconButton.propTypes = {
-  children: PropTypes.node,
+  children: PropTypes.node.isRequired,
   size: PropTypes.oneOf(['small', 'medium', 'large']),
   rounded: PropTypes.oneOf(['none', 'small', 'medium', 'large', 'full']),
   variant: PropTypes.oneOf(['text', 'outlined', 'contained']),