Bug 498005

Summary:	Provide shortcuts for videos in the slideshow
Product:	[Applications] digikam	Reporter:	Curiositas Felis <linux-stuff.drool940>
Component:	Plugin-Generic-SlideShow	Assignee:	Digikam Developers <digikam-bugs-null>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	caulier.gilles, metzpinguin
Priority:	NOR
Version First Reported In:	8.5.0
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:	https://invent.kde.org/graphics/digikam/-/commit/5a0f94d9760e3f9c1dd5e38d3b29cdc7760af5c2	Version Fixed/Implemented In:	8.6.0
Sentry Crash Report:
Attachments:	A screenshot of the seek controls Celluloid offers

Description Curiositas Felis 2024-12-29 07:51:17 UTC

Created attachment 176939 [details]
A screenshot of the seek controls Celluloid offers

SUMMARY
Feature request: currently there is no way to seek a video that is playing in the slideshow. You can play/pause by pressing the spacebar, but there is no way to seek forward/backwards in the video using the keyboard. It is possible to use the scrubber tool at the top, which after [a recent fix](https://bugs.kde.org/show_bug.cgi?id=497446) can be used to seek to a certain point in the video, but the scrubber is a small UI element at the top of the screen and is easy to misclick (which advances to the next slide). It would be good to have some keyboard shortcuts to seek the video, similar to what media players like VLC/Celluloid offer (screenshot attached).

Alternatively, it would be amazing if a chosen media player could be used as the backend player for the videos in the slideshow. So, for instance, VLC/Celluloid would be used to play the videos, which would mean their keyboard shortcuts would be usable. I'm not sure how big/difficult of a change this might be.

STEPS TO REPRODUCE
1. Open an album with multiple videos and pictures
2. Open the slideshow view (F9 or click the slideshow button)
3. Open a slide with the video

OBSERVED RESULT
The video can only be paused using the keyboard. You cannot seek 10s forwards/backwards, or jump 1 minute ahead, etc.

EXPECTED RESULT
There is a shortcut that can be used to quickly seek to different parts of the video

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Arch Zen 6.12.7-zen1 (Wayland)
KDE Plasma Version: 6.2.4
KDE Frameworks Version: 6.9.0
Qt Version: 6.8.1

ADDITIONAL INFORMATION

Comment 1 Maik Qualmann 2024-12-29 17:57:55 UTC

Git commit 5a0f94d9760e3f9c1dd5e38d3b29cdc7760af5c2 by Maik Qualmann.
Committed on 29/12/2024 at 17:57.
Pushed by mqualmann into branch 'master'.

add key shortcut to move forward/backward of Slideshow video
One step is 10% of the respective video time, the key sortcuts
are Ctrl+Right and Ctrl+Left
FIXED-IN: 8.6.0

M  +1    -1    NEWS
M  +23   -0    core/dplugins/generic/view/slideshow/common/slideshowloader.cpp
M  +2    -0    core/dplugins/generic/view/slideshow/slideshowplugin.cpp
M  +36   -0    core/libs/video/player/qtmm/slidevideo.cpp
M  +2    -0    core/libs/video/player/qtmm/slidevideo.h

https://invent.kde.org/graphics/digikam/-/commit/5a0f94d9760e3f9c1dd5e38d3b29cdc7760af5c2

Comment 2 Maik Qualmann 2024-12-29 17:59:19 UTC

Note that third-party players cannot be integrated into the slideshow.

Maik

Comment 3 Curiositas Felis 2024-12-29 20:41:48 UTC

This works great, can confirm this solves the issue with the latest appimage. One small request along with this change (I can create a separate issue if needed), it would be helpful to show the time scrubber when we use these shortcuts, so we can know where in the video we are.

Thanks again for the quick response!

Comment 4 Maik Qualmann 2024-12-29 20:50:23 UTC

I had already thought about briefly displaying the video timeline. I'll implement it tomorrow.

Maik

Comment 5 Curiositas Felis 2024-12-29 20:55:41 UTC

(In reply to Maik Qualmann from comment #4)
> I had already thought about briefly displaying the video timeline. I'll
> implement it tomorrow.
> 
> Maik

Thanks!

Comment 6 Maik Qualmann 2024-12-29 20:55:49 UTC

Git commit 1854530299413a6d9fc0e15f6cbd35f028f5b9e3 by Maik Qualmann.
Committed on 29/12/2024 at 20:55.
Pushed by mqualmann into branch 'master'.

show video timeline if forward/backward keys used

M  +4    -0    core/dplugins/generic/view/slideshow/common/slideshowloader.cpp

https://invent.kde.org/graphics/digikam/-/commit/1854530299413a6d9fc0e15f6cbd35f028f5b9e3