D11880: Add firewall-config and firewall-applet icons
Sven Mauch
noreply at phabricator.kde.org
Mon Sep 24 08:11:34 BST 2018
svenmauch added a comment.
In D11880#330810 <https://phabricator.kde.org/D11880#330810>, @ndavis wrote:
> While orange may mean "warning", it could mean any kind of warning. In this case, it's a warning that an extreme setting is being used, but there is a lock in the corner to show that it is at least safe.
Exactly what I thought!
In D11880#330865 <https://phabricator.kde.org/D11880#330865>, @ndavis wrote:
> Like this? This is #4d4d4d (icon grey), the standard color for small breeze icons, not black.
> F6281673: Screenshot_20180924_000854.png <https://phabricator.kde.org/F6281673>F6281669: Screenshot_20180924_000627.png <https://phabricator.kde.org/F6281669>
Looks great! +1
In D11880#330788 <https://phabricator.kde.org/D11880#330788>, @ngraham wrote:
> I think that `firewall-applet` looks maybe a bit too plain at its 22px size. The wall seems to need something.
I think the 22px firewall-applet looks really good and would fit perfectly with the notifications popup and task icons. I'm not sure why the 48px version is colored though?
Thank you for the superb work @ndavis & @bruns!
REPOSITORY
R266 Breeze Icons
BRANCH
firewalld-icons (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D11880
To: ndavis, #vdg, #breeze, ngraham
Cc: bruns, abetts, alex-l, svenmauch, kde-frameworks-devel, ngraham, michaelh
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180924/2d62fa1d/attachment.html>
More information about the Kde-frameworks-devel
mailing list