[MM-59899] Migrate tooltips of 'components/setting_picture.tsx' to WithTooltip (#27821)

Этот коммит содержится в:
Rita Anene
2024-08-08 20:56:09 +01:00
коммит произвёл GitHub
родитель 59873a2a50
Коммит d393bbcac0
2 изменённых файлов: 25 добавлений и 58 удалений

Просмотреть файл

@@ -739,29 +739,15 @@ exports[`components/SettingItemMin should match snapshot, team icon on source 1`
src="http://localhost:8065/api/v4/users/src_id" src="http://localhost:8065/api/v4/users/src_id"
/> />
</div> </div>
<OverlayTrigger <WithTooltip
defaultOverlayShown={false} disabled={false}
delayShow={400} id="removeIcon"
overlay={
<Tooltip
id="removeIcon"
>
<div
aria-hidden={true}
>
<Memo(MemoizedFormattedMessage)
defaultMessage="Remove This Icon"
id="setting_picture.remove"
/>
</div>
</Tooltip>
}
placement="right" placement="right"
trigger={ title={
Array [ <Memo(MemoizedFormattedMessage)
"hover", defaultMessage="Remove This Icon"
"focus", id="setting_picture.remove"
] />
} }
> >
<button <button
@@ -783,7 +769,7 @@ exports[`components/SettingItemMin should match snapshot, team icon on source 1`
/> />
</span> </span>
</button> </button>
</OverlayTrigger> </WithTooltip>
</div> </div>
</li> </li>
<div <div
@@ -908,29 +894,15 @@ exports[`components/SettingItemMin should match snapshot, user icon on source 1`
src="http://localhost:8065/api/v4/users/src_id" src="http://localhost:8065/api/v4/users/src_id"
/> />
</div> </div>
<OverlayTrigger <WithTooltip
defaultOverlayShown={false} disabled={false}
delayShow={400} id="removeIcon"
overlay={
<Tooltip
id="removeIcon"
>
<div
aria-hidden={true}
>
<Memo(MemoizedFormattedMessage)
defaultMessage="Remove Profile Picture"
id="setting_picture.remove_profile_picture"
/>
</div>
</Tooltip>
}
placement="right" placement="right"
trigger={ title={
Array [ <Memo(MemoizedFormattedMessage)
"hover", defaultMessage="Remove Profile Picture"
"focus", id="setting_picture.remove_profile_picture"
] />
} }
> >
<button <button
@@ -952,7 +924,7 @@ exports[`components/SettingItemMin should match snapshot, user icon on source 1`
/> />
</span> </span>
</button> </button>
</OverlayTrigger> </WithTooltip>
</div> </div>
</li> </li>
<div <div

Просмотреть файл

@@ -6,14 +6,14 @@ import type {ChangeEvent, CSSProperties, MouseEvent, ReactNode, RefObject} from
import {FormattedMessage} from 'react-intl'; import {FormattedMessage} from 'react-intl';
import FormError from 'components/form_error'; import FormError from 'components/form_error';
import OverlayTrigger from 'components/overlay_trigger';
import Tooltip from 'components/tooltip';
import LoadingWrapper from 'components/widgets/loading/loading_wrapper'; import LoadingWrapper from 'components/widgets/loading/loading_wrapper';
import {Constants} from 'utils/constants'; import {Constants} from 'utils/constants';
import * as FileUtils from 'utils/file_utils'; import * as FileUtils from 'utils/file_utils';
import {localizeMessage} from 'utils/utils'; import {localizeMessage} from 'utils/utils';
import WithTooltip from './with_tooltip';
type Props = { type Props = {
clientError?: ReactNode; clientError?: ReactNode;
serverError?: ReactNode; serverError?: ReactNode;
@@ -231,16 +231,11 @@ export default class SettingPicture extends Component<Props, State> {
> >
{imageElement} {imageElement}
</div> </div>
<OverlayTrigger <WithTooltip
delayShow={Constants.OVERLAY_TIME_DELAY} id='removeIcon'
title={title ?? ''}
placement='right' placement='right'
overlay={( disabled={!title}
<Tooltip id='removeIcon'>
<div aria-hidden={true}>
{title}
</div>
</Tooltip>
)}
> >
<button <button
data-testid='removeSettingPicture' data-testid='removeSettingPicture'
@@ -250,7 +245,7 @@ export default class SettingPicture extends Component<Props, State> {
<span aria-hidden={true}>{'×'}</span> <span aria-hidden={true}>{'×'}</span>
<span className='sr-only'>{title}</span> <span className='sr-only'>{title}</span>
</button> </button>
</OverlayTrigger> </WithTooltip>
</div> </div>
); );
} }