[effects] Drop the logout effect
ClosedPublic

Authored by graesslin on Aug 24 2016, 8:09 AM.

Details

Summary

The new logout design doesn't want to have the vignetting. Thus the
logout effect itself doesn't make any sense any more. All that would
still be used is the logout blur which can also be provided by the
blur effect nowadays for fullscreen windows. As the new logout is a
fullscreen window it should use that one.

The logout effect did one more thing: it kept the vignetting and the
blur once the user selected logout. Now without the vignetting this
would be weird and again doesn't make much sense any more.

So overall I think it's better to just drop the logout effect and use
blur effect in normal way. Neat side advantage: it will also work on
Wayland out-of-the-box.

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 updated this revision to Diff 6205.Aug 24 2016, 8:09 AM
graesslin retitled this revision from to [effects] Drop the logout effect.
graesslin updated this object.
graesslin edited the test plan for this revision. (Show Details)
graesslin added reviewers: KWin, Plasma, VDG.
Restricted Application added a project: KWin. · View Herald TranscriptAug 24 2016, 8:09 AM
Restricted Application added subscribers: kwin, plasma-devel. · View Herald Transcript
mart accepted this revision.Aug 24 2016, 10:36 AM
mart added a reviewer: mart.
mart added a subscriber: mart.

fine with it, greeter should be adapted as well

This revision is now accepted and ready to land.Aug 24 2016, 10:36 AM
This revision was automatically updated to reflect the committed changes.