Skip to content

ROU-12567: Fix Carousel a11y issues#1138

Merged
OS-susmitabhowmik merged 8 commits into
devfrom
ROU-12567-carousel-a11y
May 11, 2026
Merged

ROU-12567: Fix Carousel a11y issues#1138
OS-susmitabhowmik merged 8 commits into
devfrom
ROU-12567-carousel-a11y

Conversation

@OS-susmitabhowmik
Copy link
Copy Markdown
Contributor

@OS-susmitabhowmik OS-susmitabhowmik commented May 7, 2026

This PR is for...

Sample page

What was happening

  • The carousel was not WCAG 2.2 compliant

What was done

  • This PR implements the changes in this documentation PR to ensure that the carousel is WCAG 2.2 compliant

Test Steps

  1. Verify that pagination controls meet the minimum target size of 24px
Screenshot 2026-05-06 at 5 19 06 PM
  1. Verify that for carousels with images, the tabpanel role was removed from the image.
Screenshot 2026-05-06 at 6 06 45 PM
  1. Verify that lists within the carousel have a list role and each item has a listitem role
Screenshot 2026-05-06 at 5 22 46 PM
  1. Verify that each carousel has a role of presentation rather than region so that multiple carousels within the same page do not throw an a11y error
Screenshot 2026-05-08 at 3 30 29 PM
  1. Verify that for ol and ul elements within carousels there is no role attribute as these elements use semantic HTML and should be detected by a screen reader.
Screenshot 2026-05-08 at 3 32 28 PM

Checklist

  • tested locally
  • documented the code
  • clean all warnings and errors of eslint
  • requires changes in OutSystems (if so, provide a module with changes)
  • requires new sample page in OutSystems (if so, provide a module with changes)

@OS-susmitabhowmik OS-susmitabhowmik requested a review from a team as a code owner May 7, 2026 00:07
@OS-susmitabhowmik OS-susmitabhowmik requested a review from joselrio May 7, 2026 00:31
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 8, 2026

@OS-susmitabhowmik OS-susmitabhowmik merged commit c307023 into dev May 11, 2026
15 of 20 checks passed
@OS-susmitabhowmik OS-susmitabhowmik deleted the ROU-12567-carousel-a11y branch May 11, 2026 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants