Bug 437905 - Terminal icon in the toolbar is no longer a monochrome icon
Summary: Terminal icon in the toolbar is no longer a monochrome icon
Status: RESOLVED INTENTIONAL
Alias: None
Product: dolphin
Classification: Applications
Component: general (show other bugs)
Version: 21.04.1
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Dolphin Bug Assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-05-31 12:19 UTC by Lua
Modified: 2021-06-01 19:39 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
It happens with Breeze iconpack too (55.96 KB, image/png)
2021-05-31 12:20 UTC, Lua 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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.