[effects] Put Window Aperture and eye On Screen in an exclusive category
ClosedPublic

Authored by zzag on Sun, Jul 29, 1:09 PM.

Details

Summary

Having these two enabled doesn't make sense. Also, Window Aperture + eye
On Screen looks really weird.

Test Plan

Opened the Desktop Effects KCM, both the Window Aperture effect and the
eye On Screen effect are in an exclusive category:

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.
zzag created this revision.Sun, Jul 29, 1:09 PM
Restricted Application added a project: KWin. · View Herald TranscriptSun, Jul 29, 1:09 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Sun, Jul 29, 1:09 PM
zzag edited the test plan for this revision. (Show Details)Sun, Jul 29, 1:10 PM

What should we do about minimize effects? Move them to a separate category?

zzag retitled this revision from [effects] Put Window Aperture and eye On Screen in an exclusive group to [effects] Put Window Aperture and eye On Screen in an exclusive category.Sun, Jul 29, 1:12 PM
zzag edited the test plan for this revision. (Show Details)
davidedmundson accepted this revision.Sun, Jul 29, 1:24 PM
davidedmundson added a subscriber: davidedmundson.

What should we do about minimize effects? Move them to a separate category?

Go for it

This revision is now accepted and ready to land.Sun, Jul 29, 1:24 PM
ngraham accepted this revision.Sun, Jul 29, 2:08 PM
ngraham added a subscriber: ngraham.

What should we do about minimize effects? Move them to a separate category?

Go for it

+1.

Also I've never understood the "this might remind you of something" text for Eye On Screen. What is it supposed to remind me of? I don't get it.

This revision was automatically updated to reflect the committed changes.