From ba3bfe3e31ae06ba2f3db792c0784c3ee58b76dd 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) --- 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 dd16e9b775d87..5885c1f6d0575 100644 --- a/src/renderers/dom/shared/ReactDOMComponent.js +++ b/src/renderers/dom/shared/ReactDOMComponent.js @@ -624,6 +624,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 {