Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add: mejoras para la accesibilidad, sobretodo para los screen readers #135

Merged
merged 2 commits into from
Feb 26, 2024

Conversation

IreneAragon
Copy link
Contributor

@IreneAragon IreneAragon commented Feb 25, 2024

Descripción

He agregado algunos aria-labels que faltaban y he mejorado la información de otros como por ejemplo para los links o botones que se abren en una nueva pestaña, siempre hay que avisar al usuario de que va a salir de la página en la que está navegando.

He añadido además un poco de código Javascript en el archivo ThemeToggle.astro para que el screen reader pueda leerle al usuario el color theme actual y así pueda decidir mejor si quiere cambiarlo o no.

Problema solucionado

Encontré un error en la cuenta atrás, el screen reader solo leía el primer texto correspondiente al día, y ahí acababa todo, no seguía leyendo el resto de la página, se "bloqueaba" ahí el lector.
Para solucionarlo he agregado role="timer" al div que lo contiene, de esta manera el screen reader puede leer el texto de la cuenta atrás y seguir leyendo el resto de la página.

Comprobación de cambios

  • He revisado localmente los cambios para asegurarme de que no haya errores ni problemas.
  • He probado estos cambios en múltiples dispositivos y navegadores para asegurarme de que la landing page se vea y funcione correctamente.
  • He actualizado la documentación, si corresponde.

Copy link

vercel bot commented Feb 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
la-velada-web-oficial ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 26, 2024 7:33pm

Copy link

⚠️ Esta Pull Request tiene conflictos. Por favor, resuelvelos antes de que podamos evaluar los cambios.

Copy link

✅ ¡Los conflictos han sido resuletos! Un colaborador revisará pronto la Pull Request.

@midudev midudev merged commit 4eaad02 into midudev:main Feb 26, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants