beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-09-29 08:11:13 +00:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

LibWeb: Handle the cssFloat and cssOffset keyframe properties properly

Browse source
This commit is contained in:
Matthew Olsson 2024-06-02 12:27:48 -07:00 committed by Andreas Kling
parent 36ceaf38d5
commit 44bb6e8390
Notes: sideshowbarker 2024-07-18 02:47:59 +09:00 
Author: https://github.com/mattco98
Commit: https://github.com/SerenityOS/serenity/commit/44bb6e8390
Pull-request: https://github.com/SerenityOS/serenity/pull/24530
1 changed files with 33 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

45
Userland/Libraries/LibWeb/Animations/KeyframeEffect.cpp
View file

@ -159,9 +159,16 @@ static WebIDL::ExceptionOr<KeyframeType<AL>> process_a_keyframe_like_object(JS::
if (CSS::is_animatable_property(property)) if (CSS::is_animatable_property(property))
animation_properties.append(String { CSS::string_from_property_id(property) }); animation_properties.append(String { CSS::string_from_property_id(property) });
} }
} else if (auto property = CSS::property_id_from_camel_case_string(name); property.has_value()) { } else {
if (CSS::is_animatable_property(property.value())) // Handle the two special cases
if (name == "cssFloat"sv || name == "cssOffset"sv) {
animation_properties.append(name); animation_properties.append(name);
} else if (name == "float"sv || name == "offset"sv) {
// Ignore these property names
} else if (auto property = CSS::property_id_from_camel_case_string(name); property.has_value()) {
if (CSS::is_animatable_property(property.value()))
animation_properties.append(name);
}
} }
} }
@ -516,17 +523,31 @@ static WebIDL::ExceptionOr<Vector<BaseKeyframe>> process_a_keyframes_argument(JS
// highlight // highlight
BaseKeyframe::ParsedProperties parsed_properties; BaseKeyframe::ParsedProperties parsed_properties;
for (auto& [property_string, value_string] : keyframe.unparsed_properties()) { for (auto& [property_string, value_string] : keyframe.unparsed_properties()) {
if (auto property = CSS::property_id_from_camel_case_string(property_string); property.has_value()) { Optional<CSS::PropertyID> property_id;
auto maybe_parser = CSS::Parser::Parser::create(CSS::Parser::ParsingContext(realm), value_string);
if (maybe_parser.is_error())
continue;
if (auto style_value = maybe_parser.release_value().parse_as_css_value(*property)) { // Handle some special cases
// Handle 'initial' here so we don't have to get the default value of the property every frame in StyleComputer if (property_string == "cssFloat"sv) {
if (style_value->is_initial()) property_id = CSS::PropertyID::Float;
style_value = CSS::property_initial_value(realm, *property); } else if (property_string == "cssOffset"sv) {
parsed_properties.set(*property, *style_value); // FIXME: Support CSS offset property
} } else if (property_string == "float"sv || property_string == "offset"sv) {
// Ignore these properties
} else if (auto property = CSS::property_id_from_camel_case_string(property_string); property.has_value()) {
property_id = *property;
}
if (!property_id.has_value())
continue;
auto maybe_parser = CSS::Parser::Parser::create(CSS::Parser::ParsingContext(realm), value_string);
if (maybe_parser.is_error())
continue;
if (auto style_value = maybe_parser.release_value().parse_as_css_value(*property_id)) {
// Handle 'initial' here so we don't have to get the default value of the property every frame in StyleComputer
if (style_value->is_initial())
style_value = CSS::property_initial_value(realm, *property_id);
parsed_properties.set(*property_id, *style_value);
} }
} }
keyframe.properties.set(move(parsed_properties)); keyframe.properties.set(move(parsed_properties));