Skip to content

Commit 8d77a9f

Browse files
authored
better aria labels (#55929)
1 parent 794cdc0 commit 8d77a9f

File tree

3 files changed

+4
-3
lines changed

3 files changed

+4
-3
lines changed

data/ui.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ search:
3030
placeholder_no_icon: Search or ask Copilot
3131
shortcut: Type {{icon}} to search
3232
overlay:
33+
aria_label: Search overlay
3334
input_aria_label: Search or ask Copilot
3435
suggestions_list_aria_label: Search suggestions
3536
ai_suggestions_list_aria_label: Copilot search suggestions

src/fixtures/fixtures/data/ui.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ search:
3030
placeholder_no_icon: Search or ask Copilot
3131
shortcut: Type {{icon}} to search
3232
overlay:
33+
aria_label: Search overlay
3334
input_aria_label: Search or ask Copilot
3435
suggestions_list_aria_label: Search suggestions
3536
ai_suggestions_list_aria_label: Copilot search suggestions

src/search/components/input/SearchOverlay.tsx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -675,7 +675,6 @@ export function SearchOverlay({
675675
)
676676
}
677677

678-
const overlayHeadingId = 'overlay-heading'
679678
return (
680679
<>
681680
<div className={styles.overlayBackdrop} />
@@ -698,7 +697,7 @@ export function SearchOverlay({
698697
}
699698
role="dialog"
700699
aria-modal="true"
701-
aria-labelledby={overlayHeadingId}
700+
aria-label={t('search.overlay.aria_label')}
702701
ref={overlayRef}
703702
>
704703
<div className={styles.header}>
@@ -723,11 +722,11 @@ export function SearchOverlay({
723722
value={urlSearchInputQuery}
724723
onChange={handleSearchQueryChange}
725724
leadingVisual={<SearchIcon />}
726-
aria-labelledby={overlayHeadingId}
727725
role="combobox"
728726
// In AskAI the search input not longer "controls" the suggestions list, because there is no list, so we remove the aria-controls attribute
729727
aria-controls={isAskAIState ? 'ask-ai-result-container' : 'search-suggestions-list'}
730728
aria-expanded={combinedOptions.length > 0}
729+
aria-label={t('search.overlay.input_aria_label')}
731730
aria-activedescendant={
732731
selectedIndex >= 0
733732
? `search-option-${combinedOptions[selectedIndex]?.group}-${selectedIndex}`

0 commit comments

Comments
 (0)