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

Add note explicitly about mouse compat events, preventDefault, and passive events #369

Closed
patrickhlauke opened this issue Apr 22, 2021 · 1 comment · Fixed by #403
Closed
Labels

Comments

@patrickhlauke
Copy link
Member

it's implicit at the moment, but would be good to just make explicit

#360 (comment)

@mustaqahmed
Copy link
Member

I would suggest covering only the passive events part here because the rest is covered by #368 (otherwise we risk splitting one discussion in two issues).

patrickhlauke added a commit that referenced this issue Aug 3, 2021
* split the awkward "Also," into a separate paragraph (still inside the note)
* add an additional paragraph in the note about passive event listeners (which ignore `preventDefault` / where the default action can't be prevented) - see #360 (comment)

Closes #369
patrickhlauke added a commit that referenced this issue Aug 18, 2021
…normative (#403)

* Expand note about preventing compatibility mouse events

* split the awkward "Also," into a separate paragraph (still inside the note)
* add an additional paragraph in the note about passive event listeners (which ignore `preventDefault` / where the default action can't be prevented) - see #360 (comment)

Closes #369

* Add explicit [[DOM]] reference

to make clear these are DOM concepts.

* Make non-normative note normative
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants