From 18051a4f31ffcb9d7ec3d39515803edfc446aac2 Mon Sep 17 00:00:00 2001
From: Lizandro Guarnizo <77708265+lizandrogd@users.noreply.github.com>
Date: Wed, 24 Jun 2026 08:11:52 -0500
Subject: [PATCH] =?UTF-8?q?fix(sidebar):=20centrar=20labels=20(styles.css?=
 =?UTF-8?q?=20ten=C3=ADa=20mayor=20especificidad),=20preservar=20scroll=20?=
 =?UTF-8?q?al=20toggle?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 assets/css/styles.css         | 18 +++++++++++-------
 shared/components/sidebar.php | 11 +++++++++--
 2 files changed, 20 insertions(+), 9 deletions(-)

diff --git a/assets/css/styles.css b/assets/css/styles.css
index 46be704..639b43f 100644
--- a/assets/css/styles.css
+++ b/assets/css/styles.css
@@ -150,7 +150,7 @@ body {
 /* ── Etiquetas de categoría ───────────────────────────── */
 .sidebar-menu li.sidebar-section {
     border-bottom: none;
-    padding: 10px 16px 6px;
+    padding: 10px 30px 6px;
     font-size: 0.65rem;
     font-weight: 800;
     text-transform: uppercase;
@@ -158,18 +158,22 @@ body {
     color: rgba(255, 255, 255, 0.55);
     pointer-events: auto;
     cursor: pointer;
-    display: flex;
-    align-items: center;
-    justify-content: space-between;
-    gap: 5px;
+    display: block;
+    text-align: center;
     position: relative;
     z-index: 1;
     border-radius: 6px;
+    border-left: 3px solid transparent;
     margin: 4px 8px 0;
-    transition: background .15s, color .15s;
+    transition: background .15s, color .15s, border-color .15s;
 }
 .sidebar-menu li.sidebar-section:hover { color: rgba(255,255,255,.9); background: rgba(255,255,255,.07); }
-.sidebar-menu li.sidebar-section .sb-cat-label { display:flex; align-items:center; gap:4px; }
+.sidebar-menu li.sidebar-section.has-active {
+    border-left-color: rgba(255,255,255,.7);
+    background: rgba(255,255,255,.1);
+    color: #fff;
+}
+.sidebar-menu li.sidebar-section .sb-cat-label { display:inline-flex; align-items:center; gap:4px; }
 .sidebar-menu li.sidebar-section::after { display: none; }
 
 /* ── Badge dentro de un item ──────────────────────────── */
diff --git a/shared/components/sidebar.php b/shared/components/sidebar.php
index 8fafbce..564b60c 100644
--- a/shared/components/sidebar.php
+++ b/shared/components/sidebar.php
@@ -159,7 +159,7 @@ if (class_exists('ModuleRegistry', false)) {
 ?>
         <li class="sidebar-section<?= $_catHasActive ? ' has-active' : '' ?>"
             data-cat="<?= $catId ?>"
-            onclick="sbToggleCat('<?= $catId ?>', this)">
+            onclick="sbToggleCat('<?= $catId ?>',this,event)">
             <span class="sb-cat-label">
                 <i class="<?= htmlspecialchars($catData['icon'], ENT_QUOTES, 'UTF-8') ?> me-1"></i>
                 <?= htmlspecialchars($catData['label'], ENT_QUOTES, 'UTF-8') ?>
@@ -380,7 +380,11 @@ if (class_exists('ModuleRegistry', false)) {
         }
     }
 
-    window.sbToggleCat = function(catId, sectionEl) {
+    window.sbToggleCat = function(catId, sectionEl, ev) {
+        if (ev) { ev.preventDefault(); ev.stopPropagation(); }
+        var nav = document.getElementById('erp-sidebar');
+        var savedScroll = nav ? nav.scrollTop : 0;
+
         var group = document.getElementById(catId);
         if (!group) return;
         var isCollapsed = group.classList.contains('collapsed');
@@ -400,6 +404,9 @@ if (class_exists('ModuleRegistry', false)) {
             });
         }
         setCollapsed(arr);
+
+        // Restaurar scroll del sidebar para que no salte al top
+        if (nav) { requestAnimationFrame(function(){ nav.scrollTop = savedScroll; }); }
     };
 
     // ── Buscador de menú ─────────────────────────────────────────