@pingux/astro 2.163.1-alpha.2 → 2.163.1-alpha.4

package/lib/cjs/components/LinkSelectField/LinkSelectField.js

@@ -41,17 +41,19 @@ var LinkSelectField = /*#__PURE__*/(0, _react.forwardRef)(function (props, ref)
     isDisabled = props.isDisabled,
     status = props.status,
     helperText = props.helperText,
-    hasInlineLoader = props.hasInlineLoader;
+    hasInlineLoader = props.hasInlineLoader,
+    popoverWidth = props.popoverWidth;
   var _getAriaAttributeProp = (0, _ariaAttributes.getAriaAttributeProps)(props),
     ariaProps = _getAriaAttributeProp.ariaProps;
   var _useGetTheme = (0, _hooks.useGetTheme)(),
+    linkSelectFieldWidth = _useGetTheme.linkSelectFieldWidth,
     themeState = _useGetTheme.themeState;
   var isOnyx = themeState.isOnyx;
   var helperTextId = (0, _uuid.v4)();
   (0, _hooks.usePropWarning)(props, 'disabled', 'isDisabled');
   var _useSelectField = (0, _hooks.useSelectField)(_objectSpread(_objectSpread({
       listboxStyle: {
-        width: '10em'
+        width: popoverWidth || linkSelectFieldWidth
       }
     }, props), {}, {
       // Need this for not applying is-default class
@@ -162,7 +164,9 @@ LinkSelectField.propTypes = _objectSpread(_objectSpread(_objectSpread({
    */
   onSelectionChange: _propTypes["default"].func,
   /** Display an inline loader inside the select trigger while loading. */
-  hasInlineLoader: _propTypes["default"].bool
+  hasInlineLoader: _propTypes["default"].bool,
+  /** Width of the popover menu. Accepts any valid CSS unit. */
+  popoverWidth: _propTypes["default"].oneOfType([_propTypes["default"].string, _propTypes["default"].number])
 }, _statusProp.statusPropTypes), _fieldAttributes.inputFieldAttributesBasePropTypes), _ariaAttributes.ariaAttributesBasePropTypes);
 LinkSelectField.defaultProps = _objectSpread({
   placeholder: 'Select',

package/lib/cjs/components/LinkSelectField/LinkSelectField.stories.js

@@ -20,7 +20,7 @@ var _interopRequireDefault = require("@babel/runtime-corejs3/helpers/interopRequ
 _Object$defineProperty(exports, "__esModule", {
   value: true
 });
-exports["default"] = exports.WithNoneOption = exports.NoOptionsAvailable = exports.LeftLabel = exports.HelperText = exports.DynamicItems = exports.DisabledOptions = exports.DisabledField = exports.Default = exports.Controlled = exports.AsyncLoading = void 0;
+exports["default"] = exports.WithPopoverWidth = exports.WithNoneOption = exports.NoOptionsAvailable = exports.LeftLabel = exports.HelperText = exports.DynamicItems = exports.DisabledOptions = exports.DisabledField = exports.Default = exports.Controlled = exports.AsyncLoading = void 0;
 var _asyncToGenerator2 = _interopRequireDefault(require("@babel/runtime-corejs3/helpers/asyncToGenerator"));
 var _map = _interopRequireDefault(require("@babel/runtime-corejs3/core-js-stable/instance/map"));
 var _fill = _interopRequireDefault(require("@babel/runtime-corejs3/core-js-stable/instance/fill"));
@@ -324,4 +324,23 @@ var AsyncLoading = exports.AsyncLoading = function AsyncLoading() {
       key: item.id
     }, item.name.given);
   }));
+};
+var WithPopoverWidth = exports.WithPopoverWidth = function WithPopoverWidth(args) {
+  return (0, _react2.jsx)(_reactAria.OverlayProvider, null, (0, _react2.jsx)(_index.LinkSelectField, (0, _extends2["default"])({}, args, {
+    width: "100%",
+    popoverWidth: "20rem"
+  }), (0, _react2.jsx)(_index.Item, {
+    key: "red"
+  }, "Red"), (0, _react2.jsx)(_index.Item, {
+    key: "blue"
+  }, "Blue"), (0, _react2.jsx)(_index.Item, {
+    key: "yellow"
+  }, "Extra long goes here")));
+};
+WithPopoverWidth.parameters = {
+  docs: {
+    description: {
+      story: 'The `popoverWidth` prop can be used to set a custom width for the dropdown/popover.'
+    }
+  }
 };

package/lib/cjs/components/MenuItem/MenuItem.js

@@ -10,12 +10,14 @@ _Object$defineProperty(exports, "__esModule", {
 });
 exports["default"] = void 0;
 var _extends2 = _interopRequireDefault(require("@babel/runtime-corejs3/helpers/extends"));
-var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime-corejs3/helpers/slicedToArray"));
 var _objectWithoutProperties2 = _interopRequireDefault(require("@babel/runtime-corejs3/helpers/objectWithoutProperties"));
 var _react = _interopRequireWildcard(require("react"));
 var _reactAria = require("react-aria");
+var _CheckIcon = _interopRequireDefault(require("@pingux/mdi-react/CheckIcon"));
+var _focus = require("@react-aria/focus");
 var _interactions = require("@react-aria/interactions");
 var _uuid = require("uuid");
+var _ = require("../..");
 var _MenuContext = require("../../context/MenuContext");
 var _hooks = require("../../hooks");
 var _Box = _interopRequireDefault(require("../Box"));
@@ -31,7 +33,6 @@ var MenuItem = /*#__PURE__*/(0, _react.forwardRef)(function (props, ref) {
   var _item$key;
   var item = props.item,
     isPropDisabled = props.isDisabled,
-    isFocusVisible = props.isFocusVisible,
     isNotFocusedOnHover = props.isNotFocusedOnHover,
     className = props.className,
     state = props.state;
@@ -59,10 +60,6 @@ var MenuItem = /*#__PURE__*/(0, _react.forwardRef)(function (props, ref) {
       closeOnSelect: shouldCloseOnSelect
     }, state, menuItemRef),
     menuItemProps = _useMenuItem.menuItemProps;
-  var _useState = (0, _react.useState)(false),
-    _useState2 = (0, _slicedToArray2["default"])(_useState, 2),
-    isFocused = _useState2[0],
-    setFocused = _useState2[1];
   var _usePress = (0, _interactions.usePress)({
       ref: menuItemRef,
       isDisabled: isDisabled,
@@ -70,34 +67,46 @@ var MenuItem = /*#__PURE__*/(0, _react.forwardRef)(function (props, ref) {
     }),
     pressProps = _usePress.pressProps,
     isPressed = _usePress.isPressed;
-  var _useFocus = (0, _interactions.useFocus)({
-      onFocusChange: setFocused
-    }),
-    focusProps = _useFocus.focusProps;
+  var _useFocusRing = (0, _focus.useFocusRing)(),
+    focusProps = _useFocusRing.focusProps,
+    isFocusVisible = _useFocusRing.isFocusVisible;
   var _useHover = (0, _interactions.useHover)({
       isDisabled: isDisabled
     }),
     hoverProps = _useHover.hoverProps,
     isHovered = _useHover.isHovered;
   var _useStatusClasses = (0, _hooks.useStatusClasses)(className, {
-      isFocused: isFocused || isHovered && !isFocusVisible,
+      isFocused: isFocusVisible && !isNotFocusedOnHover,
       isDisabled: isDisabled,
       isSelected: isSelected,
-      isPressed: isPressed
+      isPressed: isPressed,
+      isHovered: isHovered
     }),
     classNames = _useStatusClasses.classNames;
   if (isNotFocusedOnHover) {
     delete menuItemProps.onPointerEnter;
     delete menuItemProps.onPointerLeave;
   }
+  var _useGetTheme = (0, _hooks.useGetTheme)(),
+    isOnyx = _useGetTheme.themeState.isOnyx;
   return (0, _react2.jsx)(_Box["default"], (0, _extends2["default"])({
     as: "li",
+    isRow: true,
+    alignItems: "center",
+    justifyContent: "space-between",
     className: classNames,
     ref: menuItemRef,
     variant: isSeparator ? 'menuItem.separator' : 'menuItem.item',
     "data-id": dataId,
     "aria-disabled": isDisabled
-  }, (0, _reactAria.mergeProps)(pressProps, hoverProps, focusProps, menuItemProps, others)), rendered);
+  }, (0, _reactAria.mergeProps)(pressProps, hoverProps, focusProps, menuItemProps, others)), rendered, isSelected && isOnyx && (0, _react2.jsx)(_.Icon, {
+    icon: _CheckIcon["default"],
+    title: {
+      name: 'Check Icon'
+    },
+    color: "success.bright",
+    size: "sm"
+  }));
 });
 MenuItem.displayName = 'MenuItem';
 MenuItem.defaultProps = {

package/lib/cjs/components/MenuItem/MenuItem.styles.d.ts

@@ -5,7 +5,7 @@ declare const _default: {
         outline: string;
         cursor: string;
         fontFamily: string;
-        '&.is-focused, &.is-selected, &.is-pressed': {
+        '&.is-focused, &.is-selected, &.is-pressed, &.is-hovered': {
             color: string;
             bg: string;
             '> *': {

package/lib/cjs/components/MenuItem/MenuItem.styles.js

@@ -11,7 +11,7 @@ var item = {
   outline: 'none',
   cursor: 'pointer',
   fontFamily: 'standard',
-  '&.is-focused, &.is-selected, &.is-pressed': {
+  '&.is-focused, &.is-selected, &.is-pressed, &.is-hovered': {
     color: 'white',
     bg: 'active',
     '> *': {

package/lib/cjs/components/MenuItem/MenuItem.test.js

@@ -9,23 +9,34 @@ var _forEachInstanceProperty2 = require("@babel/runtime-corejs3/core-js-stable/i
 var _Object$setPrototypeOf = require("@babel/runtime-corejs3/core-js-stable/object/set-prototype-of");
 var _Promise = require("@babel/runtime-corejs3/core-js-stable/promise");
 var _sliceInstanceProperty = require("@babel/runtime-corejs3/core-js-stable/instance/slice");
+var _Object$keys = require("@babel/runtime-corejs3/core-js-stable/object/keys");
+var _Object$getOwnPropertySymbols = require("@babel/runtime-corejs3/core-js-stable/object/get-own-property-symbols");
+var _filterInstanceProperty = require("@babel/runtime-corejs3/core-js-stable/instance/filter");
+var _Object$getOwnPropertyDescriptor = require("@babel/runtime-corejs3/core-js-stable/object/get-own-property-descriptor");
+var _Object$getOwnPropertyDescriptors = require("@babel/runtime-corejs3/core-js-stable/object/get-own-property-descriptors");
+var _Object$defineProperties = require("@babel/runtime-corejs3/core-js-stable/object/define-properties");
 var _interopRequireDefault = require("@babel/runtime-corejs3/helpers/interopRequireDefault");
+var _defineProperty2 = _interopRequireDefault(require("@babel/runtime-corejs3/helpers/defineProperty"));
 var _asyncToGenerator2 = _interopRequireDefault(require("@babel/runtime-corejs3/helpers/asyncToGenerator"));
 var _extends2 = _interopRequireDefault(require("@babel/runtime-corejs3/helpers/extends"));
-var _objectWithoutProperties2 = _interopRequireDefault(require("@babel/runtime-corejs3/helpers/objectWithoutProperties"));
 var _map = _interopRequireDefault(require("@babel/runtime-corejs3/core-js-stable/instance/map"));
 var _forEach = _interopRequireDefault(require("@babel/runtime-corejs3/core-js-stable/instance/for-each"));
 var _from = _interopRequireDefault(require("@babel/runtime-corejs3/core-js-stable/array/from"));
 var _react = _interopRequireDefault(require("react"));
 var _userEvent = _interopRequireDefault(require("@testing-library/user-event"));
+var _hooks = require("../../hooks");
 var _index = require("../../index");
 var _testWrapper = require("../../utils/testUtils/testWrapper");
-var _universalComponentTest = require("../../utils/testUtils/universalComponentTest");
 var _react2 = require("@emotion/react");
-var _excluded = ["key"],
-  _excluded2 = ["key"];
+function ownKeys(e, r) { var t = _Object$keys(e); if (_Object$getOwnPropertySymbols) { var o = _Object$getOwnPropertySymbols(e); r && (o = _filterInstanceProperty(o).call(o, function (r) { return _Object$getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
+function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var _context7, _context8; var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? _forEachInstanceProperty2(_context7 = ownKeys(Object(t), !0)).call(_context7, function (r) { (0, _defineProperty2["default"])(e, r, t[r]); }) : _Object$getOwnPropertyDescriptors ? _Object$defineProperties(e, _Object$getOwnPropertyDescriptors(t)) : _forEachInstanceProperty2(_context8 = ownKeys(Object(t))).call(_context8, function (r) { _Object$defineProperty(e, r, _Object$getOwnPropertyDescriptor(t, r)); }); } return e; }
 function _regeneratorRuntime() { "use strict"; /*! regenerator-runtime -- Copyright (c) 2014-present, Facebook, Inc. -- license (MIT): https://github.com/babel/babel/blob/main/packages/babel-helpers/LICENSE */ _regeneratorRuntime = function _regeneratorRuntime() { return r; }; var t, r = {}, e = Object.prototype, n = e.hasOwnProperty, o = "function" == typeof _Symbol ? _Symbol : {}, i = o.iterator || "@@iterator", a = o.asyncIterator || "@@asyncIterator", u = o.toStringTag || "@@toStringTag"; function c(t, r, e, n) { return _Object$defineProperty(t, r, { value: e, enumerable: !n, configurable: !n, writable: !n }); } try { c({}, ""); } catch (t) { c = function c(t, r, e) { return t[r] = e; }; } function h(r, e, n, o) { var i = e && e.prototype instanceof Generator ? e : Generator, a = _Object$create(i.prototype); return c(a, "_invoke", function (r, e, n) { var o = 1; return function (i, a) { if (3 === o) throw Error("Generator is already running"); if (4 === o) { if ("throw" === i) throw a; return { value: t, done: !0 }; } for (n.method = i, n.arg = a;;) { var u = n.delegate; if (u) { var c = d(u, n); if (c) { if (c === f) continue; return c; } } if ("next" === n.method) n.sent = n._sent = n.arg;else if ("throw" === n.method) { if (1 === o) throw o = 4, n.arg; n.dispatchException(n.arg); } else "return" === n.method && n.abrupt("return", n.arg); o = 3; var h = s(r, e, n); if ("normal" === h.type) { if (o = n.done ? 4 : 2, h.arg === f) continue; return { value: h.arg, done: n.done }; } "throw" === h.type && (o = 4, n.method = "throw", n.arg = h.arg); } }; }(r, n, new Context(o || [])), !0), a; } function s(t, r, e) { try { return { type: "normal", arg: t.call(r, e) }; } catch (t) { return { type: "throw", arg: t }; } } r.wrap = h; var f = {}; function Generator() {} function GeneratorFunction() {} function GeneratorFunctionPrototype() {} var l = {}; c(l, i, function () { return this; }); var p = _Object$getPrototypeOf, y = p && p(p(x([]))); y && y !== e && n.call(y, i) && (l = y); var v = GeneratorFunctionPrototype.prototype = Generator.prototype = _Object$create(l); function g(t) { var _context5; _forEachInstanceProperty2(_context5 = ["next", "throw", "return"]).call(_context5, function (r) { c(t, r, function (t) { return this._invoke(r, t); }); }); } function AsyncIterator(t, r) { function e(o, i, a, u) { var c = s(t[o], t, i); if ("throw" !== c.type) { var h = c.arg, f = h.value; return f && "object" == _typeof(f) && n.call(f, "__await") ? r.resolve(f.__await).then(function (t) { e("next", t, a, u); }, function (t) { e("throw", t, a, u); }) : r.resolve(f).then(function (t) { h.value = t, a(h); }, function (t) { return e("throw", t, a, u); }); } u(c.arg); } var o; c(this, "_invoke", function (t, n) { function i() { return new r(function (r, o) { e(t, n, r, o); }); } return o = o ? o.then(i, i) : i(); }, !0); } function d(r, e) { var n = e.method, o = r.i[n]; if (o === t) return e.delegate = null, "throw" === n && r.i["return"] && (e.method = "return", e.arg = t, d(r, e), "throw" === e.method) || "return" !== n && (e.method = "throw", e.arg = new TypeError("The iterator does not provide a '" + n + "' method")), f; var i = s(o, r.i, e.arg); if ("throw" === i.type) return e.method = "throw", e.arg = i.arg, e.delegate = null, f; var a = i.arg; return a ? a.done ? (e[r.r] = a.value, e.next = r.n, "return" !== e.method && (e.method = "next", e.arg = t), e.delegate = null, f) : a : (e.method = "throw", e.arg = new TypeError("iterator result is not an object"), e.delegate = null, f); } function w(t) { this.tryEntries.push(t); } function m(r) { var e = r[4] || {}; e.type = "normal", e.arg = t, r[4] = e; } function Context(t) { this.tryEntries = [[-1]], _forEachInstanceProperty2(t).call(t, w, this), this.reset(!0); } function x(r) { if (null != r) { var e = r[i]; if (e) return e.call(r); if ("function" == typeof r.next) return r; if (!isNaN(r.length)) { var o = -1, a = function e() { for (; ++o < r.length;) if (n.call(r, o)) return e.value = r[o], e.done = !1, e; return e.value = t, e.done = !0, e; }; return a.next = a; } } throw new TypeError(_typeof(r) + " is not iterable"); } return GeneratorFunction.prototype = GeneratorFunctionPrototype, c(v, "constructor", GeneratorFunctionPrototype), c(GeneratorFunctionPrototype, "constructor", GeneratorFunction), GeneratorFunction.displayName = c(GeneratorFunctionPrototype, u, "GeneratorFunction"), r.isGeneratorFunction = function (t) { var r = "function" == typeof t && t.constructor; return !!r && (r === GeneratorFunction || "GeneratorFunction" === (r.displayName || r.name)); }, r.mark = function (t) { return _Object$setPrototypeOf ? _Object$setPrototypeOf(t, GeneratorFunctionPrototype) : (t.__proto__ = GeneratorFunctionPrototype, c(t, u, "GeneratorFunction")), t.prototype = _Object$create(v), t; }, r.awrap = function (t) { return { __await: t }; }, g(AsyncIterator.prototype), c(AsyncIterator.prototype, a, function () { return this; }), r.AsyncIterator = AsyncIterator, r.async = function (t, e, n, o, i) { void 0 === i && (i = _Promise); var a = new AsyncIterator(h(t, e, n, o), i); return r.isGeneratorFunction(e) ? a : a.next().then(function (t) { return t.done ? t.value : a.next(); }); }, g(v), c(v, u, "Generator"), c(v, i, function () { return this; }), c(v, "toString", function () { return "[object Generator]"; }), r.keys = function (t) { var r = Object(t), e = []; for (var n in r) e.unshift(n); return function t() { for (; e.length;) if ((n = e.pop()) in r) return t.value = n, t.done = !1, t; return t.done = !0, t; }; }, r.values = x, Context.prototype = { constructor: Context, reset: function reset(r) { var _context6; if (this.prev = this.next = 0, this.sent = this._sent = t, this.done = !1, this.delegate = null, this.method = "next", this.arg = t, _forEachInstanceProperty2(_context6 = this.tryEntries).call(_context6, m), !r) for (var e in this) "t" === e.charAt(0) && n.call(this, e) && !isNaN(+_sliceInstanceProperty(e).call(e, 1)) && (this[e] = t); }, stop: function stop() { this.done = !0; var t = this.tryEntries[0][4]; if ("throw" === t.type) throw t.arg; return this.rval; }, dispatchException: function dispatchException(r) { if (this.done) throw r; var e = this; function n(t) { a.type = "throw", a.arg = r, e.next = t; } for (var o = e.tryEntries.length - 1; o >= 0; --o) { var i = this.tryEntries[o], a = i[4], u = this.prev, c = i[1], h = i[2]; if (-1 === i[0]) return n("end"), !1; if (!c && !h) throw Error("try statement without catch or finally"); if (null != i[0] && i[0] <= u) { if (u < c) return this.method = "next", this.arg = t, n(c), !0; if (u < h) return n(h), !1; } } }, abrupt: function abrupt(t, r) { for (var e = this.tryEntries.length - 1; e >= 0; --e) { var n = this.tryEntries[e]; if (n[0] > -1 && n[0] <= this.prev && this.prev < n[2]) { var o = n; break; } } o && ("break" === t || "continue" === t) && o[0] <= r && r <= o[2] && (o = null); var i = o ? o[4] : {}; return i.type = t, i.arg = r, o ? (this.method = "next", this.next = o[2], f) : this.complete(i); }, complete: function complete(t, r) { if ("throw" === t.type) throw t.arg; return "break" === t.type || "continue" === t.type ? this.next = t.arg : "return" === t.type ? (this.rval = this.arg = t.arg, this.method = "return", this.next = "end") : "normal" === t.type && r && (this.next = r), f; }, finish: function finish(t) { for (var r = this.tryEntries.length - 1; r >= 0; --r) { var e = this.tryEntries[r]; if (e[2] === t) return this.complete(e[4], e[3]), m(e), f; } }, "catch": function _catch(t) { for (var r = this.tryEntries.length - 1; r >= 0; --r) { var e = this.tryEntries[r]; if (e[0] === t) { var n = e[4]; if ("throw" === n.type) { var o = n.arg; m(e); } return o; } } throw Error("illegal catch attempt"); }, delegateYield: function delegateYield(r, e, n) { return this.delegate = { i: x(r), r: e, n: n }, "next" === this.method && (this.arg = t), f; } }, r; }
 var testId = 'testId';
+var mockTheme = {
+  themeState: {
+    isOnyx: false
+  }
+};
 var defaultMenuItems = [{
   key: 'MenuItem 1',
   children: 'MenuItem 1',
@@ -55,208 +66,220 @@ var getComponent = function getComponent() {
     _ref$renderFn = _ref.renderFn,
     renderFn = _ref$renderFn === void 0 ? _testWrapper.render : _ref$renderFn;
   return renderFn((0, _react2.jsx)(_index.Menu, (0, _extends2["default"])({}, defaultProps, props), (0, _map["default"])(items).call(items, function (li) {
-    var key = li.key,
-      itemProps = (0, _objectWithoutProperties2["default"])(li, _excluded);
-    return (0, _react2.jsx)(_index.Item, (0, _extends2["default"])({
-      key: key
-    }, itemProps, {
+    return (0, _react2.jsx)(_index.Item, (0, _extends2["default"])({}, li, {
       sx: {
         backgroundColor: 'orange'
       }
     }));
   })));
 };
-
-// Needs to be added to each components test file
-(0, _universalComponentTest.universalComponentTests)({
-  renderComponent: function renderComponent(props) {
-    return (0, _react2.jsx)(_index.Menu, (0, _extends2["default"])({}, defaultProps, props), (0, _map["default"])(defaultMenuItems).call(defaultMenuItems, function (li) {
-      var key = li.key,
-        itemProps = (0, _objectWithoutProperties2["default"])(li, _excluded2);
-      return (0, _react2.jsx)(_index.Item, (0, _extends2["default"])({
-        key: key
-      }, itemProps, {
-        sx: {
-          backgroundColor: 'orange'
-        }
-      }));
-    }));
-  }
+jest.mock('../../hooks/useGetTheme', function () {
+  return {
+    __esModule: true,
+    "default": jest.fn()
+  };
 });
-test('renders menu with menu items', function () {
-  getComponent();
-  var menu = _testWrapper.screen.getByRole('menu');
-  var menuItems = _testWrapper.screen.getAllByRole('menuitem');
-  expect(menu).toBeInTheDocument();
-  expect(menuItems).toHaveLength(menuItems.length);
-  (0, _forEach["default"])(menuItems).call(menuItems, function (item) {
-    return expect(item).toBeInTheDocument();
+describe('MenuItem', function () {
+  beforeEach(function () {
+    jest.clearAllMocks();
+    _hooks.useGetTheme.mockReturnValue(mockTheme);
   });
-});
-test('applies focus when tabbed to', /*#__PURE__*/(0, _asyncToGenerator2["default"])(/*#__PURE__*/_regeneratorRuntime().mark(function _callee() {
-  var itemText, menuItem;
-  return _regeneratorRuntime().wrap(function _callee$(_context) {
-    while (1) switch (_context.prev = _context.next) {
-      case 0:
-        getComponent();
-        itemText = defaultMenuItems[0].children;
-        menuItem = _testWrapper.screen.getByText(itemText);
-        expect(menuItem).not.toHaveFocus();
-        expect(menuItem).not.toHaveClass('is-focused');
-        _context.next = 7;
-        return _userEvent["default"].tab();
-      case 7:
-        expect(menuItem).toHaveFocus();
-        expect(menuItem).toHaveClass('is-focused');
-      case 9:
-      case "end":
-        return _context.stop();
-    }
-  }, _callee);
-})));
-test('can arrow through menuItems', /*#__PURE__*/(0, _asyncToGenerator2["default"])(/*#__PURE__*/_regeneratorRuntime().mark(function _callee2() {
-  var itemText1, itemText2, menuItem1, menuItem2;
-  return _regeneratorRuntime().wrap(function _callee2$(_context2) {
-    while (1) switch (_context2.prev = _context2.next) {
-      case 0:
-        getComponent();
-        itemText1 = defaultMenuItems[0].children;
-        itemText2 = defaultMenuItems[1].children;
-        menuItem1 = _testWrapper.screen.getByText(itemText1);
-        menuItem2 = _testWrapper.screen.getByText(itemText2);
-        expect(menuItem1).not.toHaveFocus();
-        expect(menuItem1).not.toHaveClass('is-focused');
-        _context2.next = 9;
-        return _userEvent["default"].tab();
-      case 9:
-        expect(menuItem1).toHaveFocus();
-        expect(menuItem1).toHaveClass('is-focused');
-        _testWrapper.fireEvent.keyDown(menuItem1, {
-          key: 'ArrowDown',
-          code: 'ArrowDown'
-        });
-        expect(menuItem2).toHaveFocus();
-        expect(menuItem2).toHaveClass('is-focused');
-      case 14:
-      case "end":
-        return _context2.stop();
-    }
-  }, _callee2);
-})));
-test('hovering menuItems applies focus class and removes focus style on unhover with "isNotFocusedOnHover"', /*#__PURE__*/(0, _asyncToGenerator2["default"])(/*#__PURE__*/_regeneratorRuntime().mark(function _callee3() {
-  var itemText1, menuItem1;
-  return _regeneratorRuntime().wrap(function _callee3$(_context3) {
-    while (1) switch (_context3.prev = _context3.next) {
-      case 0:
-        getComponent({
-          isNotFocusedOnHover: true
-        });
-        itemText1 = defaultMenuItems[0].children;
-        menuItem1 = _testWrapper.screen.getByText(itemText1);
-        expect(menuItem1).not.toHaveClass('is-focused');
-        _context3.next = 6;
-        return _userEvent["default"].hover(menuItem1);
-      case 6:
-        expect(menuItem1).toHaveClass('is-focused');
-        _context3.next = 9;
-        return _userEvent["default"].unhover(menuItem1);
-      case 9:
-        expect(menuItem1).not.toHaveClass('is-focused');
-      case 10:
-      case "end":
-        return _context3.stop();
-    }
-  }, _callee3);
-})));
-test('hovering menuItems applies focus class and leaves focus style on unhover', /*#__PURE__*/(0, _asyncToGenerator2["default"])(/*#__PURE__*/_regeneratorRuntime().mark(function _callee4() {
-  var itemText1, itemText2, menuItem1, menuItem2;
-  return _regeneratorRuntime().wrap(function _callee4$(_context4) {
-    while (1) switch (_context4.prev = _context4.next) {
-      case 0:
-        getComponent();
-        itemText1 = defaultMenuItems[0].children;
-        itemText2 = defaultMenuItems[1].children;
-        menuItem1 = _testWrapper.screen.getByText(itemText1);
-        menuItem2 = _testWrapper.screen.getByText(itemText2);
-        expect(menuItem1).not.toHaveClass('is-focused');
-        expect(menuItem2).not.toHaveClass('is-focused');
-        _context4.next = 9;
-        return _userEvent["default"].hover(menuItem1);
-      case 9:
-        expect(menuItem1).toHaveClass('is-focused');
-        _context4.next = 12;
-        return _userEvent["default"].hover(menuItem2);
-      case 12:
-        expect(menuItem2).toHaveClass('is-focused');
-        _context4.next = 15;
-        return _userEvent["default"].unhover(menuItem2);
-      case 15:
-        expect(menuItem2).toHaveClass('is-focused');
-      case 16:
-      case "end":
-        return _context4.stop();
-    }
-  }, _callee4);
-})));
-test('disables item with isDisabled applied, does not disable without prop', function () {
-  getComponent({
-    disabledKeys: [defaultMenuItems[0].key]
+  test('renders menu with menu items', function () {
+    getComponent();
+    var menu = _testWrapper.screen.getByRole('menu');
+    var menuItems = _testWrapper.screen.getAllByRole('menuitem');
+    expect(menu).toBeInTheDocument();
+    expect(menuItems).toHaveLength(menuItems.length);
+    (0, _forEach["default"])(menuItems).call(menuItems, function (item) {
+      return expect(item).toBeInTheDocument();
+    });
   });
-  var disbaledItemText = defaultMenuItems[0].children;
-  var disabledMenuItem = _testWrapper.screen.getByText(disbaledItemText);
-  expect(disabledMenuItem).toHaveClass('is-disabled');
-  var menuItemText = defaultMenuItems[1].children;
-  var menuItem = _testWrapper.screen.getByText(menuItemText);
-  expect(menuItem).not.toHaveClass('is-disabled');
-});
-test('should change styles according to isPressed prop', function () {
-  var customItems = (0, _from["default"])(defaultMenuItems);
-  customItems[0].isPressed = true;
-  getComponent({}, {
-    items: customItems
+  test('applies focus when tabbed to', /*#__PURE__*/(0, _asyncToGenerator2["default"])(/*#__PURE__*/_regeneratorRuntime().mark(function _callee() {
+    var itemText, menuItem;
+    return _regeneratorRuntime().wrap(function _callee$(_context) {
+      while (1) switch (_context.prev = _context.next) {
+        case 0:
+          getComponent();
+          itemText = defaultMenuItems[0].children;
+          menuItem = _testWrapper.screen.getByText(itemText);
+          expect(menuItem).not.toHaveFocus();
+          expect(menuItem).not.toHaveClass('is-focused');
+          _context.next = 7;
+          return _userEvent["default"].tab();
+        case 7:
+          expect(menuItem).toHaveFocus();
+          expect(menuItem).toHaveClass('is-focused');
+        case 9:
+        case "end":
+          return _context.stop();
+      }
+    }, _callee);
+  })));
+  test('can arrow through menuItems', /*#__PURE__*/(0, _asyncToGenerator2["default"])(/*#__PURE__*/_regeneratorRuntime().mark(function _callee2() {
+    var itemText1, itemText2, menuItem1, menuItem2;
+    return _regeneratorRuntime().wrap(function _callee2$(_context2) {
+      while (1) switch (_context2.prev = _context2.next) {
+        case 0:
+          getComponent();
+          itemText1 = defaultMenuItems[0].children;
+          itemText2 = defaultMenuItems[1].children;
+          menuItem1 = _testWrapper.screen.getByText(itemText1);
+          menuItem2 = _testWrapper.screen.getByText(itemText2);
+          expect(menuItem1).not.toHaveFocus();
+          expect(menuItem1).not.toHaveClass('is-focused');
+          _context2.next = 9;
+          return _userEvent["default"].tab();
+        case 9:
+          expect(menuItem1).toHaveFocus();
+          expect(menuItem1).toHaveClass('is-focused');
+          _testWrapper.fireEvent.keyDown(menuItem1, {
+            key: 'ArrowDown',
+            code: 'ArrowDown'
+          });
+          expect(menuItem2).toHaveFocus();
+          expect(menuItem2).toHaveClass('is-focused');
+        case 14:
+        case "end":
+          return _context2.stop();
+      }
+    }, _callee2);
+  })));
+  test('hovering menuItems applies hover class and removes hover style on unhover with "isNotFocusedOnHover"', /*#__PURE__*/(0, _asyncToGenerator2["default"])(/*#__PURE__*/_regeneratorRuntime().mark(function _callee3() {
+    var itemText1, menuItem1;
+    return _regeneratorRuntime().wrap(function _callee3$(_context3) {
+      while (1) switch (_context3.prev = _context3.next) {
+        case 0:
+          getComponent({
+            isNotFocusedOnHover: true
+          });
+          itemText1 = defaultMenuItems[0].children;
+          menuItem1 = _testWrapper.screen.getByText(itemText1);
+          expect(menuItem1).not.toHaveClass('is-hovered');
+          _context3.next = 6;
+          return _userEvent["default"].hover(menuItem1);
+        case 6:
+          expect(menuItem1).toHaveClass('is-hovered');
+          _context3.next = 9;
+          return _userEvent["default"].unhover(menuItem1);
+        case 9:
+          expect(menuItem1).not.toHaveClass('is-hovered');
+        case 10:
+        case "end":
+          return _context3.stop();
+      }
+    }, _callee3);
+  })));
+  test('hovering menuItems applies hover class and leaves hover style on unhover', /*#__PURE__*/(0, _asyncToGenerator2["default"])(/*#__PURE__*/_regeneratorRuntime().mark(function _callee4() {
+    var itemText1, itemText2, menuItem1, menuItem2;
+    return _regeneratorRuntime().wrap(function _callee4$(_context4) {
+      while (1) switch (_context4.prev = _context4.next) {
+        case 0:
+          getComponent();
+          itemText1 = defaultMenuItems[0].children;
+          itemText2 = defaultMenuItems[1].children;
+          menuItem1 = _testWrapper.screen.getByText(itemText1);
+          menuItem2 = _testWrapper.screen.getByText(itemText2);
+          expect(menuItem1).not.toHaveClass('is-hovered');
+          expect(menuItem2).not.toHaveClass('is-hovered');
+          _context4.next = 9;
+          return _userEvent["default"].hover(menuItem1);
+        case 9:
+          expect(menuItem1).toHaveClass('is-hovered');
+          _context4.next = 12;
+          return _userEvent["default"].hover(menuItem2);
+        case 12:
+          expect(menuItem2).toHaveClass('is-hovered');
+          _context4.next = 15;
+          return _userEvent["default"].unhover(menuItem2);
+        case 15:
+          expect(menuItem2).not.toHaveClass('is-hovered');
+        case 16:
+        case "end":
+          return _context4.stop();
+      }
+    }, _callee4);
+  })));
+  test('disables item with isDisabled applied, does not disable without prop', function () {
+    getComponent({
+      disabledKeys: [defaultMenuItems[0].key]
+    });
+    var disbaledItemText = defaultMenuItems[0].children;
+    var disabledMenuItem = _testWrapper.screen.getByText(disbaledItemText);
+    expect(disabledMenuItem).toHaveClass('is-disabled');
+    var menuItemText = defaultMenuItems[1].children;
+    var menuItem = _testWrapper.screen.getByText(menuItemText);
+    expect(menuItem).not.toHaveClass('is-disabled');
   });
-  var pressedItem = _testWrapper.screen.getAllByRole('menuitem')[0];
-  var regularItem = _testWrapper.screen.getAllByRole('menuitem')[1];
-  expect(pressedItem).toHaveClass('is-pressed');
-  expect(regularItem).not.toHaveClass('is-pressed');
-});
-test('applies selected styling with isSelected prop, does not add class without prop', function () {
-  getComponent({
-    selectionMode: 'single',
-    defaultSelectedKeys: [defaultMenuItems[0].key]
+  test('should change styles according to isPressed prop', function () {
+    var customItems = (0, _from["default"])(defaultMenuItems);
+    customItems[0].isPressed = true;
+    getComponent({}, {
+      items: customItems
+    });
+    var pressedItem = _testWrapper.screen.getAllByRole('menuitem')[0];
+    var regularItem = _testWrapper.screen.getAllByRole('menuitem')[1];
+    expect(pressedItem).toHaveClass('is-pressed');
+    expect(regularItem).not.toHaveClass('is-pressed');
   });
-  var selectedItemText = defaultMenuItems[0].children;
-  var selectedMenuItem = _testWrapper.screen.getByText(selectedItemText);
-  expect(selectedMenuItem).toHaveClass('is-selected');
-  var menuItemText = defaultMenuItems[1].children;
-  var menuItem = _testWrapper.screen.getByText(menuItemText);
-  expect(menuItem).not.toHaveClass('is-selected');
-});
-test('Item accepts a data-id and the data-id can be found in the DOM', function () {
-  getComponent();
-  var menuItems = _testWrapper.screen.queryAllByRole('menuitem');
-  expect(menuItems).toHaveLength(defaultMenuItems.length);
-  expect(menuItems[0]).toHaveAttribute('data-id', defaultMenuItems[0]['data-id']);
-});
-test('custom props are passed into menuItem', function () {
-  getComponent();
-  var itemText1 = defaultMenuItems[0].children;
-  var menuItem1 = _testWrapper.screen.getByText(itemText1);
-  expect(menuItem1).toHaveStyleRule('background-color', 'orange');
-});
-test('Selection mode given as single', function () {
-  getComponent({
-    selectionMode: 'single',
-    defaultSelectedKeys: [defaultMenuItems[0].key]
+  test('applies selected styling with isSelected prop, does not add class without prop', function () {
+    getComponent({
+      selectionMode: 'single',
+      defaultSelectedKeys: [defaultMenuItems[0].key]
+    });
+    var selectedItemText = defaultMenuItems[0].children;
+    var selectedMenuItem = _testWrapper.screen.getByText(selectedItemText);
+    expect(selectedMenuItem).toHaveClass('is-selected');
+    var menuItemText = defaultMenuItems[1].children;
+    var menuItem = _testWrapper.screen.getByText(menuItemText);
+    expect(menuItem).not.toHaveClass('is-selected');
+  });
+  test('Item accepts a data-id and the data-id can be found in the DOM', function () {
+    getComponent();
+    var menuItems = _testWrapper.screen.queryAllByRole('menuitem');
+    expect(menuItems).toHaveLength(defaultMenuItems.length);
+    expect(menuItems[0]).toHaveAttribute('data-id', defaultMenuItems[0]['data-id']);
+  });
+  test('custom props are passed into menuItem', function () {
+    getComponent();
+    var itemText1 = defaultMenuItems[0].children;
+    var menuItem1 = _testWrapper.screen.getByText(itemText1);
+    expect(menuItem1).toHaveStyleRule('background-color', 'orange');
+  });
+  test('Selection mode given as single', function () {
+    getComponent({
+      selectionMode: 'single',
+      defaultSelectedKeys: [defaultMenuItems[0].key]
+    });
+    var menuItems = _testWrapper.screen.queryAllByRole('menuitemradio');
+    expect(menuItems).toHaveLength(defaultMenuItems.length);
+  });
+  test('Selection mode given as multiple', function () {
+    getComponent({
+      selectionMode: 'multiple',
+      defaultSelectedKeys: [defaultMenuItems[0].key]
+    });
+    var menuItems = _testWrapper.screen.queryAllByRole('menuitemcheckbox');
+    expect(menuItems).toHaveLength(defaultMenuItems.length);
   });
-  var menuItems = _testWrapper.screen.queryAllByRole('menuitemradio');
-  expect(menuItems).toHaveLength(defaultMenuItems.length);
 });
-test('Selection mode given as multiple', function () {
-  getComponent({
-    selectionMode: 'multiple',
-    defaultSelectedKeys: [defaultMenuItems[0].key]
+describe('MenuItem', function () {
+  beforeEach(function () {
+    jest.clearAllMocks();
+    _hooks.useGetTheme.mockReturnValue(_objectSpread(_objectSpread({}, mockTheme), {}, {
+      themeState: {
+        isOnyx: true
+      }
+    }));
+  });
+  test('checkmark icon is shown when item is selected and theme is onyx', function () {
+    getComponent({
+      selectionMode: 'single',
+      defaultSelectedKeys: [defaultMenuItems[0].key]
+    });
+    var selectedItemText = defaultMenuItems[0].children;
+    var selectedMenuItem = _testWrapper.screen.getByText(selectedItemText);
+    var checkIcon = _testWrapper.screen.getByTitle('Check Icon');
+    expect(selectedMenuItem).toBeInTheDocument();
+    expect(checkIcon).toBeInTheDocument();
   });
-  var menuItems = _testWrapper.screen.queryAllByRole('menuitemcheckbox');
-  expect(menuItems).toHaveLength(defaultMenuItems.length);
 });

package/lib/cjs/components/MenuSection/MenuSection.js

@@ -42,7 +42,8 @@ function _interopRequireWildcard(e, t) { if ("function" == typeof _WeakMap) var
   return (0, _react2.jsx)(_react["default"].Fragment, null, section.key !== state.collection.getFirstKey() && (0, _react2.jsx)(_Separator["default"], {
     as: "li",
     p: 0,
-    m: 0
+    my: "sm",
+    mx: "0"
   }), (0, _react2.jsx)(_Box["default"], (0, _extends2["default"])({
     as: "li"
   }, itemProps, {