Skip to content

Accessibility: Fixed keyboard navigation and narrator accessibility issues for BasemapGallery #652

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

Samhitha-Burugupalli
Copy link
Contributor

@Samhitha-Burugupalli Samhitha-Burugupalli commented Jun 2, 2025

  • Added automation properties, name and help text for basemap gallery control and list view item
  • removed tab stops at control and at control template inside list view item
  • added automation id
  • Updated the name and help text of the control with localized string
  • Added AutomationProperties.Name for the image in ListItem template and GridItem template

- Added automation properties, name and help text for basemap gallery control and list view item
- removed tab stops at control and at control template inside list view item
- added automation id
- Updated the name and help text of the control with localized string
- Added AutomationProperties.Name for the image in grid template and ListItem template and GridItem template
Copy link
Collaborator

@mstefarov mstefarov left a comment

Choose a reason for hiding this comment

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

Looks great, I just have one suggestion about HelpText

Updating help text

Co-authored-by: Matvei Stefarov <[email protected]>
Copy link
Collaborator

@mstefarov mstefarov left a comment

Choose a reason for hiding this comment

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

👍

@prathameshnarkhede prathameshnarkhede merged commit f84a7af into Esri:main Jun 6, 2025
1 check passed
@Samhitha-Burugupalli Samhitha-Burugupalli deleted the sam/accessibility-basemapgallery branch June 6, 2025 22:08
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.

5 participants