D17797: Include applets/22

This revision was automatically updated to reflect the committed changes. Closed by commit R266:d5a2b47c2d37: Include applets/22 (authored by nicolasfella, committed by ndavis). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17797?vs=49294&id=49495 REVIS

D17797: Include applets/22

ndavis accepted this revision. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons BRANCH fix REVISION DETAIL https://phabricator.kde.org/D17797 To: nicolasfella, #breeze, ndavis Cc: ndavis, kde-frameworks-devel, trickyricky26, michaelh, ngraham, bruns

D17797: Include applets/22

nicolasfella updated this revision to Diff 49294. nicolasfella added a comment. - Rebase - add to Breeze dark REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17797?vs=48191&id=49294 BRANCH fix REVISION DETAIL https://phabricator.kde.org/D17797

D17797: Include applets/22

ndavis requested changes to this revision. ndavis added a comment. This revision now requires changes to proceed. This needs to be rebased REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D17797 To: nicolasfella, #breeze, ndavis Cc: ndavis, kde-frameworks-devel, t

D17797: Include applets/22

ndavis added a comment. What is questionable is the purpose for keeping those specific icons in there. All of them can be categorized as device icons, with the car being arguably a device. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D17797 To: nicolasfella,

D17797: Include applets/22

ndavis added a comment. In D17797#382232 , @nicolasfella wrote: > Thinking about it, is there a particular reason why those icons are in the 22 folder in the first place? They are all svgs after all While we do use SVGs for scalability,

D17797: Include applets/22

nicolasfella added a comment. Thinking about it, is there a particular reason why those icons are in the 22 folder in the first place? They are all svgs after all REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D17797 To: nicolasfella, #breeze Cc: kde-frameworks

D17797: Include applets/22

nicolasfella created this revision. nicolasfella added a reviewer: Breeze. nicolasfella added a project: Breeze. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. nicolasfella requested review of this revision. REVISION SUMMARY I noticed that plasma-pa doesn't