broulik created this revision. broulik added a reviewer: Plasma. broulik set the repository for this revision to rPLASMAINTEGRATION Integration for Qt applications in Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY This is part one in a series to restore global menu support in Plasma. Since we inherit from QPlatformTheme we do not get the global menu support for free like Qt's own platform themes inheriting from QGenericUnixTheme would, hence the use of private headers. TEST PLAN - platform integration cmake thing is from kwin - checkDBusGlobalMenuAvailable() et al are from Qt's upstream - requires Qt 5.7 - in KXMLGui applications you still get a traditional menu bar in addition to the global menu, it seems it explicitly sets visible true on the menu bar - in VLC it hides its own menu bar. Upcoming patches will: - restore and cleanup kappmenu (the kded module that acts as a registrar for Qt apps, so they will actually start exporting their menus) REPOSITORY rPLASMAINTEGRATION Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D3085 AFFECTED FILES CMakeLists.txt autotests/CMakeLists.txt cmake/modules/FindQt5PlatformSupport.cmake src/platformtheme/CMakeLists.txt src/platformtheme/kdeplatformtheme.cpp src/platformtheme/kdeplatformtheme.h EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas