From 248df8e7aadeca395007857dd7f82d4505ace6a6 Mon Sep 17 00:00:00 2001 From: "Claude Opus 4.7" Date: Mon, 4 May 2026 16:57:37 +0200 Subject: [PATCH 1/4] docs(input): document copy_to_clipboard arguments --- src/input/Binding.zig | 25 +++++++++++++++++++++++++ 1 file changed, 25 insertions(+) diff --git a/src/input/Binding.zig b/src/input/Binding.zig index 52ab82e93..1cf925401 100644 --- a/src/input/Binding.zig +++ b/src/input/Binding.zig @@ -347,6 +347,31 @@ pub const Action = union(enum) { reset, /// Copy the selected text to the clipboard. + /// + /// Valid arguments: + /// + /// - `plain` + /// + /// Copy the selection as plain text only. + /// + /// - `vt` + /// + /// Copy the selection as plain text, preserving terminal escape + /// sequences (such as colors and styles). + /// + /// - `html` + /// + /// Copy the selection as HTML, preserving colors and styles as + /// HTML markup. + /// + /// - `mixed` (default) + /// + /// Place multiple representations on the clipboard at once + /// (e.g. plain text and HTML), each tagged with its content type + /// so the receiving OS or application can pick the most appropriate + /// representation when pasting. + /// + /// If no argument is given, defaults to `mixed`. copy_to_clipboard: CopyToClipboard, /// Paste the contents of the default clipboard. From b1b01741f6c0d694bb9e19daa1a77874676a84ea Mon Sep 17 00:00:00 2001 From: "Claude Opus 4.7" Date: Mon, 4 May 2026 16:57:56 +0200 Subject: [PATCH 2/4] docs(input): document navigate_search arguments --- src/input/Binding.zig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/input/Binding.zig b/src/input/Binding.zig index 1cf925401..5b8750f7c 100644 --- a/src/input/Binding.zig +++ b/src/input/Binding.zig @@ -423,6 +423,8 @@ pub const Action = union(enum) { /// Navigate the search results. If there is no active search, this /// is not performed. + /// + /// Valid arguments: `previous`, `next`. navigate_search: NavigateSearch, /// Start a search if it isn't started already. This doesn't set any From df44c6dd8344544d614e4dbf121b60e7149244db Mon Sep 17 00:00:00 2001 From: "Claude Opus 4.7" Date: Mon, 4 May 2026 16:58:10 +0200 Subject: [PATCH 3/4] docs(input): document close_tab arguments --- src/input/Binding.zig | 16 +++++++++++++++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/src/input/Binding.zig b/src/input/Binding.zig index 5b8750f7c..251ce603b 100644 --- a/src/input/Binding.zig +++ b/src/input/Binding.zig @@ -711,7 +711,21 @@ pub const Action = union(enum) { /// Close the current tab and all splits therein, close all other tabs, or /// close every tab to the right of the current one depending on the mode. /// - /// If the mode is not specified, defaults to closing the current tab. + /// Valid arguments: + /// + /// - `this` (default) + /// + /// Close the current tab and all splits within it. + /// + /// - `other` + /// + /// Close every tab in the current window except the current tab. + /// + /// - `right` + /// + /// Close every tab to the right of the current tab. + /// + /// If no argument is given, defaults to `this`. /// /// This might trigger a close confirmation popup, depending on the value /// of the `confirm-close-surface` configuration setting. From 5874ce633c8e49a4b1bf38c336e1196c735f83cb Mon Sep 17 00:00:00 2001 From: Lukas <134181853+bo2themax@users.noreply.github.com> Date: Tue, 5 May 2026 09:47:06 +0200 Subject: [PATCH 4/4] Apply suggestions from code review Co-authored-by: kat <65649991+00-kat@users.noreply.github.com> Co-authored-by: Lukas <134181853+bo2themax@users.noreply.github.com> --- src/input/Binding.zig | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/src/input/Binding.zig b/src/input/Binding.zig index 251ce603b..d60f2933b 100644 --- a/src/input/Binding.zig +++ b/src/input/Binding.zig @@ -348,7 +348,7 @@ pub const Action = union(enum) { /// Copy the selected text to the clipboard. /// - /// Valid arguments: + /// Valid values: /// /// - `plain` /// @@ -370,8 +370,6 @@ pub const Action = union(enum) { /// (e.g. plain text and HTML), each tagged with its content type /// so the receiving OS or application can pick the most appropriate /// representation when pasting. - /// - /// If no argument is given, defaults to `mixed`. copy_to_clipboard: CopyToClipboard, /// Paste the contents of the default clipboard. @@ -424,7 +422,7 @@ pub const Action = union(enum) { /// Navigate the search results. If there is no active search, this /// is not performed. /// - /// Valid arguments: `previous`, `next`. + /// Valid values: `previous`, `next`. navigate_search: NavigateSearch, /// Start a search if it isn't started already. This doesn't set any @@ -708,10 +706,9 @@ pub const Action = union(enum) { /// of the `confirm-close-surface` configuration setting. close_surface, - /// Close the current tab and all splits therein, close all other tabs, or - /// close every tab to the right of the current one depending on the mode. + /// Close the specified tabs and all splits therein. /// - /// Valid arguments: + /// Valid values: /// /// - `this` (default) /// @@ -725,8 +722,6 @@ pub const Action = union(enum) { /// /// Close every tab to the right of the current tab. /// - /// If no argument is given, defaults to `this`. - /// /// This might trigger a close confirmation popup, depending on the value /// of the `confirm-close-surface` configuration setting. close_tab: CloseTabMode,