Bug 498178 - Hiding a track in an embeded sequence changes the duration of the sequence in parent sequence
Summary: Hiding a track in an embeded sequence changes the duration of the sequence in...
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: Timeline & Editing (show other bugs)
Version: 24.12.0
Platform: Mint (Ubuntu based) Linux
: HI normal
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL:
Keywords: triaged
Depends on:
Blocks:
 
Reported: 2025-01-02 18:56 UTC by steph
Modified: 2025-01-05 16:50 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 24.12.1
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description steph 2025-01-02 18:56:08 UTC
SUMMARY
In a project with two sequences and with "sequence 2" embeded in "sequence 1", if I Hide/show tracks while editing "sequence 2", the duration of "sequence 2" embeded in "sequence 1" changes. Even If the tracks are set back to visible in "sequence 2", this does not reset original duration in "sequence 1". 

STEPS TO REPRODUCE
1. Start a new project
2. Add a new sequence, name it sequence 2
3. Add a new clip to the bin (can be a color clip)
4. Add the clip to sequence 2
5. Add sequence 2 once or multiple times to sequence 1
6. Go back editing sequence 2 and hide the track with the clip (also mute the sound if it has sound)
7. Click on sequence 1 and observe the duration of sequence 2 has changed everywhere it was used and is now set to 1 frame.
8. Going back to sequence 2 and unhiding the hidden tracks does reset the duration of sequence 2 in sequence 1 to what it was before. So hiding/showing tracks in an embeded sequence destroys the editing done in parent sequence.

OBSERVED RESULT
Hiding tracks in embeded sequence changes duration of the sequence everywhere its used.

EXPECTED RESULT
Duration of the sequence should not be changed and only the visibility should be affected

SOFTWARE/OS VERSIONS
Linux mint 21.3 cinnamon using kdenlive flatpak 24.12.0
Comment 1 emohr 2025-01-04 15:15:58 UTC
Thank you for reporting. I can confirm the issue. Following your steps. 

I think sequence 1 should show sequence 2 but as hidden with the existing length.
Comment 2 Jean-Baptiste Mardelle 2025-01-05 16:47:48 UTC
Git commit 42e12551eb255d7d8aa67a026515221df0f25c26 by Jean-Baptiste Mardelle.
Committed on 05/01/2025 at 16:45.
Pushed by mardelle into branch 'master'.

Ensure sequence clips in timeline are not resized to smaller when hiding a track
FIXED-IN: 24.12.1

M  +12   -0    src/bin/projectclip.cpp
M  +5    -2    src/bin/projectclip.h
M  +25   -0    src/bin/sequenceclip.cpp
M  +3    -0    src/bin/sequenceclip.h
M  +8    -0    src/timeline2/model/timelinemodel.cpp

https://invent.kde.org/multimedia/kdenlive/-/commit/42e12551eb255d7d8aa67a026515221df0f25c26
Comment 3 Jean-Baptiste Mardelle 2025-01-05 16:50:06 UTC
Git commit 48130ff9d32c9927c82d40efa96353d5816c6039 by Jean-Baptiste Mardelle.
Committed on 05/01/2025 at 16:49.
Pushed by mardelle into branch 'release/24.12'.

Ensure sequence clips in timeline are not resized to smaller when hiding a track
FIXED-IN: 24.12.1

M  +11   -0    src/bin/projectclip.cpp
M  +5    -2    src/bin/projectclip.h
M  +25   -0    src/bin/sequenceclip.cpp
M  +3    -0    src/bin/sequenceclip.h
M  +8    -0    src/timeline2/model/timelinemodel.cpp

https://invent.kde.org/multimedia/kdenlive/-/commit/48130ff9d32c9927c82d40efa96353d5816c6039