Skip to content

Support
-in- #167

merged 1 commit into from
May 3, 2023

Conversation

annevk
Copy link
Contributor

@annevk annevk commented May 2, 2023

This is a long-standing WebKit feature that regressed as part of the standardized HTML parser effort. This suggests bringing it back with optional semantics, but a mandatory HTML parser change.

The HTML parser change is not expected to be significant for existing content or XSS. When the feature is correctly used it will also not hurt HTML parsers that have not yet incorporated the change. I.e., it should be fully backwards compatible.

Tests: html5lib/html5lib-tests#167.

Fixes #3410.
https://bugs.webkit.org/show_bug.cgi?id=80686
rdar://107656886

Reviewed by Ryosuke Niwa.

Revive support in the HTML parser for 
-in-
flavorjones reacted with thumbs up emoji

@flavorjones
Copy link
Contributor

@annevk Thanks for asking. I have to do some work on the gumbo parser to support #9124. I'll very likely get to it today, but if you'd like to merge this in the meantime I'm happy to submit a PR once I get there.

@annevk
Copy link
Contributor Author

annevk commented May 3, 2023

That works too. I'll merge this then.

@annevk annevk merged commit 55aa183 into html5lib:master May 3, 2023
@annevk annevk deleted the annevk/hr-in-select branch May 3, 2023 12:02
flavorjones added a commit to flavorjones/html5lib-tests that referenced this pull request May 3, 2023
flavorjones added a commit to sparklemotion/nokogiri that referenced this pull request May 3, 2023
Ms2ger pushed a commit that referenced this pull request May 3, 2023
flavorjones added a commit to sparklemotion/nokogiri that referenced this pull request May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants