Bug 399253 - archive-* icons could use improvement
Summary: archive-* icons could use improvement
Status: RESOLVED FIXED
Alias: None
Product: Breeze
Classification: Plasma
Component: Icons (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: visual-design
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-09-30 19:26 UTC by Nate Graham
Modified: 2018-11-18 18:01 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.53


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nate Graham 2018-09-30 19:26:57 UTC
The Breeze `archive-*` icons are very simple: just an empty rectangle with a plus sign, an arrow, or a red X in the corner

These icons could use some more visual pizzaz. In particular, the image that gets a little emblem in the corner should probably look like a monochrome version of an archive, not just a simple rectangle.
Comment 1 Noah Davis 2018-11-18 18:01:03 UTC
Git commit aace21a156679810eff5b3ebc63ab3855b7f6d70 by Noah Davis, on behalf of Rafael Brandmaier.
Committed on 18/11/2018 at 18:00.
Pushed by ndavis into branch 'master'.

Change archive-* action icons to represent archives

Summary:

FIXED-IN: 5.53

Change the `archive-*` action icons to represent actual archives instead of a generic document. The archive symbols were taken from the `folder-achive` `16px` and `22px` icons.
Change the fallback color in the embedded stylesheet to Shade Black.

Test Plan:
Before:

|size|`archive-extract`|`archive-insert`|`archive-remove`
|-----|-----|-----|-----
|`22px`|{F6429398}|{F6429400}|{F6429401}

After:

|size|`archive-extract`|`archive-insert`|`archive-remove`
|-----|-----|-----|-----
|`16px`|{F6430388}|{F6430389}|{F6430390}
|`22px` and `24px`|{F6430392}|{F6430393}|{F6430394}
|`32px`|{F6430395}|{F6430396}|{F6430397}

Reviewers: #vdg, ngraham, ndavis

Reviewed By: #vdg, ngraham, ndavis

Subscribers: ngraham, ndavis, kde-frameworks-devel

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D16964

M  +1    -13   icons-dark/actions/16/archive-extract.svg
M  +1    -13   icons-dark/actions/16/archive-insert.svg
M  +1    -22   icons-dark/actions/16/archive-remove.svg
M  +1    -14   icons-dark/actions/22/archive-extract.svg
M  +1    -14   icons-dark/actions/22/archive-insert.svg
M  +1    -18   icons-dark/actions/22/archive-remove.svg
M  +1    -13   icons-dark/actions/24/archive-extract.svg
M  +1    -13   icons-dark/actions/24/archive-insert.svg
M  +1    -20   icons-dark/actions/24/archive-remove.svg
M  +1    -18   icons-dark/actions/32/archive-extract.svg
M  +1    -18   icons-dark/actions/32/archive-insert.svg
M  +1    -17   icons-dark/actions/32/archive-remove.svg
M  +1    -13   icons/actions/16/archive-extract.svg
M  +1    -13   icons/actions/16/archive-insert.svg
M  +1    -22   icons/actions/16/archive-remove.svg
M  +1    -14   icons/actions/22/archive-extract.svg
M  +1    -14   icons/actions/22/archive-insert.svg
M  +1    -18   icons/actions/22/archive-remove.svg
M  +1    -13   icons/actions/24/archive-extract.svg
M  +1    -13   icons/actions/24/archive-insert.svg
M  +1    -20   icons/actions/24/archive-remove.svg
M  +1    -18   icons/actions/32/archive-extract.svg
M  +1    -18   icons/actions/32/archive-insert.svg
M  +1    -17   icons/actions/32/archive-remove.svg

https://commits.kde.org/breeze-icons/aace21a156679810eff5b3ebc63ab3855b7f6d70