beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-26 09:30:24 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

LibWeb: Match class selectors case insensitively in quirks mode

Browse source 
This align our implementation with the CSSWG Selectors Level 4
specification.
This commit is contained in:
Tim Ledbetter 2024-08-02 22:21:12 +01:00 committed by Andreas Kling
parent e2c9e34050
commit 82ed253477
Notes: github-actions[bot] 2024-08-03 14:46:10 +00:00 
Author: https://github.com/tcl3
Commit: https://github.com/LadybirdBrowser/ladybird/commit/82ed2534776
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/937
3 changed files with 22 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
Tests/LibWeb/Text/expected/quirks-mode-case-insensitive-class-selector.txt Normal file
View file

@ -0,0 +1 @@
ParentNode.querySelector matches class selectors case-insensitively in quirks mode: true

9
Tests/LibWeb/Text/input/quirks-mode-case-insensitive-class-selector.html Normal file
View file

@ -0,0 +1,9 @@
<!-- Quirks mode -->
<script src="include.js"></script>
<div class="tEsT"></div>
<script>
test(() => {
let divElement = document.querySelector(".test");
println(`ParentNode.querySelector matches class selectors case-insensitively in quirks mode: ${divElement instanceof Element}`);
});
</script>

16
Userland/Libraries/LibWeb/CSS/SelectorEngine.cpp
View file

@ -702,8 +702,12 @@ static inline bool matches(CSS::Selector::SimpleSelector const& component, Optio
}
case CSS::Selector::SimpleSelector::Type::Id:
return component.name() == element.id();
case CSS::Selector::SimpleSelector::Type::Class:
return element.has_class(component.name());
case CSS::Selector::SimpleSelector::Type::Class: {
// Class selectors are matched case insensitively in quirks mode.
// See: https://drafts.csswg.org/selectors-4/#class-html
auto case_sensitivity = element.document().in_quirks_mode() ? CaseSensitivity::CaseInsensitive : CaseSensitivity::CaseSensitive;
return element.has_class(component.name(), case_sensitivity);
}
case CSS::Selector::SimpleSelector::Type::Attribute:
return matches_attribute(component.attribute(), style_sheet_for_rule, element);
case CSS::Selector::SimpleSelector::Type::PseudoClass:
@ -788,8 +792,12 @@ static bool fast_matches_simple_selector(CSS::Selector::SimpleSelector const& si
return false;
}
return matches_namespace(simple_selector.qualified_name(), element, style_sheet_for_rule);
case CSS::Selector::SimpleSelector::Type::Class:
return element.has_class(simple_selector.name());
case CSS::Selector::SimpleSelector::Type::Class: {
// Class selectors are matched case insensitively in quirks mode.
// See: https://drafts.csswg.org/selectors-4/#class-html
auto case_sensitivity = element.document().in_quirks_mode() ? CaseSensitivity::CaseInsensitive : CaseSensitivity::CaseSensitive;
return element.has_class(simple_selector.name(), case_sensitivity);
}
case CSS::Selector::SimpleSelector::Type::Id:
return simple_selector.name() == element.id();
case CSS::Selector::SimpleSelector::Type::Attribute: