Remove <menuitem> and context menu specs #2730

Closed
domenic opened this Issue Jun 1, 2017 · 5 comments

Comments

4 participants
@domenic
Member

domenic commented Jun 1, 2017

Previously we had Firefox implementing and Chrome with an in-progress implementation, which was enough to keep it in the spec. But Chrome has just WONTFIXed their bug for context menus, and removed the related code: https://bugs.chromium.org/p/chromium/issues/detail?id=87553&desc=2#c66

Unless there is strong interest from Safari (@rniwa? @cdumez?) or Edge (@travisleithead?) for implementing these, we should remove them from the spec, as they do not meet the bar for features to be included in HTML.

Sincere apologies to Firefox for taking a risk that didn't work out. As detailed in https://whatwg.org/working-mode, this kind of feature without multi-implementer support should not make its way into the spec in the future.

@cdumez

This comment has been minimized.

Show comment
Hide comment
@cdumez

cdumez Jun 1, 2017

Contributor

I am not aware of interest on WebKit side to implement menu item.

Contributor

cdumez commented Jun 1, 2017

I am not aware of interest on WebKit side to implement menu item.

@domenic domenic self-assigned this Jun 6, 2017

domenic added a commit that referenced this issue Jun 6, 2017

Remove the context menu feature
This feature sadly never garnered enough implementation interest, being
only implemented in Gecko. For some time it looked as though Blink was
also interested in implementing, but they recently made the decision to
cease implementation and remove all of their code for it, per
https://bugs.chromium.org/p/chromium/issues/detail?id=87553&desc=2#c64.

Closes #2730.
@zcorpan

This comment has been minimized.

Show comment
Hide comment
@zcorpan

zcorpan Jun 8, 2017

Member

I added a comment in https://bugzilla.mozilla.org/show_bug.cgi?id=617528 to notify relevant Mozilla people about this.

Member

zcorpan commented Jun 8, 2017

I added a comment in https://bugzilla.mozilla.org/show_bug.cgi?id=617528 to notify relevant Mozilla people about this.

@domenic domenic closed this in #2742 Jun 8, 2017

domenic added a commit that referenced this issue Jun 8, 2017

Remove the context menu feature
This feature sadly never garnered enough implementation interest, being
only implemented in Gecko. For some time it looked as though Blink was
also interested in implementing, but they recently made the decision to
cease implementation and remove all of their code for it, per
https://bugs.chromium.org/p/chromium/issues/detail?id=87553&desc=2#c64.

Closes #2730.

@prlbr prlbr referenced this issue in w3c/html Jun 12, 2017

Closed

Identify sections at risk for 5.2 #893

@bzbarsky

This comment has been minimized.

Show comment
Hide comment
@bzbarsky

bzbarsky Jun 28, 2017

Collaborator

I added a comment in https://bugzilla.mozilla.org/show_bug.cgi?id=617528

For what it's worth, comments in long-resolved bugs don't really notify anyone. Filing new bugs on spec changes is much preferable...

Was there an issue reported against https://github.com/inikulin/parse5 too?

Collaborator

bzbarsky commented Jun 28, 2017

I added a comment in https://bugzilla.mozilla.org/show_bug.cgi?id=617528

For what it's worth, comments in long-resolved bugs don't really notify anyone. Filing new bugs on spec changes is much preferable...

Was there an issue reported against https://github.com/inikulin/parse5 too?

@domenic

This comment has been minimized.

Show comment
Hide comment
@domenic

domenic Jun 28, 2017

Member

I filed the bugs at #2742 (comment), including a separate one for Mozilla.

Good call on also notifying parse5.

Member

domenic commented Jun 28, 2017

I filed the bugs at #2742 (comment), including a separate one for Mozilla.

Good call on also notifying parse5.

@bzbarsky

This comment has been minimized.

Show comment
Hide comment
@bzbarsky

bzbarsky Jun 28, 2017

Collaborator

Ah, I missed that bug report in my search. Thanks!

Collaborator

bzbarsky commented Jun 28, 2017

Ah, I missed that bug report in my search. Thanks!

@domenic domenic referenced this issue in inikulin/parse5 Jun 28, 2017

Closed

Remove menuitem special-casing #204

@cpplearner cpplearner referenced this issue in Fyrd/caniuse Aug 26, 2017

Closed

<menuitem> element #3687

ExE-Boss added a commit to ExE-Boss/about-button that referenced this issue Aug 30, 2017

bbrala added a commit to swisnl/jQuery-contextMenu that referenced this issue Dec 30, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment