Skip to content

FSHSP-111 fix(tabs): expose the scroll navigators' aria-labels as inputs - #35

Merged
LBU4SH merged 2 commits into
mainfrom
claude/modest-chatterjee-5ca90b
Aug 14, 2026
Merged

FSHSP-111 fix(tabs): expose the scroll navigators' aria-labels as inputs#35
LBU4SH merged 2 commits into
mainfrom
claude/modest-chatterjee-5ca90b

Conversation

@LBU4SH

@LBU4SH LBU4SH commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

Contexte

Les deux boutons de défilement de ui-tab-list (mode scrollable) portaient un aria-label écrit en dur dans le template — les seuls libellés du kit qu'une application consommatrice ne pouvait pas surcharger. Le kit ne ship aucune couche i18n (règle « No i18n catalog in the kit », AGENTS.md) : tout nom accessible doit être une entrée avec défaut français, comme closeAriaLabel sur ui-modal ou prevPageAriaLabel sur ui-paginator. Une application non francophone héritait donc de deux libellés lecteur d'écran intraduisibles.

Changements

  • UiTabList expose prevAriaLabel / nextAriaLabel (input<string>(…)), les textes actuels servant de valeurs par défaut → aucun changement de rendu.
  • Le template passe à [attr.aria-label]="…() || null", conforme à la règle de garde contre la chaîne vide.
  • ui-tabs.stories.ts : argTypes + args, et la story Scrollable binde les deux entrées — c'est la seule où les navigateurs sont rendus, donc la seule où les contrôles Storybook sont utiles.
  • ui-tabs.mdx : deux lignes dans le tableau d'API de ui-tab-list.
  • CHANGELOG.md : entrée sous [Unreleased] / Added (l'API du package publié change).

Vérification

⚠️ Pas de build local : la worktree n'a pas de node_modules. Les modifications sont des bindings de template calqués sur ui-paginator, mais un npm ci && npm run ui-kit:build reste à passer — la CI de PR le couvre.

🤖 Generated with Claude Code

@LBU4SH
LBU4SH merged commit 47f03d1 into main Aug 14, 2026
2 of 3 checks passed
@LBU4SH
LBU4SH deleted the claude/modest-chatterjee-5ca90b branch August 14, 2026 14:06
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.

1 participant