Code review comment for lp:~vanvugt/compiz/fix-1020830

Revision history for this message
Sam Spilsbury (smspillaz) wrote :

Yay. Happy to have this in as is (as its non-core), only question would be, do you know why:

- window->alpha () && window->type () & CompWindowTypeDesktopMask)
+ window->type () & CompWindowTypeDesktopMask)

The check for alpha () was removed? I believe that was necessary for some reason, something to do with the transparent cube. Or something.

I guess if it was breaking more things than it fixed, then its not worth having.

review: Approve

« Back to merge proposal