From 02ae1fe8691d0d6fe741e3613a7255962864feab Mon Sep 17 00:00:00 2001
From: Felix Habib <fhabib@seek.com.au>
Date: Fri, 6 Dec 2024 10:27:30 +1100
Subject: [PATCH] Refactor

---
 .../OverflowMenu/OverflowMenu.css.ts          |  1 -
 .../components/OverflowMenu/OverflowMenu.tsx  | 44 ++++++++++---------
 2 files changed, 23 insertions(+), 22 deletions(-)

diff --git a/packages/braid-design-system/src/lib/components/OverflowMenu/OverflowMenu.css.ts b/packages/braid-design-system/src/lib/components/OverflowMenu/OverflowMenu.css.ts
index bc414a043b4..0d7553e44c1 100644
--- a/packages/braid-design-system/src/lib/components/OverflowMenu/OverflowMenu.css.ts
+++ b/packages/braid-design-system/src/lib/components/OverflowMenu/OverflowMenu.css.ts
@@ -2,5 +2,4 @@ import { style } from '@vanilla-extract/css';
 
 export const wrapperPositioning = style({
   margin: '-1px -6px',
-  width: 'fit-content',
 });
diff --git a/packages/braid-design-system/src/lib/components/OverflowMenu/OverflowMenu.tsx b/packages/braid-design-system/src/lib/components/OverflowMenu/OverflowMenu.tsx
index 9dee7d35ab5..2384676f89d 100644
--- a/packages/braid-design-system/src/lib/components/OverflowMenu/OverflowMenu.tsx
+++ b/packages/braid-design-system/src/lib/components/OverflowMenu/OverflowMenu.tsx
@@ -20,26 +20,28 @@ export const OverflowMenu = ({
   id,
   ...menuProps
 }: OverflowMenuProps) => (
-  <Box display="flex" justifyContent="flexEnd">
-    <Box className={styles.wrapperPositioning}>
-      <MenuRenderer
-        trigger={(triggerProps) => (
-          <ButtonIcon
-            // @ts-expect-error With no id, ButtonIcon will fallback from Tooltip to title internally.
-            // ID will no longer be required when React 18 has sufficient adoption and we can safely `useId()`
-            id={id}
-            icon={<IconOverflow />}
-            variant="transparent"
-            label={label}
-            {...triggerProps}
-          />
-        )}
-        align="right"
-        offsetSpace="small"
-        {...menuProps}
-      >
-        {children}
-      </MenuRenderer>
-    </Box>
+  <Box
+    className={styles.wrapperPositioning}
+    display="flex"
+    justifyContent="flexEnd"
+  >
+    <MenuRenderer
+      trigger={(triggerProps) => (
+        <ButtonIcon
+          // @ts-expect-error With no id, ButtonIcon will fallback from Tooltip to title internally.
+          // ID will no longer be required when React 18 has sufficient adoption and we can safely `useId()`
+          id={id}
+          icon={<IconOverflow />}
+          variant="transparent"
+          label={label}
+          {...triggerProps}
+        />
+      )}
+      align="right"
+      offsetSpace="small"
+      {...menuProps}
+    >
+      {children}
+    </MenuRenderer>
   </Box>
 );