davidedmundson created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY We need to setup our attachments when the wl_surface is created. This gets recreated when a window is hidden and shown. QPlatformSurfaceCreated is emitted once, when the backing store is created, not when the wl_surface is created, despite it's similar sounding name. QEvent::Show is too soon, as this is before the platform gets setVisible We do however, always get an expose event. QEvent::Hide weirdly comes after the platform has done its thing. TEST PLAN See attached test on Qt 5.9 Still has window decos after hide/show This needs testing on Qt 5.7 before merging REPOSITORY R135 Integration for Qt applications in Plasma BRANCH testing REVISION DETAIL https://phabricator.kde.org/D5746 AFFECTED FILES src/platformtheme/kwaylandintegration.cpp src/platformtheme/kwaylandintegration.h tests/CMakeLists.txt tests/windowdecotest.cpp To: davidedmundson, #plasma Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, lukas
