use unsafeDisableTooltip prop for iconbuttons and remove explicit too…

…ltips
primer · Jul 8, 2024 · ea8b026 · ea8b026
1 parent c9415df
commit ea8b026
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 12 deletions.
diff --git a/packages/react/src/ButtonGroup/ButtonGroup.dev.stories.tsx b/packages/react/src/ButtonGroup/ButtonGroup.dev.stories.tsx
@@ -84,17 +84,13 @@ export const LinkAndButtonWithTooltip2 = () => (
         Link
       </Button>
     </TooltipV2>
-    <TooltipV2 text="Open GitHub Copilot chat" direction="s">
-      <IconButton icon={CopilotIcon} aria-label="Open GitHub Copilot chat" />
-    </TooltipV2>
+    <IconButton icon={CopilotIcon} aria-label="Open GitHub Copilot chat" unsafeDisableTooltip={false} />
   </ButtonGroup>
 )
 
 export const ButtonAndLinkWithTooltip2 = () => (
   <ButtonGroup sx={{pl: 2}}>
-    <TooltipV2 text="Open GitHub Copilot chat" direction="s">
-      <IconButton icon={CopilotIcon} aria-label="Open GitHub Copilot chat" />
-    </TooltipV2>
+    <IconButton icon={CopilotIcon} aria-label="Open GitHub Copilot chat" unsafeDisableTooltip={false} />
     <TooltipV2 text="Additional info about the link">
       <Button as="a" href="https://primer.style">
         Link

diff --git a/packages/react/src/ButtonGroup/ButtonGroup.features.stories.tsx b/packages/react/src/ButtonGroup/ButtonGroup.features.stories.tsx
@@ -21,12 +21,8 @@ export const IconButtons = () => (
 
 export const IconButtonsWithTooltip = () => (
   <ButtonGroup>
-    <Tooltip text="Add" type="label">
-      <IconButton icon={PlusIcon} aria-label="Add" />
-    </Tooltip>
-    <Tooltip text="Subtract" type="label">
-      <IconButton icon={DashIcon} aria-label="Subtract" />
-    </Tooltip>
+    <IconButton icon={PlusIcon} aria-label="Add" unsafeDisableTooltip={false} />
+    <IconButton icon={DashIcon} aria-label="Subtract" unsafeDisableTooltip={false} />
   </ButtonGroup>
 )
 export const ButtonAndLink = () => (