From a89a40304472deb48d26d30c840239995647be33 Mon Sep 17 00:00:00 2001
From: brobert <borja@brobert.net>
Date: Fri, 17 Oct 2025 16:07:16 +0200
Subject: [PATCH] fix: corregir a11y en Hourglass/Popover; quitar role
 redundante

Co-authored-by: aider (openrouter/openai/gpt-5) <aider@aider.chat>
---
 apps/web/src/lib/ui/data/TaskItem.svelte    | 2 --
 apps/web/src/lib/ui/feedback/Popover.svelte | 9 ++++++++-
 apps/web/src/lib/ui/icons/Hourglass.svelte  | 4 ++--
 apps/web/src/lib/ui/layout/AppShell.svelte  | 2 +-
 4 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/apps/web/src/lib/ui/data/TaskItem.svelte b/apps/web/src/lib/ui/data/TaskItem.svelte
index f3b16b8..e64134e 100644
--- a/apps/web/src/lib/ui/data/TaskItem.svelte
+++ b/apps/web/src/lib/ui/data/TaskItem.svelte
@@ -576,8 +576,6 @@
 		background: var(--color-primary);
 		color: #fff;
 	}
-	.btn.secondary {
-	}
 	.btn.ghost {
 		background: transparent;
 	}
diff --git a/apps/web/src/lib/ui/feedback/Popover.svelte b/apps/web/src/lib/ui/feedback/Popover.svelte
index ef112ae..9d45811 100644
--- a/apps/web/src/lib/ui/feedback/Popover.svelte
+++ b/apps/web/src/lib/ui/feedback/Popover.svelte
@@ -72,7 +72,14 @@
 </script>
 
 {#if open}
-  <div class="popover-overlay" on:click={close} />
+  <div
+    class="popover-overlay"
+    role="button"
+    tabindex="0"
+    aria-label="Cerrar"
+    on:click={close}
+    on:keydown={(e) => { if (e.key === 'Enter' || e.key === ' ') { e.preventDefault(); close(); } }}
+  ></div>
   <div
     class="popover-panel"
     role="dialog"
diff --git a/apps/web/src/lib/ui/icons/Hourglass.svelte b/apps/web/src/lib/ui/icons/Hourglass.svelte
index a48e38a..d5abc26 100644
--- a/apps/web/src/lib/ui/icons/Hourglass.svelte
+++ b/apps/web/src/lib/ui/icons/Hourglass.svelte
@@ -1,8 +1,8 @@
 <script lang="ts">
   export let size: number = 16;
   export let className: string = '';
-  export let ariaLabel?: string;
-  export let title?: string;
+  export let ariaLabel: string | undefined;
+  export let title: string | undefined;
 </script>
 
 <svg
diff --git a/apps/web/src/lib/ui/layout/AppShell.svelte b/apps/web/src/lib/ui/layout/AppShell.svelte
index 66aa99a..783b805 100644
--- a/apps/web/src/lib/ui/layout/AppShell.svelte
+++ b/apps/web/src/lib/ui/layout/AppShell.svelte
@@ -38,7 +38,7 @@
   <slot />
 </main>
 
-<nav class="tabbar" role="navigation" aria-label="Navegación inferior">
+<nav class="tabbar" aria-label="Navegación inferior">
   <a href="/app" class:active={$page.url.pathname === '/app'} aria-label="Tareas">
     <span class="icon">✅</span>
     <span class="label">Tareas</span>