Skip to content
Merged
Show file tree
Hide file tree
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ export const App: FunctionComponent<AppProps> = ({ onSuccess }) => {
${euiShadowM};
`}
>
<EuiIcon type="logoElastic" size="xxl" />
<EuiIcon type="logoElastic" size="xxl" aria-hidden={true} />
</span>
<EuiTitle size="m">
<h1>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -330,7 +330,7 @@ export const CertificatePanel: FunctionComponent<CertificatePanelProps> = ({
<EuiPanel color={compressed ? 'subdued' : undefined} hasBorder={!compressed}>
<EuiFlexGroup responsive={false} alignItems="center" gutterSize="m">
<EuiFlexItem grow={false}>
<EuiIcon type="document" size="l" />
<EuiIcon type="document" size="l" aria-hidden={true} />
</EuiFlexItem>
<EuiFlexItem>
<EuiFlexGroup responsive={false} gutterSize="none" justifyContent="spaceBetween">
Expand Down Expand Up @@ -453,7 +453,7 @@ const CertificateChain: FunctionComponent<CertificateChainProps> = ({ certificat
<EuiSpacer size="s" />
<EuiFlexGroup responsive={false} justifyContent="center">
<EuiFlexItem grow={false}>
<EuiIcon type="sortDown" color="subdued" />
<EuiIcon type="sortDown" color="subdued" aria-hidden={true} />
</EuiFlexItem>
</EuiFlexGroup>
<EuiSpacer size="s" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ export class ChangeAllPrivilegesControl extends Component<Props, State> {
id="xpack.security.management.editRole.changeAllPrivilegesLink"
defaultMessage="Bulk actions"
/>{' '}
<EuiIcon size="s" type="chevronSingleDown" />
<EuiIcon size="s" type="chevronSingleDown" aria-hidden={true} />
</EuiText>
</EuiLink>
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ export class FeatureTable extends Component<Props, State> {
>
{category.euiIconType ? (
<EuiFlexItem grow={false}>
<EuiIcon size="m" type={category.euiIconType} />
<EuiIcon size="m" type={category.euiIconType} aria-hidden={true} />
</EuiFlexItem>
) : null}
<EuiFlexItem grow={1}>
Expand Down Expand Up @@ -279,7 +279,11 @@ export class FeatureTable extends Component<Props, State> {
</EuiText>
);

return renderFeatureMarkup(buttonContent, extraAction, <EuiIcon type="empty" />);
return renderFeatureMarkup(
buttonContent,
extraAction,
<EuiIcon type="empty" aria-hidden={true} />
);
}

if (primaryFeaturePrivileges.length === 0) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ export const FormLabel: FC<PropsWithChildren<FormLabelProps>> = (props) => {

{!isEqual ? (
<EuiFlexItem grow={false}>
<EuiIcon type="dot" color="success" size="s" />
<EuiIcon type="dot" color="success" size="s" aria-hidden={true} />
</EuiFlexItem>
) : undefined}
</EuiFlexGroup>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ export const ThemeKeyPadMenu: FunctionComponent<ThemeKeyPadMenuProps> = ({
isDisabled={isDisabled}
onChange={() => helpers.setValue(id)}
>
<EuiIcon type={icon} size="l" />
<EuiIcon type={icon} size="l" aria-hidden={true} />
</EuiKeyPadMenuItem>
);
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ export const AuthenticationStatePage: FC<PropsWithChildren<Props>> = (props) =>
const logo = customLogo ? (
<EuiImage src={customLogo} size={40} alt="logo" />
) : (
<EuiIcon type="logoElastic" size="xxl" />
<EuiIcon type="logoElastic" size="xxl" aria-hidden={true} />
);
// custom logo needs to be centered
const logoStyle = customLogo ? { padding: 0 } : {};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -344,7 +344,7 @@ export class LoginForm extends Component<LoginFormProps, State> {
>
<EuiFlexGroup alignItems="center" gutterSize="m" responsive={false}>
<EuiFlexItem grow={false}>
<EuiIcon size="xl" type={provider.icon ?? 'empty'} />
<EuiIcon size="xl" type={provider.icon ?? 'empty'} aria-hidden={true} />
</EuiFlexItem>
<EuiFlexItem>
<EuiTitle size="xs">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -187,7 +187,7 @@ export class MappingInfoPanel extends Component<Props, State> {
id="xpack.security.management.editRoleMapping.switchToRoleTemplates"
defaultMessage="Switch to role templates"
/>{' '}
<EuiIcon size="s" type="inputOutput" />
<EuiIcon size="s" type="inputOutput" aria-hidden={true} />
</Fragment>
</EuiLink>
);
Expand Down Expand Up @@ -266,7 +266,7 @@ export class MappingInfoPanel extends Component<Props, State> {
id="xpack.security.management.editRoleMapping.switchToRoles"
defaultMessage="Switch to roles"
/>{' '}
<EuiIcon size="s" type="inputOutput" />
<EuiIcon size="s" type="inputOutput" aria-hidden={true} />
</Fragment>
</EuiLink>
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ export class FieldRuleEditor extends Component<Props, {}> {
)}
/>
) : (
<EuiIcon size="s" type="empty" />
<EuiIcon size="s" type="empty" aria-hidden={true} />
)}
</EuiFormRow>
</EuiFlexItem>
Expand All @@ -196,7 +196,7 @@ export class FieldRuleEditor extends Component<Props, {}> {
onClick={() => this.onRemoveAlternateValue(valueIndex)}
/>
) : (
<EuiIcon size="s" type="empty" />
<EuiIcon size="s" type="empty" aria-hidden={true} />
)}
</EuiFormRow>
</EuiFlexItem>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,7 @@ export class RuleEditorPanel extends Component<Props, State> {
id="xpack.security.management.editRoleMapping.switchToJSONEditorLink"
defaultMessage="Switch to JSON editor"
/>{' '}
<EuiIcon type="inputOutput" size="s" />
<EuiIcon type="inputOutput" size="s" aria-hidden={true} />
</Fragment>
</EuiLink>
);
Expand All @@ -211,7 +211,7 @@ export class RuleEditorPanel extends Component<Props, State> {
id="xpack.security.management.editRoleMapping.switchToVisualEditorLink"
defaultMessage="Switch to visual editor"
/>{' '}
<EuiIcon type="inputOutput" size="s" />
<EuiIcon type="inputOutput" size="s" aria-hidden={true} />
</Fragment>
</EuiLink>
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ export const RuleGroupTitle = (props: Props) => {
data-test-subj="ruleGroupTitle"
>
{props.rule.getDisplayTitle()}
{readOnly === false && <EuiIcon type="chevronSingleDown" />}
{readOnly === false && <EuiIcon type="chevronSingleDown" aria-hidden={true} />}
</EuiLink>
);

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ import { EuiIcon, EuiLink, EuiToolTip } from '@elastic/eui';
import React from 'react';

import type { ApplicationStart } from '@kbn/core/public';
import { i18n } from '@kbn/i18n';

import type { Role } from '../../../common';
import { getExtendedRoleDeprecationNotice, isRoleDeprecated } from '../../../common/model';
Expand All @@ -31,7 +32,17 @@ export const RoleTableDisplay = ({ role, navigateToApp }: Props) => {
data-test-subj="roleDeprecationTooltip"
>
<div tabIndex={0}>
{role.name} <EuiIcon type="warning" color="warning" size="s" className={'eui-alignTop'} />
{role.name}{' '}
<EuiIcon
type="warning"
color="warning"
size="s"
className={'eui-alignTop'}
aria-label={i18n.translate(
'xpack.security.management.roles.roleTableDisplay.deprecatedIconAriaLabel',
{ defaultMessage: 'Deprecated' }
)}
/>
</div>
</EuiToolTip>
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@ const CollapsiblePanelTitle = ({
<EuiIcon
type={iconType}
size="xl"
aria-hidden={true}
css={css`
margin-right: ${euiTheme.size.s};
vertical-align: text-bottom;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -180,7 +180,7 @@ export const PrivilegeSummaryTable = (props: PrivilegeSummaryTableProps) => {
/>
);
} else {
iconTip = <EuiIcon size="s" type="empty" />;
iconTip = <EuiIcon size="s" type="empty" aria-hidden={true} />;
}
return (
<span
Expand Down Expand Up @@ -229,7 +229,7 @@ export const PrivilegeSummaryTable = (props: PrivilegeSummaryTableProps) => {
>
{category.euiIconType ? (
<EuiFlexItem grow={false}>
<EuiIcon size="m" type={category.euiIconType} />
<EuiIcon size="m" type={category.euiIconType} aria-hidden={true} />
</EuiFlexItem>
) : null}
<EuiFlexItem grow={1}>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ import _ from 'lodash';
import type { FC, ReactNode } from 'react';
import React from 'react';

import { i18n } from '@kbn/i18n';
import { constants } from '@kbn/security-ui-components';

interface Props extends PropsOf<typeof EuiText> {
Expand Down Expand Up @@ -44,7 +45,16 @@ function getDisplayValue(privilege: string | string[] | undefined) {
(privileges.length === 1 && privileges.includes(constants.NO_PRIVILEGE_VALUE));

if (isPrivilegeMissing) {
displayValue = <EuiIcon color="subdued" type={'minusCircle'} />;
displayValue = (
<EuiIcon
color="subdued"
type={'minusCircle'}
aria-label={i18n.translate(
'xpack.security.management.editRole.privilegeDisplay.noPrivilegeAriaLabel',
{ defaultMessage: 'None' }
)}
/>
);
} else {
displayValue = privileges.map((p) => _.upperFirst(p)).join(', ');
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -187,7 +187,7 @@ export class PrivilegeSpaceTable extends Component<Props, State> {
? '*'
: basePrivilege;

let icon = <EuiIcon type="empty" size="s" />;
let icon = <EuiIcon type="empty" size="s" aria-hidden={true} />;
if (privilegeCalculator.hasSupersededInheritedPrivileges(record.privilegeIndex)) {
icon = (
<span data-test-subj="spaceTablePrivilegeSupersededWarning">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,7 @@ export const ChangePasswordModal: FunctionComponent<ChangePasswordModalProps> =
>
<EuiFlexGroup alignItems="center" gutterSize="s" responsive={false}>
<EuiFlexItem grow={false}>
<EuiIcon type="user" />
<EuiIcon type="user" aria-hidden={true} />
</EuiFlexItem>
<EuiFlexItem>
<EuiSpacer size="xs" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,13 @@ export const SessionExpirationModal: FunctionComponent<SessionExpirationModalPro
>
<EuiModalHeader>
<EuiModalHeaderTitle id="session-expiration-modal-title">
<EuiIcon type="clock" color="warning" size="l" style={{ marginRight: 8 }} />
<EuiIcon
type="clock"
color="warning"
size="l"
style={{ marginRight: 8 }}
aria-hidden={true}
/>
{i18n.translate('xpack.security.sessionExpirationModal.title', {
defaultMessage: 'Session timeout',
})}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -260,7 +260,7 @@ export const CopyToSpaceFlyoutInternal = (props: CopyToSpaceFlyoutProps) => {
<EuiFlyoutHeader hasBorder>
<EuiFlexGroup alignItems="center" gutterSize="m">
<EuiFlexItem grow={false}>
<EuiIcon size="m" type="copy" />
<EuiIcon size="m" type="copy" aria-hidden={true} />
</EuiFlexItem>
<EuiFlexItem>
<EuiTitle size="m">
Expand All @@ -277,7 +277,7 @@ export const CopyToSpaceFlyoutInternal = (props: CopyToSpaceFlyoutProps) => {
<EuiFlyoutBody>
<EuiFlexGroup alignItems="center" gutterSize="m">
<EuiFlexItem grow={false}>
<EuiIcon type={savedObjectTarget.icon} />
<EuiIcon type={savedObjectTarget.icon} aria-hidden={true} />
</EuiFlexItem>
<EuiFlexItem>
<EuiText>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ export class FeatureTable extends Component<Props, {}> {
>
{category.euiIconType ? (
<EuiFlexItem grow={false}>
<EuiIcon size="m" type={category.euiIconType} />
<EuiIcon size="m" type={category.euiIconType} aria-hidden={true} />
</EuiFlexItem>
) : null}
<EuiFlexItem grow={1}>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ const SectionPanelTitle = ({
<EuiIcon
type={iconType}
size={'xl'}
aria-hidden={true}
css={css`
vertical-align: text-bottom;
margin-right: ${euiTheme.size.s};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ const getOptions = ({ size }: EuiThemeComputed): Array<EuiSuperSelectOption<Solu
value: 'es',
inputDisplay: (
<>
<EuiIcon type="logoElasticsearch" css={iconCss} />
<EuiIcon type="logoElasticsearch" css={iconCss} aria-hidden={true} />
{i18n.translate(
'xpack.spaces.management.manageSpacePage.solutionViewSelect.searchOptionLabel',
{ defaultMessage: 'Elasticsearch' }
Expand All @@ -52,7 +52,7 @@ const getOptions = ({ size }: EuiThemeComputed): Array<EuiSuperSelectOption<Solu
value: 'oblt',
inputDisplay: (
<>
<EuiIcon type="logoObservability" css={iconCss} />
<EuiIcon type="logoObservability" css={iconCss} aria-hidden={true} />
{i18n.translate(
'xpack.spaces.management.manageSpacePage.solutionViewSelect.obltOptionLabel',
{ defaultMessage: 'Observability' }
Expand All @@ -65,7 +65,7 @@ const getOptions = ({ size }: EuiThemeComputed): Array<EuiSuperSelectOption<Solu
value: 'security',
inputDisplay: (
<>
<EuiIcon type="logoSecurity" css={iconCss} />
<EuiIcon type="logoSecurity" css={iconCss} aria-hidden={true} />
{i18n.translate(
'xpack.spaces.management.manageSpacePage.solutionViewSelect.securityOptionLabel',
{ defaultMessage: 'Security' }
Expand All @@ -78,7 +78,7 @@ const getOptions = ({ size }: EuiThemeComputed): Array<EuiSuperSelectOption<Solu
value: 'classic',
inputDisplay: (
<>
<EuiIcon type="logoElasticStack" css={iconCss} />
<EuiIcon type="logoElasticStack" css={iconCss} aria-hidden={true} />
{i18n.translate(
'xpack.spaces.management.manageSpacePage.solutionViewSelect.classicOptionLabel',
{ defaultMessage: 'Classic' }
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ export const EditSpaceContentTab: FC<{ space: Space }> = ({ space }) => {
return (
<EuiFlexGroup gutterSize="m" alignItems="center">
<EuiFlexItem grow={false}>
<EuiIcon type={icon ?? 'gear'} size="m" />
<EuiIcon type={icon ?? 'gear'} size="m" aria-hidden={true} />
</EuiFlexItem>
<EuiFlexItem grow={true}>{capitalize(displayName)}</EuiFlexItem>
</EuiFlexGroup>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -482,7 +482,7 @@ export const ShareToSpaceFlyoutInternal = (props: ShareToSpaceFlyoutProps) => {
<EuiFlexGroup alignItems="center" gutterSize="m" responsive={false}>
{flyoutIcon && (
<EuiFlexItem grow={false}>
<EuiIcon size="l" type={flyoutIcon} />
<EuiIcon size="l" type={flyoutIcon} aria-hidden={true} />
</EuiFlexItem>
)}
<EuiFlexItem>
Expand All @@ -506,7 +506,7 @@ export const ShareToSpaceFlyoutInternal = (props: ShareToSpaceFlyoutProps) => {
<EuiFlexGroup alignItems="center" gutterSize="m" responsive={false}>
{savedObjectTarget.icon && (
<EuiFlexItem grow={false}>
<EuiIcon type={savedObjectTarget.icon} />
<EuiIcon type={savedObjectTarget.icon} aria-hidden={true} />
</EuiFlexItem>
)}
<EuiFlexItem>
Expand Down
Loading