[compiz] [PATCH] place plugin window matching

David Reveman davidr at novell.com
Mon Mar 5 09:24:58 PST 2007


On Mon, 2007-03-05 at 14:47 +0100, Bellegarde Cedric wrote:
> Here a patch to enable window position/viewport matching in place plugin:
> 
> http://hibbert.univ-lille3.fr/~cbellegarde/Compiz/place_matches.patch

I'm sure a lot of people find this useful. I re-worked your patch a bit
and put it in head. If you have any questions about the changes I made
please let me know. The patch was good, I just wanted to reduce some
code and make it possible to match both vertical and horizontal viewport
position. I haven't actually tried the changes I put in :) so please let
me know if it's not working correctly.

> 
> I don't know how to initialise properly x and y position max value in options 
> initialisation.

MINSHORT, MAXSHORT makes most sense and if we want to actually constrain
the position to the screen size, that must be done dynamically when
matching.

- David



More information about the compiz mailing list