Bug 477739

Summary: Free space dropdown shows empty menu if Filelight is not installed
Product: [Applications] dolphin Reporter: Timothy B <yule2000>
Component: bars: statusAssignee: Dolphin Bug Assignee <dolphin-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: kfm-devel, nate
Priority: NOR Keywords: qt6, regression
Version: 24.01.80   
Target Milestone: ---   
Platform: Neon   
OS: Linux   
Latest Commit: Version Fixed In: 24.08.0
Sentry Crash Report:
Attachments: Screenshot of observed bug

Description Timothy B 2023-11-29 23:43:05 UTC
Created attachment 163624 [details]
Screenshot of observed bug

SUMMARY
If Filelight is not installed in the system, clicking on the free space button in the status bar on Dolphin would display a tiny popup menu with no items.

STEPS TO REPRODUCE
0. Verify that Filelight is not installed. If it is, uninstall it.
1. Open any Dolphin window
2. Click on the free space button on the status bar. The button text says something like "123.4 GiB free" with a down arrow beside it.

OBSERVED RESULT
A tiny 1x1 pixel blank popup menu appears below the button.

EXPECTED RESULT
Possibly a menu with only an entry that says something like "Install Filelight" would appear.

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.81.80
KDE Frameworks Version: 5.245.0
Qt Version: 6.6.0
Dolphin Version: 24.01.80

ADDITIONAL INFORMATION
Filelight is not installed by default in KDE Neon, so this can be easily reproduced when running the live ISO image. After installing Filelight, Dolphin will populate the otherwise empty menu when the free space button is clicked.
Comment 1 Bug Janitor Service 2023-12-11 00:20:17 UTC
A possibly relevant merge request was started @ https://invent.kde.org/system/dolphin/-/merge_requests/674
Comment 2 Felix Ernst 2024-05-30 15:35:13 UTC
Git commit 0a8f84a52dcd246550b5ea0e795dd7b309f2ece1 by Felix Ernst, on behalf of Nicolas Fella.
Committed on 30/05/2024 at 15:21.
Pushed by felixernst into branch 'master'.

Offer installing Filelight if no disk usage analyzer was found

This avoids an empty menu and points the user towards a useful tool

M  +12   -1    src/statusbar/statusbarspaceinfo.cpp

https://invent.kde.org/system/dolphin/-/commit/0a8f84a52dcd246550b5ea0e795dd7b309f2ece1
Comment 3 Bug Janitor Service 2024-05-30 16:51:34 UTC
A possibly relevant merge request was started @ https://invent.kde.org/system/dolphin/-/merge_requests/783
Comment 4 Felix Ernst 2024-07-08 11:41:59 UTC
Git commit 0d981e7b7df266c867004e23a169ba8cc13c8683 by Felix Ernst.
Committed on 08/07/2024 at 11:41.
Pushed by felixernst into branch 'master'.

Improve Filelight installation UX

Before this commit pressing the free space button when Filelight
is not installed would show a singular action called "Install
Filelight to View Disk Usage Statistics…". Pressing this button
would open the store page for Filelight. This is an okay user
experience, but we can do better.

This commit makes it so pressing the free space button when
Filelight is not installed shows an attractive UI that makes clear
that freeing up disk space can be accomplished nicely by installing
Filelight. The "Install Filelight…" button on this UI is connected
to PackageKit directly, so we do not need to show a separate store
like Discover and instead trigger an installation right then and
there. For this, the recently introduced DolphinPackageInstaller
KJob is used.

Installation progress is reported through the status bar similar to
the progress reporting of slowly loading directories or searches.
Installation failure or success is ultimately shown within Dolphin
as a passive notification above the view.

On Microsoft Windows or when PackageKit is not available, the
install button will only open a store page for Filelight.

M  +1    -0    src/CMakeLists.txt
M  +3    -0    src/config-dolphin.h.cmake
M  +1    -0    src/dolphinviewcontainer.cpp
M  +2    -2    src/global.h
M  +7    -0    src/statusbar/dolphinstatusbar.cpp
M  +7    -0    src/statusbar/dolphinstatusbar.h
M  +98   -11   src/statusbar/statusbarspaceinfo.cpp
M  +32   -0    src/statusbar/statusbarspaceinfo.h

https://invent.kde.org/system/dolphin/-/commit/0d981e7b7df266c867004e23a169ba8cc13c8683