Differential D4253

redesign gwenview icon
AbandonedPublic
Actions

Authored by varlesh on Jan 23 2017, 3:37 PM.

Details

Reviewers
andreask
cfeck
sandsmark
Group Reviewers
Breeze
Summary
  • colorize
  • add shadows
  • pixelated align
  • change form
  • clear code

Compare:


Zoomed:

Diff Detail

Repository
R266 Breeze Icons
Lint
Lint Skipped
Unit
Unit Tests Skipped
varlesh updated this revision to Diff 10458.Jan 23 2017, 3:37 PM
varlesh retitled this revision from to redesign gwenview icon.
varlesh updated this object.
varlesh edited the test plan for this revision. (Show Details)
varlesh added a reviewer: Breeze.
varlesh set the repository for this revision to R266 Breeze Icons.
Restricted Application added a project: Frameworks. · View Herald TranscriptJan 23 2017, 3:37 PM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
mck182 added a subscriber: mck182.Jan 23 2017, 4:20 PM
mck182 removed a subscriber: mck182.
andreaska added a subscriber: andreaska.Jan 23 2017, 11:24 PM

first of all, thanks for starting the discussion I think also that the gwenview icon could get an update, but I miss something in the update in addition the dark eye background look to hard.

varlesh added a comment.Jan 24 2017, 4:30 AM

Ok, but maybe this?
New:


SVG:
gwenview.svg9 KBDownload

Oxygen:

varlesh added a comment.Jan 24 2017, 4:34 AM

For compare:

varlesh updated this revision to Diff 10479.Jan 24 2017, 7:29 AM

version 2, diff updated

alex-l added a subscriber: alex-l.Jan 24 2017, 1:38 PM

As mentioned on Telegram, I'm for keeping the current one (please update the screenshot of it because it's not how it currently look)

andreaska added a comment.Jan 24 2017, 2:24 PM

I also think that we shouldn't update the icon.

varlesh added a comment.Jan 26 2017, 4:12 PM

gwenview devs please tell what you like icon for Breeze 1 or 2?
1:1

zoomed

SVG's:

1-gwenview.svg13 KBDownload

2- gwenview.svg8 KBDownload

adelsarto added a subscriber: adelsarto.Jan 26 2017, 4:46 PM

I also think that we shouldn't update the icon. I like the current one.

aacid resigned from this revision.Jan 26 2017, 9:29 PM
aacid removed a reviewer: aacid.

I don't have an opinion, not good in saying which icon is nicer than others.

varlesh added a comment.Jan 28 2017, 7:37 AM

ok guys, next variant:


1:1

Also this icon use style as mime image

andreaska added a comment.Jan 28 2017, 9:06 AM

Varlesh you do awesome icon work, but gwenview was changed some time ago so no icon redesign needed sorry.
Please work on other icons.

varlesh abandoned this revision.Jan 28 2017, 1:04 PM

Revision Contents
Changeset List

PathPackages
Micons/apps/48/gwenview.svg (286 lines)

Diff 10479

View Options

icons/apps/48/gwenview.svg

Loading...