Bug 399253

Summary:	archive-* icons could use improvement
Product:	[Plasma] Breeze	Reporter:	Nate Graham <nate>
Component:	Icons	Assignee:	visual-bugs-null
Status:	RESOLVED FIXED
Severity:	normal	CC:	kainz.a, noahadvs
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://commits.kde.org/breeze-icons/aace21a156679810eff5b3ebc63ab3855b7f6d70	Version Fixed/Implemented In:	5.53
Sentry Crash Report:

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