Bug 364585

Summary: git master: after moving a set of clips using the spacer tool clips cannot be moved anymore
Product: [Applications] kdenlive Reporter: Wegwerf <wegwerf-1-2-3>
Component: User Interface & MiscellaneousAssignee: Jean-Baptiste Mardelle <jb>
Status: RESOLVED FIXED    
Severity: major    
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Wegwerf 2016-06-21 09:06:37 UTC 
When selecting the spacer tool and moving a set of clips on a track, these moved clips cannot moved anymore in selection mode unless the project is saved and reloaded.

Reproducible: Always

Steps to Reproduce:
1. Open a Kdenlive project with a set of clips on the same track.
2. Select spacer tool and move some of these clips within the same track to make room.
3. Go back to normal selection tool.
4. Try to move one of the clips that you've moved with the spacer tool.

Actual Results:  
Kdenlive complains that it cannot move the clip. While its graphical representation  moves once, it cannot be moved anymore. Only saving and reloading gets the user back to a working state ... until the spacer tools is used again...

Expected Results:  
Normaler operation in that clips moved with the spacer tool can correctly be moved also in selection mode.
Comment 1 Jean-Baptiste Mardelle 2016-06-23 22:48:03 UTC 
Cannot reproduce... is it still a problem for you ?
Comment 2 Wegwerf 2016-06-24 19:37:41 UTC 
This timeline corruption bug is unfortunately still present. Take some larger project and move parts of it around using the spacer tool: move larger parts forwards. Next, move backwards. Repeat a few times, save, reload ... timeline corrupted.
Comment 3 Jean-Baptiste Mardelle 2016-06-25 06:53:37 UTC 
Git commit 5798554e80e0fd9e327517862a4bb68e3b7210a3 by Jean-Baptiste Mardelle.
Committed on 25/06/2016 at 06:53.
Pushed by mardelle into branch 'master'.

Fix spacer timeline corruption
Related: bug 364297

M  +1    -2    src/timeline/customtrackview.cpp
M  +1    -1    src/timeline/managers/spacermanager.cpp

http://commits.kde.org/kdenlive/5798554e80e0fd9e327517862a4bb68e3b7210a3
Comment 4 Wegwerf 2016-06-25 15:13:55 UTC 
Thank you very much, Jean-Baptiste, for fixing this ugly corruption bug! I shuffled around a lot of clips in one of my current projects and couldn't find any timeline corruption anymore. Phew! Thank you again!