feat: hide all open bottom panels at once instead of closing them individually

Author: devvaannsh

src/extensionsIntegrated/NoDistractions/main.js

@@ -87,20 +87,20 @@ define(function (require, exports, module) {
      * hide all open panels
      */
     function _hidePanelsIfRequired() {
-        var panelIDs = WorkspaceManager.getAllPanelIDs();
         _previouslyOpenPanelIDs = [];
-        // Loop until no visible panels remain. In a tabbed system, hiding the
-        // active tab may reveal the next tab, so we must iterate.
-        let hiddenSomething = true;
-        while (hiddenSomething) {
-            hiddenSomething = false;
-            for (let i = 0; i < panelIDs.length; i++) {
-                let panel = WorkspaceManager.getPanelForID(panelIDs[i]);
-                if (panel && panel.isVisible()) {
-                    panel.hide();
-                    _previouslyOpenPanelIDs.push(panelIDs[i]);
-                    hiddenSomething = true;
-                }
+
+        // Batch-hide all open bottom panel tabs in one pass, avoiding O(n)
+        // intermediate tab activations and layout recalcs.
+        let hiddenBottomPanels = WorkspaceManager.hideAllOpenBottomPanels();
+        _previouslyOpenPanelIDs = hiddenBottomPanels;
+
+        // Hide any remaining visible panels (e.g. plugin side-panels)
+        let panelIDs = WorkspaceManager.getAllPanelIDs();
+        for (let i = 0; i < panelIDs.length; i++) {
+            let panel = WorkspaceManager.getPanelForID(panelIDs[i]);
+            if (panel && panel.isVisible()) {
+                panel.hide();
+                _previouslyOpenPanelIDs.push(panelIDs[i]);
             }
         }
     }

src/view/PanelView.js

@@ -376,12 +376,52 @@ define(function (require, exports, module) {
         return _openIds.slice();
     }
 
+    /**
+     * Hides every open bottom panel tab in a single batch
+     * @return {string[]} The IDs of panels that were open (useful for restoring later).
+     */
+    function hideAllOpenPanels() {
+        if (_openIds.length === 0) {
+            return [];
+        }
+        let closedIds = _openIds.slice();
+
+        // Remove visual active state from every panel
+        for (let i = 0; i < closedIds.length; i++) {
+            let panel = _panelMap[closedIds[i]];
+            if (panel) {
+                panel.$panel.removeClass("active-bottom-panel");
+            }
+        }
+
+        // Clear internal state BEFORE hiding the container so the
+        // panelCollapsed handler sees an empty _openIds and doesn't
+        // redundantly update the stacks.
+        _openIds = [];
+        _activeId = null;
+
+        if (_$container && _$container.is(":visible")) {
+            Resizer.hide(_$container[0]);
+        }
+
+        _updateBottomPanelTabBar();
+
+        // Fire one EVENT_PANEL_HIDDEN per panel for stack tracking.
+        // No intermediate EVENT_PANEL_SHOWN events are emitted.
+        for (let i = 0; i < closedIds.length; i++) {
+            exports.trigger(EVENT_PANEL_HIDDEN, closedIds[i]);
+        }
+
+        return closedIds;
+    }
+
     EventDispatcher.makeEventDispatcher(exports);
 
     // Public API
     exports.Panel = Panel;
     exports.init = init;
     exports.getOpenBottomPanelIDs = getOpenBottomPanelIDs;
+    exports.hideAllOpenPanels = hideAllOpenPanels;
     //events
     exports.EVENT_PANEL_HIDDEN = EVENT_PANEL_HIDDEN;
     exports.EVENT_PANEL_SHOWN = EVENT_PANEL_SHOWN;

src/view/WorkspaceManager.js

@@ -664,6 +664,7 @@ define(function (require, exports, module) {
     exports.getAllPanelIDs                  = getAllPanelIDs;
     exports.getPanelForID                   = getPanelForID;
     exports.getOpenBottomPanelIDs           = PanelView.getOpenBottomPanelIDs;
+    exports.hideAllOpenBottomPanels         = PanelView.hideAllOpenPanels;
     exports.addEscapeKeyEventHandler        = addEscapeKeyEventHandler;
     exports.removeEscapeKeyEventHandler     = removeEscapeKeyEventHandler;
     exports._setMockDOM                     = _setMockDOM;