[Merge] lp:~ubuntu-mate-dev/compiz/fix-1559371 into lp:compiz

[Sam Spilsbury]

Sounds like the relevant client is broken. You shouldn't have to force a
zero height and width - all that does is set the client to the minimum size.

I had a look at the UI definition for mate-menu[0] and it looks awfully
suspect. Looks as though its trying to simulate override-redirect behaviour
(for instance, disabling resize, focus, skip_taskbar, skip_pager, no
decorations) etc, but doesn't set the override redirect flag. Granted,
there are usecases for this.

I also noticed that the type hint is set to "menu". I'm not sure if this
actually forces the type to be set to "Menu" or if the toolkit only allows
that if the window is also override redirect. You'd have to use xwininfo
and xprop on the menu to find out for sure.

If I recall correctly, I remember SLED10 used to have a menu like MATE does
today. It also did not set the override redirect flag. I remember that it
had a strange window /type/ and that's how it got around being treated as a
"managed" window.

Having a look at the place plugin's source code, I saw that window types
that are "Dock", "Util", or "Toolbar" are ignored by the placement plugin.
It might be worth changing the window type of the menu to one of those
things.

I would not suggest working around these types of problems inside the
winrules plugin. That's bound to cause more pain down the line.

[0]
https://bitbucket.org/ubuntu-mate/mate-menu/src/90f004c920568ccf8d3b2fa1bfcf9537049b1b78/data/mate-menu.glade?at=master&fileviewer=file-view-default

On Fri, Apr 1, 2016 at 9:07 PM, Martin Wimpress <martin at flexion.org> wrote:

> I can't see why the application behaves this way. It appears to be
> introduced by the Compiz Placement plugin and therefore this winrule is
> required.
>
> This "fix" has been well tested by the Ubuntu MATE team and some Linux
> Mint users too.
> --
>
> https://code.launchpad.net/~ubuntu-mate-dev/compiz/fix-1559371/+merge/289588
> Your team Compiz Maintainers is requested to review the proposed merge of
> lp:~ubuntu-mate-dev/compiz/fix-1559371 into lp:compiz.
>



-- 
Sam Spilsbury

https://code.launchpad.net/~ubuntu-mate-dev/compiz/fix-1559371/+merge/289588
Your team Ubuntu MATE Developers is subscribed to branch lp:~ubuntu-mate-dev/compiz/fix-1559371.