From 452cf5549a879ff70b5f72b6801ff83613485d45 Mon Sep 17 00:00:00 2001
From: JoseSerpaCode <joseserpamedinaxd@gmail.com>
Date: Mon, 17 Aug 2026 22:52:06 -0500
Subject: [PATCH] =?UTF-8?q?fix(tablero):=20=C2=AB+=20Sprint=C2=BB=20al=20m?=
 =?UTF-8?q?en=C3=BA,=20para=20que=20=C2=AB+=20Issue=C2=BB=20deje=20de=20ca?=
 =?UTF-8?q?er=20solo?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

El menú de acciones del sprint quitó tres controles de la barra, pero seguían
siendo seis y «+ Issue» continuaba bajando a una segunda fila él solo, que es
peor que envolver del todo.

Crear un sprint también es una acción de sprint, y tampoco es algo de cada día.
Sacarlo de la fila es lo que hace que quepa lo que sí se usa a diario.

De paso, el menú se pinta ahora **siempre que se puede editar**, no solo con un
sprint activo. Antes dependía de `activeSprintId`, así que al meter «+ Sprint»
dentro dejaba de haber forma de crear el primero desde el backlog: el botón
existía solo cuando ya había un sprint. Las acciones de estado —completar,
desactivar, borrar— sí siguen atadas a que haya uno delante, porque en el
backlog no hay nada que completar.

Verificado con capturas a 1890, 1600 y 1280. A los dos primeros anchos cabe en
una fila; a 1280 solo baja «+ Issue», que es un botón y no un amontonamiento.

Tres corridas seguidas de las 199 e2e, sin intermitencias.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
---
 src/pages/w/[sys_tag]/board.astro | 54 +++++++++++++++++++------------
 tests/e2e/test-sprints.spec.ts    |  4 +++
 2 files changed, 37 insertions(+), 21 deletions(-)

diff --git a/src/pages/w/[sys_tag]/board.astro b/src/pages/w/[sys_tag]/board.astro
index 49a67c0..47e495b 100644
--- a/src/pages/w/[sys_tag]/board.astro
+++ b/src/pages/w/[sys_tag]/board.astro
@@ -323,7 +323,7 @@ const metaDescription = `${workspace.name} board on Forge OS. Track issues, spri
           porque se usan una vez cada mucho, no en cada visita. Lo que se usa a
           diario se queda fuera.
         */}
-        {acceso.rol !== 'viewer' && activeSprintId && currentSprintObj && (
+        {acceso.rol !== 'viewer' && (
           <div class="relative" id="sprint-menu-root">
             <button
               id="btn-sprint-menu"
@@ -338,36 +338,48 @@ const metaDescription = `${workspace.name} board on Forge OS. Track issues, spri
             </button>
 
             <div id="sprint-menu" role="menu" class="hidden absolute right-0 top-full mt-1 min-w-[13rem] bg-forge-panel border border-forge-border rounded-xl shadow-xl overflow-hidden z-50 py-1">
-              <button type="button" role="menuitem" id="btn-toggle-sprint" data-sprint-id={activeSprintId} data-current-status={currentSprintObj.status} class="sprint-menu-item">
-                {currentSprintObj.status === 'planned' ? t('board.sprint_start') : currentSprintObj.status === 'active' ? t('board.sprint_complete') : t('board.sprint_reopen')}
-              </button>
-
-              {currentSprintObj.status === 'active' && (
-                <button type="button" role="menuitem" id="btn-deactivate-sprint" data-sprint-id={activeSprintId} class="sprint-menu-item">
-                  {t('board.deactivate')}
-                </button>
+              {/* Crear un sprint también es una acción de sprint, y tampoco es
+                  algo de cada día: sacarlo de la barra es lo que hace que
+                  «+ Issue» —eso sí es de cada día— deje de caer a otra fila. */}
+              {(acceso.rol === 'owner' || acceso.rol === 'editor') && (
+                <>
+                  <button type="button" role="menuitem" id="btn-new-sprint" class="sprint-menu-item">
+                    {t('board.create_sprint')}
+                  </button>
+                  <div class="h-px bg-forge-border/50 my-1"></div>
+                </>
               )}
 
-              {acceso.rol === 'owner' && (
+              {/* Lo que sigue solo tiene sentido con un sprint delante: en el
+                  backlog no hay nada que completar ni que borrar. */}
+              {activeSprintId && currentSprintObj && (
                 <>
-                  <div class="h-px bg-forge-border/50 my-1"></div>
-                  {/* Borrar reorganiza el trabajo de todo el equipo, así que es
-                      de dueños. Los tickets no se van con él. */}
-                  <button type="button" role="menuitem" id="btn-delete-sprint" data-sprint-id={activeSprintId} class="sprint-menu-item hover:text-forge-error">
-                    {t('board.delete_sprint')}
+                  <button type="button" role="menuitem" id="btn-toggle-sprint" data-sprint-id={activeSprintId} data-current-status={currentSprintObj.status} class="sprint-menu-item">
+                    {currentSprintObj.status === 'planned' ? t('board.sprint_start') : currentSprintObj.status === 'active' ? t('board.sprint_complete') : t('board.sprint_reopen')}
                   </button>
+
+                  {currentSprintObj.status === 'active' && (
+                    <button type="button" role="menuitem" id="btn-deactivate-sprint" data-sprint-id={activeSprintId} class="sprint-menu-item">
+                      {t('board.deactivate')}
+                    </button>
+                  )}
+
+                  {acceso.rol === 'owner' && (
+                    <>
+                      <div class="h-px bg-forge-border/50 my-1"></div>
+                      {/* Borrar reorganiza el trabajo de todo el equipo, así que
+                          es de dueños. Los tickets no se van con él. */}
+                      <button type="button" role="menuitem" id="btn-delete-sprint" data-sprint-id={activeSprintId} class="sprint-menu-item hover:text-forge-error">
+                        {t('board.delete_sprint')}
+                      </button>
+                    </>
+                  )}
                 </>
               )}
             </div>
           </div>
         )}
 
-        {(acceso.rol === 'owner' || acceso.rol === 'editor') && (
-          <button id="btn-new-sprint" class="bg-forge-panel border border-forge-border hover:bg-forge-border/40 text-forge-text font-bold px-3 py-1.5 rounded-md text-sm transition-colors">
-            + Sprint
-          </button>
-        )}
-        
         {acceso.rol !== 'viewer' && (
           <button id="btn-new-issue" title={t('board.new_issue_hint')} class="bg-forge-primary text-forge-on-accent font-bold px-3 py-1.5 rounded-md text-sm hover:opacity-90 transition-opacity ml-2 relative group z-50">
             + Issue
diff --git a/tests/e2e/test-sprints.spec.ts b/tests/e2e/test-sprints.spec.ts
index 2a2eb65..d2a2183 100644
--- a/tests/e2e/test-sprints.spec.ts
+++ b/tests/e2e/test-sprints.spec.ts
@@ -60,6 +60,10 @@ test.describe('Sprint Organizer & Isolation', () => {
     await expect(selector).toHaveValue('backlog');
 
     // 2. Create Sprint
+    // Crear un sprint vive en el menú de sprint desde que la barra pasó de
+    // ocho controles: sacarlo de la fila es lo que hace que «+ Issue» —eso sí
+    // es de cada día— deje de caer a una segunda línea.
+    await page.click('#btn-sprint-menu');
     await page.click('#btn-new-sprint');
     await expect(page.locator('#new-sprint-modal')).toBeVisible();