From 60028d63c058831d0604c463c75f34a1299f7e3f Mon Sep 17 00:00:00 2001 From: Rita Anene <92169163+Camillarhi@users.noreply.github.com> Date: Wed, 26 Jun 2024 12:07:22 +0100 Subject: [PATCH] [MM-58409] Migrate tooltips of 'components/file_preview_modal/popover_bar/popover_bar' to WithTooltip (#27447) --- .../__snapshots__/popover_bar.test.tsx.snap | 81 ++++++------------- .../popover_bar/popover_bar.tsx | 65 +++++++-------- 2 files changed, 53 insertions(+), 93 deletions(-) diff --git a/webapp/channels/src/components/file_preview_modal/popover_bar/__snapshots__/popover_bar.test.tsx.snap b/webapp/channels/src/components/file_preview_modal/popover_bar/__snapshots__/popover_bar.test.tsx.snap index e3c4eae801..7c7191447a 100644 --- a/webapp/channels/src/components/file_preview_modal/popover_bar/__snapshots__/popover_bar.test.tsx.snap +++ b/webapp/channels/src/components/file_preview_modal/popover_bar/__snapshots__/popover_bar.test.tsx.snap @@ -8,26 +8,15 @@ exports[`components/file_preview_modal/popover_bar/PopoverBar should match snaps
- - - - } placement="top" - trigger={ - Array [ - "hover", - "focus", - ] + title={ + } > - - + - - - } placement="top" - trigger={ - Array [ - "hover", - "focus", - ] + title={ + } > - - + - - - } placement="top" - trigger={ - Array [ - "hover", - "focus", - ] + title={ + } > - +
`; diff --git a/webapp/channels/src/components/file_preview_modal/popover_bar/popover_bar.tsx b/webapp/channels/src/components/file_preview_modal/popover_bar/popover_bar.tsx index c0a617241a..c207c91848 100644 --- a/webapp/channels/src/components/file_preview_modal/popover_bar/popover_bar.tsx +++ b/webapp/channels/src/components/file_preview_modal/popover_bar/popover_bar.tsx @@ -5,10 +5,9 @@ import debounce from 'lodash/debounce'; import React from 'react'; import {FormattedMessage} from 'react-intl'; -import OverlayTrigger from 'components/overlay_trigger'; -import Tooltip from 'components/tooltip'; +import WithTooltip from 'components/with_tooltip'; -import {Constants, ZoomSettings} from 'utils/constants'; +import {ZoomSettings} from 'utils/constants'; export interface Props { scale?: number; @@ -43,21 +42,19 @@ export default class PopoverBar extends React.PureComponent { ); } zoomControls.push( - - - + id='zoom-out-icon-tooltip' + title={ + } + placement='top' > {zoomOutButton} - , + , ); if (this.props.scale && this.props.scale > ZoomSettings.DEFAULT_SCALE) { @@ -84,21 +81,19 @@ export default class PopoverBar extends React.PureComponent { ); } zoomControls.push( - - - + id='zoom-reset-icon-tooltip' + title={ + } + placement='top' > {zoomResetButton} - , + , ); if (this.props.scale && this.props.scale < ZoomSettings.MAX_SCALE) { @@ -118,21 +113,19 @@ export default class PopoverBar extends React.PureComponent { ); } zoomControls.push( - - - + id='zoom-in-icon-tooltip' + title={ + } + placement='top' > {zoomInButton} - , + , ); wrappedZoomControls = (