From 8e127335cbaa38c5753a701b16c02a69fa1bea61 Mon Sep 17 00:00:00 2001 From: Mike Blumenkrantz Date: Wed, 9 Jan 2013 13:24:24 +0000 Subject: [PATCH] force window border (theme) inset to be applied to new client placement geometry, fixing issues where windows would appear to "slide" across the desktop by the size of the border inset; this was primarily noticeable when using wine applications SVN revision: 82460 --- ChangeLog | 1 + NEWS | 1 + src/bin/e_border.c | 5 +++++ 3 files changed, 7 insertions(+) diff --git a/ChangeLog b/ChangeLog index fa37bf9d1..f8a883053 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,6 +1,7 @@ 2013-01-09 Mike Blumenkrantz * geometry_auto_move option now also corrects applications which fail in their attempts to center windows + * fixed bug where window border insets were not applied to initial positioning geometry, causing them to be placed incorrectly 2013-01-08 David H. Bronke diff --git a/NEWS b/NEWS index c1fdc423c..8f75106da 100644 --- a/NEWS +++ b/NEWS @@ -74,3 +74,4 @@ Fixes: * Fixed bug where disabled slider widgets could be changed with mouse wheel * Fixed bug where fileman config slider widgets were not properly disabled * Tasks gadgets now apply the selected style + * fixed bug where window border insets were not applied to initial positioning geometry, causing them to be placed incorrectly diff --git a/src/bin/e_border.c b/src/bin/e_border.c index dff887895..32766e0e4 100644 --- a/src/bin/e_border.c +++ b/src/bin/e_border.c @@ -8570,13 +8570,18 @@ _e_border_eval(E_Border *bd) (abs((bd->zone->container->h / 2) - bd->y - (bd->h / 2)) < 3))) //bd->y - bd->h/2 is center of container ) e_border_center(bd); + else + _e_border_move_internal(bd, bd->x, bd->y, 1); } + else + _e_border_move_internal(bd, bd->x, bd->y, 1); bd->changes.pos = 1; bd->placed = 1; } } else { + _e_border_move_internal(bd, bd->x, bd->y, 1); bd->changes.pos = 1; bd->placed = 1; }