From 8194d555ecc9f803da4796c7734732b276e25a09 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 | 2 ++ lib/awful/rules.lua.in | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/lib/awful/ewmh.lua.in b/lib/awful/ewmh.lua.in index e6928296..b53320cb 100644 --- a/lib/awful/ewmh.lua.in +++ b/lib/awful/ewmh.lua.in @@ -9,6 +9,7 @@ local client = client local screen = screen local ipairs = ipairs local math = math +local atag = require("awful.tag") --- Implements EWMH requests handling. -- awful.ewmh @@ -147,6 +148,7 @@ function ewmh.tag(c, t) if not t then c.sticky = true else + c.screen = atag.getscreen(t) c:tags({ t }) end end diff --git a/lib/awful/rules.lua.in b/lib/awful/rules.lua.in index cb657cad..32d84a4d 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