graesslin created this revision. graesslin added a reviewer: bshah. graesslin added a subscriber: plasma-devel.
REVISION SUMMARY KSld is a library which means the componentName is set to the application name. This works fine for KSMServer, but in KWin the shortcut didn't work. This change manually sets the proper component name (ksmserver) on the QAction and at the same time removes the usage of the KActionCollection. Due to that we can also drop the dependency on XmlGui. TEST PLAN Locking screen through configured shortcut works now in kwin_wayland. REPOSITORY rKSCREENLOCKER KScreenLocker BRANCH drop-action-collection REVISION DETAIL https://phabricator.kde.org/D820 AFFECTED FILES CMakeLists.txt ksldapp.cpp ksldapp.h EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: graesslin, bshah Cc: plasma-devel _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel