From 799c94ff489e9d3e2a4291bb27348b4a905cce96 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 1bae18b5..e458d047 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