Bug 363055 - "K" key does not pause playback immediately.
Summary: "K" key does not pause playback immediately.
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: User Interface & Miscellaneous (show other bugs)
Version: 16.04.0
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-05-14 06:17 UTC by Mike Ironfist
Modified: 2016-08-04 16:28 UTC (History)
1 user (show)

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 Mike Ironfist 2016-05-14 06:17:13 UTC
This seems like a very simple bug to reproduce.

Spacebar for will pause playback *immediately.*

Hitting the K key (default mapping) will take a few seconds to pause, making precise frame-by-frame editting more difficult.

A current, easy workaround is to reassaign K to be play/pause in the key shortcut settings.
It seems as though whatever command the spacebar is mapped to by default has no trouble pausing right away but some sort of buffer happens when using the "pause" command K is normally mapped to. A small annoyance is that this then makes the K-key "play" as well, which might interfere with some workflows.

Reproducible: Always

Steps to Reproduce:
1. Start playback using Spacebar or L key.
2. Stop playback using default K key.
3. Playback does not stop instantly and may take 1 or 2 seconds.

Actual Results:  
Video continues playing for a second or two before pausing.

Expected Results:  
Pressing K should pause playback immediately, just like the spacebar does.
Comment 1 Jean-Baptiste Mardelle 2016-05-16 09:45:56 UTC
Git commit 81196f727412f2020c776d9b6dd6c01096356e3c by Jean-Baptiste Mardelle.
Committed on 16/05/2016 at 09:45.
Pushed by mardelle into branch 'master'.

Fix K shortcut delay for pausing

M  +1    -1    src/monitor/monitor.cpp
M  +1    -1    src/monitor/monitormanager.h
M  +4    -5    src/renderer.cpp
M  +0    -1    src/renderer.h
M  +4    -4    src/timeline/customtrackview.cpp

http://commits.kde.org/kdenlive/81196f727412f2020c776d9b6dd6c01096356e3c
Comment 2 Vincent PINON 2016-06-07 21:47:00 UTC
Git commit 97c5359a48ecd4004b921602ac7252ea88246f18 by Vincent Pinon, on behalf of Jean-Baptiste Mardelle.
Committed on 07/06/2016 at 20:22.
Pushed by vpinon into branch 'Applications/16.04'.

Fix K shortcut delay for pausing

M  +1    -1    src/monitor/monitor.cpp
M  +1    -1    src/monitor/monitormanager.h
M  +4    -5    src/renderer.cpp
M  +0    -1    src/renderer.h
M  +4    -4    src/timeline/customtrackview.cpp

http://commits.kde.org/kdenlive/97c5359a48ecd4004b921602ac7252ea88246f18
Comment 3 Wegwerf 2016-06-13 20:19:19 UTC
Mike, can you please check and confirm that this bug has been fixed? If yes, I would like to ask you to be so kind as close this bug report as RESOLVED FIXED. Thank you very much!