Skip to content

Commit

Permalink
fix: old style utils
Browse files Browse the repository at this point in the history
  • Loading branch information
chrispader committed Dec 7, 2023
1 parent 7609021 commit a9cdc0b
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 30 deletions.
7 changes: 2 additions & 5 deletions src/components/Modal/BaseModal.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import useWindowDimensions from '@hooks/useWindowDimensions';
import ComposerFocusManager from '@libs/ComposerFocusManager';
import useNativeDriver from '@libs/useNativeDriver';
import useTheme from '@styles/themes/useTheme';
import getModalStyles from '@styles/ThemeStyleUtils/ModalStyleUtils';
import useStyleUtils from '@styles/useStyleUtils';
import useThemeStyles from '@styles/useThemeStyles';
import variables from '@styles/variables';
Expand Down Expand Up @@ -134,9 +133,7 @@ function BaseModal(
hideBackdrop,
} = useMemo(
() =>
getModalStyles(
theme,
styles,
StyleUtils.getModalStyles(
type,
{
windowWidth,
Expand All @@ -147,7 +144,7 @@ function BaseModal(
innerContainerStyle,
outerStyle,
),
[innerContainerStyle, isSmallScreenWidth, outerStyle, popoverAnchorPosition, theme, type, windowHeight, windowWidth, styles],
[StyleUtils, type, windowWidth, windowHeight, isSmallScreenWidth, popoverAnchorPosition, innerContainerStyle, outerStyle],
);

const {
Expand Down
28 changes: 12 additions & 16 deletions src/components/PopoverWithoutOverlay/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,31 +5,27 @@ import {defaultProps, propTypes} from '@components/Popover/popoverPropTypes';
import {PopoverContext} from '@components/PopoverProvider';
import withWindowDimensions from '@components/withWindowDimensions';
import useSafeAreaInsets from '@hooks/useSafeAreaInsets';
import useTheme from '@styles/themes/useTheme';
import getModalStyles from '@styles/ThemeStyleUtils/ModalStyleUtils';
import useStyleUtils from '@styles/useStyleUtils';
import useThemeStyles from '@styles/useThemeStyles';
import * as Modal from '@userActions/Modal';

function Popover(props) {
const theme = useTheme();
const styles = useThemeStyles();
const StyleUtils = useStyleUtils();
const {onOpen, close} = React.useContext(PopoverContext);
const insets = useSafeAreaInsets();
const {modalStyle, modalContainerStyle, shouldAddTopSafeAreaMargin, shouldAddBottomSafeAreaMargin, shouldAddTopSafeAreaPadding, shouldAddBottomSafeAreaPadding} = getModalStyles(
theme,
styles,
'popover',
{
windowWidth: props.windowWidth,
windowHeight: props.windowHeight,
isSmallScreenWidth: false,
},
props.anchorPosition,
props.innerContainerStyle,
props.outerStyle,
);
const {modalStyle, modalContainerStyle, shouldAddTopSafeAreaMargin, shouldAddBottomSafeAreaMargin, shouldAddTopSafeAreaPadding, shouldAddBottomSafeAreaPadding} =
StyleUtils.getModalStyles(
'popover',
{
windowWidth: props.windowWidth,
windowHeight: props.windowHeight,
isSmallScreenWidth: false,
},
props.anchorPosition,
props.innerContainerStyle,
props.outerStyle,
);

const {
paddingTop: safeAreaPaddingTop,
Expand Down
13 changes: 4 additions & 9 deletions src/components/Tooltip/TooltipRenderedOnPageBody.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,7 @@ import ReactDOM from 'react-dom';
import {Animated, View} from 'react-native';
import Text from '@components/Text';
import Log from '@libs/Log';
import useTheme from '@styles/themes/useTheme';
import getTooltipStyles from '@styles/ThemeStyleUtils/TooltipStyleUtils';
import useThemeStyles from '@styles/useThemeStyles';
import useStyleUtils from '@styles/useStyleUtils';

const propTypes = {
/** Window width */
Expand Down Expand Up @@ -84,8 +82,7 @@ function TooltipRenderedOnPageBody({
const contentRef = useRef();
const rootWrapper = useRef();

const theme = useTheme();
const styles = useThemeStyles();
const StyleUtils = useStyleUtils();

useEffect(() => {
if (!renderTooltipContent || !text) {
Expand All @@ -103,7 +100,7 @@ function TooltipRenderedOnPageBody({

const {animationStyle, rootWrapperStyle, textStyle, pointerWrapperStyle, pointerStyle} = useMemo(
() =>
getTooltipStyles({
StyleUtils.getTooltipStyles({
tooltip: rootWrapper.current,
currentSize: animation,
windowWidth,
Expand All @@ -114,12 +111,10 @@ function TooltipRenderedOnPageBody({
maxWidth,
tooltipContentWidth: contentMeasuredWidth,
tooltipWrapperHeight: wrapperMeasuredHeight,
theme,
styles,
shiftHorizontal,
shiftVertical,
}),
[animation, windowWidth, xOffset, yOffset, targetWidth, targetHeight, maxWidth, contentMeasuredWidth, wrapperMeasuredHeight, shiftHorizontal, shiftVertical, theme, styles],
[StyleUtils, animation, windowWidth, xOffset, yOffset, targetWidth, targetHeight, maxWidth, contentMeasuredWidth, wrapperMeasuredHeight, shiftHorizontal, shiftVertical],
);

let content;
Expand Down

0 comments on commit a9cdc0b

Please sign in to comment.