yukijoou/kwin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[effects/magiclamp] Don't reset timeline

Browse source 
Summary:
Resetting timeline while animating a window causes some visual "glitches",
see videos below. Even though we can't use `QTimer::toggleDirection`,
that's safe to delete code which resets timeline because `prePaintScreen`
figures out correct direction of the timeline.

Before

{F5809835, layout=center, size=full}

After

{F5809850, layout=center, size=full}

Test Plan:
* Set duration to 5000
* Furiously click an icon in the task manager

Reviewers: #kwin, davidedmundson

Reviewed By: davidedmundson

Subscribers: kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D12253
This commit is contained in:
Vlad Zagorodniy 2018-04-20 21:10:30 +03:00
parent 75fff17612
commit 913f92ae9a
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
effects/magiclamp/magiclamp.cpp
View file

@ -342,9 +342,9 @@ void MagicLampEffect::slotWindowMinimized(EffectWindow* w)
return;
if (!mTimeLineWindows.contains(w)) {
mTimeLineWindows.insert(w, new QTimeLine(mAnimationDuration, this));
mTimeLineWindows[w]->setCurrentTime(0);
mTimeLineWindows[w]->setCurveShape(QTimeLine::LinearCurve);
}
mTimeLineWindows[w]->setCurrentTime(0);
}
void MagicLampEffect::slotWindowUnminimized(EffectWindow* w)
@ -353,9 +353,9 @@ void MagicLampEffect::slotWindowUnminimized(EffectWindow* w)
return;
if (!mTimeLineWindows.contains(w)) {
mTimeLineWindows.insert(w, new QTimeLine(mAnimationDuration, this));
mTimeLineWindows[w]->setCurrentTime(mAnimationDuration);
mTimeLineWindows[w]->setCurveShape(QTimeLine::LinearCurve);
}
mTimeLineWindows[w]->setCurrentTime(mAnimationDuration);
}
bool MagicLampEffect::isActive() const