Skip to content

Conversation

@shunguoy
Copy link
Contributor

@shunguoy shunguoy commented Jun 9, 2025

  • Rule bug: figure_label_exists, element_scrollable_tabbable

This PR is related to the following issue(s):

Testing reference:

I have conducted the following for this PR:

  • I validated this code in Chrome and FF
  • I validated this fix in my local env
  • I provided details for testing
  • This PR has been reviewed and is ready for test
  • I understand that the title of this PR will be used for the next release notes.

Assets to aide review attached

  • NA Links to design artifacts
  • NA Links to video walkthrough of user experience
  • NA Other

Definition of Done

  • Peer review complete
  • Secondary review complete
  • Staging deployment verified

@shunguoy shunguoy changed the title fixrule(figure_label_exists): fix figure missing label issue fixrule(figure_label_exists): fix the figure missing label issue Jun 9, 2025
@shunguoy shunguoy changed the title fixrule(figure_label_exists): fix the figure missing label issue fixrule(figure_label_exists): deprecate fix element_scrollable_tabbable rule and fix the figure missing label issue Jun 9, 2025
@shunguoy shunguoy marked this pull request as draft June 9, 2025 22:27
@shunguoy shunguoy changed the title fixrule(figure_label_exists): deprecate fix element_scrollable_tabbable rule and fix the figure missing label issue fixrule(figure_label_exists, element_scrollable_tabbable): deprecate fix element_scrollable_tabbable rule and fix the figure missing label issue Jun 9, 2025
@shunguoy shunguoy marked this pull request as ready for review June 9, 2025 22:51
@shunguoy shunguoy self-assigned this Jun 10, 2025
@nam-singh nam-singh added Bug Something isn't working engine Issues in the accessibility-checker-engine component labels Jun 10, 2025
@shunguoy shunguoy changed the title fixrule(figure_label_exists, element_scrollable_tabbable): deprecate fix element_scrollable_tabbable rule and fix the figure missing label issue fixrule(figure_label_exists, element_scrollable_tabbable): deprecate element_scrollable_tabbable rule and fix the figure missing label issue Jun 11, 2025
Copy link
Contributor

@philljenkins philljenkins left a comment

Choose a reason for hiding this comment

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

  • QA testing complete w/ Chrome & Firefox
  • see QA testing results notes in individual issues

@philljenkins philljenkins merged commit 52ed455 into master Jun 12, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working engine Issues in the accessibility-checker-engine component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Question]: Remove requirement to set tabindex on scrollable nodes figure elements with figcaptions flagged as not having labels

5 participants