Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

addEventHandler should accept "null" to remove event handlers as per W3C specs #177

Open
Apollon77 opened this issue Mar 22, 2024 · 2 comments

Comments

@Apollon77
Copy link
Contributor

see https://html.spec.whatwg.org/multipage/webappapis.html#event-handler-attributes

a listener, which is either null or an event listener responsible for running the event handler processing algorithm. Initially, an event handler's listener must be set to null.

I can provide a PR once #56 is in if wanted

PS: I curently work on https://github.com/project-chip/matter.js and want to get a web-bluetooth compatible BLE handler for Matter commissioning, so that developers can choose if they use it in a browser or node.js :-)

@Apollon77
Copy link
Contributor Author

@thegecko i can also do a PR then for the types package to get tthat in sync because it is missing there too

@thegecko
Copy link
Owner

Yes please

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants