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