Use Meta modifier action for edges actions
AbandonedPublic
Actions

Authored by mvourlakos on Mar 8 2018, 11:11 PM.

Details

Reviewers

Group Reviewers

Summary

--Update the activateLauncherMenu edge action in order to take
into account the user-set modifier dbus options. This way
kwin can forward to external apps(except plasmashell)
the activateLauncherMenu signal.

BUG: 391575

Test Plan

Check that the excecution to external apps is working

Diff Detail

Repository

R108 KWin

Branch

edgeapp

Lint

No Linters Available

Unit

No Unit Test Coverage

mvourlakos created this revision.Mar 8 2018, 11:11 PM

Restricted Application added a project: KWin. · View Herald TranscriptMar 8 2018, 11:11 PM

Restricted Application added a subscriber: kwin. · View Herald Transcript

mvourlakos requested review of this revision.Mar 8 2018, 11:11 PM

Sorry, but this is magic. You cannot just redefine the application launcher setting to use a random other config value. You don't know what the user has configured. Imagine the user has set the meta key to a destructive action....

This revision now requires changes to proceed.Mar 9 2018, 5:18 AM

ok no prob...

I thought that

list.at(3) == "activateLauncherMenu"

might be sufficient for this...

next stop a kwin script then...

Revision Contents
Changeset List

			Path	Packages
M			screenedge.h (14 lines)
M			screenedge.cpp (34 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	29049	90762ad		Mar 8 2018, 11:11 PM	★	★

Use Meta modifier action for edges actionsAbandonedPublicActions

Details

Diff Detail

Revision ContentsChangeset List

Diff 29049

screenedge.h

screenedge.cpp

Use Meta modifier action for edges actions
AbandonedPublic
Actions

Revision Contents
Changeset List