KDF currently relies on having Dolphin installed to be able to
open a disk in file explorer. The choice seems to be hardcoded
in source code
A better way to open local paths might be to use
QUrl::fromLocalFile and pass it to QDesktopServices::openUrl
This will ensure that disk is opened in whichever file manager
is available on user's system, rather than assuming that
Dolphin is always available.
STEPS TO REPRODUCE
Install KDF on a system without Dolphin. In KDF, right click
on a disk and select "Open in File Manager".
Disk should open in whichever file manager is available
on user's system.
You can set a file manager in the settings, but yes, it should just open the default one
Since you already know what needs to be done, would you like to submit a patch for it?
Thanks for the acknowledgement about opening default file manager. I'll send a merge request soon.
A possibly relevant merge request was started @ https://invent.kde.org/utilities/kdf/-/merge_requests/4
A possibly relevant merge request was started @ https://invent.kde.org/utilities/kdf/-/merge_requests/5
Git commit 7ca2f908a5dc5285c5b94ae053e32ff0016d1afe by Albert Astals Cid, on behalf of Snehit Sah.
Committed on 23/01/2022 at 11:45.
Pushed by aacid into branch 'master'.
Add feature to use system default file manager to open drive
Added an option in settings to use system default file manager to
open drives. If selected, KDF will use QDesktopServices to open
disk. If unchecked, KDF will use the previous behaviour of executing
supplied command in settings.
Signed-off-by: Snehit Sah <email@example.com>
M +10 -0 src/kdfconfig.cpp
M +11 -4 src/kdfconfig.ui
M +7 -1 src/kdfwidget.cpp
M +16 -0 src/stdoption.cpp
M +3 -0 src/stdoption.h