Skip to content

Commit

Permalink
Fixed a11y issue on rollup jobs table selection (elastic#84567) (elas…
Browse files Browse the repository at this point in the history
…tic#84880)

Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Kibana Machine <[email protected]>
  • Loading branch information
yuliacech and kibanamachine authored Dec 3, 2020
1 parent 0b8e7a0 commit 7c4ec08
Showing 1 changed file with 7 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -310,6 +310,10 @@ export class JobTable extends Component {
this.toggleItem(id);
}}
data-test-subj={`indexTableRowCheckbox-${id}`}
aria-label={i18n.translate('xpack.rollupJobs.jobTable.selectRow', {
defaultMessage: 'Select this row {id}',
values: { id },
})}
/>
</EuiTableRowCellCheckbox>

Expand Down Expand Up @@ -380,6 +384,9 @@ export class JobTable extends Component {
checked={this.areAllItemsSelected()}
onChange={this.toggleAll}
type="inList"
aria-label={i18n.translate('xpack.rollupJobs.jobTable.selectAllRows', {
defaultMessage: 'Select all rows',
})}
/>
</EuiTableHeaderCellCheckbox>
{this.buildHeader()}
Expand Down

0 comments on commit 7c4ec08

Please sign in to comment.