[compiz] [PATCH] Add option to Move for disabling server positionsync on each notify

David Reveman
Wed, 11 Jul 2007 23:29:09 -0000

On Mon, 2007-06-25 at 03:26 +0200, Trevi?o wrote:
> The new move plugin interface that makes the applications sync to
> Xserver position at each move notify, also if it could be considered
> important, caused - as negative point - a poor wobbly quality while
> moving windows...
> That's why some kinds of "forks" were made to disable this
new feature
> (see the "bouge" plugin), so I decided, instead of use
"another" plugin
> simply to patch the git move plugins to add an option that will allow
> users to decide which move type should be used.
>
> So pratically, I've looked what has been really changed in commit [1]
> (the one in which David applied the new sync-move), and pratically I
> edited it until I got all working in both the ways. To made this
> selectable by users I've added an "compare_server_motion"
option, that
> is enabled in the xml file by default (so without user interaction, all
> works like before).
>
> Look at the diff, and let me know! :)

OK, just to give a little update on the situation with the move plugin.

Not actually changing the server side position when moving a window is a
very useful feature and something that the move plugin should definitely
support.

However, the way that's been done previously, by modifying w->attrib.x
and w->attrib.y is very bad. It should instead be done by modifying the
CompTransform when painting the window but the reason I haven't
implemented this yet is that the wobbly functionality is not going to
work with this until the new animation framework is in place. So here's
the steps that need to be made before this movement mode will be
supported:

1. Add animation framework
2. Update wobbly plugin to use new animation framework
3. Update move plugin to use new animation framework

-David