Skip to content

[Cmdpal] Screen reader announces "Enter Alias" instead of "Alias" in ExtensionPage#45906

Open
chatasweetie wants to merge 1 commit intomainfrom
jearleycha-alias-annouce-once
Open

[Cmdpal] Screen reader announces "Enter Alias" instead of "Alias" in ExtensionPage#45906
chatasweetie wants to merge 1 commit intomainfrom
jearleycha-alias-annouce-once

Conversation

@chatasweetie
Copy link
Contributor

Summary of the Pull Request

Fixes a screen reader accessibility issue where the Alias TextBox in the ExtensionPage settings was announcing "Enter Alias" as its accessible name instead of just "Alias", making it confusing for users relying on assistive technology.

This is part of the a11y bug batch.
User Impact:
Duplicate announcements can confuse users, increase cognitive load, and slow down interaction for users relying on assistive technologies.

PR Checklist

  • Closes: #xxx
  • Communication: I've discussed this with core contributors already. If the work hasn't been agreed, this work might be rejected
  • Tests: Added/updated and all pass
  • Localization: All end-user-facing strings can be localized
  • Dev docs: Added/updated
  • New binaries: Added on the required places
  • Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

bug-60948009.mp4

@chatasweetie chatasweetie added the Product-Command Palette Refers to the Command Palette utility label Mar 3, 2026
Copy link
Collaborator

@jiripolasek jiripolasek left a comment

Choose a reason for hiding this comment

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

LGTM (except for the XAML Styler issue, which must be fixed in order to get this build).

</controls:SettingsCard>
<controls:SettingsCard x:Uid="Settings_ExtensionPage_Alias_SettingsCard">
<controls:SettingsCard
x:Name="AliasSettingsCard"
Copy link
Collaborator

Choose a reason for hiding this comment

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

XAML Styler failed the build; it requires these two properties to be on a same line

<controls:SettingsCard x:Name="AliasSettingsCard" x:Uid="Settings_ExtensionPage_Alias_SettingsCard">

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Product-Command Palette Refers to the Command Palette utility

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants