Skip to content

Commit 0d10c0d

Browse files
[ACS-10253] Screen Reader: Search: Search control is focused multiple times and incorrectly announced by SR (#4846)
1 parent 4fed9ff commit 0d10c0d

File tree

2 files changed

+11
-2
lines changed

2 files changed

+11
-2
lines changed

projects/aca-content/src/lib/components/search/search-input/search-input.component.html

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
<button class="app-search-container"
1+
<div class="app-search-container"
2+
role="search"
23
[matMenuTriggerFor]="searchOptionsMenu"
34
(menuOpened)="onMenuOpened()"
45
(menuClosed)="syncInputValues()"
@@ -9,6 +10,7 @@
910
</button>
1011

1112
<input
13+
class="app-search-input"
1214
matInput
1315
[attr.aria-label]="'SEARCH.INPUT.ARIA-LABEL' | translate"
1416
[type]="'text'"
@@ -30,7 +32,7 @@
3032
</button>
3133
</div>
3234
</mat-form-field>
33-
</button>
35+
</div>
3436

3537
<mat-menu #searchOptionsMenu="matMenu" [overlapTrigger]="true" class="app-search-options-menu">
3638
<div (keydown.tab)="$event.stopPropagation()" (keydown.shift.tab)="$event.stopPropagation()" tabindex=0>

projects/aca-content/src/lib/components/search/search-input/search-input.component.scss

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,13 @@ $search-border-radius: 4px;
2222
display: none;
2323
}
2424
}
25+
26+
.app-search-input {
27+
&:focus-within {
28+
outline: 2px solid var(--theme-blue-button-color);
29+
border-radius: 4px;
30+
}
31+
}
2532
}
2633

2734
.app-search-options-menu {

0 commit comments

Comments
 (0)