Repeated snapping causes a cumulative offset about bunsen-utilities HOT 8 CLOSED

Can you test this, too? 😄 I think it is fixed, but I'm not sure how you reproduce this issue exactly.

from bunsen-utilities.

Now that I switched to using the parent window (Hjdskes@5bc730a), this issue is back:

~/code/bunsen-utilities master $= 12:54:57
> ./bl-aerosnap --right
Restoring to: 201 115 852 615

~/code/bunsen-utilities master $= 12:54:58
> cat /tmp/bl-aerosnap-1000 
{}
~/code/bunsen-utilities master $= 12:55:01
> ./bl-aerosnap --right
Snapping window from : 201 115 852 629
Snapping window to: 683.0 0 683.0

~/code/bunsen-utilities master $= 12:55:03
> cat /tmp/bl-aerosnap-1000 
{"<class 'Xlib.display.Window'>(0x00200543)-0": [201, 115, 852, 629]}
~/code/bunsen-utilities master $= 12:55:09
> ./bl-aerosnap --right
Restoring to: 201 115 852 629

~/code/bunsen-utilities master $= 12:55:15
> cat /tmp/bl-aerosnap-1000 
{}
~/code/bunsen-utilities master $= 12:55:25
> ./bl-aerosnap --right
Snapping window from : 201 114 852 643
Snapping window to: 683.0 0 683.0

Cumulative snapping increases the height by (in my case, perhaps this is my window frame's height) 14 pixels every time.

from bunsen-utilities.

Indeed, this ugly hack resolves my issue:

diff --git a/bl-aerosnap b/bl-aerosnap
index 785b16c..de7cb6f 100755
--- a/bl-aerosnap
+++ b/bl-aerosnap
@@ -198,7 +198,7 @@ def get_window_geometry(win):

     :param win: the X Window object"""
     geom = win.get_geometry()
-    return [geom.x, geom.y, geom.width, geom.height]
+    return [geom.x, geom.y, geom.width, geom.height-14]

 def get_window_id(ewmh, win):
     """Returns a unique identifier for an X Window, made up of its "to string" and proces' PID.

Not sure what to do about this. Perhaps I need to use the x,y from the parent but the width,height from the actual window. Will test this now.

from bunsen-utilities.

It is definitely the window frames that are messing things up: undecorated windows work fine still.

from bunsen-utilities.

Latest push seems to have fixed the issue. However, I occasionally get this (bl-aerosnap says it is moving the window to the right half of the screen, but as you can see it does not occupy the full right half, even though the coordinates and dimensions reported are correct):

from bunsen-utilities.

@Hjdskes is your code now ready to merge into deuterium?

from bunsen-utilities.

No, it's not done yet.

from bunsen-utilities.

For lack of interest, bl-aerosnap has been removed from bunsen-utilities.

from bunsen-utilities.