chore(Dialog): Add scrollable prop type defintion to DialogBody

rmwc · Feb 15, 2018 · 15b8b48 · 15b8b48
1 parent 8b3d6bc
commit 15b8b48
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 15 deletions.
diff --git a/src/Dialog/index.js b/src/Dialog/index.js
@@ -41,8 +41,14 @@ export const DialogHeaderTitle = simpleTag({
   classNames: 'mdc-dialog__header__title'
 });
 
+
+type DialogBodyT = {
+  /** Make it scrollable. */
+  scrollable?: boolean
+} & SimpleTagPropsT;
+
 /** The Dialog body */
-export const DialogBody = simpleTag({
+export class DialogBody extends simpleTag({
   displayName: 'DialogBody',
   tag: 'section',
   classNames: props => [
@@ -52,7 +58,11 @@ export const DialogBody = simpleTag({
     }
   ],
   consumeProps: ['scrollable']
-});
+})<DialogBodyT> {
+  render() {
+    return super.render();
+  }
+}
 
 /** The Dialog footer */
 export const DialogFooter = simpleTag({

diff --git a/src/docs/docgen.json b/src/docs/docgen.json
@@ -344,20 +344,16 @@
     {
       "description": "The Dialog body",
       "displayName": "DialogBody",
-      "methods": [
-        {
-          "name": "classNames",
-          "docblock": null,
-          "modifiers": [],
-          "params": [
-            {
-              "name": "props",
-              "type": null
-            }
-          ],
-          "returns": null
+      "methods": [],
+      "props": {
+        "scrollable": {
+          "flowType": {
+            "name": "boolean"
+          },
+          "required": false,
+          "description": "Make it scrollable."
         }
-      ]
+      }
     },
     {
       "description": "The Dialog footer",