From b44cb2a81191a2df88bc13208f001d1baf48a826 Mon Sep 17 00:00:00 2001
From: Raviteja Lingineni <rlingineni@pinterest.com>
Date: Fri, 18 Aug 2023 14:51:12 -0500
Subject: [PATCH] SideNavigation: Remove Tooltip on dropdown open (#3134)

---
 .../src/SideNavigation/PrimaryActionIconButton.js        | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/packages/gestalt/src/SideNavigation/PrimaryActionIconButton.js b/packages/gestalt/src/SideNavigation/PrimaryActionIconButton.js
index e46ea26d3a..5771376b6a 100644
--- a/packages/gestalt/src/SideNavigation/PrimaryActionIconButton.js
+++ b/packages/gestalt/src/SideNavigation/PrimaryActionIconButton.js
@@ -3,7 +3,7 @@ import { cloneElement, type Element, type Node, useEffect, useId, useRef, useSta
 import Dropdown from '../Dropdown.js';
 import Pog from '../Pog.js';
 import TapArea from '../TapArea.js';
-import Tooltip from '../Tooltip.js';
+import MaybeTooltip from '../utils/MaybeTooltip.js';
 import { CompositeZIndex, FixedZIndex, type Indexable } from '../zIndex.js';
 
 type Props = {|
@@ -83,7 +83,10 @@ function ItemIconButton({
     : new FixedZIndex(1);
   const dropdownZIndex = new CompositeZIndex([tooltipZIndex]);
   return (
-    <Tooltip accessibilityLabel="" text={tooltip.text} zIndex={tooltipZIndex}>
+    <MaybeTooltip
+      disabled={open}
+      tooltip={{ text: tooltip.text, accessibilityLabel: '', zIndex: tooltipZIndex }}
+    >
       {/* Interactive elements require an a11yLabel on them or their children. That's why we set`accessibilityLabel` on `TapArea` instead of `Tooltip` */}
       <TapArea
         accessibilityControls={id}
@@ -155,7 +158,7 @@ function ItemIconButton({
           </Dropdown>
         )}
       </TapArea>
-    </Tooltip>
+    </MaybeTooltip>
   );
 }