-
Notifications
You must be signed in to change notification settings - Fork 2
feat: add sponsors info page #36
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unable to see the page at: http://localhost:4321/sponsors
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
General a11y issues
Barcelonalooks like a button, but is not.>looks as expandable, but is not.
> Nivel de Seniorityshould beh2(noth3)- Check if
Extras (Add-ons)match headers hierarchy level, maybe ish2(?) - The 'progress bar' for seniority is not accessible, set role
img:
<div
class="flex w-full h-10 rounded-md overflow-hidden mb-4"
role="img"
aria-label="Experience distribution: Junior 20 percent, Senior or Pro 60 percent, Expert 20 percent"
>Accessibility review Sponsors table
What’s good
- Correct semantic structure:
<table>,<thead>,<tbody>,<tr>. - This is real tabular data (benefits × plans), so a table is the right choice.
Issues & fixes
1) Add proper header associations (scope="col" / scope="row")
Screen readers need explicit row/column relationships.
Columns (plans)
<thead>
<tr>
<th scope="col">Beneficios</th>
<th scope="col">Bronce</th>
<th scope="col">Plata</th>
<th scope="col">Oro</th>
<th scope="col">Platino</th>
<th scope="col">Principal</th>
</tr>
</thead>Rows (benefits) — use <th scope="row"> in the first column
<tbody>
<tr>
<th scope="row">Stand físico</th>
<td>…</td>
<td>…</td>
</tr>
</tbody>2) Complex <th> content triggers duplicate VoiceOver reading
Current headers contain multiple nested elements (plan name, price, limit), which VoiceOver may read:
- once as a single concatenated header, then
- again element-by-element.
Fix: one accessible label + hide visual composition
<th
scope="col"
aria-label="Plan Bronce. Precio 1.000 euros. Ilimitado."
>
<div aria-hidden="true">Bronce</div>
<div aria-hidden="true">1.000€</div>
<div aria-hidden="true">Ilimitado</div>
</th>3) Emojis / symbols need text alternatives (🚫 / ✅ / -)
Don’t rely on emoji or punctuation alone to convey meaning.
Fix pattern
<td>
<span aria-hidden="true">🚫</span>
<span class="sr-only">No incluido</span>
</td>Or (for checkmarks):
<td>
<span aria-hidden="true">✅</span>
<span class="sr-only">Incluido</span>
</td>|
@ctrl-alt-d puedes aplicar los cambios que propones y mandar otro screenshot? |
|
Como curiosidad, respecto a:
Pero vamos, no me parece mal añadir lo que comentas. |
No hay ningún problema con los emojis. Pero yo interpreto que se han usado emojis para representación visual, como si fuera una imagen. Cuando lo he probado con el lector de pantalla se me hacía muy rara la lectura de la tabla. Te pongo un ejemplo que va más allá de los emojis, la tabla tenía en la primera versión |
He aplicado los cambios. Para screen readers solo he escondido los emojis de la tabla, no los otros, aunque creo que todos esos emojis son puramente decorativos y deberían esconderse. Ejemplo, la captura de debajo, el lector lo lee como
|
ctrl-alt-d
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Para mbl exploraría alternatives al table. Quizá cards o, al haber tantas opciones, acordeón. Es solo una idea.



Uh oh!
There was an error while loading. Please reload this page.