hein added a comment.
This is a good patch, but I think it's actually doing too much :). I'd be OK with hardcoding the icon names in the code. I don't think they need to be configurable. Could you simplify the patch? REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D15170 To: marcusb, #plasma Cc: hein, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
