A column can be empty, as acme's can; new panes go where acme's makenewwindow puts them

Newcol makes an empty column; closing a column's last pane leaves it empty with
the keyboard on its tag; Delcol and Joincol alone take a column away; the
session's last pane closing quits. The +New stand-in and replaceStillborn /
stillborn_joiner are gone. Every new pane goes through exec.placeNew, which
follows acme's makenewwindow (active column, empty column whole, blank space,
halving the biggest) or, with Placement pardes, the old rules. ColumnTags is
gone: column tags are always shown. A grip drag shows acme's box cursor.

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

1 files changed, 4 insertions, 3 deletions

diff --git a/src/draw.zig b/src/draw.zig
index 2d757635..b12ab52c 100644
--- a/src/draw.zig
+++ b/src/draw.zig
@@ -163,6 +163,7 @@ pub fn render(p: *Pardes, arena: std.mem.Allocator) !*Surface {
     // the pointer per-text (the macOS I-beam) read this as "target" and
     // afford it the arrow instead.
     if (p.look_hover_preview != null) s.pointer_shape = .target;
+    if (p.drag == .move or p.drag == .column_move) s.pointer_shape = .box;
     for (&s.body_layers) |*layer| layer.rows = 0;
     for (&s.tag_layers) |*layer| layer.rows = 0;
     s.nimages = 0;
@@ -974,14 +975,14 @@ test "place puts every part where the painters and layers put it, in every layou
     const p = try Pardes.init(std.testing.allocator, .{ .tty_only = true, .cols = 100, .rows = 30 });
     defer p.deinit();
     _ = try p.setTestFile("placed\n" ** 80);
-    p.newScratchColumn(0);
+    p.newColumn(0);
+    p.newScratchIn(0, p.activeColumn().?);
     p.newScratchBelow(0);
     p.sync();
     var arena: std.heap.ArenaAllocator = .init(std.testing.allocator);
     defer arena.deinit();
     for (0..16) |step| {
         p.settings.tag_bottom = step & 1 != 0;
-        p.settings.column_tags = step & 2 != 0;
         if (step & 4 != 0) try std.testing.expect(p.executeBuiltinLine(p.active, "Msg a placed notice"));
         if (step == 8) try std.testing.expect(p.executeBuiltinLine(1, "Collapse"));
         if (step == 12) try std.testing.expect(p.executeBuiltinLine(1, "Collapse"));
@@ -1121,7 +1122,7 @@ test "a monkey over notices, prompts, tags and tiny panes at a gui's metrics nev
     var arena: std.heap.ArenaAllocator = .init(std.testing.allocator);
     defer arena.deinit();
     const words = [_][]const u8{ "error:", "\u{754c}", "x", "Shader /tmp/a.glsl:", "e\u{301}", "\t", "long-" ** 20, " " };
-    const commands = [_][]const u8{ "New", "Newcol", "Collapse", "Tagbottom", "ColumnTags", "TreeContext", "Wrap", "Find", "Del", "FocusTint", "Debug" };
+    const commands = [_][]const u8{ "New", "Newcol", "Collapse", "Tagbottom", "TreeContext", "Wrap", "Find", "Del", "FocusTint", "Debug" };
     var text: std.ArrayListUnmanaged(u8) = .empty;
     defer text.deinit(std.testing.allocator);
     for (0..steps) |_| {