Add: mejoras para la accesibilidad, sobretodo para los screen readers #135
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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