Update according to spec #29827
Update according to spec #29827
Conversation
see https://html.spec.whatwg.org/#the-select-element Content model: Zero or more option, optgroup, hr, and script-supporting elements.
danield770
requested review from
chrisdavidmills
and removed request for
a team
| {{HTMLElement("optgroup")}} elements, {{HTMLElement("hr")}}, and <a href="https://html.spec.whatwg.org/#script-supporting-elements-2" | ||
| >script-supporting</a | ||
| > elements |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
There was a problem hiding this comment.
This I think won't work because it's inside a <table>.
There was a problem hiding this comment.
This I think won't work because it's inside a
<table>.
yes, my mistake
|
Most non-empty elements support script supporting elements (script and template), so we should either include it on all the elements that do (body, ol, li, hgroup, h1-h6, etc), or create a page for script supporting elements similar to global attributes (which are global to all element, not just "not empty" elements), and then refer to it as we refer to the page that explains script and template being able to be nested almost anywhere. |
|
(I'm commenting here because I'm tagged for review, but @estelle if you want to take it from me I'm happy for you to do that too.)
From the spec it looks like the following are explicitly listed as including script supporting elements in their content model: hgroup (this does not include elements that include them indirectly in "phrasing content"). Given that this isn't a massive list, I'd just vote to include it on these pages. |
| @@ -581,7 +581,9 @@ document.forms[0].onsubmit = (e) => { | |||
| <th scope="row">Permitted content</th> | |||
| <td> | |||
| Zero or more {{HTMLElement("option")}} or | |||
| {{HTMLElement("optgroup")}} elements. | |||
| {{HTMLElement("optgroup")}} elements, {{HTMLElement("hr")}}, and <a href="https://html.spec.whatwg.org/#script-supporting-elements-2" | |||
There was a problem hiding this comment.
hr-in-select is part of #26536. We would preferably have compatibility data, but I guess Firefox is the only browser that doesn't support this now, so it can just go through the typical Firefox release process.
|
Preview URLs External URLs (1)URL:
|
|
This pull request has merge conflicts that must be resolved before it can be merged. |
|
Thank you for your contribution. The hr in select was added, along with an example, inhttps://github.com//pull/31169/. It's live: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/select#technical_summary. I'll close this PR. Thank you though. |
see https://html.spec.whatwg.org/#the-select-element Content model:
Zero or more option, optgroup, hr, and script-supporting elements.
Description
Motivation
Additional details
Related issues and pull requests