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

What are the event.type strings? #245

Closed
martijnthe opened this issue Jul 26, 2017 · 4 comments
Closed

What are the event.type strings? #245

martijnthe opened this issue Jul 26, 2017 · 4 comments
Assignees
Milestone

Comments

@martijnthe
Copy link

I looking at the test suite in the hope of finding out answers but, I don't see sensor.addEventListener(...) being used there...

There was some discussion here, but it's unclear to me from the spec doc what various the Event's type strings are supposed to be. In case one wants to use the EventTarget interface instead of the onreading, etc. properties this is quite important 😄 It would be nice if this could be documented more clearly and also be added to the test cases.

cc @jiangzidong

@alexshalamov
Copy link

@martijnthe yes, spec is missing section that describes event handlers. I will take a look at it.

@jiangzidong
Copy link

Does that means there is no sensor.onreading property in the future? only sensor.addEventListener(...)

@alexshalamov
Copy link

alexshalamov commented Jul 27, 2017

@jiangzidong Specification always had both, EventHandler properties as well as EventTarget interface with corresponding eventtypes.

Edit: There are no plans to remove EventHandler properties.

@rwaldron
Copy link
Contributor

yes, spec is missing section that describes event handlers.

Right, basically this hasn't been done because there was still enough churn that it made some sense to let the naming of events settle before doing all the work to write up the spec prose.

alexshalamov pushed a commit to alexshalamov/sensors that referenced this issue Aug 3, 2017
alexshalamov pushed a commit to alexshalamov/sensors that referenced this issue Aug 4, 2017
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

4 participants