From a5d7fc84eeb779568c86d355c344341a4cbeddbb Mon Sep 17 00:00:00 2001 From: Jim Date: Fri, 27 May 2016 09:19:25 -0700 Subject: [PATCH] Followup to 6896, add explanation of workaround. (#6905) (cherry picked from commit ba3bfe3e31ae06ba2f3db792c0784c3ee58b76dd) --- src/renderers/dom/shared/ReactDOMComponent.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/renderers/dom/shared/ReactDOMComponent.js b/src/renderers/dom/shared/ReactDOMComponent.js index aaab8467c9..344d368963 100644 --- a/src/renderers/dom/shared/ReactDOMComponent.js +++ b/src/renderers/dom/shared/ReactDOMComponent.js @@ -617,6 +617,9 @@ ReactDOMComponent.Mixin = { } else if (props.is) { el = ownerDocument.createElement(this._currentElement.type, props.is); } else { + // Separate else branch instead of using `props.is || undefined` above becuase of a Firefox bug. + // See discussion in https://github.com/facebook/react/pull/6896 + // and discussion in https://bugzilla.mozilla.org/show_bug.cgi?id=1276240 el = ownerDocument.createElement(this._currentElement.type); } } else {