Addon-docs: Fix TS false default value in prop table (#9560)

Addon-docs: Fix TS false default value in prop table
storybookjs · Jan 21, 2020 · 850ca06 · 850ca06
1 parent 2039097
commit 850ca06
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/examples/cra-ts-kitchen-sink/src/stories/Button.tsx b/examples/cra-ts-kitchen-sink/src/stories/Button.tsx
@@ -9,7 +9,12 @@ interface ButtonProps {
   /**
    * Is primary?
    */
-  primary: boolean;
+  primary?: boolean;
+
+  /**
+   * default is false
+   */
+  secondary: boolean;
 }
 
 /**
@@ -23,4 +28,5 @@ export const Button: FC<ButtonProps> = ({ children, onClick }) => (
 
 Button.defaultProps = {
   primary: true,
+  secondary: false,
 };
diff --git a/lib/components/src/blocks/PropsTable/PropValue.tsx b/lib/components/src/blocks/PropsTable/PropValue.tsx
@@ -75,11 +75,12 @@ const PropSummary: FC<PropSummaryProps> = ({ value }) => {
   const { summary, detail } = value;
 
   const [isOpen, setIsOpen] = useState(false);
-
   // summary is used for the default value
   // below check fixes not displaying default values for boolean typescript vars
   const summaryAsString =
-    summary && typeof summary.toString === 'function' ? summary.toString() : summary;
+    summary !== undefined && summary !== null && typeof summary.toString === 'function'
+      ? summary.toString()
+      : summary;
   if (isNil(detail)) {
     return <PropText text={summaryAsString} />;
   }