Skip to content

Make carousel indicator aria-label translatable#110

Merged
malberts merged 1 commit into
masterfrom
fix/carousel-indicator-aria-label-i18n
Jul 16, 2026
Merged

Make carousel indicator aria-label translatable#110
malberts merged 1 commit into
masterfrom
fix/carousel-indicator-aria-label-i18n

Conversation

@malberts

Copy link
Copy Markdown
Collaborator

Make carousel indicator aria-label translatable.

The carousel indicator buttons emitted a hardcoded English "Slide N" aria-label regardless of wiki language. Resolve it through a new bootstrap-components-carousel-slide-label message in the content language, matching how other component controls (such as the alert close button) localise their aria-labels.

AI-authored: Claude Code, Opus 4.8; detailed spec from @malberts, one clarifying question on test strategy; diff not yet human-reviewed; php -l and JSON valid, existing carousel indicator PHPUnit pins unchanged (English output identical); full MW-PHPUnit suite not run locally (no BootstrapComponents env), CI to exercise it.

The carousel indicator buttons emitted a hardcoded English "Slide N"
aria-label regardless of wiki language. Resolve it through a new
bootstrap-components-carousel-slide-label message in the content
language, matching how other component controls (such as the alert
close button) localise their aria-labels.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@malberts
malberts force-pushed the fix/carousel-indicator-aria-label-i18n branch from b080d11 to 7a47487 Compare July 16, 2026 10:29
@malberts
malberts marked this pull request as ready for review July 16, 2026 10:30
@malberts
malberts merged commit 21f6769 into master Jul 16, 2026
5 of 6 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.

1 participant