Skip to content

Commit

Permalink
[MM-59890] Migrate tooltips of 'components/admin_console/brand_image_…
Browse files Browse the repository at this point in the history
…setting/brand_image_setting.tsx' to WithTooltip (mattermost#27854)
  • Loading branch information
KvngMikey authored Aug 7, 2024
1 parent 1bb8298 commit 5238d40
Showing 1 changed file with 9 additions and 15 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,7 @@ import {Client4} from 'mattermost-redux/client';
import {uploadBrandImage, deleteBrandImage} from 'actions/admin_actions.jsx';

import FormError from 'components/form_error';
import OverlayTrigger from 'components/overlay_trigger';
import Tooltip from 'components/tooltip';
import WithTooltip from 'components/with_tooltip';

import {Constants} from 'utils/constants';

Expand Down Expand Up @@ -193,19 +192,14 @@ export default class BrandImageSetting extends React.PureComponent<Props, State>
let overlay;
if (!this.props.disabled) {
overlay = (
<OverlayTrigger
delayShow={Constants.OVERLAY_TIME_DELAY}
<WithTooltip
id='removeIcon'
title={
<FormattedMessage
id='admin.team.removeBrandImage'
defaultMessage='Remove brand image'
/>}
placement='right'
overlay={
<Tooltip id='removeIcon'>
<div aria-hidden={true}>
<FormattedMessage
id='admin.team.removeBrandImage'
defaultMessage='Remove brand image'
/>
</div>
</Tooltip>
}
>
<button
type='button'
Expand All @@ -214,7 +208,7 @@ export default class BrandImageSetting extends React.PureComponent<Props, State>
>
<span aria-hidden={true}>{'×'}</span>
</button>
</OverlayTrigger>
</WithTooltip>
);
}
img = (
Expand Down

0 comments on commit 5238d40

Please sign in to comment.