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

contenteditable missing some allowed values description #22048

Closed
dabuside opened this issue Nov 4, 2022 · 6 comments
Closed

contenteditable missing some allowed values description #22048

dabuside opened this issue Nov 4, 2022 · 6 comments
Labels
Content:HTML Hypertext Markup Language docs effort: medium This task is a medium effort. help wanted If you know something about this topic, we would love your help!

Comments

@dabuside
Copy link
Contributor

dabuside commented Nov 4, 2022

MDN URL

https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/contenteditable

What specific section or headline is this issue about?

main content

What information was incorrect, unhelpful, or incomplete?

The above page says contenteditable has 2 enumerable values true, false.
But the browser compatibility table says it's values can be caret, events, plaintext-only, typing
Also note that page says the values also can be inherit

What did you expect to see?

  1. Add some description about caret, events, plaintext-only, typing values
  2. Correct the allowed values

Do you have any supporting links, references, or citations?

https://w3c.github.io/contentEditable/

Do you have anything more you want to share?

No response

MDN metadata

Page report details
@github-actions github-actions bot added Content:HTML Hypertext Markup Language docs Content:WebAPI Web API docs needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Nov 4, 2022
@sideshowbarker sideshowbarker added help wanted If you know something about this topic, we would love your help! effort: medium This task is a medium effort. and removed Content:WebAPI Web API docs needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Nov 7, 2022
@OnkarRuikar
Copy link
Contributor

As per the latest spec on 7th December 2023 only valid values are "", true, false, and plaintext-only. Implicit value is inherit.
And both the documents cover these.

@dabuside
Copy link
Contributor Author

dabuside commented Dec 7, 2023

As per the latest spec on 7th December 2023 only valid values are "", true, false, and plaintext-only. Implicit value is inherit. And both the documents cover these.

w3c's spec(Editor's draft) didn't match whatwg's spec.
They are conflicted.

@OnkarRuikar
Copy link
Contributor

w3c's spec(Editor's draft) didn't match whatwg's spec. They are conflicted.

@dancer1325
Copy link
Contributor

dancer1325 commented Jan 25, 2024

Could we take advantage of this issue, and create a PR to remove the reference about inherit or specify it better in https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/contentEditable ?

I can create the PR if you agree with.
@OnkarRuikar / @dabuside

@OnkarRuikar
Copy link
Contributor

Could we take advantage of this issue, and create a PR to remove the reference about inherit or specify it better in https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/contentEditable ?

I can create the PR if you agree with. @OnkarRuikar / @dabuside

Yes, we need to update the doc.
As per the specs inherit is not a value but the default state:

The attribute's missing value default and invalid value default are both the inherit state. The inherit state indicates that the element is editable (or not) based on the parent element's state.

@Josh-Cena
Copy link
Member

Fixed by #31935

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 effort: medium This task is a medium effort. help wanted If you know something about this topic, we would love your help!
Projects
None yet
Development

No branches or pull requests

5 participants