Skip to content

fix(onboarding): Name SCM provider pills by their provider - #124833

Open
jaydgoss wants to merge 1 commit into
jaygoss/scm-create-project-a11yfrom
jaygoss/scm-provider-pill-names
Open

jaydgoss wants to merge 1 commit into
jaygoss/scm-create-project-a11yfrom
jaygoss/scm-provider-pill-names

Conversation

@jaydgoss

Copy link
Copy Markdown
Member

AddIntegrationButton sets a static aria-label of "Add integration" on every instance. In the SCM onboarding and project creation flows, where it renders as a provider pill, a screen reader user heard "Add integration, button" three times in a row and could not tell GitHub from GitLab from Bitbucket. The overflow menu trigger next to them was named only "More", which says nothing about what it holds.

Each pill is now named "Add" plus the provider name, for example "Add GitHub", and the overflow trigger is named "More providers". Both names keep the visible text inside them so voice control users can still speak the label they see. The static "Add integration" label moves before the spread of caller props in AddIntegrationButton so a caller can override it, and IntegrationButton passes aria-label through. Every other caller passes no label, so the settings pages keep the "Add integration" name they have today.

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Sep 17, 2026
@jaydgoss
jaydgoss marked this pull request as ready for review September 17, 2026 21:14
@jaydgoss
jaydgoss requested a review from a team as a code owner September 17, 2026 21:14
@jaydgoss
jaydgoss requested review from a team September 17, 2026 21:14
AddIntegrationButton sets a static aria-label of "Add integration" on every instance, so the GitHub, GitLab and Bitbucket pills in the SCM onboarding and project creation flows all had the same accessible name and the More menu trigger was named only "More". Each pill is now named "Add" plus the provider name and the trigger is named "More providers". The static label stays the default for every other caller.
@jaydgoss
jaydgoss force-pushed the jaygoss/scm-provider-pill-names branch from 98afc33 to 5928658 Compare September 17, 2026 21:23
@jaydgoss
jaydgoss changed the base branch from master to jaygoss/scm-create-project-a11y September 17, 2026 21:23
@jaydgoss
jaydgoss added this pull request to stack #124480 September 17, 2026 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant