broulik created this revision. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel.
REVISION SUMMARY FrameSvgItem already updates the margins in response to that. TEST PLAN `FrameSvgItem` has connect(m_frameSvg, SIGNAL(repaintNeeded()), this, SLOT(doUpdate())); and in doUpdate does m_margins->update(); m_fixedMargins->update(); I still get tons of binding loop warnings on e.g. "topMargin" in Plasma REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D4688 AFFECTED FILES src/declarativeimports/core/framesvgitem.cpp EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma, drosca Cc: plasma-devel, #frameworks, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol