Skip to content

fix(ui5-dynamic-date-range): add accessibleName to the ResponsivePopover#12965

Merged
hinzzx merged 1 commit intomainfrom
ddr-acc-fix
Feb 4, 2026
Merged

fix(ui5-dynamic-date-range): add accessibleName to the ResponsivePopover#12965
hinzzx merged 1 commit intomainfrom
ddr-acc-fix

Conversation

@hinzzx
Copy link
Contributor

@hinzzx hinzzx commented Jan 27, 2026

Problem

The ui5-dynamic-date-range popover (role="dialog") lacks an accessible name:

  1. Screen readers cannot announce the purpose of the popover dialog when it opens
  2. Accessibility tools (AXE/AccessContinuum) flag this as a violation

Solution

  • Added accessibleName to the ResponsivePopover in the template, using i18n text

What This Fixes

  • ✅ Screen readers now announce "Dynamic Date Range" when the popover opens (i18n-enabled)
  • ✅ Popover dialog has a proper accessible name per WAI-ARIA requirements
  • ✅ Accessibility automation tools no longer flag this as a violation

@hinzzx hinzzx changed the title fix(ui5-dynamic-date-range): add accessibleName to the ResponsivePopover fix(ui5-dynamic-date-range): add accessibleName to the ResponsivePopover Jan 27, 2026
@ui5-webcomponents-bot
Copy link
Collaborator

ui5-webcomponents-bot commented Jan 27, 2026

🧹 Preview deployment cleaned up: https://pr-12965--ui5-webcomponents.netlify.app

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview January 27, 2026 08:20 Inactive
@unazko unazko requested a review from tsanislavgatev January 29, 2026 07:34
@hinzzx hinzzx merged commit a674966 into main Feb 4, 2026
35 of 38 checks passed
@hinzzx hinzzx deleted the ddr-acc-fix branch February 4, 2026 07:32
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview February 4, 2026 07:32 Inactive
@ui5-webcomponents-bot
Copy link
Collaborator

🎉 This PR is included in version v2.19.0-rc.3 🎉

The release is available on v2.19.0-rc.3

Your semantic-release bot 📦🚀

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.

4 participants