Add app icon for the Kirogi Drone control app
ClosedPublic

Authored by trickyricky26 on May 12 2019, 10:31 PM.

Details

Summary

Adds an app icon for @hein's upcoming Kirogi app.

Test Plan

The Kirogi icon:

At 100% size:

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.May 12 2019, 10:31 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptMay 12 2019, 10:31 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
trickyricky26 requested review of this revision.May 12 2019, 10:31 PM
trickyricky26 edited the test plan for this revision. (Show Details)May 12 2019, 10:33 PM
trickyricky26 added a reviewer: VDG.
apol added a subscriber: apol.May 12 2019, 10:50 PM

Let's add it when the application exists?

hein added a comment.May 13 2019, 2:14 PM

@trickyricky26 It's great! What do you think about adding the detailed SVG as 256 version?

In D21169#464291, @apol wrote:

Let's add it when the application exists?

We should do it beforehand so the app gets the proper Breeze icon on launch day.

hein added a comment.May 13 2019, 5:02 PM

I'd say it's up to the Breeze maintainers. From my end I'm very interested in the app having a Breeze-compatible icon, because I'd like it to integrate well with KDE's workspaces and other apps. Review by the Breeze community is very appreciated to help with that. Application code needs to ship the icon anyway though for systems that don't have the theme and can't be reasonably expected to provide every app icon, so it's not vital to be included with the theme.

the icon theme can be considered to be VDG-maintained these days, andI think this passes the visual design test, but also the most important thing is that you as the app author like it!

However if this is the actual icon you want to ship with the app, do we even need it to be in the breeze-icons framework? If it's the exact same icon, and you're going to ship it in your app's packaging/bundle anyway, I'm wondering what the value is.

Should I land this or abandon it?
It doesn't really make a difference if the icon is in Breeze in addition to being shipped with the app, but IMO it would also be strange to not have this Breeze-style icon for a KDE app in Breeze.

ognarb added a subscriber: ognarb.Sep 21 2019, 5:09 PM

kde.org/applications use the icons from breeze-icons. So I think it's worth adding to breeze-icons or improving how the icon are fetched in kde.org/applications.

ngraham accepted this revision.Sep 21 2019, 5:16 PM

Shipit!

This revision is now accepted and ready to land.Sep 21 2019, 5:16 PM
  • A tiny bit more optimized
Closed by commit R266:faff4ff246af: Add app icon for the Kirogi Drone control app (authored by Rafael Brandmaier <rafael.brandmaier@kdemail.net>). · Explain WhySep 21 2019, 5:29 PM
This revision was automatically updated to reflect the committed changes.