it is possible to see a window in it's pre-animation state during unmap animations
Bug #864476 reported by
Sam Spilsbury
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Compiz Animations Plugin |
Fix Released
|
Medium
|
Sam Spilsbury | ||
| Compiz Main Plugins |
Fix Released
|
Medium
|
Sam Spilsbury | ||
| Unity |
Fix Released
|
Medium
|
Sam Spilsbury | ||
| compiz-plugins-main (Ubuntu) |
Fix Released
|
Undecided
|
Sam Spilsbury | ||
| Oneiric |
Fix Released
|
Undecided
|
Sam Spilsbury | ||
| unity (Ubuntu) |
Fix Released
|
Medium
|
Sam Spilsbury | ||
| Oneiric |
Invalid
|
Undecided
|
Unassigned | ||
Bug Description
This is visible in eg, shade and unshade animations where a slight artefact is displayed of what the window's full decoration state would have been like before the window frame is actually adjusted post animation, because we disable the animation plugin painting windows in preparePaint rather than donePaint, so there's a race condition where one frame will be painted with the artefact.
TESTCASE:
No artefacts when shading windows
Related branches
lp:~compiz-team/compiz-animation-plugin/compiz-animation-plugin.fix_864476
- Robert Carr (community): Approve
-
Diff: 91 lines (+45/-23)1 file modifiedsrc/animation.cpp (+45/-23)
| Changed in compiz-animation-plugin: | |
| status: | New → In Progress |
| importance: | Undecided → Medium |
| assignee: | nobody → Sam Spilsbury (smspillaz) |
| milestone: | none → 0.9.6 |
| Changed in unity: | |
| status: | New → In Progress |
| importance: | Undecided → High |
| assignee: | nobody → Sam Spilsbury (smspillaz) |
| importance: | High → Medium |
| milestone: | none → 4.22.0 |
| Changed in compiz-plugins-main: | |
| status: | New → In Progress |
| assignee: | nobody → Sam Spilsbury (smspillaz) |
| importance: | Undecided → Medium |
| milestone: | none → 0.9.6 |
| Changed in compiz-plugins-main (Ubuntu): | |
| status: | New → In Progress |
| assignee: | nobody → Sam Spilsbury (smspillaz) |
| Changed in unity (Ubuntu): | |
| status: | New → In Progress |
| Changed in compiz-plugins-main: | |
| status: | In Progress → Fix Committed |
| Changed in compiz-animation-plugin: | |
| status: | In Progress → Fix Committed |
| Changed in unity: | |
| status: | In Progress → Fix Committed |
| Changed in compiz-plugins-main (Ubuntu): | |
| status: | In Progress → Fix Committed |
| Changed in unity: | |
| milestone: | 4.22.0 → 4.24.0 |
| Changed in unity (Ubuntu Oneiric): | |
| status: | In Progress → Invalid |
Revision history for this message
| Martin Pitt (pitti) wrote Please test proposed package | #1 |
| tags: | added: verification-needed |
Revision history for this message
| Pierre-Olivier Megret (sorrodje) wrote | #2 |
| tags: |
added: verification-done removed: verification-needed |
| description: | updated |
| Changed in compiz-animation-plugin: | |
| status: | Fix Committed → Fix Released |
| Changed in compiz-plugins-main: | |
| status: | Fix Committed → Fix Released |
| Changed in unity: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #3 |
| Changed in compiz-plugins-main (Ubuntu): | |
| status: | Fix Committed → Fix Released |
| Changed in compiz-plugins-main (Ubuntu Oneiric): | |
| status: | Fix Committed → Fix Released |
| Changed in unity (Ubuntu): | |
| status: | Invalid → Fix Released |
| assignee: | nobody → Sam Spilsbury (smspillaz) |
| importance: | Undecided → Medium |
To post a comment you must log in.
Hello Sam, or anyone else affected,
Accepted compiz-plugins-main into oneiric-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https:/