⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@ctrl-alt-d
Copy link
Collaborator

@ctrl-alt-d ctrl-alt-d commented Jan 22, 2026

Accessible Navigation Component

Summary

This PR implements a fully accessible navigation system following WCAG 2.1 guidelines, based on the accessible-astro-starter patterns.

Changes

New Components

Refactored

  • Nav.astro - Simplified wrapper that imports the Navigation component
  • Moved components from src/layouts/components/ to src/components/ (Astro convention)

Accessibility Features

Keyboard Navigation (Desktop)

Key Action
Tab Navigate between focusable elements
ArrowLeft / ArrowRight Move between top-level menu items (closes open dropdowns)
ArrowDown Open dropdown and focus first item
ArrowUp / ArrowDown Navigate within dropdown items
Enter / Space Open dropdown and focus first item
Escape Close dropdown and return focus to trigger

Mobile Navigation

  • Uses native <details> / <summary> elements for expandable menu sections
  • No complex JavaScript required for accordion behavior
  • Escape closes the entire mobile menu
  • Simple <a> links for all menu items

ARIA Support

  • aria-expanded on dropdown triggers
  • aria-haspopup on buttons with submenus
  • aria-controls linking buttons to their menus
  • aria-current="page" on active menu items
  • aria-label on navigation landmarks

@ctrl-alt-d ctrl-alt-d changed the title Accessible Navigation Component fix: Accessible Navigation Component Jan 22, 2026
@ctrl-alt-d ctrl-alt-d added the 🦮 a11y Accessibility Issue label Jan 22, 2026
@ctrl-alt-d ctrl-alt-d changed the title fix: Accessible Navigation Component Fix: #35 - Feat: add dynamic menu Jan 26, 2026
@ctrl-alt-d ctrl-alt-d closed this Jan 26, 2026
@ctrl-alt-d ctrl-alt-d changed the title Fix: #35 - Feat: add dynamic menu Feat: Ejemplo menú nav con navegación accesible Jan 26, 2026
@ctrl-alt-d ctrl-alt-d reopened this Jan 26, 2026
@ctrl-alt-d ctrl-alt-d changed the base branch from feat_dynamic_menu to main January 26, 2026 16:48
@ctrl-alt-d ctrl-alt-d marked this pull request as draft January 26, 2026 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🦮 a11y Accessibility Issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants