mart added inline comments. INLINE COMMENTS
> ndavis wrote in configuration-icons.svg:17 > I couldn't see any issues in the SVG code. @mart Where is `showbackground` > used in the UI so that I can test it? in the patch, it adds a new icon in the handle, tough you need this plus D25591 <https://phabricator.kde.org/D25591> and D25592 <https://phabricator.kde.org/D25592> to have it working. the idea is to have this button that can add and remove the background from a plasmoid switching between the full frame and inverted colors with drop shadow. right now is like an a with a rectangular background behing, that could mean literally anything (was also thinking to switch the icon between this one and an a with a shadow without rectangle) it's shown in the screenshot in D25591 <https://phabricator.kde.org/D25591> tough i fear there doesn't exist an icon that can communicate this unambiguously REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D25590 To: mart, #plasma, davidedmundson Cc: davidedmundson, ndavis, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
