[ACA-2789] - edit button a11y fixes (#5399)

* fix: remove title property from button

* use tooltip instead of aria label

* fix aria label

Co-authored-by: Taylor Richards <taylor.richards@deque.com>
Co-authored-by: Denys Vuika <denys.vuika@gmail.com>
This commit is contained in:
Taylor
2020-03-20 17:31:30 -04:00
committed by GitHub
parent fc6a80c45a
commit 78c3db5db7
2 changed files with 2 additions and 1 deletions

View File

@@ -17,7 +17,7 @@
mat-icon-button mat-icon-button
(click)="toggleEdit()" (click)="toggleEdit()"
[attr.title]="'CORE.METADATA.ACTIONS.EDIT' | translate" [attr.title]="'CORE.METADATA.ACTIONS.EDIT' | translate"
[attr.aria-label]="'CORE.METADATA.ACTIONS.EDIT' | translate" [attr.aria-label]="'CORE.METADATA.ACCESSIBILITY.EDIT' | translate"
data-automation-id="meta-data-card-toggle-edit"> data-automation-id="meta-data-card-toggle-edit">
<mat-icon>mode_edit</mat-icon> <mat-icon>mode_edit</mat-icon>
</button> </button>

View File

@@ -201,6 +201,7 @@
"TOGGLE": "Toggle value" "TOGGLE": "Toggle value"
}, },
"ACCESSIBILITY": { "ACCESSIBILITY": {
"EDIT": "Edit button",
"DATEPICKER": "Use the arrow keys to navigate between dates. Up and down move to the next or previous week but on the same day. Left and right move to the next or previous day. Press Enter or Return to select a date." "DATEPICKER": "Use the arrow keys to navigate between dates. Up and down move to the next or previous week but on the same day. Left and right move to the next or previous day. Press Enter or Return to select a date."
} }
}, },