From 9ca57a8364893541eef3a56689f6597d6f01f38b Mon Sep 17 00:00:00 2001 From: blueusername Date: Fri, 2 Sep 2016 20:15:00 +0100 Subject: [PATCH] Fix a bug moving between a tiled and a floating screen. Check if any of the window is on the screen it is moved to, not just the topmost leftmost edge. Signed-off-by: blueusername --- objects/client.c | 2 +- objects/screen.c | 14 ++++++++++++++ objects/screen.h | 1 + 3 files changed, 16 insertions(+), 1 deletion(-) diff --git a/objects/client.c b/objects/client.c index 838112d18..84de9daf3 100644 --- a/objects/client.c +++ b/objects/client.c @@ -1506,7 +1506,7 @@ client_resize_do(client_t *c, area_t geometry, bool force_notice) bool java_is_broken = true; screen_t *new_screen = c->screen; - if(!screen_coord_in_screen(new_screen, geometry.x, geometry.y)) + if(!screen_area_in_screen(new_screen, geometry)) new_screen = screen_getbycoord(geometry.x, geometry.y); if(c->geometry.width == geometry.width diff --git a/objects/screen.c b/objects/screen.c index 09ba56bc3..bee9aee74 100644 --- a/objects/screen.c +++ b/objects/screen.c @@ -731,6 +731,20 @@ screen_coord_in_screen(screen_t *s, int x, int y) && (y >= s->geometry.y && y < s->geometry.y + s->geometry.height); } +/** Is there any overlap between the given geometry and a given screen? + * \param screen The logical screen number. + * \param geom The geometry + * \return True if there is any overlap between the geometry and a given screen. + */ +bool +screen_area_in_screen(screen_t *s, area_t geom) +{ + return (geom.x < s->geometry.x + s->geometry.width) + && (geom.x + geom.width > s->geometry.x ) + && (geom.y < s->geometry.y + s->geometry.height) + && (geom.y + geom.height > s->geometry.y); +} + void screen_update_workarea(screen_t *screen) { area_t area = screen->geometry; diff --git a/objects/screen.h b/objects/screen.h index 6b419c801..45876062c 100644 --- a/objects/screen.h +++ b/objects/screen.h @@ -50,6 +50,7 @@ void screen_class_setup(lua_State *L); void screen_scan(void); screen_t *screen_getbycoord(int, int); bool screen_coord_in_screen(screen_t *, int, int); +bool screen_area_in_screen(screen_t *, area_t); int screen_get_index(screen_t *); area_t display_area_get(void); void screen_client_moveto(client_t *, screen_t *, bool);