Improve search bar accessibility

Also remove the "X" icon from Chrome we don't want
pull/6000/head
Chocobozzz 2023-10-06 09:59:21 +02:00
parent e049e3ec73
commit ce99886db6
No known key found for this signature in database
GPG Key ID: 583A612D890159BE
3 changed files with 25 additions and 14 deletions

View File

@ -4,10 +4,10 @@
[(ngModel)]="search" (ngModelChange)="onSearchChange()" (keydown)="handleKey($event)" [(ngModel)]="search" (ngModelChange)="onSearchChange()" (keydown)="handleKey($event)"
aria-label="Search" autocomplete="off" aria-label="Search" autocomplete="off"
> >
<my-global-icon
title="Search" i18n-title role="button" <button class="border-0 search-button" title="Search" i18n-title (click)="doSearch()">
iconName="search" class="icon icon-search" (click)="doSearch()" (keydown.enter)="doSearch()" <my-global-icon iconName="search"></my-global-icon>
></my-global-icon> </button>
<div class="position-absolute jump-to-suggestions"> <div class="position-absolute jump-to-suggestions">

View File

@ -12,16 +12,21 @@
} }
} }
.icon-search { .search-button {
@include icon(25px); display: inline-flex;
@include margin-left(-35px); align-self: center;
height: 18px;
// yolo
position: absolute; position: absolute;
margin-top: 3.5px; right: 5px;
right: 10px;
&:hover {
opacity: 0.8;
}
my-global-icon {
@include icon(18px);
display: inline-flex;
}
} }
.jump-to-suggestions { .jump-to-suggestions {
@ -52,7 +57,7 @@ li.suggestion {
#typeahead-help, #typeahead-help,
#typeahead-instructions { #typeahead-instructions {
margin-top: 10px; margin-top: 9px;
width: 100%; width: 100%;
padding: .5rem 1rem; padding: .5rem 1rem;
white-space: normal; white-space: normal;

View File

@ -18,6 +18,12 @@
@use './class-helpers/index.scss'; @use './class-helpers/index.scss';
/* clears the X from Chrome */
input[type="search"]::-webkit-search-decoration,
input[type="search"]::-webkit-search-cancel-button,
input[type="search"]::-webkit-search-results-button,
input[type="search"]::-webkit-search-results-decoration { display: none; }
body { body {
/*** theme ***/ /*** theme ***/
// now beware sass requires interpolation // now beware sass requires interpolation