tui: disable iTerm inline images by default

1 file · +10 -1

M internal/tui/image.go +10 -1


@@ -69,9 +69,18 @@ func detectImageProtocolAuto() ImageProtocol {
69	69		if termProgram == "ghostty" \|\| termProgram == "kitty" {
70	70		return ImageProtocolKitty
71	71		}
72		-	if termProgram == "iTerm.app" \|\| termProgram == "WezTerm" {
	72	+	if termProgram == "WezTerm" {
73	73		return ImageProtocolITerm2
74	74		}
	75	+	// iTerm2's OSC 1337 inline image rendering is incompatible with
	76	+	// zot's full-screen per-row redraw: subsequent row clears erase
	77	+	// all but the first rendered image row, and doNotMoveCursor=1 is
	78	+	// not reliable across iTerm versions. Leave ghostty/kitty untouched
	79	+	// and fall back to the text placeholder in iTerm by default. Users
	80	+	// can still force the protocol with ZOT_INLINE_IMAGES=iterm.
	81	+	if termProgram == "iTerm.app" {
	82	+	return ImageProtocolNone
	83	+	}
75	84		if strings.Contains(strings.ToLower(termProgram), "ghostty") {
76	85		return ImageProtocolKitty
77	86		}

 	if termProgram == "ghostty" || termProgram == "kitty" {
 		return ImageProtocolKitty
 	}
-	if termProgram == "iTerm.app" || termProgram == "WezTerm" {
+	if termProgram == "WezTerm" {
 		return ImageProtocolITerm2
 	}
+	// iTerm2's OSC 1337 inline image rendering is incompatible with
+	// zot's full-screen per-row redraw: subsequent row clears erase
+	// all but the first rendered image row, and doNotMoveCursor=1 is
+	// not reliable across iTerm versions. Leave ghostty/kitty untouched
+	// and fall back to the text placeholder in iTerm by default. Users
+	// can still force the protocol with ZOT_INLINE_IMAGES=iterm.
+	if termProgram == "iTerm.app" {
+		return ImageProtocolNone
+	}
 	if strings.Contains(strings.ToLower(termProgram), "ghostty") {
 		return ImageProtocolKitty
 	}