Bug 433618

Summary:	Limit of keyframes ignored when importing keyframes
Product:	[Applications] kdenlive	Reporter:	Larsen <larsen007>
Component:	User Interface & Miscellaneous	Assignee:	Jean-Baptiste Mardelle <jb>
Status:	RESOLVED FIXED
Severity:	normal	CC:	fritzibaby, julius.kuenzel
Priority:	NOR	Flags:	fritzibaby: Brainstorm+
Version First Reported In:	20.12.2
Target Milestone:	---
Platform:	Microsoft Windows
OS:	Microsoft Windows
Latest Commit:	https://invent.kde.org/multimedia/kdenlive/commit/24278b9a83f5bd5279faa202d81a6e318066f566	Version Fixed/Implemented In:
Sentry Crash Report:

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