Skip to content

Commit 1c3dca4

Browse files
authored
use placeholder for aria label is same as visual label (#55994)
1 parent f5aeb11 commit 1c3dca4

3 files changed

Lines changed: 2 additions & 4 deletions

File tree

data/ui.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ release_notes:
2525
banner_text: GitHub began rolling these changes out to enterprises on
2626
search:
2727
input:
28-
aria_label: Open search overlay
2928
placeholder: Search or ask {{icon}} Copilot
3029
placeholder_no_icon: Search or ask Copilot
3130
shortcut: Type {{icon}} to search

src/fixtures/fixtures/data/ui.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ release_notes:
2525
banner_text: GitHub began rolling these changes out to enterprises on
2626
search:
2727
input:
28-
aria_label: Open search overlay
2928
placeholder: Search or ask {{icon}} Copilot
3029
placeholder_no_icon: Search or ask Copilot
3130
shortcut: Type {{icon}} to search

src/search/components/input/SearchBarButton.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,14 +58,14 @@ export function SearchBarButton({ isSearchOpen, setIsSearchOpen, params, searchB
5858
className={styles.searchIconButton}
5959
onClick={handleClick}
6060
tabIndex={0}
61-
aria-label={t('search.input.aria_label')}
61+
aria-label={t('search.input.placeholder_no_icon')}
6262
icon={SearchIcon}
6363
/>
6464
{/* On large and up the SearchBarButton is shown */}
6565
<button
6666
data-testid="search"
6767
tabIndex={0}
68-
aria-label={t`search.input.aria_label`}
68+
aria-label={t('search.input.placeholder_no_icon')}
6969
className={styles.searchInputButton}
7070
onKeyDown={handleKeyDown}
7171
onClick={handleClick}

0 commit comments

Comments
 (0)