fix(compat): prefer {add,remove}EventListener over legacy fns

Author: STRML

lib/utils/domFns.js

@@ -39,24 +39,26 @@ export function matchesSelectorAndParentsTo(el: Node, selector: string, baseNode
   return false;
 }
 
-export function addEvent(el: ?Node, event: string, handler: Function): void {
-  if (!el) { return; }
-  if (el.attachEvent) {
+export function addEvent(el: ?Node, event: string, handler: Function, inputOptions?: Object): void {
+  if (!el) return;
+  const options = {capture: true, ...inputOptions};
+  if (el.addEventListener) {
+    el.addEventListener(event, handler, options);
+  } else if (el.attachEvent) {
     el.attachEvent('on' + event, handler);
-  } else if (el.addEventListener) {
-    el.addEventListener(event, handler, true);
   } else {
     // $FlowIgnore: Doesn't think elements are indexable
     el['on' + event] = handler;
   }
 }
 
-export function removeEvent(el: ?Node, event: string, handler: Function): void {
-  if (!el) { return; }
-  if (el.detachEvent) {
+export function removeEvent(el: ?Node, event: string, handler: Function, inputOptions?: Object): void {
+  if (!el) return;
+  const options = {capture: true, ...inputOptions};
+  if (el.removeEventListener) {
+    el.removeEventListener(event, handler, options);
+  } else if (el.detachEvent) {
     el.detachEvent('on' + event, handler);
-  } else if (el.removeEventListener) {
-    el.removeEventListener(event, handler, true);
   } else {
     // $FlowIgnore: Doesn't think elements are indexable
     el['on' + event] = null;