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

Remove/update menuitems and menu #188

Closed
4 tasks done
scottaohara opened this issue May 21, 2019 · 3 comments · Fixed by #214
Closed
4 tasks done

Remove/update menuitems and menu #188

scottaohara opened this issue May 21, 2019 · 3 comments · Fixed by #214

Comments

@scottaohara
Copy link
Member

scottaohara commented May 21, 2019

remove menuitems as they are not part of the HTML living standard:

  • menuitem type=command
  • menuitem type=checkbox
  • menuitem type=radio

menu is part of the HTML living standard, so this should stay in AAM, but it does not accept a type attribute.

  • revise menu to remove reference to type
@scottaohara scottaohara changed the title Remove the following HTML elements Remove/update menuitems and menu May 21, 2019
@scottaohara
Copy link
Member Author

Initial work done to remove menuitem types from the spec. Still need to review menu

@scottaohara
Copy link
Member Author

scottaohara commented Jun 17, 2019

Note: <menu> should map to role="list" per the Living Standard definition.

Per the note in the menu definition:

The menu element is simply a semantic alternative to ul to express an unordered list of commands (a "toolbar").

@scottaohara
Copy link
Member Author

scottaohara commented Sep 7, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant