graesslin created this revision. graesslin added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY So for the _KDE_NET_WM_COLOR_SCHEME property was set in KStyle. This has a few disadvantages: - it only works for QWidgets - it requires a style inheriting from KStyle This change brings the behavior into the plasma-integration QPT. It's an integration feature for our platform theme as it's a hint for KWin. The existing event filter in the X11Integration is used to monitor for all PlatformSurface events and sets the required property on the newly created X11 window. In addition it monitors for the ApplicationPaletteChange to update the property as needed on all toplevel windows. TEST PLAN Removed code from kstyle and used the KColorSchemeManager test application. REPOSITORY rPLASMAINTEGRATION Integration for Qt applications in Plasma BRANCH color-scheme-atom REVISION DETAIL https://phabricator.kde.org/D3044 AFFECTED FILES CMakeLists.txt autotests/CMakeLists.txt src/platformtheme/CMakeLists.txt src/platformtheme/x11integration.cpp src/platformtheme/x11integration.h EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: graesslin, #plasma Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
