Bug 356209 - Copying Effects messes up parameters (Fade from black + Fade in)
Summary: Copying Effects messes up parameters (Fade from black + Fade in)
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: Video Effects & Transitions (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Vincent PINON
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-12-02 19:55 UTC by qubodup
Modified: 2015-12-14 21:42 UTC (History)
0 users

See Also:
Latest Commit: http://commits.kde.org/kdenlive/d9b115f306102af528581ff3071c3c4e91a77372
Version Fixed In: 15.12.0
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description qubodup 2015-12-02 19:55:13 UTC 
Steps to reproduce:
1. Start Kdenlive
2. Add video clip (with audio stream)
3. Place clip twice in timeline
4. Add Fade from black and Fade in effects to first clip in timeline in this order
5. Change effect duration parameter values to 10 and 15 frames
6. Copy the clip that has the effects
7. Paste the effects to the second clip

Expected result:
1. Second clip has identical effect parameters

Actual result:
1. Second clip's Fade in effect has 10 frames duration instead of 15.

Reproduceable always.

Video: https://youtu.be/b4VYyhZ5Pkw
Comment 1 Jean-Baptiste Mardelle 2015-12-06 13:38:43 UTC 
Git commit d9b115f306102af528581ff3071c3c4e91a77372 by Jean-Baptiste Mardelle.
Committed on 06/12/2015 at 13:38.
Pushed by mardelle into branch 'Applications/15.12'.

Fix fades sometimes incorrectly displayed / messed up when 2 fades were added to a clip

M  +71   -45   src/timeline/clipitem.cpp
M  +110  -91   src/timeline/customtrackview.cpp

http://commits.kde.org/kdenlive/d9b115f306102af528581ff3071c3c4e91a77372
Comment 2 qubodup 2015-12-06 23:25:23 UTC 
I can confirm that this appears to be working now:
https://www.youtube.com/watch?v=Qi8DA3OHta4

Thanks!