Linux Enso doesn't behave with multiple monitor setup

[GoogleCodeExporter]

Running Enso (r. 160) on Kubuntu Hardy Heron. On a dual-monitor setup (set
up using urandr), Enso windows calculate position based on the entire
viewport.  Thus, centered windows show up split with two halves on the
inside edge of the two monitors.

Here is my proposed patch, though I'm new to python so you'll have to
excuse anything I break.  However, on my system, Enso windows show up on
the monitor that the mouse cursor is currently on.  The patch covers
multiple files, but also includes Stuart Langridge's 3 patches (I'm also
new to svn, and my local setup uses his code, and I'm not sure how to
separate them all).

The patch will break Windows and OSX versions, as their
platform/graphics.py doesn't have my new getDesktopOffset() function. 
Adding in a dummy function to return (0,0) should do the trick.  Also,
instead of using the mouse cursor, it would probably be more 'humane' to
find the monitor where the cursor (or window) focus is on.  I haven't
figured out how to do that yet.

Original issue reported on code.google.com by sirpe...@gmail.com on 24 Jul 2008 at 9:10

Attachments:

• linux_centering.patch

[GoogleCodeExporter]

Here's an updated version of my patch.  It uses xlib in conjunction with gtk to
determine the monitor that has the current focus.  It also falls back to the 
mouse
cursor position if there is no window focus (i.e. focus is on the desktop)

Original comment by sirpe...@gmail.com on 24 Jul 2008 at 11:29

Attachments:

• linux_centering_2.patch

[GoogleCodeExporter]

I'm loath to apply the patch to the community branch if it contains the other 
patches
(since I've applied my patches to the community branch) and especially if it'll 
break
Windows/OSX. Can you split the patch out so it just does this? Since I've 
applied the
other patches to community, you ought to be able to svn switch to the community
branch, which will merge the two and just show your changes. Grab me by email if
you're unsure.

Original comment by stuart.l...@gmail.com on 22 Aug 2008 at 6:34

[GoogleCodeExporter]

I've fixed my patch and applied it to the community.  I added in the OSX dummy 
code
so that things on that OS should work the same.  I'll have to figure out an OSX 
box
to work with so I can double check that.  I'm not sure how it'll react in 
Windows.

Original comment by sirpe...@gmail.com on 2 Sep 2008 at 9:38

• Changed state: Fixed

Attachments:

• linux_centering_final.patch