Use a Kolf icon with nice background and shadows
ClosedPublic

Authored by trickyricky26 on Jan 4 2019, 8:05 PM.

Details

Summary

Improve the Kolf icon with a golf ball with hills and the sky in the background as well as the Breeze-typical shadows.
As suggested in T10243: Some KDE applications could use better icons.

oldnew

Diff Detail

Repository
R266 Breeze Icons
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
trickyricky26 created this revision.Jan 4 2019, 8:05 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptJan 4 2019, 8:05 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
trickyricky26 requested review of this revision.Jan 4 2019, 8:05 PM
trickyricky26 edited the summary of this revision. (Show Details)Jan 4 2019, 8:11 PM
trickyricky26 added reviewers: VDG, KDE Games.
yurchor added a subscriber: yurchor.Jan 4 2019, 8:17 PM

+1 Much better than old, as for me.

ndavis accepted this revision.Jan 4 2019, 8:38 PM
ndavis added a subscriber: ndavis.

Yes, this is much better. I think I'll accept it as it is.

This is how it looks at 100% size.

This revision is now accepted and ready to land.Jan 4 2019, 8:38 PM
ndavis added a comment.Jan 4 2019, 9:24 PM

It seems there are not actually any members in the KDE Games group. I think I'll land this now so that it can be included with KF5.54.

This revision was automatically updated to reflect the committed changes.
aacid added a subscriber: aacid.Jan 4 2019, 9:52 PM

It seems there are not actually any members in the KDE Games group. I think I'll land this now so that it can be included with KF5.54.

There's a whole mailing list ;)

ndavis added a comment.EditedJan 4 2019, 10:21 PM

It seems there are not actually any members in the KDE Games group. I think I'll land this now so that it can be included with KF5.54.

There's a whole mailing list ;)

I suppose there is. Should I have waited for someone from that mailing list to accept the diff even if it meant waiting until after KF5.54 gets tagged?

ngraham added a subscriber: ngraham.Jan 5 2019, 3:17 AM

I love it! This new icon is great--head and shoulders above the previous one.