davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. davidedmundson requested review of this revision.
REVISION SUMMARY Breeze has if (!(widget->testAttribute(Qt::WA_WState_Created) || widget->internalWinId())) KWindowSystem::blur(widget->winId(), ...) which makes sense as the style should not be creating surfaces in polish and I don't want to remove it. This is problematic with QMenu::polish which has a subtle behavioural change compared to QMenu::exec in that it calls ensurePolish before creating the surface. Plasma::ContainmentInterface accidentally has this workaround already for parent window setting. BUG: 395262 TEST PLAN Right clicked on desktop icon Looked amazing REPOSITORY R119 Plasma Desktop BRANCH davidedmundson/kcm_clock_qml REVISION DETAIL https://phabricator.kde.org/D14174 AFFECTED FILES containments/desktop/plugins/folder/foldermodel.cpp To: davidedmundson, #plasma Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart