Refactor getPosition function call

seek-oss · Dec 2, 2024 · 9623132 · 9623132
1 parent 067ed4d
commit 9623132
Showing 1 changed file with 5 additions and 10 deletions.
diff --git a/packages/braid-design-system/src/lib/components/MenuRenderer/MenuRenderer.tsx b/packages/braid-design-system/src/lib/components/MenuRenderer/MenuRenderer.tsx
@@ -153,8 +153,7 @@ export const MenuRenderer = ({
             open: true,
             closeReason: CLOSE_REASON_EXIT,
             highlightIndex: getNextIndex(-1, state.highlightIndex, itemCount),
-            triggerPosition:
-              menuContainerRef && getPosition(menuContainerRef.current),
+            triggerPosition: getPosition(menuContainerRef.current),
           };
         }
         case MENU_TRIGGER_DOWN:
@@ -164,8 +163,7 @@ export const MenuRenderer = ({
             open: true,
             closeReason: CLOSE_REASON_EXIT,
             highlightIndex: getNextIndex(1, state.highlightIndex, itemCount),
-            triggerPosition:
-              menuContainerRef && getPosition(menuContainerRef.current),
+            triggerPosition: getPosition(menuContainerRef.current),
           };
         }
         case BACKDROP_CLICK:
@@ -210,8 +208,7 @@ export const MenuRenderer = ({
             open: nextOpen,
             closeReason: CLOSE_REASON_EXIT,
             highlightIndex: nextOpen ? 0 : CLOSED_INDEX,
-            triggerPosition:
-              menuContainerRef && getPosition(menuContainerRef.current),
+            triggerPosition: getPosition(menuContainerRef.current),
           };
         }
         case MENU_TRIGGER_CLICK: {
@@ -221,15 +218,13 @@ export const MenuRenderer = ({
             ...state,
             open: nextOpen,
             closeReason: CLOSE_REASON_EXIT,
-            triggerPosition:
-              menuContainerRef && getPosition(menuContainerRef.current),
+            triggerPosition: getPosition(menuContainerRef.current),
           };
         }
         case WINDOW_RESIZE: {
           return {
             ...state,
-            triggerPosition:
-              menuContainerRef && getPosition(menuContainerRef.current),
+            triggerPosition: getPosition(menuContainerRef.current),
           };
         }
         default: