At 1236896661 time_t, Gregor Best wrote:
> +        local s = c.screen
> +        if s == true then
> +            c:geometry(property.get(c, "floating_geometry"))
> +        end
> +        c.screen = s

No no, your declaration of s shadows a previous one.

I did not test it, but I think your patch has an "unexpected" behaviour
when you move the window from one screen to another: the
floating_geometry property will ne be updated so as soon as you reset
the window to be floating it will go back to its previous screen.

Cheers,
-- 
Julien Danjou
// ᐰ <[email protected]>   http://julien.danjou.info
// 9A0D 5FD9 EB42 22F6 8974  C95C A462 B51E C2FE E5CD
// Trust me.

Attachment: signature.asc
Description: Digital signature

Reply via email to