make target required

stipsan · Nov 22, 2016 · bcb36d0 · bcb36d0
1 parent 4b9b96b
commit bcb36d0
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -10,7 +10,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 * `Notify` component was not passing all the props.
 
 ### Breaking Changes
-* `Modal` component needs to pass `target` as Element instead of Node
+* `Modal` component needs to pass `target` as Element instead of Node. `target` is required. 
 
 ### Coverage
 * uikit-react has 100% test coverage now.

diff --git a/src/Modal/index.js b/src/Modal/index.js
@@ -9,11 +9,11 @@ import Prompt from './Prompt'
 export default class Modal extends Component {
 
   static propTypes = {
+    target: PropTypes.element.isRequired,
     cancelLabel: PropTypes.string,
     children: PropTypes.node,
     confirmLabel: PropTypes.string,
     isOpen: PropTypes.bool,
-    target: PropTypes.element,
     type: PropTypes.string,
     onCancel: PropTypes.func,
     onConfirm: PropTypes.func,