add pix icon
AcceptedPublic

Authored by mbruchert on Feb 7 2020, 5:39 PM.

Details

Reviewers
camiloh
ndavis
Group Reviewers
VDG

Diff Detail

Repository
R266 Breeze Icons
Branch
pix (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 22187
Build 22205: arc lint + arc unit
mbruchert created this revision.Feb 7 2020, 5:39 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptFeb 7 2020, 5:39 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
mbruchert requested review of this revision.Feb 7 2020, 5:39 PM
mbruchert updated this revision to Diff 75186.Feb 7 2020, 5:42 PM
  • fix pix icon
mbruchert updated this revision to Diff 75270.Feb 8 2020, 5:21 PM
  • change shadow color

Hello, Mathis!

Pix is a gallery application, right? From the proposed icon, one cannot tell that...

the Icon is supposed to be just a remake of the original icon which looks similar

The original icon (on my Android device, at least) looks like a framed pixelated painting, so still holds the "picture" association.

In your proposal, it looks like a whiteboard with sticky notes, or a window with sunset, or...

mbruchert updated this revision to Diff 78228.Mar 22 2020, 3:18 PM
  • change Icon to look like a partly pixalated photograph
mbruchert updated this revision to Diff 78229.Mar 22 2020, 3:20 PM
  • change shadow color

@camiloh, how do you feel about this?

ndavis accepted this revision.Apr 20 2020, 3:37 AM
This revision is now accepted and ready to land.Apr 20 2020, 3:37 AM