Replace Workspace::slotWindowQuickTileFoo by a quickTileWindow with argument
ClosedPublic

Authored by graesslin on Jul 19 2017, 4:44 PM.

Details

Summary

Thanks to std::bind we don't need that many different slots to setup the
global shortcut connections. Instead we can have one shared
implementation which takes the argument and passes it to the window.

To support std::bind arguments in kwinbindings the initShortcut method
and dependencies are adjusted as well as a new macro is added.

As I don't want to include abstract_client.h in workspace.h a new enum
is created for the quick tiling flags used in Workspace. This caused a
larger refactoring as the change to an enum class also caused quite some
changes.

Test Plan

Affected test cases still pass

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
graesslin created this revision.Jul 19 2017, 4:44 PM
Restricted Application added a project: KWin. · View Herald TranscriptJul 19 2017, 4:44 PM
Restricted Application added subscribers: kwin, plasma-devel. · View Herald Transcript
This revision was automatically updated to reflect the committed changes.