[Differential] [Request, 23 lines] D820: Improve setting global lock shortcut

graesslin (Martin Gräßlin) noreply at phabricator.kde.org
Thu Jan 14 14:43:51 UTC 2016


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


More information about the Plasma-devel mailing list