From 003369e649e3ffd59fb6f31180aced39be9c6ebb Mon Sep 17 00:00:00 2001 From: "Adrian C. (anrxc)" Date: Wed, 30 Dec 2009 02:42:42 +0100 Subject: [PATCH] awful.wibox: take wibox border into account when updating struts Function wibox_update_strut would not take the border width into account when calculating struts. When a wibox border was in use clients would overlap the wibox. With a border of 1px we loose 1px of the wibox, but as the wibox border increases it is "pushed" by the border nearest the screen edge and clients steal more and more space. Signed-off-by: Adrian C. (anrxc) Signed-off-by: Julien Danjou --- lib/awful/wibox.lua.in | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/awful/wibox.lua.in b/lib/awful/wibox.lua.in index a55a6c32..8f3bd1ef 100644 --- a/lib/awful/wibox.lua.in +++ b/lib/awful/wibox.lua.in @@ -87,13 +87,13 @@ local function wibox_update_strut(wibox) if not wibox.visible then wibox:struts { left = 0, right = 0, bottom = 0, top = 0 } elseif wprop.position == "top" then - wibox:struts { left = 0, right = 0, bottom = 0, top = wibox.height } + wibox:struts { left = 0, right = 0, bottom = 0, top = wibox.height + wibox.border_width } elseif wprop.position == "bottom" then - wibox:struts { left = 0, right = 0, bottom = wibox.height, top = 0 } + wibox:struts { left = 0, right = 0, bottom = wibox.height + wibox.border_width, top = 0 } elseif wprop.position == "left" then - wibox:struts { left = wibox.width, right = 0, bottom = 0, top = 0 } + wibox:struts { left = wibox.width + wibox.border_width, right = 0, bottom = 0, top = 0 } elseif wprop.position == "right" then - wibox:struts { left = 0, right = wibox.width, bottom = 0, top = 0 } + wibox:struts { left = 0, right = wibox.width + wibox.border_width, bottom = 0, top = 0 } end break end