-
Notifications
You must be signed in to change notification settings - Fork 161
Add InputEvent
, StaticRange
, AbstractRange
#695
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
Conversation
After we land this one I'm hoping we can cut a v2.2.0, now that we've accumulated a handful of contributor PRs :) |
Polite ping @japgolly :) if you don't have time to review for a while, no big deal, just let me know and I won't block v2.2.0 on this if that's okay with you! |
Oh my! Apologies @armanbilge ! I haven't been keeping up with my email lately so I missed this. I'll get to this very son |
No worries, whenever you get to it! I hope you don't mind I released v2.2.0, it was really wonderful to get a bunch of contributions and I wanted to get it out :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for the follow-up PR and your patience 😄
Just wanted to add
InputEvent
and it sort of snowballed :)https://www.w3.org/TR/input-events-2/#interface-InputEvent-AttributesMDN links to this, which is different (?)
https://rawgit.com/w3c/input-events/v1/index.html#interface-InputEvent-Attributes