From c7621587f607f717a3bafc496de08d51112f2ba2 Mon Sep 17 00:00:00 2001
From: Romain Tripault <rt@seda.digital>
Date: Tue, 6 May 2025 16:53:15 +0200
Subject: [PATCH] moved logic to add/remove collapsed css class in appropriate
 event handlers

---
 manager/assets/modext/core/modx.layout.js | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/manager/assets/modext/core/modx.layout.js b/manager/assets/modext/core/modx.layout.js
index 89730c030f..d379a2149b 100644
--- a/manager/assets/modext/core/modx.layout.js
+++ b/manager/assets/modext/core/modx.layout.js
@@ -427,18 +427,25 @@ Ext.extend(MODx.Layout, Ext.Viewport, {
                 },
                 afterrender: function() {
                     const trigger = Ext.get('modx-leftbar-trigger');
-                    if (this.collapsed) {
-                        trigger.addClass('collapsed');
-                    }
                     trigger.on('click', function() {
                         if (this.collapsed) {
-                            trigger.removeClass('collapsed');
                             this.expand(true);
                         } else {
-                            trigger.addClass('collapsed');
                             this.collapse(true);
                         }
                     }, this);
+                },
+                expand: function () {
+                    const trigger = Ext.get('modx-leftbar-trigger');
+                    if (trigger && trigger.hasClass('collapsed')) {
+                        trigger.removeClass('collapsed');
+                    }
+                },
+                collapse: function () {
+                    const trigger = Ext.get('modx-leftbar-trigger');
+                    if (trigger && !trigger.hasClass('collapsed')) {
+                        trigger.addClass('collapsed');
+                    }
                 }
             }
         };