A new pane with no room under the last goes last all the same

A pane from pane/new or New takes the bottom half of its column's last
pane, as acme's coladd does (cols.c:62-65); when that pane is too short
to give it a tag and 2 rows, the fallback halved the tallest pane and
put the new one under it, mid-column (the 7 9 8 found). It now goes last
all the same, with half the tallest pane's rows (its tag measured at the
column's width). Tty from a pane's ctl already follows acme's
makenewwindow (the active column first, util.c:456-467); fs.md now says
so rather than beside that pane.

Co-Authored-By: Claude Opus 5.5 <[email protected]>

1 files changed, 12 insertions, 4 deletions

diff --git a/src/layout.zig b/src/layout.zig
index bdef9675..644054ed 100644
--- a/src/layout.zig
+++ b/src/layout.zig
@@ -447,8 +447,9 @@ pub fn everyPaneRoomy(p: *Pardes) bool {
     return true;
 }
 
-/// Places `id` in column `c` by halving its tallest unfolded pane, when
-/// that leaves both their rows; false, and `id` in no column, when not.
+/// Places `id` last in column `c`, where acme's coladd puts a new window,
+/// with half the rows of the column's tallest unfolded pane, when that
+/// leaves both their rows; false, and `id` in no column, when not.
 pub fn splitTallest(p: *Pardes, c: usize, id: usize) bool {
     compute(p);
     var tall: ?usize = null;
@@ -457,8 +458,15 @@ pub fn splitTallest(p: *Pardes, c: usize, id: usize) bool {
         if (tall == null or p.rects[k].h > p.rects[tall.?].h) tall = k;
     }
     const v = tall orelse return false;
-    if (p.rects[v].h < minRows(p, v) + minRows(p, id)) return false;
-    splitAt(p, c, id, p.rects[v].y + p.rects[v].h / 2);
+    const h = p.rects[v].h;
+    // Its tag wraps at the column's width: measured there, not at none.
+    p.rects[id].w = p.col_w[c];
+    if (h < minRows(p, v) + minRows(p, id)) return false;
+    snapColWeights(p, c);
+    insert(p, c, p.col_n[c], id);
+    const given = std.math.clamp(h / 2, minRows(p, id), h - minRows(p, v));
+    p.panes[v].?.vweight = @floatFromInt(h - given);
+    p.panes[id].?.vweight = @floatFromInt(given);
     return true;
 }