fix(Modal): add workaround for Modal auto focus

TheSharpieOne · TheSharpieOne · commit f12057d702b9 · 2017-08-16T16:29:18.000-04:00
Check if the dialog has a parent which can receive focus before trying to focus it.
This workaround will allow react 16 to use Modal without forcing autoFocus={false}
diff --git a/src/Modal.js b/src/Modal.js
@@ -228,7 +228,9 @@ class Modal extends React.Component {
 
     // check if modal should receive focus
     if (this._focus) {
-      this._dialog.parentNode.focus();
+      if (this._dialog && this._dialog.parentNode && typeof this._dialog.parentNode.focus === 'function') {
+        this._dialog.parentNode.focus();
+      }
       this._focus = false;
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -228,7 +228,9 @@ class Modal extends React.Component {`
`228`	`228`
`229`	`229`	`// check if modal should receive focus`
`230`	`230`	`if (this._focus) {`
`231`		`- this._dialog.parentNode.focus();`
	`231`	`+ if (this._dialog && this._dialog.parentNode && typeof this._dialog.parentNode.focus === 'function') {`
	`232`	`+ this._dialog.parentNode.focus();`
	`233`	`+ }`
`232`	`234`	`this._focus = false;`
`233`	`235`	`}`
`234`	`236`	`}`