From 6280998306f31064db50a21eac98d193901c546e Mon Sep 17 00:00:00 2001 From: Daniel Hahler Date: Mon, 24 Mar 2014 00:18:35 +0100 Subject: [PATCH] Set c.screen in ewmh.tag and before tags in rules.execute The current premise is that c.screen should be the same as awful.tag.getscreen(t). The addition in `ewmh.tag` appears to be the important part here, changing the order in awful.rules.execute is (maybe) only for consistency across the codebase. --- lib/awful/ewmh.lua.in | 1 + lib/awful/rules.lua.in | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/awful/ewmh.lua.in b/lib/awful/ewmh.lua.in index e69282969..e2fc0b4e3 100644 --- a/lib/awful/ewmh.lua.in +++ b/lib/awful/ewmh.lua.in @@ -147,6 +147,7 @@ function ewmh.tag(c, t) if not t then c.sticky = true else + c.screen = awful.tag.getscreen(t) c:tags({ t }) end end diff --git a/lib/awful/rules.lua.in b/lib/awful/rules.lua.in index cb657cada..32d84a4d1 100644 --- a/lib/awful/rules.lua.in +++ b/lib/awful/rules.lua.in @@ -206,8 +206,8 @@ function rules.execute(c, props, callbacks) if property == "floating" then aclient.floating.set(c, value) elseif property == "tag" then - c:tags({ value }) c.screen = atag.getscreen(value) + c:tags({ value }) elseif property == "switchtotag" and value and props.tag then atag.viewonly(props.tag) elseif property == "height" or property == "width" or