OSSForks/ladybird
1
0
Fork 0
mirror of https://github.com/zebrajr/ladybird.git synced 2025-12-06 12:20:00 +01:00
Code Issues Actions 3 Packages Projects Releases Wiki Activity

LibWeb/CSS: Treat *|* selector like * when serializing

Browse Source 
1 new WPT pass.
This commit is contained in:
Sam Atkins 2025-05-16 13:41:12 +01:00 committed by Andreas Kling
parent eb98bd1a36
commit 26d71207d4
2 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
Libraries/LibWeb/CSS/Selector.cpp
View File

@ -501,7 +501,8 @@ String Selector::serialize() const
for (auto& simple_selector : compound_selector.simple_selectors) {
if (simple_selector.type == SimpleSelector::Type::Universal) {
auto qualified_name = simple_selector.qualified_name();
if (qualified_name.namespace_type == SimpleSelector::QualifiedName::NamespaceType::Default)
if (qualified_name.namespace_type == SimpleSelector::QualifiedName::NamespaceType::Default
|| qualified_name.namespace_type == SimpleSelector::QualifiedName::NamespaceType::Any)
continue;
// FIXME: I *think* if we have a namespace prefix that happens to equal the same as the default namespace,
// we also should skip it. But we don't have access to that here. eg:

6
Tests/LibWeb/Text/expected/wpt-import/css/selectors/parsing/parse-not.txt
View File

@ -2,12 +2,12 @@ Harness status: OK
Found 26 tests
23 Pass
3 Fail
24 Pass
2 Fail
Pass "button:not([disabled])" should be a valid selector
Pass "*:not(foo)" should be a valid selector
Pass ":not(:link):not(:visited)" should be a valid selector
Fail "*|*:not(*)" should be a valid selector
Pass "*|*:not(*)" should be a valid selector
Pass ":not(:hover)" should be a valid selector
Pass ":not(*|*)" should be a valid selector
Pass "foo:not(bar)" should be a valid selector