Merge pull request #81 from sandersn/add-EventListener-overloads-from…

…-HtmlElement Add add/removeEventListener overloads from HTMLElement
electron · Nov 4, 2017 · f304ea6 · f304ea6
2 parents 483eb2f + 087a0e7
commit f304ea6
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/lib/module-declaration.js b/lib/module-declaration.js
@@ -105,6 +105,12 @@ const generateModuleDeclaration = (module, index, API) => {
         moduleAPI.push(`${method}(event: '${domEvent.name}', listener: (event: ${eventType}) => void${method === 'addEventListener' ? ', useCapture?: boolean' : ''}): this;`)
       }
     })
+
+    // original overloads copied from HTMLElement, because they are not inherited
+    moduleAPI.push(`addEventListener<K extends keyof HTMLElementEventMap>(type: K, listener: (this: HTMLElement, ev: HTMLElementEventMap[K]) => any, useCapture?: boolean): void;`)
+    moduleAPI.push(`addEventListener(type: string, listener: EventListenerOrEventListenerObject, useCapture?: boolean): void;`)
+    moduleAPI.push(`removeEventListener<K extends keyof HTMLElementEventMap>(type: K, listener: (this: HTMLElement, ev: HTMLElementEventMap[K]) => any, useCapture?: boolean): void;`)
+    moduleAPI.push(`removeEventListener(type: string, listener: EventListenerOrEventListenerObject, useCapture?: boolean): void;`)
   }
 
   const returnsThis = (moduleMethod) => ['on', 'once', 'removeAllListeners', 'removeListener'].includes(moduleMethod.name)