diff options
author | Ryan Dwyer <ryandwyer1@gmail.com> | 2018-08-20 13:24:57 +1000 |
---|---|---|
committer | Ryan Dwyer <ryandwyer1@gmail.com> | 2018-08-20 13:24:57 +1000 |
commit | 771ad7bffafdb9fc06aa81f8fc1f33c05f1026d2 (patch) | |
tree | 3e776d5688bcec25102f540db2e04293cdf6bcbc /sway/tree/view.c | |
parent | 08a69f267a96a5b0591222ef1d0f40208e46d484 (diff) |
Fix jumping views
Fixes #2490.
To be honest I'm not sure why this fixes the issue.
I observed that I could only make the view jump if I resized it to the
smallest possible size first. Then I had a suspicion that we were
accidentally factoring in the title and border sizes into the view size
when it uses CSD. So I changed that and it appears to have fixed the
jumping issue.
I guess when we factor the title and borders in, we send a configure to
the surface with a size smaller than the minimum, and it comes back with
a surface at the minimum size. We interpret this as an unexpected
resize, and this somehow makes it jump.
Diffstat (limited to 'sway/tree/view.c')
-rw-r--r-- | sway/tree/view.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/sway/tree/view.c b/sway/tree/view.c index 2c0c1aa9..7bf7325a 100644 --- a/sway/tree/view.c +++ b/sway/tree/view.c @@ -236,7 +236,12 @@ void view_autoconfigure(struct sway_view *view) { view->border_top = false; } - switch (view->border) { + enum sway_container_border border = view->border; + if (view->using_csd) { + border = B_NONE; + } + + switch (border) { case B_NONE: x = con->x; y = con->y + y_offset; |