From a6e2839785ce295dc716e60d0ef2737c7d04c8e3 Mon Sep 17 00:00:00 2001 From: Adrien Destugues Date: Sun, 21 Aug 2016 16:38:47 +0200 Subject: [PATCH] Terminal: remove limitation to 6 tabs - This has been bothering me for quite a while. My screen is wide enough to accomodate more, and even if they don't fit the BTabView, it's still possible to navigate many more tabs with the keyboard shortcuts. - We should still remember to fix BTabView to accomodate more tabs than it can fit with some scrolling support or similar, however. --- src/apps/terminal/TermWindow.cpp | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/src/apps/terminal/TermWindow.cpp b/src/apps/terminal/TermWindow.cpp index 5a233e8411..6350e6a4d6 100644 --- a/src/apps/terminal/TermWindow.cpp +++ b/src/apps/terminal/TermWindow.cpp @@ -64,7 +64,6 @@ #include "TitlePlaceholderMapper.h" -const static int32 kMaxTabs = 6; const static int32 kTermViewOffset = 3; const static int32 kMinimumFontSize = 8; @@ -1213,13 +1212,11 @@ TermWindow::_DoPrint() void TermWindow::_NewTab() { - if (fTabView->CountTabs() < kMaxTabs) { - ActiveProcessInfo info; - if (_ActiveTermView()->GetActiveProcessInfo(info)) - _AddTab(NULL, info.CurrentDirectory()); - else - _AddTab(NULL); - } + ActiveProcessInfo info; + if (_ActiveTermView()->GetActiveProcessInfo(info)) + _AddTab(NULL, info.CurrentDirectory()); + else + _AddTab(NULL); }