Apply suggestions from code review

Refactor snippets Co-authored-by: Michael Taranto <[email protected]>
seek-oss · Dec 23, 2024 · cfba997 · cfba997
1 parent 7001a76
commit cfba997
Showing 1 changed file with 14 additions and 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
@@ -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" />