gladhorn added inline comments.
INLINE COMMENTS
> broulik wrote in daemon.cpp:94
> You could also connect it to the method directly
>
> connect(action, &QAction::triggered, this, &KScreenDaemon::displayButton);
Of course, that was pretty blind of me.
REPOSITORY
R104 KScreen
REVISION DETAI
broulik added inline comments.
INLINE COMMENTS
> daemon.cpp:94
> KGlobalAccel::self()->setGlobalShortcut(action, switchDisplayShortcuts);
> -connect(action, &QAction::triggered, [&](bool) { displayButton(); });
> +connect(action, &QAction::triggered, this, [&](bool) { displayButton()
This revision was automatically updated to reflect the committed changes.
Closed by commit R104:439feca241e8: Use context argument in lambda connect
(authored by gladhorn).
REPOSITORY
R104 KScreen
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D14074?vs=37636&id=37648
REVISION DETAIL
gladhorn created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
gladhorn requested review of this revision.
REPOSITORY
R104 KScreen
BRANCH
x
REVISION DETAIL
https://phabricator.kde.org/D14074
AFFECTED FILES
kded/da
