beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-22 15:40:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

LibWeb: Add hack for :host in pseudo-class serialization

Browse source 
The spec for this algorithm is quite outdated, we already have some
other pseudo-classes in here which the spec doesn't cover.
This commit is contained in:
Sam Atkins 2023-08-12 13:58:19 +01:00 committed by Andreas Kling
parent a31a0934b8
commit 337754ae64
Notes: sideshowbarker 2024-07-17 22:09:47 +09:00 
Author: https://github.com/AtkinsSJ
Commit: https://github.com/SerenityOS/serenity/commit/337754ae64
Pull-request: https://github.com/SerenityOS/serenity/pull/20519
1 changed files with 15 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
Userland/Libraries/LibWeb/CSS/Selector.cpp
View file

@ -229,9 +229,22 @@ ErrorOr<String> Selector::SimpleSelector::serialize() const
auto& pseudo_class = this->pseudo_class();
auto metadata = pseudo_class_metadata(pseudo_class.type);
// HACK: `:host()` has both a function and a non-function form, so handle that first.
// It's also not in the spec.
if (pseudo_class.type == PseudoClass::Host) {
if (pseudo_class.argument_selector_list.is_empty()) {
TRY(s.try_append(':'));
TRY(s.try_append(pseudo_class_name(pseudo_class.type)));
} else {
TRY(s.try_append(':'));
TRY(s.try_append(pseudo_class_name(pseudo_class.type)));
TRY(s.try_append('('));
TRY(s.try_append(TRY(serialize_a_group_of_selectors(pseudo_class.argument_selector_list))));
TRY(s.try_append(')'));
}
}
// If the pseudo-class does not accept arguments append ":" (U+003A), followed by the name of the pseudo-class, to s.
if (metadata.is_valid_as_identifier) {
else if (metadata.is_valid_as_identifier) {
TRY(s.try_append(':'));
TRY(s.try_append(pseudo_class_name(pseudo_class.type)));
}