From bab932431f64117ab40397c3007e313e038e4d13 Mon Sep 17 00:00:00 2001
From: Mitchell Hashimoto <mitchell.hashimoto@gmail.com>
Date: Fri, 19 Jan 2024 14:08:09 -0800
Subject: [PATCH 1/2] terminal: erase line right should reset soft wrap state

---
 src/terminal/Terminal.zig  | 34 +++++++++++++++++++++++++++++++++-
 website/app/vt/el/page.mdx |  4 +++-
 2 files changed, 36 insertions(+), 2 deletions(-)

diff --git a/src/terminal/Terminal.zig b/src/terminal/Terminal.zig
index 070196900..b05c6ec91 100644
--- a/src/terminal/Terminal.zig
+++ b/src/terminal/Terminal.zig
@@ -1370,6 +1370,9 @@ pub fn eraseLine(
                 if (cell.attrs.wide_spacer_tail) x -= 1;
             }
 
+            // This resets the soft-wrap of this line
+            row.setWrapped(false);
+
             break :right .{ x, row.lenCells() };
         },
 
@@ -1387,6 +1390,8 @@ pub fn eraseLine(
             break :left .{ 0, x + 1 };
         },
 
+        // Note that it seems like complete should reset the soft-wrap
+        // state of the line but in xterm it does not.
         .complete => .{ 0, row.lenCells() },
 
         else => {
@@ -5193,7 +5198,7 @@ test "Terminal: eraseLine simple erase right" {
     }
 }
 
-test "Terminal: eraseLine resets wrap" {
+test "Terminal: eraseLine resets pending wrap" {
     const alloc = testing.allocator;
     var t = try init(alloc, 5, 5);
     defer t.deinit(alloc);
@@ -5211,6 +5216,33 @@ test "Terminal: eraseLine resets wrap" {
     }
 }
 
+test "Terminal: eraseLine resets wrap" {
+    const alloc = testing.allocator;
+    var t = try init(alloc, 5, 5);
+    defer t.deinit(alloc);
+
+    for ("ABCDE123") |c| try t.print(c);
+    {
+        const row = t.screen.getRow(.{ .active = 0 });
+        try testing.expect(row.isWrapped());
+    }
+
+    t.setCursorPos(1, 1);
+    t.eraseLine(.right, false);
+
+    {
+        const row = t.screen.getRow(.{ .active = 0 });
+        try testing.expect(!row.isWrapped());
+    }
+    try t.print('X');
+
+    {
+        const str = try t.plainString(testing.allocator);
+        defer testing.allocator.free(str);
+        try testing.expectEqualStrings("X\n123", str);
+    }
+}
+
 test "Terminal: eraseLine right preserves background sgr" {
     const alloc = testing.allocator;
     var t = try init(alloc, 5, 5);
diff --git a/website/app/vt/el/page.mdx b/website/app/vt/el/page.mdx
index 45b16bbfd..2d690a1b1 100644
--- a/website/app/vt/el/page.mdx
+++ b/website/app/vt/el/page.mdx
@@ -27,7 +27,9 @@ first or second cell of the two-cell character, both cells should be erased.
 If `n` is `0`, perform an **erase line right** operation. Erase line right
 is equivalent to [Erase Character (ECH)](/vt/ech) with `n` set to the total
 remaining columns from the cursor to the end of the line (and including
-the cursor).
+the cursor). If the line is softwrapped, only the single row is erased;
+it does not erase through the wrap. Further, the wrap state of the row is
+reset such that the line is no longer considered wrapped.
 
 If `n` is `1`, perform an **erase line left** operation. This replaces
 the `n` cells left of and including the cursor with a blank character and

From e9fe14c7501cdbe75ea27f41e8f14918372aa990 Mon Sep 17 00:00:00 2001
From: Mitchell Hashimoto <mitchell.hashimoto@gmail.com>
Date: Fri, 19 Jan 2024 14:31:33 -0800
Subject: [PATCH 2/2] terminal: ECH resets line wrap state in any scenario

---
 src/terminal/Terminal.zig   | 32 +++++++++++++++++++++++++++++++-
 website/app/vt/ech/page.mdx |  3 ++-
 2 files changed, 33 insertions(+), 2 deletions(-)

diff --git a/src/terminal/Terminal.zig b/src/terminal/Terminal.zig
index b05c6ec91..6e8d18ec8 100644
--- a/src/terminal/Terminal.zig
+++ b/src/terminal/Terminal.zig
@@ -1497,6 +1497,9 @@ pub fn eraseChars(self: *Terminal, count_req: usize) void {
         break :end end;
     };
 
+    // This resets the soft-wrap of this line
+    row.setWrapped(false);
+
     const pen: Screen.Cell = .{
         .bg = self.screen.cursor.pen.bg,
     };
@@ -4838,7 +4841,7 @@ test "Terminal: deleteChars split wide character tail" {
     }
 }
 
-test "Terminal: eraseChars resets wrap" {
+test "Terminal: eraseChars resets pending wrap" {
     const alloc = testing.allocator;
     var t = try init(alloc, 5, 5);
     defer t.deinit(alloc);
@@ -4856,6 +4859,33 @@ test "Terminal: eraseChars resets wrap" {
     }
 }
 
+test "Terminal: eraseChars resets wrap" {
+    const alloc = testing.allocator;
+    var t = try init(alloc, 5, 5);
+    defer t.deinit(alloc);
+
+    for ("ABCDE123") |c| try t.print(c);
+    {
+        const row = t.screen.getRow(.{ .active = 0 });
+        try testing.expect(row.isWrapped());
+    }
+
+    t.setCursorPos(1, 1);
+    t.eraseChars(1);
+
+    {
+        const row = t.screen.getRow(.{ .active = 0 });
+        try testing.expect(!row.isWrapped());
+    }
+    try t.print('X');
+
+    {
+        const str = try t.plainString(testing.allocator);
+        defer testing.allocator.free(str);
+        try testing.expectEqualStrings("XBCDE\n123", str);
+    }
+}
+
 test "Terminal: eraseChars simple operation" {
     const alloc = testing.allocator;
     var t = try init(alloc, 5, 5);
diff --git a/website/app/vt/ech/page.mdx b/website/app/vt/ech/page.mdx
index 4d340b528..ba7481075 100644
--- a/website/app/vt/ech/page.mdx
+++ b/website/app/vt/ech/page.mdx
@@ -12,7 +12,8 @@ or equal to 0, adjust `n` to be 1. If `n` is omitted, `n` defaults to 1.
 The rightmost column that can be erased is the rightmost column of the screen.
 The [right margin](#) has no effect on this sequence.
 
-This sequence always unsets the pending wrap state.
+This sequence always unsets the pending wrap state. If the row under the cursor
+is soft-wrapped, then the soft-wrap state is also reset.
 
 For `n` cells up to the rightmost column, blank the cell by replacing it
 with an empty character with the background color colored according to the