Skip to content

Commit 7912fb3

Browse files
committed
fix(a11y): use role="status" for the Vertex credentials path warning
role="alert" implies aria-live="assertive" by default, which mixed awkwardly with the explicit aria-live="polite" override (urgent role + don't-interrupt liveness). For an informational warning that fires as the user types into a settings field, role="status" is the conventional pattern — implicitly polite, semantically correct for a non-urgent dynamic message, and removes the role/live conflict.
1 parent 6b0e582 commit 7912fb3

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

  • webview-ui/src/components/settings/providers

webview-ui/src/components/settings/providers/Vertex.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -94,8 +94,7 @@ export const Vertex = ({ apiConfiguration, setApiConfigurationField }: VertexPro
9494
{credentialsLooksLikePath && (
9595
<div
9696
data-testid="vertex-credentials-path-warning"
97-
role="alert"
98-
aria-live="polite"
97+
role="status"
9998
className="text-sm text-vscode-errorForeground">
10099
<Trans
101100
i18nKey="settings:providers.googleCloudCredentialsPathWarning"

0 commit comments

Comments
 (0)