Bug 365035 - Regression: folder-tar and folder-scripts missing on smallest size
Summary: Regression: folder-tar and folder-scripts missing on smallest size
Status: RESOLVED FIXED
Alias: None
Product: Breeze
Classification: Plasma
Component: Icons (show other bugs)
Version: 5.6.95
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: visual-design
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-07-03 11:43 UTC by Christian (Fuchs)
Modified: 2016-07-20 22:16 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Screenshot of the issue (50.27 KB, image/png)
2016-07-03 11:44 UTC, Christian (Fuchs)
Details

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