Differential D3077

Call new toggle signal for applet de-/activation in order to fix non-closing launchers on Meta (and also on Active Screenedges)
AbandonedPublic
Actions

Authored by romangg on Oct 17 2016, 6:26 AM.

Details

Reviewers
graesslin
Summary

Let KWin use the new toggle signal for showing / hiding the launcher.

Dependent on this diff in plasma-framework: https://git.reviewboard.kde.org/r/129204/
A more extensive description of the problem and the proposed solution can be found there.

Diff Detail

Repository
R108 KWin
Lint
Lint Skipped
Unit
Unit Tests Skipped
romangg updated this revision to Diff 7437.Oct 17 2016, 6:26 AM
romangg retitled this revision from to Call new toggle signal for applet de-/activation in order to fix non-closing launchers on Meta (and also on Active Screenedges).
romangg updated this object.
romangg edited the test plan for this revision. (Show Details)
romangg added a reviewer: graesslin.
romangg set the repository for this revision to R108 KWin.
romangg added a project: Plasma.
romangg added a subscriber: Plasma.
Restricted Application edited projects, added KWin; removed Plasma. · View Herald TranscriptOct 17 2016, 6:26 AM
Restricted Application added a subscriber: kwin. · View Herald Transcript
romangg updated this revision to Diff 7473.Oct 17 2016, 6:48 PM

Adjustements to second revision of the Diff in Plasma-Frameworks on Reviewboard: https://git.reviewboard.kde.org/r/129204/

But it reverts all changes here, since no changes are necessary anymore in KWin. In the end I will abandon this Diff, if the concept of the second revision of the Plasma-Frameworks Diff is accepted.

romangg abandoned this revision.Oct 24 2016, 1:37 PM

Abandoned because we now use the other solution in https://git.reviewboard.kde.org/r/129204/ making this diff not necessary anymore.

Revision Contents
Changeset List

PathPackages
Moptions.cpp (2 lines)
Mscreenedge.cpp (2 lines)

Diff 7473

View Options

options.cpp

Loading...
View Options

screenedge.cpp

Loading...