Skip to content

fix(ui5-media-gallery): acc findings fixed #13555

Open
PetyaMarkovaBogdanova wants to merge 1 commit into
mainfrom
fix-acc-mg
Open

fix(ui5-media-gallery): acc findings fixed #13555
PetyaMarkovaBogdanova wants to merge 1 commit into
mainfrom
fix-acc-mg

Conversation

@PetyaMarkovaBogdanova
Copy link
Copy Markdown
Contributor

@PetyaMarkovaBogdanova PetyaMarkovaBogdanova commented May 20, 2026

4) MediaGallery (22 failing)

  • Repeated continuum violations, including:
  • LI with role="option" missing aria-checked (ID 1079).
  • UL child-structure violation for list semantics (ID 246).
  • Failing examples include thumbnail options and overflow scenarios.
MediaGallery option items missing required state ID 1079: LI role option missing aria-checked Option semantics implemented but selection state attribute omitted on some items Likely real High Provide consistent state attribute where role requires it, or switch to correct role-state pair All option-like items expose required state attributes consistently
MediaGallery UL child-structure violation ID 246 list child structure Children inside list container are not valid list children in overflow/thumbnail variants Likely real High Normalize list markup so only valid child structure exists in all modes Thumbnail and overflow DOM both satisfy list parent-child constraints

@ui5-webcomponents-bot
Copy link
Copy Markdown
Collaborator

ui5-webcomponents-bot commented May 20, 2026

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview May 20, 2026 07:09 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview May 20, 2026 07:22 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview May 20, 2026 12:21 Inactive
@PetyaMarkovaBogdanova PetyaMarkovaBogdanova changed the title Fix acc mg fix(ui5-media-gallery): acc findings fixed May 20, 2026
@PetyaMarkovaBogdanova PetyaMarkovaBogdanova marked this pull request as ready for review May 20, 2026 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants