Bug 501797 - Dim icons in Folder View widget in the panel, only with view mode "list"
Summary: Dim icons in Folder View widget in the panel, only with view mode "list"
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Desktop icons & Folder View widget (show other bugs)
Version: git-stable-Plasma/6.3
Platform: Other Linux
: NOR minor
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2025-03-20 13:56 UTC by David de Cos
Modified: 2025-03-24 14:50 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.3.4
Sentry Crash Report:


Attachments
Folder View widget in Plasma 6.1 vs Plasma 6.3 (466.59 KB, image/png)
2025-03-20 14:09 UTC, David de Cos
Details

Note You need to log in before you can comment on or make changes to this bug.
Description David de Cos 2025-03-20 13:56:49 UTC
SUMMARY

I have a Folder View widget in my panel, configured in list mode. In the last weeks, probably since updating to Plasma 6.3, the icons in the widget have become very dim. Ever since Plasma 5, with the exact same themes, configurations, etc, those icons always looked normal (i.e. bright). Besides that, I know it's not something to do with my specific configuration because I have checked on a new user with default themes and settings, and the Folder View icons look dim there too.

STEPS TO REPRODUCE
1. Add a Folder View widget to your panel. Set it as View mode = List (the bug doesn't happen with Grid).
2. Click on the widget and have a look at the icons in the list.
 

OBSERVED RESULT
The icons are very dim.

EXPECTED RESULT
The icons should be as bright as they were in all previous versions, and as bright as they are in Grid mode.

SOFTWARE/OS VERSIONS
Operating System: openSUSE Tumbleweed 20250318
KDE Plasma Version: 6.3.3
KDE Frameworks Version: 6.12.0
Qt Version: 6.8.2
Kernel Version: 6.13.6-1-default (64-bit)
Graphics Platform: Wayland


ADDITIONAL INFORMATION

- The Folder View must be inside a panel, since the list view mode isn't available otherwise.

- In the next comment I will show a comparison between the same Folder View widget (also same Plasma theme, icons, etc) in Plasma 6.1 and Plasma 6.3.
Comment 1 David de Cos 2025-03-20 14:09:52 UTC
Created attachment 179602 [details]
Folder View widget in Plasma 6.1 vs Plasma 6.3
Comment 2 Nate Graham 2025-03-21 17:30:14 UTC
Can confirm.
Comment 3 Nate Graham 2025-03-21 18:27:35 UTC
Aha, it regressed in https://invent.kde.org/plasma/plasma-desktop/-/commit/f7a92f7e65ac09cfd82a01835ac9f955c1b25744.

Previously selectionButton could be null, so checking for its existence was fine. In that commit, it's always initialized to something, so now we should check for visibility, the way the rest of the code in this file does.

Patch incoming.
Comment 4 Bug Janitor Service 2025-03-21 18:33:41 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2898
Comment 5 Nate Graham 2025-03-21 19:45:11 UTC
Git commit 13125b69d838200e624de9208a21af30188cbd59 by Nate Graham.
Committed on 21/03/2025 at 18:30.
Pushed by ngraham into branch 'master'.

applets/folderview: fix list view icon opacity

Previously selectionButton could be null, so checking for its existence
was fine. In f7a92f7e65ac09cfd82a01835ac9f955c1b25744, it's now always
initialized to something, so we should check for visibility, the way
the rest of the code in this file does.
FIXED-IN: 6.3.4

M  +1    -1    containments/desktop/package/contents/ui/FolderItemDelegate.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/13125b69d838200e624de9208a21af30188cbd59
Comment 6 Nate Graham 2025-03-21 19:46:26 UTC
Git commit 9a230caf440c364a56ce37c17725c029b52ceeca by Nate Graham.
Committed on 21/03/2025 at 19:46.
Pushed by ngraham into branch 'Plasma/6.3'.

applets/folderview: fix list view icon opacity

Previously selectionButton could be null, so checking for its existence
was fine. In f7a92f7e65ac09cfd82a01835ac9f955c1b25744, it's now always
initialized to something, so we should check for visibility, the way
the rest of the code in this file does.
FIXED-IN: 6.3.4


(cherry picked from commit 13125b69d838200e624de9208a21af30188cbd59)

Co-authored-by: Nate Graham <nate@kde.org>

M  +1    -1    containments/desktop/package/contents/ui/FolderItemDelegate.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/9a230caf440c364a56ce37c17725c029b52ceeca
Comment 7 David de Cos 2025-03-24 09:54:11 UTC
Thank you for the quick patch, Nate!
Comment 8 Nate Graham 2025-03-24 14:50:22 UTC
You're welcome!