Use addEventListener instead of attachEvent in IE9

facebook · Jun 4, 2018 · 7c3d3ac · 7c3d3ac
1 parent fcc3c89
commit 7c3d3ac
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/react-dom/src/events/ChangeEventPlugin.js b/packages/react-dom/src/events/ChangeEventPlugin.js
@@ -135,7 +135,7 @@ if (ExecutionEnvironment.canUseDOM) {
 function startWatchingForValueChange(target, targetInst) {
   activeElement = target;
   activeElementInst = targetInst;
-  activeElement.attachEvent('onpropertychange', handlePropertyChange);
+  activeElement.addEventListener('onpropertychange', handlePropertyChange);
 }
 
 /**
@@ -146,7 +146,7 @@ function stopWatchingForValueChange() {
   if (!activeElement) {
     return;
   }
-  activeElement.detachEvent('onpropertychange', handlePropertyChange);
+  activeElement.removeEventListener('onpropertychange', handlePropertyChange);
   activeElement = null;
   activeElementInst = null;
 }

diff --git a/packages/react-dom/src/events/getEventTarget.js b/packages/react-dom/src/events/getEventTarget.js
@@ -17,7 +17,7 @@ import {TEXT_NODE} from '../shared/HTMLNodeType';
 function getEventTarget(nativeEvent) {
   // Fallback to nativeEvent.srcElement for IE9
   // https://github.com/facebook/react/issues/12506
-  let target = nativeEvent.target || nativeEvent.srcElement || window;
+  let target = nativeEvent.target || window;
 
   // Normalize SVG <use> element events #4963
   if (target.correspondingUseElement) {