Differential D14688 Diff 42743 autotests/integration/effects/scripts/fullScreenEffectTest.js

Changeset View

Standalone View

View Options

autotests/integration/effects/scripts/fullScreenEffectTest.js

  • This file was added.
1​effects['desktopChanged(int,int)'].connect(function(old, current) {
2​ var stackingOrder = effects.stackingOrder;
3​ for (var i=0; i<stackingOrder.length; i++) {
4​ var w = stackingOrder[i];
5​ //1 second long random animation. Final param indicates fullscreen
6​ w.anim1 = effect.animate(w, Effect.Scale, 1000, 1.4, 0.2, 0, QEasingCurve.Linear, 0, true);
zzagUnsubmitted
Done

Also, it would be great to pass an object because it's really hard to say what each number is, e.g.

effect.animate({
    window: w,
    type: Effect.Scale,
    ...,
    fullScreen: true
});
zzag: Also, it would be great to pass an object because it's really hard to say what each number is…
davidedmundsonAuthorUnsubmitted
Done

Sure for production code, but the goal here is to test the public API.

I've added a second test with the unboxing version and I've added a comment here.

davidedmundson: Sure for production code, but the goal here is to test the public API. I've added a second…
zzagUnsubmitted
Done

Looks like the second test can't be located:

testdata ./scripts/fullScreenEffectTestMulti.js could not be located!
zzag: Looks like the second test can't be located: testdata ./scripts/fullScreenEffectTestMulti.
7​ }
8​});