Bug 357752

Summary:	Incorrect color for hovered selected items with GTK+ Qt5 style
Product:	[Applications] dolphin	Reporter:	Pulfer <pulfer>
Component:	general	Assignee:	Dolphin Bug Assignee <dolphin-bugs-null>
Status:	RESOLVED NOT A BUG
Severity:	normal	CC:	mail
Priority:	NOR
Version First Reported In:	15.12.0
Target Milestone:	---
Platform:	ROSA RPMs
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	Incorrect colors for hovered selection in Dolphin Colors in Dolphin compared to colors in file open dialogs Patch to disable background color change for hovered items

Description Pulfer 2016-01-09 17:44:45 UTC

KF5-based Dolphin sets incorrect color for hovered selected items with GTK+ Qt5 style. Only Dolphin is affected, file dialogs (like Kate -> Open file...) are fine.

Reproducible: Always

Steps to Reproduce:
1. Set Qt5 style to GTK+
2. Open Dolphin
3. Select file(s)
4. Move mouse cursor over selected item
5. See incorrect selection color (neither background nor text shouldn't change but they do)

See the attached screenshot.

Comment 1 Pulfer 2016-01-09 17:45:56 UTC

Created attachment 96541 [details]
Incorrect colors for hovered selection in Dolphin

Comment 2 Pulfer 2016-01-09 18:08:20 UTC

Created attachment 96543 [details]
Colors in Dolphin compared to colors in file open dialogs

This screenshot should be more helpful. I played with GTK2 theme colors and now it's easy to see which color Dolphin uses for hovered items (the same color is used for backgrounds in file dialogs).

I suggest to avoid changing text color and background color for hovered items. It's enough to highlight icons for hovered items (like it works in KDE4-based Dolphin and KF5-based file dialogs).

Comment 3 Pulfer 2016-01-10 05:01:25 UTC

Created attachment 96560 [details]
Patch to disable background color change for hovered items

Comment 4 Julian Steinmann 2018-04-19 15:15:12 UTC

Does this issue still exist? If yes, feel free to submit your patch at phabricator.kde.org. Thanks!

Comment 5 Julian Steinmann 2018-05-06 09:36:15 UTC

Any updates? It'd be great to have a patch for this issue if it does indeed still exist.

Comment 6 Julian Steinmann 2018-05-23 10:32:35 UTC

No response -> closing. @Pulfer, your patch is obviously still welcome, so please submit it to Phabricator if you find time to do so. Thanks!