fix(tabbookmark): avoid unexpected behavior when keep_last_tab is unset

Bush2021 · Bush2021 · commit e5d902be4028 · 2026-01-01T00:27:10.000-05:00
For `HandleMiddleClick`, expensive UI operations (like
`GetTopContainerView` and `IsOnOneTab`) are now bypassed before checking
if the feature was disabled.

For `HandleKeepTab`, the function will not force exit full-screen mode
and close the find bar before verifying if the feature was disabled.
diff --git a/src/tabbookmark.cc b/src/tabbookmark.cc
@@ -150,6 +150,10 @@ bool HandleRightClick(const MOUSEHOOKSTRUCT* pmouse) {
 
 // Preserve the last tab when the middle button is clicked on the tab.
 bool HandleMiddleClick(const MOUSEHOOKSTRUCT* pmouse) {
+  if (!config.IsKeepLastTab()) {
+    return false;
+  }
+
   const POINT pt = pmouse->pt;
   HWND hwnd = WindowFromPoint(pt);
   const NodePtr top_container_view = HandleFindBar(hwnd, pt);
@@ -304,6 +308,10 @@ bool TabBookmarkMouseHandler(WPARAM wParam, LPARAM lParam) {
 }
 
 int HandleKeepTab(WPARAM wParam) {
+  if (!config.IsKeepLastTab()) {
+    return 0;
+  }
+
   if (!(wParam == 'W' && IsKeyPressed(VK_CONTROL) && !IsKeyPressed(VK_SHIFT)) &&
       !(wParam == VK_F4 && IsKeyPressed(VK_CONTROL))) {
     return 0;
