feat: add teams page#174
Merged
Merged
Conversation
Adds /teams route for all locales (es, en, ca) listing all organization teams with name, description and contact email. Includes nav menu links and WCAG-compliant markup (role="list", focus-visible, aria-label with team context, aria-labelledby on sections). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
ctrl-alt-d
reviewed
May 10, 2026
Collaborator
ctrl-alt-d
left a comment
ctrl-alt-d
left a comment
There was a problem hiding this comment.
-
El diseño está muy bien!
-
Pensar si colapsamos el menú en un breakpoint más grande, adelantar el breakpoint del navbar:
- Hay un
h2que parece fake:
<h2 id="teams-list-heading" class="sr-only">Equips</h2>
Pensar si eliminar ese h2 y cambiar los h3 de nombres de equipos a h2.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Collaborator
Author
Gracias @ctrl-alt-d . He hecho lo siguiente:
|
ctrl-alt-d
requested changes
May 10, 2026
Collaborator
ctrl-alt-d
left a comment
ctrl-alt-d
left a comment
There was a problem hiding this comment.
Glups. Els lectors de pantalla tenen shortcuts per anar al h1. És comú navegar per headers. Si la pàgina no té h1 és molt confús:
Common Navigation Commands: Heading level 1 VO + Cmd + 1
Collaborator
Author
|
Uis, my fault @ctrl-alt-d ! Es que leí mail tus comentarios. Revertido el h1. Y el h3 pasado a h2. |
itziarZG
approved these changes
May 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Summary
/teamspage for all locales (/es/teams,/en/teams,/ca/teams) listing all organization teams in an accessible 2-column card grid (1 column on mobile)mailto:contact linkEquipos/Teams/Equips) after the accommodation linksrc/data/teams/)Accessibility
role="list"on<ul>to restore list semantics stripped by Tailwind'slist-nonein Safari/VoiceOverfocus-visibleoutline on email links for keyboard navigationaria-labelon email links includes the team name for screen reader contextaria-labelledbyon<section>elements linked to their headingsh1→ hiddenh2(region label) →h3(team names)Test plan
/es/teams,/en/teams,/ca/teams— all 11 team cards visibleFixes #88