Skip to content

fix: improve live region behavior for screen readers#2865

Open
balco0110 wants to merge 6 commits intomainfrom
fix/carousel-live-region-should-not-be-activated-when-auto-play-is-active
Open

fix: improve live region behavior for screen readers#2865
balco0110 wants to merge 6 commits intomainfrom
fix/carousel-live-region-should-not-be-activated-when-auto-play-is-active

Conversation

@balco0110
Copy link
Copy Markdown
Contributor

@balco0110 balco0110 commented Mar 18, 2026 •
edited
Loading

Description:

Closes: #2412

Definition of Reviewable:

  • relevant tickets are linked

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 18, 2026 •
edited
Loading

🦋 Changeset detected

Latest commit: bb1c978

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

💥 An error occurred when fetching the changed packages and changesets in this PR 
Some errors occurred when validating the changesets config:
The package or glob expression "@solid-design-system/components" specified in the `fixed` option does not match any package in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.
The package or glob expression "@solid-design-system/styles" specified in the `fixed` option does not match any package in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.
The package or glob expression "@solid-design-system/tokens" specified in the `fixed` option does not match any package in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 18, 2026 •
edited
Loading

🚀 Storybook has been deployed for branch fix_carousel-live-region-should-not-be-activated-when-auto-play-is-active

@balco0110 balco0110 moved this from 📋 Backlog to 👀 In review in Solid Design System Project Board Mar 18, 2026
@tega-ekakitie
Copy link
Copy Markdown

tega-ekakitie commented Mar 19, 2026

@balco0110 Looks good to me
Validated when autoplay is paused and navigated as well as when it's active

@smfonseca
Copy link
Copy Markdown
Collaborator

smfonseca commented Mar 20, 2026

@balco0110 @tega-ekakitie I've tested in Safari where we usually get better results with VoiceOver, and I'm getting announcements of the image alt text when auto-playis paused which is good. What I noticed is that if I press next or previous in a carousel without auto-play I don't get announcements. Maybe it's connected with the current change, in prod it's working.

I tested on the carousel template page where we have both scenarios.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@mariohamann mariohamann

@smfonseca smfonseca

@martascorreia martascorreia

@rahulsuvarna18 rahulsuvarna18

@balco0110 balco0110

@auroraVasconcelos auroraVasconcelos

@tega-ekakitie tega-ekakitie

Labels

None yet

Projects

Solid Design System Project Board
Status: 👀 In review

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

fix: 🤔 Carousel's live region should not be activated when auto play is active

7 participants

@balco0110 @tega-ekakitie @smfonseca @mariohamann @martascorreia @rahulsuvarna18 @auroraVasconcelos