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 603045 - [regression] viewer no longer displays after dropping files
[regression] viewer no longer displays after dropping files
Status: RESOLVED FIXED
Product: pitivi
Classification: Other
Component: User interface
Git
Other Linux
: Normal major
: 0.13.4
Assigned To: Pitivi maintainers
Pitivi maintainers
Depends on:
Blocks:
 
 
Reported: 2009-11-26 13:23 UTC by Edward Hervey
Modified: 2009-12-10 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Edward Hervey 2009-11-26 13:23:38 UTC
The following commit breaks the behaviour by which when you put objects in the timeline, the viewer would display the timeline.

Reproducable by:
* loading a project
* or adding a file to a blank timeline by using [insert]
* or by dragging a file on a blank timeline

commit af7c3b26a9dea1ab3f201b791a482aa49bab3f9e
Author: Brandon Lewis <brandon_lewis@alum.berkeley.edu>
Date:   Mon Nov 23 22:26:01 2009 -0800

    ruler.py: remove unnecessary seek request, ensures previous commit always works
Comment 1 Brandon Lewis 2009-11-27 07:58:21 UTC
This seek request occurs every time the timeline duration changes (which is often), and will override the seek-on-click behavior (returning the playhead to its original position). Wouldn't it be better to issue this request directly after adding sources to the timeline. And we should figure out why setShadedDuration() is called so frequently.
Comment 2 Edward Hervey 2009-11-27 11:39:27 UTC
if there's code that already returns the playhead to its original position, then great.

And yes, we should definitely:
* only do a initial seek when adding sources
* figure out why setShadedDuration is called so frequently
Comment 3 Brandon Lewis 2009-12-01 07:16:46 UTC
think i know what's behind this. I think it is also related to 60342, and 603102. Will investigate further tomorrow.
Comment 4 Brandon Lewis 2009-12-02 06:25:20 UTC
please review my branch fix_seeking_6030405_603424
Comment 5 Brandon Lewis 2009-12-02 06:30:19 UTC
correction: fix_seeking_603405_603424
Comment 6 Brandon Lewis 2009-12-10 19:30:31 UTC
commit 837f0d73b7d510b695ee1c9a1cb961c779ad6534
Author: Brandon Lewis <brandon_lewis@alum.berkeley.edu>
Date:   Wed Dec 9 15:10:10 2009 -0800

    mainwindow.py,test_integration.py: issue preliminary seek on timeline
    closes bugs 603045, 603424, 603102