Depends on D13740
Details
Details
- Reviewers
davidedmundson - Group Reviewers
KWin - Commits
- R108:0be5010c0a6f: [effects/minimizeanimation] Port to TimeLine
Minimized/Unminimized System Setting window, still works.
Diff Detail
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.
Comment Actions
Drop
if (!m_animations.isEmpty()) { effects->addRepaintFull(); }
in favor of
effects->addRepaintFull();
!m_animations.isEmpty() is redundant because when postPaintScreen is
called, we know that at least one animation is active. If there is only
one animation and it is about to be removed in the while loop, we still
would like to repaint(to "clean" up after the animation). So, in either case,
we would like to schedule repainting.
Yet, scheduling full repaint is not the greatest thing, but that was before.