Skip to content

ROU-12274: Add aria-label to pagination buttons#475

Merged
JoaoFerreira-FrontEnd merged 2 commits intodevfrom
ROU-12274
Dec 11, 2025
Merged

ROU-12274: Add aria-label to pagination buttons#475
JoaoFerreira-FrontEnd merged 2 commits intodevfrom
ROU-12274

Conversation

@JoaoFerreira-FrontEnd
Copy link
Contributor

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd commented Dec 10, 2025

This PR is for adding aria-labels to the pagination buttons

What was done

  • add aria-labels to the elements returned by createPageButtons method.

Test Steps

  1. Go to Sample
  2. Run Axe Dev Tools scan
  3. Check warning “Ensure buttons have discernible text” no longer exist
  4. Check attributes applied by inspecting the DOM

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)

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd requested a review from a team as a code owner December 10, 2025 12:33
@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd added the bug Something isn't working label Dec 10, 2025
@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd changed the title ROU-12274: add aria-label to pagination buttons; ROU-12274: add aria-label to pagination buttons Dec 10, 2025
@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd changed the title ROU-12274: add aria-label to pagination buttons ROU-12274: Add aria-label to pagination buttons Dec 10, 2025
@sonarqubecloud
Copy link

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd merged commit dcd445b into dev Dec 11, 2025
10 checks passed
@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd deleted the ROU-12274 branch December 11, 2025 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants