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.

Sign up for GitHub

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

Jump to bottom

Relax <select> parser #10557

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
+130 −327
Open

Relax <select> parser #10557

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
1330ad5
Relax <select> parser
josepharhar Jul 15, 2024
b87d93d
make rules more permissive
josepharhar Sep 18, 2024
9c72f6c
dont allow hr in option
josepharhar Sep 20, 2024
e07cb01
re-add out of select cases
josepharhar Oct 2, 2024
5117d13
re-add reconstruct formatting elements
josepharhar Oct 3, 2024
33370b7
fix non-select case
josepharhar Oct 9, 2024
76e0d7f
fix variable names
josepharhar Oct 9, 2024
58b9ca9
re-add reconstruct active formatting elements
josepharhar Oct 10, 2024
b8ba8e6
remove pop optgroup
josepharhar Oct 10, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 10 additions & 2 deletions source
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -129408,8 +129408,14 @@ document.body.appendChild(text);
<code>option</code> element in scope</span>, then this is a <span>parse error</span>.</p></li>
</ol>

<p>Otherwise, if the <span>current node</span> is an <code>option</code> element, then pop the
<span>current node</span> off the <span>stack of open elements</span>.</p>
<p>Otherwise:</p>

<ol>
<li><p>If the <span>current node</span> is an <code>option</code> element, then pop the
<span>current node</span> off the <span>stack of open elements</span>.</p></li>

<li><p><span>Reconstruct the active formatting elements</span>, if any.</p></li>
</ol>

<p><span>Insert an HTML element</span> for the token.</p>
</dd>
Expand Down Expand Up @@ -129438,6 +129444,8 @@ document.body.appendChild(text);

<li><p>If the <span>current node</span> is an <code>optgroup</code> element, then pop that node
from the <span>stack of open elements</span>.</p></li>
zcorpan marked this conversation as resolved.
Show resolved Hide resolved

<li><p><span>Reconstruct the active formatting elements</span>, if any.</p></li>
</ol>

<p><span>Insert an HTML element</span> for the token.</p>
Expand Down