Bug 437905

Summary:	Terminal icon in the toolbar is no longer a monochrome icon
Product:	[Applications] dolphin	Reporter:	Lua <lv215bd5>
Component:	general	Assignee:	Dolphin Bug Assignee <dolphin-bugs-null>
Status:	RESOLVED INTENTIONAL
Severity:	normal	CC:	kfm-devel, nate
Priority:	NOR
Version:	21.04.1
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	It happens with Breeze iconpack too

Description Lua 2021-05-31 12:19:29 UTC

SUMMARY


STEPS TO REPRODUCE
1. Install a custom icon pack like papirus-icon-theme;
2. Open Dolphin.

OBSERVED RESULT
The terminal icon is no longer monochrome. papirus-icon-theme wasn't updated because is tied to the Ubuntu 20.04 LTS update policy (I'm using Neon), so this is likely an issue with Dolphin.

EXPECTED RESULT


SOFTWARE/OS VERSIONS
Operating System: KDE neon 5.21
KDE Plasma Version: 5.21.5
KDE Frameworks Version: 5.82.0
Qt Version: 5.15.2
Kernel Version: 5.4.0-73-generic
OS Type: 64-bit
Graphics Platform: X11
Processors: 8 × Intel® Core™ i5-8265U CPU @ 1.60GHz
Memory: 15,5 GiB of RAM
Graphics Processor: Mesa Intel® UHD Graphics 620
ADDITIONAL INFORMATION

Comment 1 Lua 2021-05-31 12:20:47 UTC

Created attachment 138898 [details]
It happens with Breeze iconpack too

Nevermind about the iconpack. It happens with Breeze icons too.

Comment 2 Nate Graham 2021-06-01 13:59:48 UTC

What happens when you click on that icon? Does it open a terminal panel within Dolphin, or does it open up a separate app?

Comment 3 Lua 2021-06-01 17:47:12 UTC

It opens Konsole instead of embedding the terminal in Dolphin.

Comment 4 Nate Graham 2021-06-01 19:03:57 UTC

Then the current appearance is intentional; we *want* to show the icon of the app that will be launched. It's intended to be a visual cue to tell the user "hey, this will open another app."

Comment 5 Lua 2021-06-01 19:39:48 UTC

Understood o/ It makes sense.