[chore] specifiy the object when overriding class names.

check for a precise object when setting class names (className and overlayClassName) as object.
jaksenko · Oct 18, 2017 · 59fbdb3 · 59fbdb3
1 parent 4ca54df
commit 59fbdb3
Showing 1 changed file with 16 additions and 2 deletions.
diff --git a/src/components/Modal.js b/src/components/Modal.js
@@ -31,8 +31,22 @@ export default class Modal extends Component {
     }),
     portalClassName: PropTypes.string,
     bodyOpenClassName: PropTypes.string,
-    className: PropTypes.oneOfType([PropTypes.string, PropTypes.object]),
-    overlayClassName: PropTypes.oneOfType([PropTypes.string, PropTypes.object]),
+    className: PropTypes.oneOfType([
+      PropTypes.string,
+      PropTypes.shape({
+        base: PropTypes.string.isRequired,
+        afterOpen: PropTypes.string.isRequired,
+        beforeClose: PropTypes.string.isRequired
+      })
+    ]),
+    overlayClassName: PropTypes.oneOfType([
+      PropTypes.string,
+      PropTypes.shape({
+        base: PropTypes.string.isRequired,
+        afterOpen: PropTypes.string.isRequired,
+        beforeClose: PropTypes.string.isRequired
+      })
+    ]),
     appElement: PropTypes.instanceOf(SafeHTMLElement),
     onAfterOpen: PropTypes.func,
     onRequestClose: PropTypes.func,