Bug 365035

Summary:	Regression: folder-tar and folder-scripts missing on smallest size
Product:	[Plasma] Breeze	Reporter:	Christian (Fuchs) <kde>
Component:	Icons	Assignee:	visual-design
Status:	RESOLVED FIXED
Severity:	normal	CC:	kainz.a
Priority:	NOR
Version First Reported In:	5.6.95
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	http://commits.kde.org/breeze-icons/6fafea8e754ae4637cbc0755a311c2a11ea6470c	Version Fixed In:
Sentry Crash Report:
Attachments:	Screenshot of the issue

Description Christian (Fuchs) 2016-07-03 11:43:45 UTC

Yet another missing icon regression: 

folder-tar and folder-script are lacking at the smallest icon size, which is especially bad as the default select folder dialogue needs these. See screenshot.

Reproducible: Always

Steps to Reproduce:
1. Define folders with the above mentioned icons
2. Either open a folder chooser or dolphin with the smallest icon size


Actual Results:  
Icons are missing

Expected Results:  
Icons are displayed

Comment 1 Christian (Fuchs) 2016-07-03 11:44:33 UTC

Created attachment 99823 [details]
Screenshot of the issue

Comment 2 andreas 2016-07-03 11:49:01 UTC

did I destroy the icon theme what's up there?

I will have a look

Comment 3 andreas 2016-07-03 12:55:51 UTC

Git commit 6fafea8e754ae4637cbc0755a311c2a11ea6470c by andreas kainz.
Committed on 03/07/2016 at 12:55.
Pushed by andreask into branch 'master'.

fix broken svg files thanks fuchs

M  +1    -1    icons-dark/places/16/folder-important.svg
M  +2    -4    icons-dark/places/16/folder-locked.svg
M  +4    -20   icons-dark/places/16/folder-script.svg
M  +3    -8    icons-dark/places/16/folder-tar.svg
M  +1    -1    icons/places/16/folder-important.svg
M  +2    -4    icons/places/16/folder-locked.svg
M  +4    -20   icons/places/16/folder-script.svg
M  +3    -8    icons/places/16/folder-tar.svg

http://commits.kde.org/breeze-icons/6fafea8e754ae4637cbc0755a311c2a11ea6470c

Comment 4 andreas 2016-07-03 12:59:31 UTC

sorry this bug is fixed but there are more broken icons out there. I have to look at all icons. I broke some icons when I add the stylesheed stuff for plasma 5.7 and KF 5.24.

Thanks for your help Fuchs.

Comment 5 Christian (Fuchs) 2016-07-16 10:11:17 UTC

Hmm, did the fix not make it into Plasma 5.7.1 / frameworks 24? 
While the other icons I reported are fixed, these two appear to be still broken in dolphin.

Comment 6 andreas 2016-07-20 22:16:27 UTC

It's fixed in master and will be released with KF 5.25.