[fixed] ensure last focused item can be focused

fixes react-bootstrap#917
kyletsang · Jul 24, 2015 · e5155c6 · e5155c6
1 parent 596f40c
commit e5155c6
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/src/Modal.js b/src/Modal.js
@@ -360,15 +360,14 @@ const Modal = React.createClass({
     let current = domUtils.activeElement(this);
     let focusInModal = current && domUtils.contains(modalContent, current);
 
-    if (this.props.autoFocus && !focusInModal) {
+    if (modalContent && this.props.autoFocus && !focusInModal) {
       this.lastFocus = current;
-
       modalContent.focus();
     }
   },
 
   restoreLastFocus () {
-    if (this.lastFocus) {
+    if (this.lastFocus && this.lastFocus.focus) {
       this.lastFocus.focus();
       this.lastFocus = null;
     }
@@ -382,7 +381,7 @@ const Modal = React.createClass({
     let active = domUtils.activeElement(this);
     let modal = React.findDOMNode(this.refs.dialog);
 
-    if (modal !== active && !domUtils.contains(modal, active)){
+    if (modal && modal !== active && !domUtils.contains(modal, active)){
       modal.focus();
     }
   },