diff --git a/src-docs/src/components/guide_page/guide_page_chrome.js b/src-docs/src/components/guide_page/guide_page_chrome.js index 869ab97ab280..d0670f378170 100644 --- a/src-docs/src/components/guide_page/guide_page_chrome.js +++ b/src-docs/src/components/guide_page/guide_page_chrome.js @@ -123,7 +123,7 @@ export class GuidePageChrome extends Component { ); diff --git a/src-docs/src/routes.js b/src-docs/src/routes.js index e470cc093a27..09827b70a8d4 100644 --- a/src-docs/src/routes.js +++ b/src-docs/src/routes.js @@ -194,8 +194,6 @@ import { ToastExample } from './views/toast/toast_example'; import { ToolTipExample } from './views/tool_tip/tool_tip_example'; -// import { ToggleExample } from './views/toggle/toggle_example'; - import { WindowEventExample } from './views/window_event/window_event_example'; import { Changelog } from './views/package/changelog'; @@ -423,7 +421,6 @@ const navigation = [ PortalExample, ResizeObserverExample, ResponsiveExample, - // ToggleExample, WindowEventExample, ].map(example => createExample(example)), }, diff --git a/src-docs/src/views/accordion/accordion_form.js b/src-docs/src/views/accordion/accordion_form.js index c1ead2da4e19..213101ac4cdb 100644 --- a/src-docs/src/views/accordion/accordion_form.js +++ b/src-docs/src/views/accordion/accordion_form.js @@ -72,7 +72,7 @@ const extraAction = ( iconType="cross" color="danger" className="euiAccordionForm__extraAction" - label="Delete" + aria-label="Delete" /> ); diff --git a/src-docs/src/views/button/button_as_link.js b/src-docs/src/views/button/button_as_link.js index b35e4b9e1b04..cf9f3a3aa8ff 100644 --- a/src-docs/src/views/button/button_as_link.js +++ b/src-docs/src/views/button/button_as_link.js @@ -25,7 +25,7 @@ export default () => ( @@ -47,7 +47,7 @@ export default () => ( diff --git a/src-docs/src/views/button/button_ghost.js b/src-docs/src/views/button/button_ghost.js index 54ac3d5004f4..8ebc44ead291 100644 --- a/src-docs/src/views/button/button_ghost.js +++ b/src-docs/src/views/button/button_ghost.js @@ -62,7 +62,7 @@ export default class extends Component { color="ghost" iconType="user" onClick={() => window.alert('Button clicked')} - label="Your account" + aria-label="Your account" /> diff --git a/src-docs/src/views/button/button_icon.js b/src-docs/src/views/button/button_icon.js index b481b3c9ec78..b39e96bd0e23 100644 --- a/src-docs/src/views/button/button_icon.js +++ b/src-docs/src/views/button/button_icon.js @@ -24,7 +24,7 @@ export default () => ( color={color} onClick={() => window.alert('Button clicked')} iconType="arrowRight" - label="Next" + aria-label="Next" disabled={color === 'disabled' ? true : false} /> diff --git a/src-docs/src/views/button/button_toggle.js b/src-docs/src/views/button/button_toggle.js index cdc9139e83f8..0158f7d3e014 100644 --- a/src-docs/src/views/button/button_toggle.js +++ b/src-docs/src/views/button/button_toggle.js @@ -48,13 +48,13 @@ export default class extends Component {     { @@ -118,7 +118,7 @@ const SelectionButton = () => { @@ -220,7 +220,7 @@ const FlyoutRowCell = rowIndex => { color="text" iconType="eye" iconSize="s" - label="View details" + aria-label="View details" onClick={() => setIsFlyoutOpen(!isFlyoutOpen)} /> {flyout} @@ -258,7 +258,7 @@ const trailingControlColumns = [ panelPaddingSize="s" button={ setIsPopoverOpen(!isPopoverOpen)} @@ -275,7 +275,7 @@ const trailingControlColumns = [ gutterSize="s"> @@ -291,7 +291,7 @@ const trailingControlColumns = [ gutterSize="s"> diff --git a/src-docs/src/views/datagrid/datagrid.js b/src-docs/src/views/datagrid/datagrid.js index cf0e74da6e16..6597826bbd74 100644 --- a/src-docs/src/views/datagrid/datagrid.js +++ b/src-docs/src/views/datagrid/datagrid.js @@ -33,7 +33,7 @@ const columns = [ alert('Email Icon Clicked!')} @@ -103,7 +103,7 @@ const trailingControlColumns = [ anchorPosition="upCenter" button={ setIsPopoverOpen(!isPopoverOpen)} @@ -120,7 +120,7 @@ const trailingControlColumns = [ gutterSize="s"> @@ -136,7 +136,7 @@ const trailingControlColumns = [ gutterSize="s"> diff --git a/src-docs/src/views/datagrid/schema.js b/src-docs/src/views/datagrid/schema.js index dc21cea7cdbe..44192d309a28 100644 --- a/src-docs/src/views/datagrid/schema.js +++ b/src-docs/src/views/datagrid/schema.js @@ -152,7 +152,7 @@ export default class DataGridSchema extends Component { dummyIcon = () => ( ); diff --git a/src-docs/src/views/drag_and_drop/drag_and_drop_clone.js b/src-docs/src/views/drag_and_drop/drag_and_drop_clone.js index af8f4c1c8f75..4056a8b0c440 100644 --- a/src-docs/src/views/drag_and_drop/drag_and_drop_clone.js +++ b/src-docs/src/views/drag_and_drop/drag_and_drop_clone.js @@ -103,7 +103,7 @@ export default () => { ) : ( remove('DROPPABLE_AREA_COPY_2', idx)} /> )} diff --git a/src-docs/src/views/drag_and_drop/drag_and_drop_complex.js b/src-docs/src/views/drag_and_drop/drag_and_drop_complex.js index 2f69f7632ed6..fa66efafc42c 100644 --- a/src-docs/src/views/drag_and_drop/drag_and_drop_complex.js +++ b/src-docs/src/views/drag_and_drop/drag_and_drop_complex.js @@ -75,7 +75,7 @@ export default () => { { } - append={} + append={} compressed={isCompressed} disabled={isDisabled} readOnly={isReadOnly} @@ -103,11 +103,11 @@ export default () => { placeholder="Icons in buttons and popovers and tooltips" prepend={[ , - , + , ]} append={[ } + button={} closePopover={() => {}} />, , @@ -127,7 +127,7 @@ export default () => { } prepend={ - + } compressed={isCompressed} @@ -138,7 +138,10 @@ export default () => { , 'String']} - append={['String', ]} + append={[ + 'String', + , + ]} compressed={isCompressed} disabled={isDisabled} readOnly={isReadOnly} @@ -149,12 +152,12 @@ export default () => { prepend={[ 'String', - + , ]} append={[ } + button={} closePopover={() => {}} />, 'String', diff --git a/src-docs/src/views/guidelines/button.js b/src-docs/src/views/guidelines/button.js index 0c21210087ea..26a8a7656609 100644 --- a/src-docs/src/views/guidelines/button.js +++ b/src-docs/src/views/guidelines/button.js @@ -104,7 +104,7 @@ export default () => ( color="danger" onClick={() => window.alert('Button clicked')} iconType="trash" - label="Next" + aria-label="Next" /> @@ -264,9 +264,9 @@ export default () => ( panel={false} frame="frame">
- +      - +
( size="s" color="danger" iconType="trash" - label="delete" + aria-label="delete" /> @@ -492,7 +492,7 @@ export default () => ( size="s" color="danger" iconType="cross" - label="Remove" + aria-label="Remove" /> diff --git a/src-docs/src/views/suggest/global_filter_options.js b/src-docs/src/views/suggest/global_filter_options.js index 3a8ce5dc4372..ee0e7a276afc 100644 --- a/src-docs/src/views/suggest/global_filter_options.js +++ b/src-docs/src/views/suggest/global_filter_options.js @@ -106,7 +106,7 @@ export default class GlobalFilterOptions extends Component { onClick={this.togglePopover} color="text" iconType="filter" - label="Change all filters" + aria-label="Change all filters" title="Change all filters" /> } diff --git a/src-docs/src/views/tables/custom/custom.js b/src-docs/src/views/tables/custom/custom.js index 8dd1f68d2556..43036e20fa0b 100644 --- a/src-docs/src/views/tables/custom/custom.js +++ b/src-docs/src/views/tables/custom/custom.js @@ -527,7 +527,7 @@ export default class extends Component { id={`${item.id}-actions`} button={ ( this.toggleDetails(item)} - label={itemIdToExpandedRowMap[item.id] ? 'Collapse' : 'Expand'} + aria-label={itemIdToExpandedRowMap[item.id] ? 'Collapse' : 'Expand'} iconType={itemIdToExpandedRowMap[item.id] ? 'arrowUp' : 'arrowDown'} /> ), diff --git a/src/components/basic_table/__snapshots__/collapsed_item_actions.test.tsx.snap b/src/components/basic_table/__snapshots__/collapsed_item_actions.test.tsx.snap index 9b403d682b88..3268b0b4941a 100644 --- a/src/components/basic_table/__snapshots__/collapsed_item_actions.test.tsx.snap +++ b/src/components/basic_table/__snapshots__/collapsed_item_actions.test.tsx.snap @@ -12,6 +12,7 @@ exports[`CollapsedItemActions render 1`] = ` class="euiToolTipAnchor" > diff --git a/src/components/basic_table/__snapshots__/default_item_action.test.tsx.snap b/src/components/basic_table/__snapshots__/default_item_action.test.tsx.snap index 2ec07540c8ca..b4c335cda111 100644 --- a/src/components/basic_table/__snapshots__/default_item_action.test.tsx.snap +++ b/src/components/basic_table/__snapshots__/default_item_action.test.tsx.snap @@ -43,10 +43,10 @@ exports[`DefaultItemAction render - icon 1`] = ` position="top" > diff --git a/src/components/basic_table/__snapshots__/in_memory_table.test.tsx.snap b/src/components/basic_table/__snapshots__/in_memory_table.test.tsx.snap index e409c27ad8b8..626acbbb17a9 100644 --- a/src/components/basic_table/__snapshots__/in_memory_table.test.tsx.snap +++ b/src/components/basic_table/__snapshots__/in_memory_table.test.tsx.snap @@ -437,14 +437,15 @@ exports[`EuiInMemoryTable behavior pagination 1`] = ` token="euiPagination.previousPage" > @@ -571,14 +565,15 @@ exports[`EuiInMemoryTable behavior pagination 1`] = ` token="euiPagination.nextPage" > diff --git a/src/components/basic_table/collapsed_item_actions.tsx b/src/components/basic_table/collapsed_item_actions.tsx index 18f827744bed..7586465172c2 100644 --- a/src/components/basic_table/collapsed_item_actions.tsx +++ b/src/components/basic_table/collapsed_item_actions.tsx @@ -149,7 +149,7 @@ export class CollapsedItemActions extends Component< {(allActions: string) => ( ({ button = (
`; @@ -29,46 +29,49 @@ exports[`EuiButtonGroup props buttonSize compressed is rendered 1`] = ` class="euiButtonGroup euiButtonGroup--compressed" >
+ -
+ -
+ -
@@ -84,49 +87,52 @@ exports[`EuiButtonGroup props buttonSize m is rendered 1`] = ` test
+ -
+ -
+ -
@@ -142,49 +148,52 @@ exports[`EuiButtonGroup props buttonSize s is rendered 1`] = ` test
+ -
+ -
+ -
@@ -200,49 +209,52 @@ exports[`EuiButtonGroup props color danger is rendered 1`] = ` test
+ -
+ -
+ -
@@ -258,49 +270,52 @@ exports[`EuiButtonGroup props color ghost is rendered 1`] = ` test
+ -
+ -
+ -
@@ -316,49 +331,52 @@ exports[`EuiButtonGroup props color primary is rendered 1`] = ` test
+ -
+ -
+ -
@@ -374,49 +392,52 @@ exports[`EuiButtonGroup props color secondary is rendered 1`] = ` test
+ -
+ -
+ -
@@ -432,49 +453,52 @@ exports[`EuiButtonGroup props color text is rendered 1`] = ` test
+ -
+ -
+ -
@@ -490,49 +514,52 @@ exports[`EuiButtonGroup props color warning is rendered 1`] = ` test
+ -
+ -
+ -
@@ -548,50 +575,53 @@ exports[`EuiButtonGroup props idSelected is rendered 1`] = ` test
- +
+ -
+ -
@@ -607,52 +637,55 @@ exports[`EuiButtonGroup props isDisabled is rendered 1`] = ` test
+ +
+
-
-
+
+
-
-
-
@@ -668,49 +701,52 @@ exports[`EuiButtonGroup props isFullWidth is rendered 1`] = ` test
+ -
+ -
+ -
@@ -726,12 +762,18 @@ exports[`EuiButtonGroup props isIconOnly is rendered 1`] = ` test
+ -
+ -
+ -
@@ -796,49 +835,52 @@ exports[`EuiButtonGroup props legend is rendered 1`] = ` test
+ -
+ -
+ -
@@ -854,52 +896,55 @@ exports[`EuiButtonGroup props name is rendered 1`] = ` test
+ +
+
-
-
+
+
-
-
-
@@ -915,49 +960,52 @@ exports[`EuiButtonGroup props options are rendered 1`] = ` test
+ -
+ -
+ -
@@ -973,23 +1021,24 @@ exports[`EuiButtonGroup props options can pass down data-test-subj 1`] = ` test
- +
@@ -1005,10 +1054,10 @@ exports[`EuiButtonGroup props type of multi idToSelectedMap is rendered 1`] = ` test
+/> `; exports[`EuiButtonIcon props color danger is rendered 1`] = ` +/> `; exports[`EuiButtonIcon props color disabled is rendered 1`] = ` +/> `; exports[`EuiButtonIcon props color ghost is rendered 1`] = ` +/> `; exports[`EuiButtonIcon props color primary is rendered 1`] = ` +/> `; exports[`EuiButtonIcon props color subdued is rendered 1`] = ` +/> `; exports[`EuiButtonIcon props color success is rendered 1`] = ` +/> `; exports[`EuiButtonIcon props color text is rendered 1`] = ` +/> `; exports[`EuiButtonIcon props color warning is rendered 1`] = ` +/> `; exports[`EuiButtonIcon props href secures the rel attribute when the target is _blank 1`] = ` - - button - - +/> `; exports[`EuiButtonIcon props iconType is rendered 1`] = ` `; exports[`EuiButtonIcon props isDisabled is rendered 1`] = ` +/> `; exports[`EuiButtonIcon props isDisabled renders a button even when href is defined 1`] = ` +/> `; diff --git a/src/components/button/button_icon/button_icon.test.tsx b/src/components/button/button_icon/button_icon.test.tsx index da954b11ac66..8a6e3d5b2c98 100644 --- a/src/components/button/button_icon/button_icon.test.tsx +++ b/src/components/button/button_icon/button_icon.test.tsx @@ -7,7 +7,7 @@ import { EuiButtonIcon, COLORS } from './button_icon'; describe('EuiButtonIcon', () => { test('is rendered', () => { const component = render( - + ); expect(component).toMatchSnapshot(); @@ -16,14 +16,16 @@ describe('EuiButtonIcon', () => { describe('props', () => { describe('isDisabled', () => { it('is rendered', () => { - const component = render(); + const component = render( + + ); expect(component).toMatchSnapshot(); }); it('renders a button even when href is defined', () => { const component = render( - + ); expect(component).toMatchSnapshot(); @@ -33,7 +35,7 @@ describe('EuiButtonIcon', () => { describe('iconType', () => { it('is rendered', () => { const component = render( - + ); expect(component).toMatchSnapshot(); @@ -44,7 +46,7 @@ describe('EuiButtonIcon', () => { COLORS.forEach(color => { test(`${color} is rendered`, () => { const component = render( - + ); expect(component).toMatchSnapshot(); @@ -55,7 +57,7 @@ describe('EuiButtonIcon', () => { describe('href', () => { it('secures the rel attribute when the target is _blank', () => { const component = render( - + ); expect(component).toMatchSnapshot(); @@ -66,7 +68,7 @@ describe('EuiButtonIcon', () => { it('supports onClick and href', () => { const handler = jest.fn(); const component = mount( - + ); component.find('a').simulate('click'); expect(handler.mock.calls.length).toEqual(1); @@ -75,7 +77,7 @@ describe('EuiButtonIcon', () => { it('supports onClick as a button', () => { const handler = jest.fn(); const component = mount( - + ); component.find('button').simulate('click'); expect(handler.mock.calls.length).toEqual(1); diff --git a/src/components/button/button_icon/button_icon.tsx b/src/components/button/button_icon/button_icon.tsx index aca785d65298..88740f456a4a 100644 --- a/src/components/button/button_icon/button_icon.tsx +++ b/src/components/button/button_icon/button_icon.tsx @@ -3,10 +3,9 @@ import React, { ButtonHTMLAttributes, FunctionComponent, Ref, - ReactNode, } from 'react'; import classNames from 'classnames'; -import { EuiScreenReaderOnly } from '../../accessibility/screen_reader'; + import { getSecureRelForTarget } from '../../../services'; import { CommonProps, @@ -32,12 +31,14 @@ export type EuiButtonIconColor = export type HideOrLabel = ExclusiveUnion< { 'aria-hidden': true }, - ExclusiveUnion<{ label: ReactNode }, { 'aria-labelledby': string }> + ExclusiveUnion<{ 'aria-label': string }, { 'aria-labelledby': string }> >; export type EuiButtonIconProps = { iconType?: IconType; color?: EuiButtonIconColor; + 'aria-label'?: string; + 'aria-labelledby'?: string; isDisabled?: boolean; size?: ButtonSize; iconSize?: IconSize; @@ -87,7 +88,6 @@ export const EuiButtonIcon: FunctionComponent = ({ target, rel, buttonRef, - label, ...rest }) => { const classes = classNames( @@ -124,11 +124,6 @@ export const EuiButtonIcon: FunctionComponent = ({ ref={buttonRef as Ref} {...rest as AnchorHTMLAttributes}> {buttonIcon} - {label && ( - - {label} - - )} ); } @@ -142,11 +137,6 @@ export const EuiButtonIcon: FunctionComponent = ({ ref={buttonRef as Ref} {...rest as ButtonHTMLAttributes}> {buttonIcon} - {label && ( - - {label} - - )} ); }; diff --git a/src/components/button/index.ts b/src/components/button/index.ts index 3a431140e38a..a2eebfd7f093 100644 --- a/src/components/button/index.ts +++ b/src/components/button/index.ts @@ -5,6 +5,7 @@ export { ButtonIconSide, EuiButton, EuiButtonProps, + colorToClassNameMap, } from './button'; export { diff --git a/src/components/code/__snapshots__/code_block.test.tsx.snap b/src/components/code/__snapshots__/code_block.test.tsx.snap index 89f3a081743c..8b2bf9a15f60 100644 --- a/src/components/code/__snapshots__/code_block.test.tsx.snap +++ b/src/components/code/__snapshots__/code_block.test.tsx.snap @@ -101,6 +101,7 @@ console.log(some); class="euiToolTipAnchor" >
@@ -158,6 +154,7 @@ console.log(some); class="euiCodeBlock__controls" > diff --git a/src/components/code/_code_block.tsx b/src/components/code/_code_block.tsx index 5d2d96cf96f3..14ecd0993dc7 100644 --- a/src/components/code/_code_block.tsx +++ b/src/components/code/_code_block.tsx @@ -214,7 +214,7 @@ export class EuiCodeBlockImpl extends Component { onClick={copy} iconType="copy" color="text" - label={copyButton} + aria-label={copyButton} /> )} @@ -244,7 +244,7 @@ export class EuiCodeBlockImpl extends Component { onClick={this.toggleFullScreen} iconType={this.state.isFullScreen ? 'cross' : 'fullScreen'} color="text" - label={ + aria-label={ this.state.isFullScreen ? fullscreenCollapse : fullscreenExpand } /> diff --git a/src/components/color_picker/color_stops/color_stop_thumb.tsx b/src/components/color_picker/color_stops/color_stop_thumb.tsx index 832ade7ce50c..4aa27edae0c2 100644 --- a/src/components/color_picker/color_stops/color_stop_thumb.tsx +++ b/src/components/color_picker/color_stops/color_stop_thumb.tsx @@ -334,7 +334,7 @@ export const EuiColorStopThumb: FunctionComponent = ({ ); } diff --git a/src/components/datagrid/__snapshots__/data_grid.test.tsx.snap b/src/components/datagrid/__snapshots__/data_grid.test.tsx.snap index 4c4447170ed0..359844937b4e 100644 --- a/src/components/datagrid/__snapshots__/data_grid.test.tsx.snap +++ b/src/components/datagrid/__snapshots__/data_grid.test.tsx.snap @@ -84,6 +84,7 @@ exports[`EuiDataGrid pagination renders 1`] = ` role="group" > diff --git a/src/components/datagrid/column_sorting_draggable.tsx b/src/components/datagrid/column_sorting_draggable.tsx index efbf4d71fb5e..8d92e24994d5 100644 --- a/src/components/datagrid/column_sorting_draggable.tsx +++ b/src/components/datagrid/column_sorting_draggable.tsx @@ -96,7 +96,7 @@ export const EuiDataGridColumnSortingDraggable: FunctionComponent< color="text" size="s" className="euiDataGridColumnSorting__button" - label={`${removeSortLabel} ${id}`} + aria-label={`${removeSortLabel} ${id}`} iconType="cross" onClick={() => { const nextColumns = [...sorting.columns]; diff --git a/src/components/flyout/__snapshots__/flyout.test.tsx.snap b/src/components/flyout/__snapshots__/flyout.test.tsx.snap index de9978be98ec..047a7510a836 100644 --- a/src/components/flyout/__snapshots__/flyout.test.tsx.snap +++ b/src/components/flyout/__snapshots__/flyout.test.tsx.snap @@ -23,6 +23,7 @@ exports[`EuiFlyout is rendered 1`] = ` tabindex="0" > @@ -70,6 +66,7 @@ exports[`EuiFlyout max width can be set to a custom number 1`] = ` tabindex="0" > @@ -117,6 +109,7 @@ exports[`EuiFlyout max width can be set to a custom value and measurement 1`] = tabindex="0" > @@ -163,6 +151,7 @@ exports[`EuiFlyout max width can be set to a default 1`] = ` tabindex="0" > @@ -210,6 +194,7 @@ exports[`EuiFlyout props accepts div props 1`] = ` tabindex="0" > @@ -285,6 +265,7 @@ exports[`EuiFlyout size l is rendered 1`] = ` tabindex="0" > @@ -331,6 +307,7 @@ exports[`EuiFlyout size m is rendered 1`] = ` tabindex="0" > @@ -377,6 +349,7 @@ exports[`EuiFlyout size s is rendered 1`] = ` tabindex="0" > diff --git a/src/components/flyout/flyout.test.tsx b/src/components/flyout/flyout.test.tsx index 44940c8148ba..eb32c45275f1 100644 --- a/src/components/flyout/flyout.test.tsx +++ b/src/components/flyout/flyout.test.tsx @@ -29,7 +29,7 @@ describe('EuiFlyout', () => { const component = render( {}} />); const label = component .find('[data-test-subj="euiFlyoutCloseButton"]') - .text(); + .prop('aria-label'); expect(label).toBe('Closes this dialog'); }); @@ -42,7 +42,7 @@ describe('EuiFlyout', () => { ); const label = component .find('[data-test-subj="euiFlyoutCloseButton"]') - .text(); + .prop('aria-label'); expect(label).toBe('Closes specific flyout'); }); }); diff --git a/src/components/flyout/flyout.tsx b/src/components/flyout/flyout.tsx index 18f87c558612..20592ecab0b8 100644 --- a/src/components/flyout/flyout.tsx +++ b/src/components/flyout/flyout.tsx @@ -99,11 +99,12 @@ export class EuiFlyout extends Component { let closeButton; if (onClose && !hideCloseButton) { closeButton = ( + // @ts-ignore Help? diff --git a/src/components/list_group/__snapshots__/list_group.test.tsx.snap b/src/components/list_group/__snapshots__/list_group.test.tsx.snap index b7073bce670f..89a24f3907a6 100644 --- a/src/components/list_group/__snapshots__/list_group.test.tsx.snap +++ b/src/components/list_group/__snapshots__/list_group.test.tsx.snap @@ -44,6 +44,7 @@ exports[`EuiListGroup is rendered with listItems 1`] = `
    • `; diff --git a/src/components/list_group/list_group.test.tsx b/src/components/list_group/list_group.test.tsx index 952425e632cb..854a40201a94 100644 --- a/src/components/list_group/list_group.test.tsx +++ b/src/components/list_group/list_group.test.tsx @@ -15,7 +15,7 @@ const someListItems: EuiListGroupItemProps[] = [ extraAction: { iconType: 'bell', alwaysShow: true, - label: 'Notification', + 'aria-label': 'Notification', }, }, { diff --git a/src/components/list_group/list_group_item.test.tsx b/src/components/list_group/list_group_item.test.tsx index 1d433b1ede25..40a903f4b8f7 100644 --- a/src/components/list_group/list_group_item.test.tsx +++ b/src/components/list_group/list_group_item.test.tsx @@ -98,7 +98,7 @@ describe('EuiListGroupItem', () => { extraAction={{ iconType: 'empty', alwaysShow: true, - label: 'label', + 'aria-label': 'label', }} /> ); diff --git a/src/components/modal/__snapshots__/confirm_modal.test.tsx.snap b/src/components/modal/__snapshots__/confirm_modal.test.tsx.snap index 4e24b3f926fd..1f40a73883a1 100644 --- a/src/components/modal/__snapshots__/confirm_modal.test.tsx.snap +++ b/src/components/modal/__snapshots__/confirm_modal.test.tsx.snap @@ -22,6 +22,7 @@ Array [ tabindex="0" >
    = ({ onClick={onClose} className="euiModal__closeIcon" color="text" - label={closeModal} + aria-label={closeModal} /> )} diff --git a/src/components/pagination/__snapshots__/pagination.test.tsx.snap b/src/components/pagination/__snapshots__/pagination.test.tsx.snap index 29f0ab5815e3..af20406cb58d 100644 --- a/src/components/pagination/__snapshots__/pagination.test.tsx.snap +++ b/src/components/pagination/__snapshots__/pagination.test.tsx.snap @@ -8,6 +8,7 @@ exports[`EuiPagination is rendered 1`] = ` role="group" >
    `; diff --git a/src/components/pagination/pagination.tsx b/src/components/pagination/pagination.tsx index d8dea01a2231..1f31a8298ccb 100644 --- a/src/components/pagination/pagination.tsx +++ b/src/components/pagination/pagination.tsx @@ -86,7 +86,7 @@ export const EuiPagination: FunctionComponent = ({ iconType="arrowLeft" disabled={activePage === 0} color="text" - label={previousPage} + aria-label={previousPage} data-test-subj="pagination-button-previous" /> )} @@ -165,7 +165,7 @@ export const EuiPagination: FunctionComponent = ({
    @@ -175,6 +167,7 @@ exports[`EuiTablePagination is rendered when hiding the per page options 1`] = ` role="group" >