fix(atoms): set default padding and background to the 'none' value

8base · Jun 13, 2018 · 445d2f8 · 445d2f8
1 parent dfb6b2d
commit 445d2f8
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/atoms/Dropdown/DropdownBody.theme.js b/src/atoms/Dropdown/DropdownBody.theme.js
@@ -35,8 +35,8 @@ const theme = createTheme(name, {
     },
   },
   defaults: {
-    background: 'white',
-    padding: 'sm',
+    background: 'none',
+    padding: 'none',
     borderRadius: 'md',
   },
 });

diff --git a/src/atoms/Dropdown/DropdownPlate.js b/src/atoms/Dropdown/DropdownPlate.js
@@ -14,8 +14,8 @@ import { DropdownContext } from './DropdownContext';
  */
 type DropdownControlledProps = {|
   isOpen: boolean,
-  onCloseDropdown: () => void,
-  onOpenDropdown: () => void,
+  onCloseDropdown?: () => void,
+  onOpenDropdown?: () => void,
   children: React$Node,
 |}
 
@@ -30,7 +30,7 @@ type DropdownUncontroledProps = {|
 type DropdownContextData = {|
   isOpen: boolean,
   toggleDropdown: () => void,
-  closeDropdown: () => void,
+  closeDropdown?: () => void,
   targetWidth?: number,
   outsideClickIgnoreClass: string,
 |}
@@ -52,8 +52,8 @@ class DropdownPlateBase extends Component<DropdownControlledProps> {
     const { isOpen, onOpenDropdown, onCloseDropdown } = this.props;
 
     isOpen
-      ? fp.isFunction(onCloseDropdown) && onCloseDropdown()
-      : fp.isFunction(onOpenDropdown) && onOpenDropdown();
+      ? onCloseDropdown && onCloseDropdown()
+      : onOpenDropdown && onOpenDropdown();
   }
 
   setDropdownRef = (ref: *) => {