jpike88 opened a new issue #244:
URL: https://github.com/apache/cordova-js/issues/244


   The ```document.addEventListener```, ```document.removeEventListener```, 
```window.addEventListener```, ```window.removeEventListener``` overloads in 
```cordova.js``` error if the ```evt``` variable is undefined. While this is 
something that also be fixed upstream in the relevant calling code, these 
functions should not be erroring as that is not the behaviour of what they are 
overloading.
   
   ```
   window.addEventListener(undefined, () => {})
   ```
   
   In cordova.js, the above would cause an error.
   In safari/chrome console, no error is triggered.
   
   I have create a PR to address this.
   
   Related:
   https://github.com/ionic-team/capacitor/issues/4178
   https://github.com/mapbox/mapbox-gl-js/issues/11433
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to