Code review comment for lp:~cimi/overlay-scrollbar/fix-compiz-stacking

Revision history for this message
Andrea Cimitan (cimi) wrote :

> Hey Cimi, I've got some question about that one.
>
> Can you please explain why that patch is needed?
>
> Why have you actually chosen to implement it that way?
>
> Isn't that EWMH atom already wrapped as a simple GDK function?

I've decided to implement it in that way, because it's the only one I got it working with compiz :-) sam spilsbury approved it.
Do you have other ideas? Another one I had was to use XQueryTree to get the real parent, but was working for metacity (with bugs) and not for compiz, and was more expensive.

« Back to merge proposal