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
- Branch
- effects-minimizeanimation-port-to-timeline
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 427 Build 427: arc lint + arc unit
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.