After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 644313 - [regression] Splitting does not work if no clip is selected
[regression] Splitting does not work if no clip is selected
Status: RESOLVED FIXED
Product: pitivi
Classification: Other
Component: Timeline
Git
Other Linux
: Normal normal
: 0.14
Assigned To: Jean-François Fortin Tam
Pitivi maintainers
Depends on:
Blocks:
 
 
Reported: 2011-03-09 14:58 UTC by Jean-François Fortin Tam
Modified: 2011-04-06 14:08 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
one-liner patch (845 bytes, patch)
2011-03-09 15:10 UTC, Jean-François Fortin Tam
none Details | Review

Description Jean-François Fortin Tam 2011-03-09 14:58:14 UTC
Splitting is non-modal and when you have a clip selected, it splits only that clip. However, it is *also* intended to work when no clips are selected: when that is the case, splitting will split all clips at the position of the timeline. As such, the timeline button for it should never be made insensitive.

This used to work until recently. I have located the source with git bisect:

9b755cf08e4abb0cc6cc52a661570c4ca6111aa7 is the first bad commit
commit 9b755cf08e4abb0cc6cc52a661570c4ca6111aa7
Author: Luis de Bethencourt <luis@debethencourt.com>
Date:   Fri Sep 3 10:59:11 2010 +0200

    ui/timeline: fixing the sensitivity of the timeline actions.

:040000 040000 823c67331f4b02fc8828f20292a22a25787eb268 9c27551fab89ca48f4875eee243b4736a9e5b97b M	pitivi
Comment 1 Jean-François Fortin Tam 2011-03-09 15:10:58 UTC
Created attachment 182992 [details] [review]
one-liner patch

Trivial patch is trivial. I could have removed lines such as "split = True" entirely, as well as the "self.split_action.set_sensitive(split)" line, but I thought that leaving those there with a comment would be better for preventing future regressions.
Comment 2 Jean-François Fortin Tam 2011-04-06 14:08:46 UTC
Fixed by a slightly modified

commit 52b6f6d3a16a6b7511c983f782303d1c82f94968
Author: Jean-François Fortin Tam <nekohayo@gmail.com>
Date:   Wed Mar 9 10:07:35 2011 -0500

    Allow splitting even when there is no selection. Fixes bug #644313.