Skip to content

Commit

Permalink
Merge pull request #54627 from ishpaul777/fix-54600
Browse files Browse the repository at this point in the history
[fix] Web- Search- Two filter tooltips are shown when navigating to Reports page for the second time

(cherry picked from commit a5feee8)

(CP triggered by jasperhuangg)
  • Loading branch information
puneetlath authored and OSBotify committed Dec 27, 2024
1 parent 54b9941 commit 51516a4
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/components/ProductTrainingContext/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -205,8 205,8 @@ const useProductTrainingContext = (tooltipName: ProductTrainingTooltipName, shou
]);

const shouldShowProductTrainingTooltip = useMemo(() => {
return shouldRenderTooltip(tooltipName);
}, [shouldRenderTooltip, tooltipName]);
return shouldShow && shouldRenderTooltip(tooltipName);
}, [shouldRenderTooltip, tooltipName, shouldShow]);

const hideProductTrainingTooltip = useCallback(() => {
const tooltip = TOOLTIPS[tooltipName];
Expand Down

0 comments on commit 51516a4

Please sign in to comment.