[ACS-3548] Accessibility issues (#7939)

* Accessibility issues

* requested changes

* breadcrumb changes

* review comments resolved of mattooltip

* e2e test cases fixed
This commit is contained in:
jatin2008 2022-11-18 11:35:28 +05:30 committed by GitHub
parent 5be9558ef2
commit 44f54b2a61
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
9 changed files with 22 additions and 11 deletions

View File

@ -273,7 +273,7 @@ export class TaskDetailsPage {
} }
async checkIsEmptyCommentListDisplayed(): Promise<void> { async checkIsEmptyCommentListDisplayed(): Promise<void> {
const emptyList = element(by.cssContainingText('h1[id="comment-header"]', '(0)')); const emptyList = element(by.cssContainingText('div[id="comment-header"]', '(0)'));
await BrowserVisibility.waitUntilElementIsVisible(emptyList); await BrowserVisibility.waitUntilElementIsVisible(emptyList);
} }

View File

@ -5,8 +5,8 @@
[expanded]="canExpandProperties()" [expanded]="canExpandProperties()"
[attr.data-automation-id]="'adf-metadata-group-properties'"> [attr.data-automation-id]="'adf-metadata-group-properties'">
<mat-expansion-panel-header> <mat-expansion-panel-header>
<mat-panel-title> <mat-panel-title class="adf-metadata-properties-title">
<h1 class="adf-metadata-properties-title">{{ 'CORE.METADATA.BASIC.HEADER' | translate }}</h1> {{ 'CORE.METADATA.BASIC.HEADER' | translate }}
</mat-panel-title> </mat-panel-title>
</mat-expansion-panel-header> </mat-expansion-panel-header>
<adf-card-view <adf-card-view

View File

