system/serenity
system/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity synced 2024-07-22 10:36:24 +00:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

LibWeb: Copy the keyframes in KeyframeEffect's copy constructor

Browse source
This commit is contained in:
Matthew Olsson 2024-06-02 10:09:30 -07:00 committed by Andreas Kling
parent 5b84bd6e45
commit b5c682bc2e
3 changed files with 15 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
Tests/LibWeb/Text/expected/WebAnimations/misc/keyframe-effect-cloning.txt Normal file
View file

@ -0,0 +1,2 @@
anim count: 1
keyframe: {"offset":null,"computedOffset":1,"easing":"linear","composite":"auto","marginLeft":"10px"}

12
Tests/LibWeb/Text/input/WebAnimations/misc/keyframe-effect-cloning.html Normal file
View file

@ -0,0 +1,12 @@
<!DOCTYPE html>
<script src="../../include.js"></script>
<script>
test(() => {
let effect = new KeyframeEffect(null, [{ marginLeft: "10px" }]);
let copiedEffect = new KeyframeEffect(effect);
let copiedKeyframes = copiedEffect.getKeyframes();
println(`anim count: ${copiedKeyframes.length}`);
if (copiedKeyframes.length > 0)
println(`keyframe: ${JSON.stringify(copiedKeyframes[0])}`);
})
</script>

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

@ -706,8 +706,8 @@ WebIDL::ExceptionOr<JS::NonnullGCPtr<KeyframeEffect>> KeyframeEffect::construct_
// - effect target,
effect->m_target_element = source->target();
// FIXME:
// - keyframes,
effect->m_keyframes = source->m_keyframes;
// - composite operation, and
effect->set_composite(source->composite());