Bug 345957

Summary:	Breeze-dark icons theme missing inherited icons
Product:	[Plasma] Breeze	Reporter:	Eugene Shalygin <eugene.shalygin+bugzilla.kde>
Component:	Icons	Assignee:	Unassigned bugs mailing-list <unassigned-bugs>
Status:	RESOLVED FIXED
Severity:	major	CC:	a.pronobis, alexander.kowalski, andre.vmatos, chrno-sphered, dev.dorrejo, hrvoje.senjan, jamboarder, kainz.a, luis.davila.pc, thomas.pfeiffer
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Gentoo Packages
OS:	Linux
Latest Commit:		Version Fixed In:	5.4.0
Sentry Crash Report:

Description Eugene Shalygin 2015-04-07 17:43:32 UTC

Running apps with breeze-dark icons theme results in a) message that "Breeze" icons them can not be found, and b) then many icons are missing, e.g., icons for mimetypes in Dolphin.
Perhaps this happens because breeze-dark/index.theme contains "Inherits=Breeze"
while the directory name is "breeze"? Correcting the case in the index file or symlinking breeze -> Breeze makes apps happy and icons appear.

Comment 1 André M 2015-04-09 14:08:06 UTC

Confirming issue in Arch Linux with breeze-git package. Work around fixed the issue.

Comment 2 Soukyuu 2015-04-15 22:55:01 UTC

Confirming. The issue is also present in the 5.2.95 package on arch linux [kde-unstable].
According to [1] and [2] the KDE packaging is at fault here. 
Workaround works.

[1] https://github.com/NitruxSA/plasma-next-icons/issues/127
[2] https://bugs.archlinux.org/task/44592

Comment 3 Eugene Shalygin 2015-06-25 17:02:20 UTC

And now Breeze-dark inherits hicolor. This is even worse then before...

Comment 4 a.pronobis 2015-08-12 16:13:52 UTC

Indeed in 5.3.95, it inherits hicolor. What's up with that?

Comment 5 Hrvoje Senjan 2015-08-16 09:45:42 UTC

VDG people, please take a look at this one. Without the proper inherits, the breeze dark won't use any missing icon from breeze but rather hicolor.

Comment 6 andreas 2015-08-16 10:48:11 UTC

change the index.theme file should now work.
https://quickgit.kde.org/?p=breeze.git&a=commit&h=6c9319e52e7f85b43b960eb8223f68d5890d1896

about breeze and Breeze where can I change the folder name, because the folder is called Breeze and not breeze as packagers will name it.

Comment 7 Hrvoje Senjan 2015-08-16 11:02:17 UTC

Andreas, please push the fix to Plasma/5.4 branch also. Tnx!

Regarding dir name, it's set in CMakeLists:
https://projects.kde.org/projects/kde/workspace/breeze/repository/revisions/master/entry/icons/CMakeLists.txt#L17

Comment 8 andreas 2015-08-16 11:03:16 UTC

how can I make this instead of :master :plasma5.4?

Comment 9 andreas 2015-08-18 00:22:41 UTC

can you please make the push. really don't know how to do it. sorry.

Comment 10 Hrvoje Senjan 2015-08-18 05:37:00 UTC

Sure, will push later today.

Comment 11 Hrvoje Senjan 2015-08-18 15:00:10 UTC

Pushed.