@ -413,7 +413,8 @@
"ADD-ACTION": "ADD", "ADD-ACTION": "ADD",
"CLOSE-ACTION": "CANCEL", "CLOSE-ACTION": "CANCEL",
"BASE-DIALOG-TITLE": "Search a group or people to add...", "BASE-DIALOG-TITLE": "Search a group or people to add...",
"EVERYONE": "EVERYONE" "EVERYONE": "EVERYONE",
"USER-GROUP-LIST": "User or Group List"
}, },
"COLUMN": { "COLUMN": {
"NAME": "Users and Groups ({{ count }})", "NAME": "Users and Groups ({{ count }})",
@ -428,7 +429,8 @@
"DIRECT-PERMISSIONS": "Directly Applied Permission", "DIRECT-PERMISSIONS": "Directly Applied Permission",
"INHERITED-PERMISSIONS": "Inherited Permission", "INHERITED-PERMISSIONS": "Inherited Permission",
"INHERITED-SUBTITLE": "{{count}} users or groups are inheriting permission from a parent folder", "INHERITED-SUBTITLE": "{{count}} users or groups are inheriting permission from a parent folder",
"SELECT-ROLE": "Select role" "SELECT-ROLE": "Select role",
"INHERITED_PERMISSION_TOGGLE": "Toggle Inherited Permission"
}, },
"MESSAGE": { "MESSAGE": {
"EMPTY-PERMISSION": "No users/groups", "EMPTY-PERMISSION": "No users/groups",

View File

@ -4,6 +4,7 @@
id="searchInput" id="searchInput"
[formControl]="searchInput" [formControl]="searchInput"
type="text" type="text"
title="{{'PERMISSION_MANAGER.ADD-PERMISSION.SEARCH' | translate}}"
placeholder="{{'PERMISSION_MANAGER.ADD-PERMISSION.SEARCH' | translate}}" placeholder="{{'PERMISSION_MANAGER.ADD-PERMISSION.SEARCH' | translate}}"
[value]="searchedWord"> [value]="searchedWord">
@ -34,7 +35,8 @@
class="adf-permission-result-list" class="adf-permission-result-list"
[class.adf-permission-result-list-search]="searchedWord.length === 0"> [class.adf-permission-result-list-search]="searchedWord.length === 0">
<ng-template let-data> <ng-template let-data>
<mat-selection-list class="adf-permission-result-list-elements" (keydown.control.a)="onSelectionChange()" <mat-selection-list class="adf-permission-result-list-elements"
title="{{'PERMISSION_MANAGER.ADD-PERMISSION.USER-GROUP-LIST' | translate}}"
(selectionChange)="onSelectionChange()"> (selectionChange)="onSelectionChange()">
<mat-list-option id="adf-add-permission-group-everyone" <mat-list-option id="adf-add-permission-group-everyone"
class="adf-list-option-item" class="adf-list-option-item"

View File

@ -25,6 +25,7 @@
<mat-slide-toggle <mat-slide-toggle
class="adf-inherit-toggle" class="adf-inherit-toggle"
title="{{'PERMISSION_MANAGER.LABELS.INHERITED_PERMISSION_TOGGLE' | translate }}"
color="primary" color="primary"
data-automation-id="adf-inherit-toggle-button" data-automation-id="adf-inherit-toggle-button"
[checked]="model.node.permissions.isInheritanceEnabled" [checked]="model.node.permissions.isInheritanceEnabled"
@ -72,7 +73,7 @@
<button <button
mat-button mat-button
[matTooltip]="'PERMISSION_MANAGER.ACTION.ADD-PERMISSION' | translate" title="{{'PERMISSION_MANAGER.ACTION.ADD-PERMISSION' | translate}}"
(click)="openAddPermissionDialog()" (click)="openAddPermissionDialog()"
data-automation-id="adf-add-permission-button"> data-automation-id="adf-add-permission-button">
<mat-icon>person_add_outline</mat-icon> <mat-icon>person_add_outline</mat-icon>
@ -80,7 +81,7 @@
<button <button
mat-button mat-button
[matTooltip]="'PERMISSION_MANAGER.ACTION.DELETE' | translate" title="{{'PERMISSION_MANAGER.ACTION.DELETE' | translate}}"
[disabled]="!selectedPermissions?.length" [disabled]="!selectedPermissions?.length"
(click)="deleteSelection()" (click)="deleteSelection()"
data-automation-id="adf-delete-selected-permission"> data-automation-id="adf-delete-selected-permission">

View File

@ -1,4 +1,5 @@
<div role="listbox" id="adf-search-results-content" [ngClass]="_classList" #panel> <div role="listbox" id="adf-search-results-content" title="{{'PERMISSION_MANAGER.ADD-PERMISSION.USER-GROUP-LIST' | translate}}"
[ngClass]="_classList" #panel>
<ng-template <ng-template
[ngTemplateOutlet]="template" [ngTemplateOutlet]="template"
[ngTemplateOutletContext]="{ $implicit: results }"> [ngTemplateOutletContext]="{ $implicit: results }">

View File

@ -12,6 +12,7 @@
<input matInput <input matInput
*ngIf="!property.multiline" *ngIf="!property.multiline"
class="adf-property-value" class="adf-property-value"
title="{{property.label | translate }}"
[placeholder]="property.default" [placeholder]="property.default"
[attr.aria-label]="property.label | translate" [attr.aria-label]="property.label | translate"
[formControl]="textInput" [formControl]="textInput"
@ -22,6 +23,7 @@
[attr.data-automation-id]="'card-textitem-value-' + property.key"> [attr.data-automation-id]="'card-textitem-value-' + property.key">
<textarea matInput <textarea matInput
*ngIf="property.multiline" *ngIf="property.multiline"
title="{{property.label | translate }}"
[cdkTextareaAutosize]="true" [cdkTextareaAutosize]="true"
[cdkAutosizeMaxRows]="1" [cdkAutosizeMaxRows]="1"
[cdkAutosizeMaxRows]="5" [cdkAutosizeMaxRows]="5"
@ -74,6 +76,7 @@
[floatLabel]="'never'"> [floatLabel]="'never'">
<input matInput <input matInput
class="adf-property-value" class="adf-property-value"
title="{{property.label | translate }}"
[placeholder]="editedValue ? '' : property.default | translate" [placeholder]="editedValue ? '' : property.default | translate"
[attr.aria-label]="property.label | translate" [attr.aria-label]="property.label | translate"
[matChipInputFor]="chipList" [matChipInputFor]="chipList"
@ -98,6 +101,7 @@
<input matInput <input matInput
[type]=property.inputType [type]=property.inputType
class="adf-property-value" class="adf-property-value"
title="{{property.label | translate }}"
[ngClass]="{ 'adf-textitem-clickable-value': !isEditable }" [ngClass]="{ 'adf-textitem-clickable-value': !isEditable }"
[placeholder]="property.default" [placeholder]="property.default"
[attr.aria-label]="property.label | translate" [attr.aria-label]="property.label | translate"

View File

@ -1,7 +1,7 @@
<div class="adf-comments-container"> <div class="adf-comments-container">
<h1 id="comment-header" class="adf-comments-header"> <div id="comment-header" class="adf-comments-header">
{{'COMMENTS.HEADER' | translate: { count: comments?.length} }} {{'COMMENTS.HEADER' | translate: { count: comments?.length} }}
</h1> </div>
<div class="adf-comments-input-container" *ngIf="!isReadOnly()"> <div class="adf-comments-input-container" *ngIf="!isReadOnly()">
<mat-form-field class="adf-full-width"> <mat-form-field class="adf-full-width">
<textarea id="comment-input" <textarea id="comment-input"

View File

@ -34,6 +34,7 @@
[attr.data-automation-id]="'auto_id_' + col.key" [attr.data-automation-id]="'auto_id_' + col.key"
[class.adf-datatable__header--sorted-asc]="isColumnSorted(col, 'asc')" [class.adf-datatable__header--sorted-asc]="isColumnSorted(col, 'asc')"
[class.adf-datatable__header--sorted-desc]="isColumnSorted(col, 'desc')" [class.adf-datatable__header--sorted-desc]="isColumnSorted(col, 'desc')"
[attr.aria-label]="col.title | translate"
(click)="onColumnHeaderClick(col)" (click)="onColumnHeaderClick(col)"
(keyup.enter)="onColumnHeaderClick(col)" (keyup.enter)="onColumnHeaderClick(col)"
role="columnheader" role="columnheader"