Bug 416917 - Fast forward and backward activating wrong monitor
Summary: Fast forward and backward activating wrong monitor
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: Rendering & Export (show other bugs)
Version: Appimage - Refactoring
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-01-29 16:40 UTC by Massimo Stella
Modified: 2020-01-31 15:43 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Massimo Stella 2020-01-29 16:40:05 UTC
SUMMARY
Fast forward and backward activating the wrong monitor

STEPS TO REPRODUCE
1. Open a project and load one clip in the Clip Monitor and put one clip in the timeline
2. Click the play button by mouse on one of the 2 Monitor (Clip Monitor or the Program - the 2 have to be visible on the GUI at the same time)
3. Click on the fast forward or backward on the window which is not playing back

OBSERVED RESULT
The video in the window you are playing back changes playback speed following the direction you clicked
EXPECTED RESULT
The video in the window you are playing back stops and the fast playback in the chosen direction start in the window you clicked on.
A similar behaviour for loop zone was corrected here: https://cgit.kde.org/kdenlive.git/commit/?id=6a068663a4f210a5872d780f92273e2902907353
SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
Comment 1 Jean-Baptiste Mardelle 2020-01-31 15:43:51 UTC
Git commit 75664632f2012f49e75b3e1fe30723407073a91a by Jean-Baptiste Mardelle.
Committed on 31/01/2020 at 15:43.
Pushed by mardelle into branch 'master'.

Fix forward/rewind affecting wrong monitor

M  +2    -0    src/monitor/abstractmonitor.h
M  +12   -7    src/monitor/monitor.cpp
M  +2    -2    src/monitor/monitor.h
M  +6    -10   src/monitor/monitormanager.cpp

https://invent.kde.org/kde/kdenlive/commit/75664632f2012f49e75b3e1fe30723407073a91a