Skip to content

Conversation

@aardrian
Copy link
Contributor

@aardrian aardrian commented Jan 8, 2026

Description

This MDN article is making some unfounded claims about the baseline accessibility of CSS carousels, so this PR strikes them. I've left the other statements about what considerations and concerns are in place for accessibility.

Motivation

Devs are starting to try to implement these and creating WCAG violations without realizing it because this, along with lots of promotion in dev sites, wrongly claims these are "accessible" (without qualifying how or in what way).

Additional details

Recent evidence of their baseline inaccessibility: https://www.sarasoueidan.com/blog/css-carousels-accessibility/

This is making some unfounded claims about the baseline accessibility of CSS carousels, so I've stricken them. I've left the other statements about what considerations and concerns are in place for accessibility.

Recent evidence of their baseline inaccessibility:
https://www.sarasoueidan.com/blog/css-carousels-accessibility/
@aardrian aardrian requested a review from a team as a code owner January 8, 2026 00:29
@aardrian aardrian requested review from chrisdavidmills and removed request for a team January 8, 2026 00:29
@github-actions github-actions bot added Content:CSS Cascading Style Sheets docs size/s [PR only] 6-50 LoC changed labels Jan 8, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

Preview URLs

Flaws (1)

URL: /en-US/docs/Web/CSS/Guides/Overflow/Carousels
Title: Creating CSS carousels
Flaw count: 1

  • unknown:
    • Error parsing /shared-assets/images/diagrams/css/carousels/carousel.svg: No such file or directory (os error 2)

(comment last updated: 2026-01-08 02:38:00)

As noted in comment, this is kinda moot considering overall context: "When carousels are created using JavaScript, the accessibility of the carousel and the associated controls has to be added in." Struck it.
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @aardrian — this seems totally fair to me.

@chrisdavidmills chrisdavidmills merged commit 8bc7e01 into mdn:main Jan 8, 2026
7 checks passed
@aardrian
Copy link
Contributor Author

aardrian commented Jan 8, 2026

Just for my own later spelunking, I filed a more detailed PR to address accName issues: #42691

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

Labels

Content:CSS Cascading Style Sheets docs size/s [PR only] 6-50 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants