D16983: Add 48px firewall-applet* icons

Noah Davis noreply at phabricator.kde.org
Sun Nov 18 22:20:22 GMT 2018


ndavis added a comment.


  In D16983#361758 <https://phabricator.kde.org/D16983#361758>, @aacid wrote:
  
  > I'm sorry but this makes no sense at all, why is firewall-applet-error now suddenly firewall-config?
  >
  > And why do you say users will never need a 48px verison of it?
  
  
  firewall-applet and firewall-config are closely accociated: https://github.com/firewalld/firewalld/tree/master/src
  
  firewall-applet is just an applet for the system tray showing the state of firewalld with an option to open firewall-config, so it makes sense to re-use the firewall-config icon. firewall-applet doesn't show up in the application menu or application menu searches either. Even if it did, only `firewall-applet.svg` would be used. The other `firewall-applet-*` icons will only be seen in the system tray applet itself and never at 48px.

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D16983

To: ndavis, #vdg, #breeze, ngraham
Cc: aacid, ngraham, kde-frameworks-devel, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181118/94c8c908/attachment.html>


More information about the Kde-frameworks-devel mailing list