mart created this revision. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel.
REVISION SUMMARY if a framesvgitem has an array as prefix, like FrameSvg { prefix: ["toolbutton-hover", "hover"] } it will use the first available prefix, so on themes that have toolbutton-hover, that one will be used, old themes will continue to use "hover" as prefix TEST PLAN tested the calendar with breeze theme has breeze-widget style toolbuttons, with olt themes as air, the old behavior is still there, switching on the fly works REPOSITORY R242 Plasma Framework (Library) BRANCH phab/framesvgFallback REVISION DETAIL https://phabricator.kde.org/D4827 AFFECTED FILES src/declarativeimports/core/framesvgitem.cpp src/declarativeimports/core/framesvgitem.h src/declarativeimports/plasmastyle/ToolButtonStyle.qml src/desktoptheme/breeze/widgets/button.svgz EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: mart, #plasma Cc: plasma-devel, #frameworks, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol