Tooltip Both Fix with Dynamic State #7288
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 2 Skipped Deployments
|
| @@ -14,6 +14,7 @@ export const Tooltip = React.memo( | |||
| const [visibleState, setVisibleState] = React.useState(false); | |||
| const [positionState, setPositionState] = React.useState(props.position || 'right'); | |||
| const [classNameState, setClassNameState] = React.useState(''); | |||
| const [multipleFocusEvents, setMultipleFocusEvents] = React.useState(false); | |||
There was a problem hiding this comment.
Actually you can switch this to a ref it doesn't need to be useState as its not used in any hooks you are just tracking a boolean state.
There was a problem hiding this comment.
I can update it but there is something I didn't understand help me if I'm missing something. I put boolean inside of state because I wanted trigger a boolean value with using ref how can I achieve that?
There was a problem hiding this comment.
you jiust do yourRef.current = true or false
There was a problem hiding this comment.
Yeap, I did the same but like I said we should trigger the component. You can try it in your local or I can commit.
There was a problem hiding this comment.
interesting i was not expecting that based on looking at this code. You are just using a boolean flag. Changing the state is not used by any hook?
There was a problem hiding this comment.
@melloware you merged but there is prettier error what should I do?
There was a problem hiding this comment.
Oh I can fix it. Next time run npm run format on the codebase it fixes it
There was a problem hiding this comment.
@Luko248 that is up to PrimeTek I don't have any insight on release dates.
fixes #7260
before
Screen.Recording.2024-10-01.at.20.27.31.mov
after
Screen.Recording.2024-10-01.at.20.28.20.mov