Skip to content
This repository has been archived by the owner on Jul 30, 2019. It is now read-only.

remove <menu> and <menuitem> #1043

Merged
merged 2 commits into from
Oct 10, 2017
Merged

remove <menu> and <menuitem> #1043

merged 2 commits into from
Oct 10, 2017

Conversation

siusin
Copy link
Contributor

@siusin siusin commented Oct 10, 2017

Clean up the at-risk features in #893 :

  • remove the<menu>element;
  • remove the <menuitem> element;
  • remove Context menu;
  • remove the contextmenu event;
  • keep menubar which is defined in wai-aria-1.1;

Copy link
Collaborator

@chaals chaals left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the changes to sections/changes.include

Otherwise, Looks good to me, although it's hard to check that everything came out :S

@siusin
Copy link
Contributor Author

siusin commented Oct 10, 2017

Added a "Changes since the 2017-08-08 Candidate Recommendation" section.

Copy link
Collaborator

@chaals chaals left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you - and for fixing stuff that I should have done in my own PRs...

@chaals chaals merged commit 269cf4c into w3c:master Oct 10, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants