From cfba99715417415340186ecca5dd95b5ad9cbb03 Mon Sep 17 00:00:00 2001
From: Felix Habib <33821218+felixhabib@users.noreply.github.com>
Date: Mon, 23 Dec 2024 13:53:32 +1100
Subject: [PATCH] Apply suggestions from code review

Refactor snippets

Co-authored-by: Michael Taranto <michaeltaranto@users.noreply.github.com>
---
 .../MenuRenderer/MenuRenderer.snippets.tsx    | 31 +++++++++----------
 1 file changed, 14 insertions(+), 17 deletions(-)

diff --git a/packages/braid-design-system/src/lib/components/MenuRenderer/MenuRenderer.snippets.tsx b/packages/braid-design-system/src/lib/components/MenuRenderer/MenuRenderer.snippets.tsx
index c86b61f5c7..ed13135f6d 100644
--- a/packages/braid-design-system/src/lib/components/MenuRenderer/MenuRenderer.snippets.tsx
+++ b/packages/braid-design-system/src/lib/components/MenuRenderer/MenuRenderer.snippets.tsx
@@ -64,13 +64,12 @@ export const snippets: Snippets = [
       <MenuRenderer
         offsetSpace="small"
         trigger={(triggerProps) => (
-          <Box userSelect="none" cursor="pointer" {...triggerProps}>
-            <ButtonIcon
-              icon={<IconList />}
-              id="buttonicon-menurenderer"
-              label="Menu"
-            />
-          </Box>
+          <ButtonIcon
+            icon={<IconList />}
+            id="buttonicon-menurenderer"
+            label="Menu"
+            {...triggerProps}
+          />
         )}
       >
         <MenuItem onClick={() => {}}>Button</MenuItem>
@@ -85,14 +84,13 @@ export const snippets: Snippets = [
         size="small"
         offsetSpace="xxsmall"
         trigger={(triggerProps) => (
-          <Box userSelect="none" cursor="pointer" {...triggerProps}>
-            <ButtonIcon
-              size="small"
-              icon={<IconList />}
-              id="buttonicon-menurenderer"
-              label="Menu"
-            />
-          </Box>
+          <ButtonIcon
+            size="small"
+            icon={<IconList />}
+            id="buttonicon-menurenderer"
+            label="Menu"
+            {...triggerProps}
+          />
         )}
       >
         <MenuItem onClick={() => {}}>Button</MenuItem>
@@ -104,8 +102,7 @@ export const snippets: Snippets = [
     name: 'Placeholder trigger',
     code: source(
       <MenuRenderer
-        size="small"
-        offsetSpace="xsmall"
+        offsetSpace="small"
         trigger={(triggerProps) => (
           <Box userSelect="none" cursor="pointer" {...triggerProps}>
             <Placeholder height={50} label="Menu Trigger" />