rsuite 5.2.4 → 5.4.2

package/esm/Input/Input.d.ts

@@ -1,9 +1,10 @@
 import React from 'react';
 import { WithAsProps, RsRefForwardingComponent, TypeAttributes, FormControlBaseProps } from '../@types/common';
+import { PrependParameters } from '../@types/utils';
 export interface LocaleType {
     unfilled: string;
 }
-export interface InputProps extends WithAsProps, Omit<React.InputHTMLAttributes<HTMLInputElement>, 'onChange' | 'size'>, FormControlBaseProps {
+export interface InputProps extends WithAsProps, Omit<React.InputHTMLAttributes<HTMLInputElement>, 'onChange' | 'size'>, Omit<FormControlBaseProps, 'onChange'> {
     /** The HTML input type */
     type?: string;
     /** The HTML input id */
@@ -12,7 +13,7 @@ export interface InputProps extends WithAsProps, Omit<React.InputHTMLAttributes<
     size?: TypeAttributes.Size;
     /** Ref of input element */
     inputRef?: React.Ref<any>;
-    onChange?: (value: string, event: React.ChangeEvent<HTMLInputElement>) => void;
+    onChange?: PrependParameters<React.ChangeEventHandler<HTMLInputElement>, [value: string]>;
     /** Called on press enter */
     onPressEnter?: React.KeyboardEventHandler<HTMLInputElement>;
 }

package/esm/InputGroup/InputGroup.d.ts

@@ -5,7 +5,7 @@ import { WithAsProps, TypeAttributes, RsRefForwardingComponent } from '../@types
 export declare const InputGroupContext: React.Context<{
     onFocus: () => void;
     onBlur: () => void;
-}>;
+} | null>;
 export interface InputGroupProps extends WithAsProps {
     /** Sets the composition content internally */
     inside?: boolean;

package/esm/InputNumber/InputNumber.js

@@ -107,11 +107,15 @@ var InputNumber = /*#__PURE__*/React.forwardRef(function (props, ref) {
       value = _useControlled[0],
       setValue = _useControlled[1];
 
-  var _useState = useState(disableMaxValue(value, max)),
+  var _useState = useState(function () {
+    return disableMaxValue(value, max);
+  }),
       disabledUpButton = _useState[0],
       setDisabledUpButton = _useState[1];
 
-  var _useState2 = useState(disableMinValue(value, min)),
+  var _useState2 = useState(function () {
+    return disableMinValue(value, min);
+  }),
       disabledDownButton = _useState2[0],
       setDisabledDownButton = _useState2[1];
 

package/esm/InputPicker/InputAutosize.js

@@ -78,10 +78,10 @@ var InputAutosize = /*#__PURE__*/React.forwardRef(function (props, ref) {
       _props$inputId = props.inputId,
       inputId = _props$inputId === void 0 ? uniqueId : _props$inputId,
       tabIndex = props.tabIndex;
-  var rootRef = useRef();
-  var inputRef = useRef();
-  var sizerRef = useRef();
-  var placeholderRef = useRef();
+  var rootRef = useRef(null);
+  var inputRef = useRef(null);
+  var sizerRef = useRef(null);
+  var placeholderRef = useRef(null);
   useImperativeHandle(ref, function () {
     return {
       root: rootRef.current,

package/esm/InputPicker/InputPicker.d.ts

@@ -9,11 +9,11 @@ export interface InputPickerContextProps {
     /** Multiple selections are allowed */
     multi?: boolean;
     /** Tag related props. */
-    tagProps?: TagProps;
+    tagProps: TagProps;
     /**
      * Set the trigger for creating tags. only valid when creatable
      */
-    trigger?: TriggerType | TriggerType[];
+    trigger: TriggerType | TriggerType[];
     /**
      * No overlay provides options
      */

package/esm/InputPicker/InputPicker.js

@@ -23,7 +23,10 @@ import { DropdownMenu, DropdownMenuItem, DropdownMenuCheckItem, PickerToggle, Pi
 import Tag from '../Tag';
 import InputAutosize from './InputAutosize';
 import InputSearch from './InputSearch';
-export var InputPickerContext = /*#__PURE__*/React.createContext({});
+export var InputPickerContext = /*#__PURE__*/React.createContext({
+  tagProps: {},
+  trigger: 'Enter'
+});
 var InputPicker = /*#__PURE__*/React.forwardRef(function (props, ref) {
   var _merge;
 
@@ -43,7 +46,8 @@ var InputPicker = /*#__PURE__*/React.forwardRef(function (props, ref) {
       readOnly = props.readOnly,
       plaintext = props.plaintext,
       defaultValue = props.defaultValue,
-      defaultOpen = props.defaultOpen,
+      _props$defaultOpen = props.defaultOpen,
+      defaultOpen = _props$defaultOpen === void 0 ? false : _props$defaultOpen,
       _props$disabledItemVa = props.disabledItemValues,
       disabledItemValues = _props$disabledItemVa === void 0 ? [] : _props$disabledItemVa,
       overrideLocale = props.locale,
@@ -104,9 +108,9 @@ var InputPicker = /*#__PURE__*/React.forwardRef(function (props, ref) {
     throw Error('`groupBy` can not be equal to `valueKey` and `labelKey`');
   }
 
-  var overlayRef = useRef();
-  var targetRef = useRef();
-  var triggerRef = useRef();
+  var overlayRef = useRef(null);
+  var targetRef = useRef(null);
+  var triggerRef = useRef(null);
   var inputRef = useRef();
 
   var _useCustom = useCustom(['Picker', 'InputPicker'], overrideLocale),
@@ -433,8 +437,8 @@ var InputPicker = /*#__PURE__*/React.forwardRef(function (props, ref) {
   var events = {
     onMenuPressBackspace: multi ? removeLastItem : handleClean,
     onMenuKeyDown: onKeyDown,
-    onMenuPressEnter: null,
-    onKeyDown: null
+    onMenuPressEnter: undefined,
+    onKeyDown: undefined
   };
   var handleKeyPress = useCallback(function (event) {
     // When typing a space, create a tag.
@@ -494,7 +498,7 @@ var InputPicker = /*#__PURE__*/React.forwardRef(function (props, ref) {
 
   var renderDropdownMenuItem = function renderDropdownMenuItem(label, item) {
     // 'Create option "{0}"' =>  Create option "xxxxx"
-    var newLabel = item.create ? /*#__PURE__*/React.createElement("span", null, tplTransform(locale === null || locale === void 0 ? void 0 : locale.createOption, label)) : label;
+    var newLabel = item.create ? /*#__PURE__*/React.createElement("span", null, tplTransform(locale.createOption, label)) : label;
     return renderMenuItem ? renderMenuItem(newLabel, item) : newLabel;
   };
 
@@ -608,7 +612,8 @@ var InputPicker = /*#__PURE__*/React.forwardRef(function (props, ref) {
       maxHeight: menuMaxHeight,
       data: items,
       group: !isUndefined(groupBy),
-      onSelect: multi ? handleCheckTag : handleSelectItem,
+      onSelect: multi ? handleCheckTag : handleSelectItem // fixme don't use any
+      ,
       renderMenuGroup: renderMenuGroup,
       renderMenuItem: renderDropdownMenuItem,
       virtualized: virtualized
@@ -701,7 +706,7 @@ var InputPicker = /*#__PURE__*/React.forwardRef(function (props, ref) {
     plaintext: plaintext,
     ref: targetRef,
     as: toggleAs,
-    tabIndex: null,
+    tabIndex: undefined,
     onClean: handleClean,
     cleanable: cleanable && !disabled,
     hasValue: hasValue,

package/esm/List/ListContext.d.ts

@@ -4,7 +4,7 @@ import useManager from './helper/useManager';
 export declare type ListContextType = {
     bordered?: boolean;
     size?: ListProps['size'];
-    register?: ReturnType<typeof useManager>['listItemRegister'];
+    register: ReturnType<typeof useManager>['listItemRegister'];
 };
 declare const ListContext: React.Context<ListContextType>;
 export default ListContext;

package/esm/List/ListContext.js

@@ -1,6 +1,12 @@
 import React from 'react';
+import noop from 'lodash/noop';
 var ListContext = /*#__PURE__*/React.createContext({
   bordered: false,
-  size: 'md'
+  size: 'md',
+  register: function register() {
+    return {
+      unregister: noop
+    };
+  }
 });
 export default ListContext;

package/esm/List/helper/AutoScroller.js

@@ -17,8 +17,10 @@ var AutoScroller = /*#__PURE__*/function () {
   var _proto = AutoScroller.prototype;
 
   _proto.clear = function clear() {
-    clearInterval(this.interval);
-    this.interval = null;
+    if (this.interval !== null) {
+      clearInterval(this.interval);
+      this.interval = null;
+    }
   };
 
   _proto.update = function update(_ref) {

package/esm/List/helper/useManager.d.ts

@@ -2,10 +2,10 @@ import { Offset } from '../../@types/common';
 export declare type Collection = string | number;
 export interface ManagedItem {
     node: HTMLElement;
-    edgeOffset: Offset;
+    edgeOffset: Offset | null;
     info: {
         collection: Collection;
-        index: number;
+        index?: number;
         disabled?: boolean;
     };
 }
@@ -13,7 +13,7 @@ declare const useManager: () => {
     listItemRegister: (item: ManagedItem) => {
         unregister: () => void;
     };
-    getManagedItem: (node: HTMLElement) => ManagedItem;
+    getManagedItem: (node: HTMLElement) => ManagedItem | undefined;
     getOrderedItems: (collection: any) => ManagedItem[];
 };
 export default useManager;

package/esm/List/helper/useManager.js

@@ -31,7 +31,7 @@ var useManager = function useManager() {
   }, []);
   var getOrderedItems = useCallback(function (collection) {
     return collection != null ? [].concat(collectionMapRef.current[collection]).sort(function (nodeInfo1, nodeInfo2) {
-      return nodeInfo1.info.index - nodeInfo2.info.index;
+      return Number(nodeInfo1.info.index) - Number(nodeInfo2.info.index);
     }) : [];
   }, []);
   return {

package/esm/List/helper/useSortHelper.d.ts

@@ -1,5 +1,5 @@
 import { MouseEventHandler } from 'react';
-import { Collection } from './useManager';
+import { Collection, ManagedItem } from './useManager';
 interface MovedItemInfo {
     collection: Collection;
     node: HTMLElement;
@@ -18,9 +18,9 @@ export declare type SortConfig = {
 declare const useSortHelper: (config: SortConfig) => {
     handleStart: MouseEventHandler<Element>;
     handleEnd: MouseEventHandler<Element>;
-    containerRef: import("react").MutableRefObject<HTMLDivElement>;
+    containerRef: import("react").RefObject<HTMLDivElement>;
     sorting: boolean;
-    register: (item: import("./useManager").ManagedItem) => {
+    register: (item: ManagedItem) => {
         unregister: () => void;
     };
 };

package/esm/List/helper/useSortHelper.js

@@ -38,7 +38,7 @@ var useSortHelper = function useSortHelper(config) {
    * */
 
   var handlePress = useCallback(function (mouseDownEvent, targetNode, curManagedItem) {
-    var _activeNodeHelper;
+    var _curManagedItem$info$, _curManagedItem$info$2, _activeNodeHelper;
 
     if (!isMounted()) return;
     var listItemBaseClassName = targetNode.classList[0]; // get list item base className
@@ -48,8 +48,8 @@ var useSortHelper = function useSortHelper(config) {
 
     var containerElement = containerRef.current;
     var activeNode = curManagedItem.node;
-    var activeNodeOldIndex = curManagedItem.info.index;
-    var activeNodeNextIndex = curManagedItem.info.index;
+    var activeNodeOldIndex = (_curManagedItem$info$ = curManagedItem.info.index) !== null && _curManagedItem$info$ !== void 0 ? _curManagedItem$info$ : 0;
+    var activeNodeNextIndex = (_curManagedItem$info$2 = curManagedItem.info.index) !== null && _curManagedItem$info$2 !== void 0 ? _curManagedItem$info$2 : 0;
     var activeNodeHolderTranslate = {
       x: 0,
       y: 0
@@ -120,8 +120,10 @@ var useSortHelper = function useSortHelper(config) {
       var activeNodeHeight = parseFloat(activeNodeStyle.height) || 0;
 
       for (var i = 0, len = listItemManagerRefs.length; i < len; i++) {
+        var _listItemManagerRefs$;
+
         var currentNode = listItemManagerRefs[i].node;
-        var currentNodeIndex = listItemManagerRefs[i].info.index;
+        var currentNodeIndex = (_listItemManagerRefs$ = listItemManagerRefs[i].info.index) !== null && _listItemManagerRefs$ !== void 0 ? _listItemManagerRefs$ : 0;
         var offsetY = activeNodeBoundingClientRect.height > currentNode.offsetHeight ? currentNode.offsetHeight / 2 : activeNodeBoundingClientRect.height / 2;
         var translate = {
           x: 0,

package/esm/List/helper/utils.d.ts

@@ -11,6 +11,6 @@ export declare function setTransitionDuration(node: HTMLElement, duration?: numb
 /**
  * find closest target node from source node
  * */
-export declare function closestNode(sourceNode: HTMLElement, judge: (target: HTMLElement) => boolean): HTMLElement;
+export declare function closestNode(sourceNode: HTMLElement, judge: (target: HTMLElement) => boolean): HTMLElement | null;
 export declare function getEdgeOffset(node: HTMLElement, parent: HTMLElement, offset?: Offset): Offset;
-export declare function getScrollingParent(el: HTMLElement): HTMLElement;
+export declare function getScrollingParent(el: HTMLElement): HTMLElement | null;

package/esm/MaskedInput/TextMask.d.ts

@@ -34,10 +34,10 @@ export declare type TextMaskProps = TextMaskBaseProps & React.HTMLAttributes<HTM
 };
 declare const TextMask: React.ForwardRefExoticComponent<TextMaskBaseProps & React.HTMLAttributes<HTMLInputElement> & {
     /** Custom rendering DOM */
-    render?: (ref: React.Ref<HTMLInputElement>, props: React.HTMLAttributes<HTMLInputElement>) => any;
-    onChange?: (event: React.ChangeEvent<HTMLInputElement>) => void;
-    value?: string | number;
-    readOnly?: boolean;
-    disabled?: boolean;
+    render?: ((ref: React.Ref<HTMLInputElement>, props: React.HTMLAttributes<HTMLInputElement>) => any) | undefined;
+    onChange?: ((event: React.ChangeEvent<HTMLInputElement>) => void) | undefined;
+    value?: string | number | undefined;
+    readOnly?: boolean | undefined;
+    disabled?: boolean | undefined;
 } & React.RefAttributes<HTMLInputElement>>;
 export default TextMask;

package/esm/MaskedInput/adjustCaretPosition.d.ts

@@ -1,11 +1,11 @@
 export default function adjustCaretPosition({ previousConformedValue, previousPlaceholder, currentCaretPosition, conformedValue, rawValue, placeholderChar, placeholder, indexesOfPipedChars, caretTrapIndexes }: {
-    previousConformedValue?: string;
-    previousPlaceholder?: string;
-    currentCaretPosition?: number;
+    previousConformedValue?: string | undefined;
+    previousPlaceholder?: string | undefined;
+    currentCaretPosition?: number | undefined;
     conformedValue: any;
     rawValue: any;
     placeholderChar: any;
     placeholder: any;
-    indexesOfPipedChars?: any[];
-    caretTrapIndexes?: any[];
-}): number;
+    indexesOfPipedChars?: string[] | undefined;
+    caretTrapIndexes?: number[] | undefined;
+}): number | undefined;

package/esm/MaskedInput/conformToMask.js

@@ -41,7 +41,8 @@ export default function conformToMask(rawValue, mask, config) {
       placeholderChar = _config$placeholderCh === void 0 ? defaultPlaceholderChar : _config$placeholderCh,
       _config$placeholder = _config.placeholder,
       placeholder = _config$placeholder === void 0 ? convertMaskToPlaceholder(mask, placeholderChar) : _config$placeholder,
-      currentCaretPosition = _config.currentCaretPosition,
+      _config$currentCaretP = _config.currentCaretPosition,
+      currentCaretPosition = _config$currentCaretP === void 0 ? 0 : _config$currentCaretP,
       keepCharPositions = _config.keepCharPositions; // The configs below indicate that the user wants the algorithm to work in *no guide* mode
 
   var suppressGuide = guide === false && previousConformedValue !== undefined; // Calculate lengths once for performance
@@ -122,9 +123,9 @@ export default function conformToMask(rawValue, mask, config) {
         // or we find at least one character that we can map.
         while (rawValueArr.length > 0) {
           // Let's retrieve the first user character in the queue of characters we have left
-          var _rawValueArr$shift = rawValueArr.shift(),
-              rawValueChar = _rawValueArr$shift.char,
-              isNew = _rawValueArr$shift.isNew; // If the character we got from the user input is a placeholder character (which happens
+          var _ref = rawValueArr.shift(),
+              rawValueChar = _ref.char,
+              isNew = _ref.isNew; // If the character we got from the user input is a placeholder character (which happens
           // regularly because user input could be something like (540) 90_-____, which includes
           // a bunch of `_` which are placeholder characters) and we are not in *no guide* mode,
           // then we map this placeholder character to the current spot in the placeholder

package/esm/MaskedInput/createTextMaskInputElement.d.ts

@@ -1,7 +1,7 @@
 export default function createTextMaskInputElement(config: any): {
     state: {
-        previousConformedValue: any;
-        previousPlaceholder: any;
+        previousConformedValue: undefined;
+        previousPlaceholder: undefined;
     };
     update(rawValue: any, { inputElement, mask: providedMask, guide, pipe, placeholderChar, keepCharPositions, showMask }?: any): void;
 };

package/esm/MaskedInput/utilities.d.ts

@@ -3,5 +3,5 @@ export declare const defaultPlaceholderChar = "_";
 export declare function convertMaskToPlaceholder(mask?: MaskType | MaskFunctionType, placeholderChar?: string): string;
 export declare function processCaretTraps(mask: any): {
     maskWithoutCaretTraps: any;
-    indexes: any[];
+    indexes: number[];
 };

package/esm/Menu/Menu.js

@@ -1,6 +1,7 @@
 import _extends from "@babel/runtime/helpers/esm/extends";
 import React, { useCallback, useContext, useRef, useMemo } from 'react';
 import PropTypes from 'prop-types';
+import isNil from 'lodash/isNil';
 import MenuContext, { MenuActionTypes, MoveFocusTo } from './MenuContext';
 import { KEY_VALUES, useCustom } from '../utils';
 import useUniqueId from '../utils/useUniqueId';
@@ -25,8 +26,8 @@ function Menu(props) {
       renderMenuButton = props.renderMenuButton,
       renderMenuPopup = props.renderMenuPopup,
       onToggleMenu = props.onToggleMenu;
-  var buttonElementRef = useRef();
-  var menuElementRef = useRef();
+  var buttonElementRef = useRef(null);
+  var menuElementRef = useRef(null);
   var parentMenu = useContext(MenuContext);
   var isSubmenu = !!parentMenu;
   var menu = useMenu();
@@ -39,7 +40,7 @@ function Menu(props) {
   var _useCustom = useCustom('Menu'),
       rtl = _useCustom.rtl;
 
-  var activeItem = (_items$activeItemInde = items[activeItemIndex]) === null || _items$activeItemInde === void 0 ? void 0 : _items$activeItemInde.element;
+  var activeItem = isNil(activeItemIndex) ? null : (_items$activeItemInde = items[activeItemIndex]) === null || _items$activeItemInde === void 0 ? void 0 : _items$activeItemInde.element;
 
   var _useFocus = useFocus(menuElementRef),
       grabFocus = _useFocus.grab;
@@ -74,7 +75,9 @@ function Menu(props) {
     onToggleMenu === null || onToggleMenu === void 0 ? void 0 : onToggleMenu(false, event);
 
     if (returnFocusToButton) {
-      buttonElementRef.current.focus({
+      var _buttonElementRef$cur;
+
+      (_buttonElementRef$cur = buttonElementRef.current) === null || _buttonElementRef$cur === void 0 ? void 0 : _buttonElementRef$cur.focus({
         preventScroll: true
       });
     }
@@ -89,7 +92,9 @@ function Menu(props) {
   useClickOutside({
     enabled: open,
     isOutside: function isOutside(event) {
-      return !buttonElementRef.current.contains(event.target) && !menuElementRef.current.contains(event.target);
+      var _buttonElementRef$cur2, _menuElementRef$curre;
+
+      return !((_buttonElementRef$cur2 = buttonElementRef.current) !== null && _buttonElementRef$cur2 !== void 0 && _buttonElementRef$cur2.contains(event.target)) && !((_menuElementRef$curre = menuElementRef.current) !== null && _menuElementRef$curre !== void 0 && _menuElementRef$curre.contains(event.target));
     },
     // fixme if clicking on a focusable element, don't move focus to menu button
     handle: function handle(event) {
@@ -330,7 +335,8 @@ function Menu(props) {
 
   var customMenuPopup = renderMenuPopup === null || renderMenuPopup === void 0 ? void 0 : renderMenuPopup(_extends({}, menuProps, {
     open: open
-  }), menuElementRef);
+  }), menuElementRef); // fixme Wrong children here
+
   var menuElement = customMenuPopup !== null && customMenuPopup !== void 0 ? customMenuPopup : /*#__PURE__*/React.createElement("ul", _extends({
     ref: menuElementRef
   }, menuProps, {
@@ -346,7 +352,7 @@ function Menu(props) {
       closeMenu(e);
     }
   }, [disabled, closeMenu]);
-  var rootElementRef = useRef();
+  var rootElementRef = useRef(null);
   var handleContainerBlur = useCallback(function (event) {
     if (isFocusLeaving(event)) {
       closeMenu(event, false);

package/esm/Menu/MenuContext.d.ts

@@ -47,5 +47,5 @@ export declare type MenuAction = {
     to: MoveFocusTo.Specific;
     id: string;
 };
-declare const MenuContext: React.Context<MenuContextProps>;
+declare const MenuContext: React.Context<MenuContextProps | null>;
 export default MenuContext;

package/esm/Menu/MenuItem.d.ts

@@ -8,7 +8,7 @@ export interface MenuItemProps {
     /** Render prop */
     children: (menuitem: React.LiHTMLAttributes<HTMLLIElement> & MenuitemRenderProps, ref: React.Ref<HTMLLIElement>) => React.ReactElement;
     /** Callback when menuitem is being activated */
-    onActivate?: (event: React.SyntheticEvent) => void;
+    onActivate?: React.MouseEventHandler;
 }
 export interface MenuitemRenderProps {
     selected: boolean;

package/esm/Menu/MenuItem.js

@@ -1,5 +1,6 @@
 import { useCallback, useContext, useEffect, useRef } from 'react';
 import PropTypes from 'prop-types';
+import isNil from 'lodash/isNil';
 import useUniqueId from '../utils/useUniqueId';
 import MenuContext, { MenuActionTypes, MoveFocusTo } from './MenuContext';
 
@@ -7,22 +8,24 @@ import MenuContext, { MenuActionTypes, MoveFocusTo } from './MenuContext';
  * Headless ARIA `menuitem`
  */
 function MenuItem(props) {
-  var _menuState$items$menu;
-
   var children = props.children,
-      selected = props.selected,
-      disabled = props.disabled,
+      _props$selected = props.selected,
+      selected = _props$selected === void 0 ? false : _props$selected,
+      _props$disabled = props.disabled,
+      disabled = _props$disabled === void 0 ? false : _props$disabled,
       onActivate = props.onActivate;
-  var menuitemRef = useRef();
+  var menuitemRef = useRef(null);
   var menuitemId = useUniqueId('menuitem-');
-  var menu = useContext(MenuContext); // fixme make sure <MenuItem> is used inside a <Menu>
+  var menu = useContext(MenuContext);
 
-  var _ref = menu !== null && menu !== void 0 ? menu : [],
-      menuState = _ref[0],
-      dispatch = _ref[1]; // Whether this menuitem has focus (indicated by `aria-activedescendant` from parent menu)
+  if (!menu) {
+    throw new Error('<MenuItem> must be rendered within a <Menu>');
+  }
 
+  var menuState = menu[0],
+      dispatch = menu[1]; // Whether this menuitem has focus (indicated by `aria-activedescendant` from parent menu)
 
-  var hasFocus = (menuState === null || menuState === void 0 ? void 0 : (_menuState$items$menu = menuState.items[menuState === null || menuState === void 0 ? void 0 : menuState.activeItemIndex]) === null || _menuState$items$menu === void 0 ? void 0 : _menuState$items$menu.element) === menuitemRef.current && !!menuitemRef.current;
+  var hasFocus = !isNil(menuitemRef.current) && !isNil(menuState.activeItemIndex) && menuState.items[menuState.activeItemIndex].element === menuitemRef.current;
   var handleClick = useCallback(function (event) {
     if (disabled) {
       return;
@@ -32,17 +35,17 @@ function MenuItem(props) {
   }, [disabled, onActivate]); // Gain/release focus on mousedown in `menubar`
 
   var handleMouseDown = useCallback(function () {
-    if (!hasFocus) {
+    if (!isNil(menuitemRef.current) && !hasFocus) {
       dispatch({
         type: MenuActionTypes.MoveFocus,
         to: MoveFocusTo.Specific,
         id: menuitemRef.current.id
       });
     }
-  }, [hasFocus, dispatch]); // Gain/release focus on mouseenter/mouseleave in `menu`
+  }, [dispatch, hasFocus]); // Gain/release focus on mouseenter/mouseleave in `menu`
 
   var handleMouseMove = useCallback(function () {
-    if (!hasFocus) {
+    if (!isNil(menuitemRef.current) && !hasFocus) {
       dispatch({
         type: MenuActionTypes.MoveFocus,
         to: MoveFocusTo.Specific,
@@ -58,7 +61,7 @@ function MenuItem(props) {
   }, [dispatch]);
   useEffect(function () {
     var menuitemElement = menuitemRef.current;
-    dispatch === null || dispatch === void 0 ? void 0 : dispatch({
+    dispatch({
       type: MenuActionTypes.RegisterItem,
       element: menuitemElement,
       props: {
@@ -66,7 +69,7 @@ function MenuItem(props) {
       }
     });
     return function () {
-      dispatch === null || dispatch === void 0 ? void 0 : dispatch({
+      dispatch({
         type: MenuActionTypes.UnregisterItem,
         id: menuitemElement.id
       });

package/esm/Menu/Menubar.js

@@ -1,12 +1,11 @@
 // Headless ARIA `menubar`
 import React, { useCallback, useRef } from 'react';
+import isNil from 'lodash/isNil';
 import MenuContext, { MenuActionTypes, MoveFocusTo } from './MenuContext';
 import { KEY_VALUES, useCustom } from '../utils';
 import { isFocusEntering, isFocusLeaving } from '../utils/events';
 import useMenu from './useMenu';
 export default function Menubar(_ref) {
-  var _items$activeItemInde;
-
   var _ref$vertical = _ref.vertical,
       vertical = _ref$vertical === void 0 ? false : _ref$vertical,
       children = _ref.children,
@@ -18,7 +17,7 @@ export default function Menubar(_ref) {
       items = _menubar$.items,
       activeItemIndex = _menubar$.activeItemIndex,
       dispatch = menubar[1];
-  var menubarElementRef = useRef();
+  var menubarElementRef = useRef(null);
   var onFocus = useCallback(function (event) {
     // Focus moves inside Menubar
     if (isFocusEntering(event)) {
@@ -44,7 +43,7 @@ export default function Menubar(_ref) {
       rtl = _useCustom.rtl;
 
   var onKeyDown = useCallback(function (event) {
-    var activeItem = items[activeItemIndex];
+    var activeItemElement = isNil(activeItemIndex) ? null : items[activeItemIndex].element;
 
     switch (true) {
       case !vertical && !rtl && event.key === KEY_VALUES.RIGHT:
@@ -90,10 +89,10 @@ export default function Menubar(_ref) {
       case !vertical && event.key === KEY_VALUES.DOWN:
       case vertical && !rtl && event.key === KEY_VALUES.RIGHT:
       case vertical && rtl && event.key === KEY_VALUES.LEFT:
-        if ((activeItem === null || activeItem === void 0 ? void 0 : activeItem.element.getAttribute('aria-haspopup')) === 'menu') {
+        if ((activeItemElement === null || activeItemElement === void 0 ? void 0 : activeItemElement.getAttribute('aria-haspopup')) === 'menu') {
           event.preventDefault();
           event.stopPropagation();
-          activeItem.element.click();
+          activeItemElement.click();
         }
 
         break;
@@ -102,7 +101,7 @@ export default function Menubar(_ref) {
       case event.key === KEY_VALUES.SPACE:
         event.preventDefault();
         event.stopPropagation();
-        activeItem === null || activeItem === void 0 ? void 0 : activeItem.element.click();
+        activeItemElement === null || activeItemElement === void 0 ? void 0 : activeItemElement.click();
         break;
     }
   }, [rtl, items, activeItemIndex, dispatch, vertical]); // Only used for handling click events bubbling from children
@@ -124,7 +123,7 @@ export default function Menubar(_ref) {
     onBlur: onBlur,
     onKeyDown: onKeyDown,
     onClick: onClick,
-    'aria-activedescendant': (_items$activeItemInde = items[activeItemIndex]) === null || _items$activeItemInde === void 0 ? void 0 : _items$activeItemInde.element.id,
+    'aria-activedescendant': isNil(activeItemIndex) ? undefined : items[activeItemIndex].element.id,
     'aria-orientation': vertical ? 'vertical' : undefined // implicitly set 'horizontal'
 
   }, menubarElementRef));