Falkon icon - Removing circle & adding outline
Needs RevisionPublic

Authored by saligari on Apr 21 2020, 7:51 PM.

Details

Reviewers
abetts
ndavis
Group Reviewers
VDG
Falkon
Summary

Removed the purple gradient circle that surrounded the falkon, added it as an outline - made some crude adjustments, I'm not proud of. But I did this because, a) I didn't like the circle and b) there is no good reason I can think to have it, this way it has a distinct shape.

Diff Detail

Repository
R266 Breeze Icons
Branch
falkon-icon-patch (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 25635
Build 25653: arc lint + arc unit
saligari created this revision.Apr 21 2020, 7:51 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptApr 21 2020, 7:51 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
saligari requested review of this revision.Apr 21 2020, 7:51 PM
saligari retitled this revision from Removed the purple gradient circle that surrounded the falkon, added it as an outline - made some crude adjustments, I'm not proud of. But I did this because, a) I didn't like the circle and b) there is no good reason I can think to have it, this... to Falkon icon - Removing circle & adding outline.Apr 21 2020, 7:54 PM
saligari edited the summary of this revision. (Show Details)
tfella added a reviewer: VDG.Apr 21 2020, 7:57 PM

Uploaded in 128x128 the old (left) and new (right) icon next to each other.

ndavis requested changes to this revision.Apr 21 2020, 10:35 PM
ndavis added a subscriber: ndavis.

I don't think this is necessary.

This revision now requires changes to proceed.Apr 21 2020, 10:35 PM

I have to agree. This icon isn't my favorite one ever because of the lavender color, but I really like the background circle that the head is slightly escaping from. I think it's a pretty good composition.