-
Notifications
You must be signed in to change notification settings - Fork 200
Added UI Events KeyboardEvent #301
base: production
Are you sure you want to change the base?
Conversation
For code and an updated key attributes.
I think, rather than add a new entry, we just need to reset (and generalize a bit) the exist entry named "DOM3 Keyboard Events". Can you reformulate your change to update this existing entry? Specifically, updating the name from DOM3 to UI Events, and adding your prose about .key and .code, as well as changing the state back to Under Consideration. |
The chromestatus.com ID is problematic... Can you add support for multiple IDs (how would that even look?)? For my reference, this update keeps the same (ambiguous) ID -
|
Works for me, but maybe @jacobrossi has strong feelings? |
@phistuck sorry this got stale. Both of these specs are shipped in Chrome. So I think we can just link to one of their IDs and it'll be ok. I don't think we'll have too many instances of many:1 relationships between the sites, so it's not really worth designing it to support that. If that's good with you, can you just update this to resolve merge conflicts? |
Only |
Hmm. Alternatively, we could just list Chrome as "In Development" since the feature is only partially supported? I really don't like splitting it into two entries since they're so tightly related and we want to keep the granularity mostly coarse on Status. |
a59a79b
to
654d321
Compare
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.
Reviewed 1 of 1 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @phistuck)
a discussion (no related file):
Long day short nights will do.
For code and an updated key attributes.