Bug 433618 - Limit of keyframes ignored when importing keyframes
Summary: Limit of keyframes ignored when importing keyframes
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: User Interface & Miscellaneous (other bugs)
Version First Reported In: 20.12.2
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-02-25 21:04 UTC by Larsen
Modified: 2021-04-08 20:26 UTC (History)
2 users (show)

See Also:
Latest Commit: https://invent.kde.org/multimedia/kdenlive/commit/24278b9a83f5bd5279faa202d81a6e318066f566
Version Fixed In:
Sentry Crash Report:
fritzibaby: Brainstorm+

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Larsen 2021-02-25 21:04:07 UTC 
STEPS TO REPRODUCE
- Add effect "Auto Mask" to a video clip with ~10 seconds in the timeline
- Analyze
- Copy resulting tracking data
- Add a "Transform" transition to the clip
- Options > "Import keyframes from clipboard"
- Limit to 2
- After the import, check the resulting keyframes in the transition

OBSERVED RESULT
Seems to ignore the limit. Still imports all keyframes.


EXPECTED RESULT
There should be only two keyframes.

SOFTWARE/OS VERSIONS
Windows 10
Comment 1 Julius Künzel 2021-02-26 16:52:36 UTC 
Confirmed on Linux too
Comment 2 Jean-Baptiste Mardelle 2021-04-08 20:26:33 UTC 
Git commit 24278b9a83f5bd5279faa202d81a6e318066f566 by Jean-Baptiste Mardelle.
Committed on 08/04/2021 at 20:26.
Pushed by mardelle into branch 'release/21.04'.

Fix keyframe limit on import from clipboard.

M  +91   -7    src/assets/view/widgets/keyframeimport.cpp

https://invent.kde.org/multimedia/kdenlive/commit/24278b9a83f5bd5279faa202d81a6e318066f566