fix(NavMenuButton): Add missing changes from previous PR

kronostechnologies · Jan 5, 2021 · e49c757 · e49c757
1 parent bb5d153
commit e49c757
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/packages/react/src/components/nav-menu-button/nav-menu-button.tsx b/packages/react/src/components/nav-menu-button/nav-menu-button.tsx
@@ -87,7 +87,9 @@ export function NavMenuButton({
     }, [isOpen]);
 
     useEffect(() => {
-        setFocusedValue(isOpen ? options[0].value : '');
+        if (options.length > 0) {
+            setFocusedValue(isOpen ? options[0].value : '');
+        }
         document.addEventListener('mouseup', handleClickOutside);
 
         return () => document.removeEventListener('mouseup', handleClickOutside);

diff --git a/packages/react/src/components/nav-menu/nav-menu.tsx b/packages/react/src/components/nav-menu/nav-menu.tsx
@@ -127,7 +127,7 @@ export const NavMenu = forwardRef(({
                         innerRef={option.ref}
                         $device={device}
                         to={option.href}
-                        onClick={() => onChange && onChange(option)}
+                        onClick={() => onChange?.(option)}
                         onKeyDown={(event) => handleKeyDown(event, option)}
                     >
                         {option.label || option.value}