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 page for <menuitem> element #24886

Closed
wants to merge 1 commit into from
Closed

Remove page for <menuitem> element #24886

wants to merge 1 commit into from

Conversation

queengooborg
Copy link
Collaborator

This PR removes the pages for the <menuitem> element. Correlates with mdn/browser-compat-data#18983.

@queengooborg queengooborg requested review from a team as code owners February 26, 2023 08:58
@queengooborg queengooborg requested review from wbamberg and schalkneethling and removed request for a team February 26, 2023 08:58
@github-actions github-actions bot added Content:HTML Hypertext Markup Language docs Content:WebAPI Web API docs labels Feb 26, 2023
@github-actions
Copy link
Contributor

Preview URLs

Flaws (7)

URL: /en-US/docs/Web/API/HTMLMenuElement
Title: HTMLMenuElement
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/HTMLMenuElement/HTMLMenuElement does not exist

URL: /en-US/docs/Web/API/HTML_DOM_API
Title: The HTML DOM API
Flaw count: 6

  • macros:
    • /en-US/docs/Web/API/HTMLDirectoryElement does not exist
    • /en-US/docs/Web/API/HTMLFrameElement does not exist
    • /en-US/docs/Web/API/HTMLIsIndexElement does not exist
    • /en-US/docs/Web/API/External does not exist
    • /en-US/docs/Web/API/ApplicationCache does not exist
    • and 1 more flaws omitted

@estelle
Copy link
Member

estelle commented Feb 26, 2023

I don't feel 100% comfortable removing the element page for <menuitem>.

While the element has been deprecated, we generally don't remove deprecated HTML elements, I don't think.
Also, there is a menuitem role in ARIA, along with a checkbox and radio.

Unlike <marquee>, though, this element was only ever supported in FF, and support has been removed.

We sure?

@queengooborg
Copy link
Collaborator Author

We do if no browser supports it within the last two years of releases (per BCD's irrelevance guidelines)! Firefox, the only browser to support this element, dropped support in 85.

@queengooborg
Copy link
Collaborator Author

Closed in favor of #24927.

@queengooborg queengooborg deleted the api/HTMLMenuItemElement branch February 27, 2023 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:HTML Hypertext Markup Language docs Content:WebAPI Web API docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants