[Breeze] [Bug 399253] archive-* icons could use improvement

[Noah Davis]

https://bugs.kde.org/show_bug.cgi?id=399253

Noah Davis <noahadvs at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REPORTED                    |RESOLVED
         Resolution|---                         |FIXED
   Version Fixed In|                            |5.53
      Latest Commit|                            |https://commits.kde.org/bre
                   |                            |eze-icons/aace21a156679810e
                   |                            |ff5b3ebc63ab3855b7f6d70

--- Comment #1 from Noah Davis <noahadvs at gmail.com> ---
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

-- 
You are receiving this mail because:
You are the assignee for the bug.