D11880: Add firewall-config and firewall-applet icons

2018-09-21 Thread Sven Mauch
svenmauch added a comment. I like the regular icon, looks great! Though I'm unsure about the panic icon. It communicates "all is well" to me instead of "all network traffic blocked". Maybe there is a better way to achieve this. An orange icon with a spread out hand in front of it? Just

D11880: Add firewall-config and firewall-applet icons

2018-09-21 Thread Sven Mauch
svenmauch added a comment. Yup! +1 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

D11880: Add firewall-config and firewall-applet icons

2018-09-24 Thread Sven Mauch
svenmauch added a comment. In 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

D7446: [Places panel] Add a Recently Used item by default

2018-09-25 Thread Sven Mauch
svenmauch added a comment. While I wouldn't use it (yet?) I think it's a great addition and would benefit a lot of people. I give it a +1, especially if D15739 gets approved aswell. ;) REPOSITORY R241 KIO REVISION DETAIL

D15739: [Places panel] Don't show Root by default

2018-09-25 Thread Sven Mauch
svenmauch added a comment. All arguments make sense. It even looks a lot better without the red folder icon. +1 REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D15739 To: ngraham, #dolphin, #vdg Cc: svenmauch, broulik, acrouthamel, kde-frameworks-devel, michaelh,