D16983: Add 48px firewall-applet icon

2018-11-18 Thread Noah Davis
ndavis updated this revision to Diff 45746. ndavis added a comment. Add more symlinks REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16983?vs=45739&id=45746 BRANCH add-firewall-applet-48px (branched from master) REVISION DETAIL https://phabricato

D16983: Add 48px firewall-applet icon

2018-11-18 Thread Noah Davis
ndavis added a comment. @ngraham Realistically, users will never need the 48px versions of each icon. Can I just make more symlinks of `firewall-config` or should I make icons that actually represent the other icons? REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.o

D16983: Add 48px firewall-applet icon

2018-11-18 Thread Nathaniel Graham
ngraham requested changes to this revision. ngraham added a comment. This revision now requires changes to proceed. The scalable test still fails; you need to add more symlinks: FAIL! : ScalableTest::test_scalable(icons:Applications) The following icons are not available in a scalable

D16983: Add 48px firewall-applet icon

2018-11-18 Thread Noah Davis
ndavis created this revision. ndavis added reviewers: VDG, Breeze. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ndavis requested review of this revision. REVISION SUMMARY Added a symlink from `firewall-config.svg` to `firewall-applet.svg` breeze-icons