Bug 486198

Summary:	In the latest update, pressing the space bar starts recording on the recording track instead of listening.
Product:	[Applications] kdenlive	Reporter:	Konstantin <jon9097>
Component:	User Interface & Miscellaneous	Assignee:	Jean-Baptiste Mardelle <jb>
Status:	RESOLVED FIXED
Severity:	normal	CC:	snd.noise
Priority:	NOR
Version:	git-master
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Konstantin 2024-04-27 12:41:56 UTC

Everything was working fine a few days ago. It broke down after the update.

How it works now:

Right−click on the track -> Insert Track −> Audio record track.
When you press the space bar, recording starts immediately (This should not be the case).

How it should work:

Right−click on the track -> Insert Track −> Audio record track.
A sound recording track is being created. When we press the record key (red circle on the track) − record and stop recording. On the space bar − listening.

Comment 1 Konstantin 2024-04-27 16:25:17 UTC

Everything works as it should in this version:
git+https://invent.kde.org/multimedia/kdenlive#commit=5b15c6c3

Comment 2 Bug Janitor Service 2024-04-28 10:58:08 UTC

A possibly relevant merge request was started @ https://invent.kde.org/multimedia/kdenlive/-/merge_requests/502

Comment 3 Jean-Baptiste Mardelle 2024-04-28 12:50:47 UTC

Git commit 776a2890a87bdc687d4772b4b2382c4cd2b09e2d by Jean-Baptiste Mardelle.
Committed on 28/04/2024 at 12:50.
Pushed by mardelle into branch 'master'.

Audio record: allow playing timeline when monitoring, clicking track rec...

Audio record: allow playing timeline when monitoring, clicking track rec button will start recording.
Related to #1861 
Related: bug 485660

M  +1    -2    src/audiomixer/mixermanager.cpp
M  +34   -23   src/capture/mediacapture.cpp
M  +7    -6    src/capture/mediacapture.h
M  +13   -5    src/core.cpp
M  +5    -1    src/core.h
M  +12   -0    src/mainwindow.cpp
M  +10   -11   src/monitor/monitor.cpp
M  +0    -1    src/monitor/monitor.h
M  +1    -1    src/monitor/recmanager.cpp
M  +9    -2    src/timeline2/view/qml/AudioLevels.qml
M  +17   -9    src/timeline2/view/timelinecontroller.cpp
M  +4    -1    src/timeline2/view/timelinecontroller.h

https://invent.kde.org/multimedia/kdenlive/-/commit/776a2890a87bdc687d4772b4b2382c4cd2b09e2d

Comment 4 Jean-Baptiste Mardelle 2024-04-28 12:51:44 UTC

Git commit 9840c3e8000b3b32427f66794569ada92248d5a2 by Jean-Baptiste Mardelle.
Committed on 28/04/2024 at 12:51.
Pushed by mardelle into branch 'release/24.05'.

Audio record: allow playing timeline when monitoring, clicking track rec...

Audio record: allow playing timeline when monitoring, clicking track rec button will start recording.
Related to #1861 
Related: bug 485660

M  +1    -2    src/audiomixer/mixermanager.cpp
M  +34   -23   src/capture/mediacapture.cpp
M  +7    -6    src/capture/mediacapture.h
M  +13   -5    src/core.cpp
M  +5    -1    src/core.h
M  +12   -0    src/mainwindow.cpp
M  +10   -11   src/monitor/monitor.cpp
M  +0    -1    src/monitor/monitor.h
M  +1    -1    src/monitor/recmanager.cpp
M  +9    -2    src/timeline2/view/qml/AudioLevels.qml
M  +17   -9    src/timeline2/view/timelinecontroller.cpp
M  +4    -1    src/timeline2/view/timelinecontroller.h

https://invent.kde.org/multimedia/kdenlive/-/commit/9840c3e8000b3b32427f66794569ada92248d5a2

Comment 5 Konstantin 2024-04-28 14:16:51 UTC

Installed it. It works as it should. I'll test it for a couple more days. If I find something, I'll write here.

Thanks for fixing the problem