Fix ARIA labels and tabindex on date picker in submission form. Tabindex is unnecessary & throws accessibility errors from AXE tools

This commit is contained in:
Tim Donohue
2023-09-01 09:48:17 -05:00
parent aeea1cd592
commit 5dad8bec39
3 changed files with 6 additions and 6 deletions

View File

@@ -4,7 +4,7 @@
{{model.placeholder}} <span *ngIf="model.required">*</span> {{model.placeholder}} <span *ngIf="model.required">*</span>
</legend> </legend>
<ds-number-picker <ds-number-picker
tabindex="1" tabindex="0"
[id]="model.id + '_year'" [id]="model.id + '_year'"
[disabled]="model.disabled" [disabled]="model.disabled"
[min]="minYear" [min]="minYear"
@@ -21,7 +21,7 @@
></ds-number-picker> ></ds-number-picker>
<ds-number-picker <ds-number-picker
tabindex="2" tabindex="0"
[id]="model.id + '_month'" [id]="model.id + '_month'"
[min]="minMonth" [min]="minMonth"
[max]="maxMonth" [max]="maxMonth"
@@ -37,7 +37,7 @@
></ds-number-picker> ></ds-number-picker>
<ds-number-picker <ds-number-picker
tabindex="3" tabindex="0"
[id]="model.id + '_day'" [id]="model.id + '_day'"
[min]="minDay" [min]="minDay"
[max]="maxDay" [max]="maxDay"

View File

@@ -6,7 +6,7 @@
[disabled]="disabled" [disabled]="disabled"
(click)="toggleUp()"> (click)="toggleUp()">
<span class="chevron"></span> <span class="chevron"></span>
<span class="sr-only">Increment</span> <span class="sr-only">Increment {{placeholder}}</span>
</button> </button>
<input <input
id="{{id}}" id="{{id}}"
@@ -24,6 +24,7 @@
[disabled]="disabled" [disabled]="disabled"
[ngClass]="{'is-invalid': invalid}" [ngClass]="{'is-invalid': invalid}"
title="{{placeholder}}" title="{{placeholder}}"
[attr.aria-label]="placeholder"
> >
<button <button
class="btn btn-link-focus" class="btn btn-link-focus"
@@ -32,6 +33,6 @@
[disabled]="disabled" [disabled]="disabled"
(click)="toggleDown()"> (click)="toggleDown()">
<span class="chevron bottom"></span> <span class="chevron bottom"></span>
<span class="sr-only">Decrement</span> <span class="sr-only">Decrement {{placeholder}}</span>
</button> </button>
</div> </div>

View File

@@ -42,7 +42,6 @@ describe('NumberPickerComponent test suite', () => {
beforeEach(() => { beforeEach(() => {
html = ` html = `
<ds-number-picker <ds-number-picker
tabindex="1"
[disabled]="disabled" [disabled]="disabled"
[min]="min" [min]="min"
[max]="max" [max]="max"