Merge pull request #2482 from max397574/push-snqtmwzyxmwx

feat: make too big tab indices go to last tab
This commit is contained in:
Mitchell Hashimoto
2024-10-24 20:02:57 -07:00
committed by GitHub
3 changed files with 12 additions and 9 deletions

View File

@ -514,11 +514,14 @@ class TerminalController: BaseTerminalController {
return
}
} else {
// Tabs are 0-indexed here, so we subtract one from the key the user hit.
finalIndex = Int(tabIndex - 1)
// The configured value is 1-indexed.
guard tabIndex >= 1 else { return }
// If our index is outside our boundary then we use the max
finalIndex = min(Int(tabIndex - 1), tabbedWindows.count - 1)
}
guard finalIndex >= 0 && finalIndex < tabbedWindows.count else { return }
guard finalIndex >= 0 else { return }
let targetWindow = tabbedWindows[finalIndex]
targetWindow.makeKeyAndOrderFront(nil)
}

View File

@ -466,11 +466,10 @@ pub fn gotoLastTab(self: *Window) void {
pub fn gotoTab(self: *Window, n: usize) void {
if (n == 0) return;
const max = self.notebook.nPages();
if (max == 0) return;
const page_idx = std.math.cast(c_int, n - 1) orelse return;
if (page_idx < max) {
self.notebook.gotoNthTab(page_idx);
self.focusCurrentTab();
}
self.notebook.gotoNthTab(@min(page_idx, max - 1));
self.focusCurrentTab();
}
/// Toggle tab overview (if present)

View File

@ -297,7 +297,8 @@ pub const Action = union(enum) {
/// Go to the last tab (the one with the highest index)
last_tab: void,
/// Go to the tab with the specific number, 1-indexed.
/// Go to the tab with the specific number, 1-indexed. If the tab number
/// is higher than the number of tabs, this will go to the last tab.
goto_tab: usize,
/// Toggle the tab overview.