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

HTML global attribute contextmenu removed #22593

Merged
merged 1 commit into from Mar 12, 2024

Conversation

hamishwillee
Copy link
Collaborator

According to data the contextmenu attribute was removed from last browser more than two years ago, and is therefore a candidate for removal from BCD and the docs.
This removes the entry.

Confirmed by @queengooborg in mdn/content#32448 (comment)

Related to mdn/content#32448

@github-actions github-actions bot added the data:html 📄 Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML label Mar 11, 2024
@queengooborg queengooborg added the needs content update 📝 This PR needs a corresponding update to mdn/content to update the documentation label Mar 11, 2024
Copy link
Collaborator

@queengooborg queengooborg left a comment

Choose a reason for hiding this comment

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

Thanks! Approving but not merging until the content PR has been created and approved.

@hamishwillee
Copy link
Collaborator Author

Approving but not merging until the content PR has been created and approved.

@queengooborg FWIW I use BCD as the trigger for confirming that something can be removed. I don't feel BCD needs to wait on content PRs to be approved, but it is probably good practice to wait on them being created.

@queengooborg queengooborg merged commit 1466423 into mdn:main Mar 12, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:html 📄 Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML needs content update 📝 This PR needs a corresponding update to mdn/content to update the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants