@aurora-ds/components 1.8.1 → 1.8.2

package/dist/esm/index.js

@@ -3466,6 +3466,13 @@ width = '100%', minWidth = 0, maxWidth, flex, flexGrow, flexShrink, flexBasis, .
 };
 TextField.displayName = 'TextField';
 
+/**
+ * Vertical padding (in px) applied to the top and bottom of the menu panel.
+ * Matches `theme.spacing.xs` (0.25rem = 4px at 16px base font).
+ * Exported so submenus can offset their position upward by this amount to align
+ * their first item with the trigger item.
+ */
+const MENU_PANEL_PADDING_Y_PX = 6;
 const MENU_PANEL_STYLES = createStyles((theme) => ({
     backdrop: {
         position: 'fixed',
@@ -3539,7 +3546,7 @@ const MENU_MIN_WIDTH_PX = 224;
  * Uses a two-pass strategy: first render with `menuHeight = 0`, then a rAF
  * recompute with the actual rendered size to apply any overflow offset.
  */
-const useMenuPosition = ({ anchorEl, open, menuRef, minWidth, gap = 4, placement = 'bottom', }) => {
+const useMenuPosition = ({ anchorEl, open, menuRef, minWidth, gap = 4, verticalOffset = 0, placement = 'bottom', }) => {
     const [style, setStyle] = useState({});
     // Stays false until the rAF second pass has run with the real panel dimensions.
     // Keeps the panel invisible during the initial style={} state and the first
@@ -3556,8 +3563,10 @@ const useMenuPosition = ({ anchorEl, open, menuRef, minWidth, gap = 4, placement
         const viewportRight = window.innerWidth - VIEWPORT_MARGIN_PX;
         const viewportBottom = window.innerHeight - VIEWPORT_MARGIN_PX;
         if (placement === 'right' || placement === 'left') {
-            // --- Vertical: align with the anchor top, clamp inside the viewport ---
-            let top = anchor.top;
+            // --- Vertical: align with the anchor top, offset upward by verticalOffset
+            // (e.g. the panel's paddingTop) so the first item lines up with the trigger,
+            // then clamp inside the viewport. ---
+            let top = anchor.top - verticalOffset;
             if (menuHeight > 0 && top + menuHeight > viewportBottom) {
                 top = Math.max(VIEWPORT_MARGIN_PX, viewportBottom - menuHeight);
             }
@@ -3617,7 +3626,7 @@ const useMenuPosition = ({ anchorEl, open, menuRef, minWidth, gap = 4, placement
         const maxLeft = window.innerWidth - effectiveWidth - VIEWPORT_MARGIN_PX;
         const left = Math.max(VIEWPORT_MARGIN_PX, Math.min(anchor.left, maxLeft));
         setStyle({ top, left, minWidth: minWidth ?? fallbackMinWidth });
-    }, [anchorEl, menuRef, minWidth, gap, placement]);
+    }, [anchorEl, menuRef, minWidth, gap, verticalOffset, placement]);
     // First pass: compute as soon as the menu opens (menuHeight = 0)
     useLayoutEffect(() => {
         if (open) {
@@ -3690,12 +3699,12 @@ const ITEM_SELECTOR = '[role^="menuitem"]:not([data-disabled]), [role="option"]:
  * `aria-activedescendant`. Item order is read from the DOM (read-only) so the
  * logic stays correct across groups and dynamic content.
  */
-const useMenuPanel = ({ open, onClose, anchorEl, minWidth, placement = 'bottom', onArrowLeft, }) => {
+const useMenuPanel = ({ open, onClose, anchorEl, minWidth, placement = 'bottom', verticalOffset = 0, onArrowLeft, }) => {
     const panelRef = useRef(null);
     const [focusedId, setFocusedId] = useState(undefined);
     // True while a descendant submenu is open → pause this panel's keyboard nav.
     const [childOpen, setChildOpen] = useState(false);
-    const { style } = useMenuPosition({ anchorEl, open, menuRef: panelRef, minWidth, placement });
+    const { style } = useMenuPosition({ anchorEl, open, menuRef: panelRef, minWidth, placement, verticalOffset });
     /** Read this panel's focusable items in DOM order (excluding nested submenus). */
     const getItems = useCallback(() => {
         const panel = panelRef.current;
@@ -3909,6 +3918,9 @@ const MenuPanel = ({ open, onClose, anchorEl, minWidth, maxHeight = '20rem', pla
         anchorEl,
         minWidth,
         placement,
+        // Shift submenus upward by the panel's paddingTop so the first item aligns
+        // visually with the trigger item in the parent menu.
+        verticalOffset: isSubmenu ? MENU_PANEL_PADDING_Y_PX : 0,
         onArrowLeft,
     });
     const contextValue = useMemo(() => ({ setChildOpen, closeMenu }), [setChildOpen, closeMenu]);