diff options
author | Drew DeVault <sir@cmpwn.com> | 2018-10-20 01:18:23 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-10-20 01:18:23 +0200 |
commit | 3b8dd4958311f40701e352bf49b62ae850cdd2f9 (patch) | |
tree | 2a47f763e69105908b5421297b4430af9eb176ee | |
parent | Merge pull request #2884 from c-edw/feature/2867_FixScalingParameter (diff) | |
parent | Fix crash when ending tiling drag (diff) | |
download | sway-3b8dd4958311f40701e352bf49b62ae850cdd2f9.tar.gz sway-3b8dd4958311f40701e352bf49b62ae850cdd2f9.tar.zst sway-3b8dd4958311f40701e352bf49b62ae850cdd2f9.zip |
Merge pull request #2885 from RyanDwyer/fix-tiling-drag-crash
Fix crash when ending tiling drag
-rw-r--r-- | sway/input/seat.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/sway/input/seat.c b/sway/input/seat.c index cffceaae..659ec89b 100644 --- a/sway/input/seat.c +++ b/sway/input/seat.c | |||
@@ -1110,9 +1110,6 @@ static void seat_end_move_tiling(struct sway_seat *seat) { | |||
1110 | int after = edge != WLR_EDGE_TOP && edge != WLR_EDGE_LEFT; | 1110 | int after = edge != WLR_EDGE_TOP && edge != WLR_EDGE_LEFT; |
1111 | 1111 | ||
1112 | container_detach(con); | 1112 | container_detach(con); |
1113 | if (old_parent) { | ||
1114 | container_reap_empty(old_parent); | ||
1115 | } | ||
1116 | 1113 | ||
1117 | // Moving container into empty workspace | 1114 | // Moving container into empty workspace |
1118 | if (target_node->type == N_WORKSPACE && edge == WLR_EDGE_NONE) { | 1115 | if (target_node->type == N_WORKSPACE && edge == WLR_EDGE_NONE) { |
@@ -1135,6 +1132,10 @@ static void seat_end_move_tiling(struct sway_seat *seat) { | |||
1135 | workspace_insert_tiling(new_ws, con, after); | 1132 | workspace_insert_tiling(new_ws, con, after); |
1136 | } | 1133 | } |
1137 | 1134 | ||
1135 | if (old_parent) { | ||
1136 | container_reap_empty(old_parent); | ||
1137 | } | ||
1138 | |||
1138 | // This is a bit dirty, but we'll set the dimensions to that of a sibling. | 1139 | // This is a bit dirty, but we'll set the dimensions to that of a sibling. |
1139 | // I don't think there's any other way to make it consistent without | 1140 | // I don't think there's any other way to make it consistent without |
1140 | // changing how we auto-size containers. | 1141 | // changing how we auto-size containers. |