Bug 363055

Summary:	"K" key does not pause playback immediately.
Product:	[Applications] kdenlive	Reporter:	Mike Ironfist <mikeklapak>
Component:	User Interface & Miscellaneous	Assignee:	Jean-Baptiste Mardelle <jb>
Status:	RESOLVED FIXED
Severity:	normal	CC:	wegwerf-1-2-3
Priority:	NOR
Version First Reported In:	16.04.0
Target Milestone:	---
Platform:	Ubuntu
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

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!