Skip to content

Conversation

@shunguoy
Copy link
Contributor

@shunguoy shunguoy commented May 5, 2025

  • Rule bug: svg_graphics_labelled, img_alt_valid

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

Testing reference:

  • test/v2/checker/accessibility/rules/svg_graphics_labelled_ruleunit/svg_name_ignore.html
    Before the fix, 2 violations on "The SVG element has no accessible name".
    After the fix, only 1 of the violations shows up.
  • test/v2/checker/accessibility/rules/img_alt_valid_ruleunit/img_name_ignore.html
    Before the fix, 5 violations on "The image has neither an accessible name nor is marked as decorativ"
    After the fix, only 3 of the violations show up.

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 & QA testing complete
  • Staging deployment verified

@shunguoy shunguoy requested review from philljenkins and tombrunet May 5, 2025 16:00
@shunguoy shunguoy marked this pull request as ready for review May 5, 2025 16:01
@shunguoy shunguoy changed the title fixrule(svg_graphics_labelled, img_alt_valid) Ignore SVG accessible name requirement when its ancestor is a widget or requires presentational child V4 fixrule(svg_graphics_labelled, img_alt_valid) Ignore SVG & Image accessible name requirement when its ancestor is a widget or requires presentational child V4 May 5, 2025
@philljenkins
Copy link
Contributor

@philljenkins
Copy link
Contributor

@shunguoy See blocking comment in #2224

@philljenkins philljenkins changed the title fixrule(svg_graphics_labelled, img_alt_valid) Ignore SVG & Image accessible name requirement when its ancestor is a widget or requires presentational child V4 fixrule(svg_graphics_labelled, img_alt_valid): Ignore SVG & image accessible name requirement when its ancestor is a widget or requires presentational child V4 May 13, 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 complete, but build failing on extension-test

@tombrunet tombrunet merged commit 0dfba02 into main-4.x May 20, 2025
119 of 128 checks passed
@tombrunet tombrunet deleted the dev-syan-v4 branch May 20, 2025 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ignore SVG accessible name requirement when the svg is used as the image of a button with an accessible name

4 participants