@pingux/astro 2.33.0 → 2.34.0-alpha.1

package/lib/cjs/components/Breadcrumbs/Breadcrumb.styles.js

@@ -15,36 +15,37 @@ _Object$defineProperty(exports, "__esModule", {
 exports["default"] = void 0;
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime-corejs3/helpers/defineProperty"));
 var _Buttons = require("../Button/Buttons.styles");
+var _Text = require("../Text/Text.styles");
 function ownKeys(object, enumerableOnly) { var keys = _Object$keys(object); if (_Object$getOwnPropertySymbols) { var symbols = _Object$getOwnPropertySymbols(object); enumerableOnly && (symbols = _filterInstanceProperty(symbols).call(symbols, function (sym) { return _Object$getOwnPropertyDescriptor(object, sym).enumerable; })), keys.push.apply(keys, symbols); } return keys; }
 function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var _context, _context2; var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? _forEachInstanceProperty(_context = ownKeys(Object(source), !0)).call(_context, function (key) { (0, _defineProperty2["default"])(target, key, source[key]); }) : _Object$getOwnPropertyDescriptors ? _Object$defineProperties(target, _Object$getOwnPropertyDescriptors(source)) : _forEachInstanceProperty(_context2 = ownKeys(Object(source))).call(_context2, function (key) { _Object$defineProperty(target, key, _Object$getOwnPropertyDescriptor(source, key)); }); } return target; }
-var link = {
-  color: 'active',
-  fontFamily: 'standard',
-  fontSize: '15px',
-  textDecoration: 'none',
-  outline: 'none',
-  '&.is-hovered': {
-    textDecoration: 'underline'
-  },
-  '&.is-focused': {
-    textDecoration: 'underline'
-  },
-  '&.is-disabled': {
-    pointerEvents: 'none'
-  },
-  '&.is-current': _objectSpread(_objectSpread({
-    pointerEvents: 'none'
-  }, _Buttons.base), {}, {
-    fontWeight: 1
-  })
-};
-var containerLi = {
+var breadcrumbMinWidth = '40px';
+var link = _objectSpread(_objectSpread(_objectSpread({}, _Text.text.textEllipsis), _Buttons.link), {}, {
+  display: 'block',
+  minWidth: breadcrumbMinWidth,
+  '&.is-current': {
+    color: 'text.primary',
+    textDecoration: 'none',
+    cursor: 'default'
+  }
+});
+var containerOl = {
+  alignItems: 'center',
+  minHeight: 42,
+  paddingInlineStart: 'unset',
   'a.is-hovered.is-current ': {
     textDecoration: 'none'
   }
 };
+var containerLi = {
+  flex: '0 1000000 auto',
+  minWidth: breadcrumbMinWidth,
+  '&.is-current': {
+    flex: '0 1 auto'
+  }
+};
 var _default = {
-  link: link,
-  containerLi: containerLi
+  containerLi: containerLi,
+  containerOl: containerOl,
+  link: link
 };
 exports["default"] = _default;

package/lib/cjs/components/Breadcrumbs/BreadcrumbItem.js

@@ -14,7 +14,7 @@ var _interopRequireDefault = require("@babel/runtime-corejs3/helpers/interopRequ
 _Object$defineProperty(exports, "__esModule", {
   value: true
 });
-exports["default"] = void 0;
+exports["default"] = exports.ELEMENT_TYPE = void 0;
 var _includes = _interopRequireDefault(require("@babel/runtime-corejs3/core-js-stable/instance/includes"));
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime-corejs3/helpers/defineProperty"));
 var _objectWithoutProperties2 = _interopRequireDefault(require("@babel/runtime-corejs3/helpers/objectWithoutProperties"));
@@ -30,6 +30,14 @@ function _getRequireWildcardCache(nodeInterop) { if (typeof _WeakMap !== "functi
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || _typeof(obj) !== "object" && typeof obj !== "function") { return { "default": obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = _Object$defineProperty && _Object$getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? _Object$getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { _Object$defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj["default"] = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
 function ownKeys(object, enumerableOnly) { var keys = _Object$keys(object); if (_Object$getOwnPropertySymbols) { var symbols = _Object$getOwnPropertySymbols(object); enumerableOnly && (symbols = _filterInstanceProperty(symbols).call(symbols, function (sym) { return _Object$getOwnPropertyDescriptor(object, sym).enumerable; })), keys.push.apply(keys, symbols); } return keys; }
 function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var _context, _context2; var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? _forEachInstanceProperty(_context = ownKeys(Object(source), !0)).call(_context, function (key) { (0, _defineProperty2["default"])(target, key, source[key]); }) : _Object$getOwnPropertyDescriptors ? _Object$defineProperties(target, _Object$getOwnPropertyDescriptors(source)) : _forEachInstanceProperty(_context2 = ownKeys(Object(source))).call(_context2, function (key) { _Object$defineProperty(target, key, _Object$getOwnPropertyDescriptor(source, key)); }); } return target; }
+var ELEMENT_TYPE = {
+  BUTTON: 'Button',
+  ICON_BUTTON: 'IconButton',
+  TEXT: 'Text',
+  LINK: 'Link',
+  FRAGMENT: 'Fragment'
+};
+exports.ELEMENT_TYPE = ELEMENT_TYPE;
 var BreadcrumbItem = /*#__PURE__*/(0, _react.forwardRef)(function (props, ref) {
   var children = props.children,
     elementType = props.elementType,
@@ -47,15 +55,15 @@ var BreadcrumbItem = /*#__PURE__*/(0, _react.forwardRef)(function (props, ref) {
     itemProps = _useBreadcrumbItem.itemProps;
   var ElementType = (0, _react.useMemo)(function () {
     switch (elementType) {
-      case 'Button':
+      case ELEMENT_TYPE.BUTTON:
         return _index.Button;
-      case 'IconButton':
+      case ELEMENT_TYPE.ICON_BUTTON:
         return _index.IconButton;
-      case 'Text':
+      case ELEMENT_TYPE.TEXT:
         return _index.Text;
-      case 'Link':
+      case ELEMENT_TYPE.LINK:
         return _index.Link;
-      case 'Fragment':
+      case ELEMENT_TYPE.FRAGMENT:
         return _react.Fragment;
       default:
         return elementType;
@@ -77,9 +85,15 @@ var BreadcrumbItem = /*#__PURE__*/(0, _react.forwardRef)(function (props, ref) {
     }
     return (0, _object.omit)(elementTypeProps, 'onClick', 'onKeyDown', 'onKeyUp');
   }, [elementType, itemProps, others, onPressHandler]);
-  return /*#__PURE__*/_react["default"].cloneElement((0, _react2.jsx)(ElementType, null), _objectSpread(_objectSpread({}, elementProps), {}, {
-    ref: ref
-  }), children);
+  var elementVariantProps = elementType !== ELEMENT_TYPE.FRAGMENT && _objectSpread({
+    variant: elementType === ELEMENT_TYPE.LINK ? 'variants.breadcrumb.link' : '',
+    ref: itemRef
+  }, elementProps);
+  return (0, _react2.jsx)(_index.Box, {
+    as: "li",
+    className: isCurrent && 'is-current',
+    variant: "variants.breadcrumb.containerLi"
+  }, (0, _react2.jsx)(ElementType, elementVariantProps, children));
 });
 BreadcrumbItem.propTypes = {
   actionKey: _propTypes["default"].string,

package/lib/cjs/components/Breadcrumbs/Breadcrumbs.js

@@ -44,37 +44,30 @@ var Breadcrumbs = /*#__PURE__*/(0, _react.forwardRef)(function (props, ref) {
   });
   var createBreadcrumb = (0, _react.useCallback)(function (child, idx) {
     var isCurrentItem = (0, _isArray["default"])(filteredChildren) && filteredChildren.length > 1 ? idx === children.length - 1 : true;
-    return (0, _react2.jsx)(_index.Box, {
-      isRow: true,
-      sx: {
-        alignItems: 'center',
-        minHeight: 42
-      },
-      as: "li",
-      key: "li-".concat(child.key),
-      variant: "variants.breadcrumb.containerLi"
+    return (0, _react2.jsx)(_react["default"].Fragment, {
+      key: "li-".concat(child.key)
     }, (0, _react2.jsx)(_BreadcrumbItem["default"], (0, _extends2["default"])({
+      actionKey: child.key,
       "data-id": child['data-id'],
       isCurrent: isCurrentItem,
-      onAction: onAction,
-      actionKey: child.key,
-      variant: "variants.breadcrumb.link"
+      onAction: onAction
     }, child.props), child.props.children), icon && !isCurrentItem && (0, _react2.jsx)(_index.Icon, (0, _extends2["default"])({
       "aria-hidden": "true",
       icon: icon,
       mx: 5,
-      size: "xs"
+      size: "xs",
+      title: {
+        name: 'Breadcrumb Separator'
+      }
     }, iconProps)));
   }, [children.length, filteredChildren, icon, iconProps, onAction]);
   return (0, _react2.jsx)("nav", {
     "aria-label": "Breadcrumb"
   }, (0, _react2.jsx)(_index.Box, (0, _extends2["default"])({
-    ref: breadcrumbsRef,
+    as: "ol",
     isRow: true,
-    sx: {
-      paddingInlineStart: 'unset'
-    },
-    as: "ol"
+    ref: breadcrumbsRef,
+    variant: "variants.breadcrumb.containerOl"
   }, (0, _reactAria.mergeProps)(wrapperProps, others)), (0, _isArray["default"])(filteredChildren) ? (0, _map["default"])(filteredChildren).call(filteredChildren, createBreadcrumb) : createBreadcrumb(children)));
 });
 Breadcrumbs.propTypes = {

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

@@ -5,7 +5,7 @@ var _interopRequireDefault = require("@babel/runtime-corejs3/helpers/interopRequ
 _Object$defineProperty(exports, "__esModule", {
   value: true
 });
-exports["default"] = exports.Default = void 0;
+exports["default"] = exports.Overflowing = exports.Default = void 0;
 var _extends2 = _interopRequireDefault(require("@babel/runtime-corejs3/helpers/extends"));
 var _react = _interopRequireDefault(require("react"));
 var _reactStately = require("react-stately");
@@ -43,9 +43,6 @@ var _default = {
         type: 'none'
       }
     }
-  },
-  args: {
-    icon: _ChevronRightIcon["default"]
   }
 };
 exports["default"] = _default;
@@ -55,23 +52,22 @@ var Default = function Default(args) {
   };
   return (0, _react2.jsx)(_index.Breadcrumbs, (0, _extends2["default"])({
     onAction: onAction
-  }, args), (0, _react2.jsx)(_reactStately.Item, {
+  }, args, {
+    icon: _ChevronRightIcon["default"]
+  }), (0, _react2.jsx)(_reactStately.Item, {
     "aria-label": "home",
     "data-id": "home",
     href: "https://www.pingidentity.com",
-    key: "home",
-    variant: "buttons.link"
+    key: "home"
   }, "Home"), (0, _react2.jsx)(_reactStately.Item, {
     "aria-label": "trendy",
     "data-id": "trendy",
     href: "https://www.pingidentity.com",
-    key: "trendy",
-    variant: "buttons.link"
+    key: "trendy"
   }, "Trendy"), (0, _react2.jsx)(_reactStately.Item, {
     "aria-label": "march-2020-assets",
     "data-id": "march",
-    key: "march 2020 assets",
-    variant: "buttons.link"
+    key: "march 2020 assets"
   }, "March 2020 Assets"));
 };
 exports.Default = Default;
@@ -80,4 +76,50 @@ Default.parameters = {
     type: 'figma',
     url: _figmaLinks.FIGMA_LINKS.breadcrumbs["default"]
   }
+};
+var Overflowing = function Overflowing() {
+  var reallyLongText = 'If This Text Were Really Very Much So Extremely Long';
+  return (0, _react2.jsx)(_index.Box, {
+    gap: "md",
+    maxWidth: "385px"
+  }, (0, _react2.jsx)(_index.Breadcrumbs, {
+    icon: _ChevronRightIcon["default"]
+  }, (0, _react2.jsx)(_reactStately.Item, {
+    "aria-label": "breadcrumb_1",
+    "data-id": "breadcrumb_1",
+    href: "https://www.pingidentity.com",
+    key: "breadcrumb_1"
+  }, reallyLongText), (0, _react2.jsx)(_reactStately.Item, {
+    "aria-label": "breadcrumb_2",
+    "data-id": "breadcrumb_2",
+    href: "https://www.pingidentity.com",
+    key: "breadcrumb_2"
+  }, "Edit")), (0, _react2.jsx)(_index.Breadcrumbs, {
+    icon: _ChevronRightIcon["default"]
+  }, (0, _react2.jsx)(_reactStately.Item, {
+    "aria-label": "breadcrumb_1",
+    "data-id": "breadcrumb_1",
+    href: "https://www.pingidentity.com",
+    key: "breadcrumb_1"
+  }, reallyLongText), (0, _react2.jsx)(_reactStately.Item, {
+    "aria-label": "breadcrumb_1",
+    "data-id": "breadcrumb_2",
+    href: "https://www.pingidentity.com",
+    key: "breadcrumb_2"
+  }, reallyLongText)));
+};
+exports.Overflowing = Overflowing;
+Overflowing.parameters = {
+  design: {
+    type: 'figma',
+    url: _figmaLinks.FIGMA_LINKS.breadcrumbs.overflowing
+  },
+  a11y: {
+    config: {
+      rules: [{
+        id: 'landmark-unique',
+        enabled: false
+      }]
+    }
+  }
 };

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

@@ -11,6 +11,7 @@ var _CreateIcon = _interopRequireDefault(require("@pingux/mdi-react/CreateIcon")
 var _userEvent = _interopRequireDefault(require("@testing-library/user-event"));
 var _testWrapper = require("../../utils/testUtils/testWrapper");
 var _universalComponentTest = require("../../utils/testUtils/universalComponentTest");
+var _BreadcrumbItem = require("./BreadcrumbItem");
 var _Breadcrumbs = _interopRequireDefault(require("./Breadcrumbs"));
 var _react2 = require("@emotion/react");
 var testId = 'test-breadcrumbs';
@@ -60,14 +61,14 @@ test('should render correct amount of icons', function () {
 });
 test('breadcrumbItem should render breadcrumbItem as a Text component when appropriate elementType passed', function () {
   getComponent({}, {
-    elementType: 'Text',
+    elementType: _BreadcrumbItem.ELEMENT_TYPE.TEXT,
     'data-testid': testItemId
   });
   expect(_testWrapper.screen.getAllByTestId(testItemId)[0]).toBeInstanceOf(HTMLSpanElement);
 });
 test('breadcrumbItem should render breadcrumbItem as a IconButton component when appropriate elementType passed', function () {
   getComponent({}, {
-    elementType: 'IconButton',
+    elementType: _BreadcrumbItem.ELEMENT_TYPE.ICON_BUTTON,
     'data-testid': testItemId,
     icon: _CreateIcon["default"]
   });
@@ -84,7 +85,7 @@ test('breadcrumbItem should render breadcrumbItem as a html tag when appropriate
 test('breadcrumbs will use onAction if provided', function () {
   var mockOnAction = jest.fn();
   getComponent({}, {
-    elementType: 'Link',
+    elementType: _BreadcrumbItem.ELEMENT_TYPE.LINK,
     onAction: mockOnAction
   });
   _userEvent["default"].click(_testWrapper.screen.getByText(testItemsArr[0]));

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

@@ -1,13 +1,19 @@
 "use strict";
 
+var _typeof = require("@babel/runtime-corejs3/helpers/typeof");
+var _WeakMap = require("@babel/runtime-corejs3/core-js-stable/weak-map");
+var _Object$defineProperty = require("@babel/runtime-corejs3/core-js-stable/object/define-property");
+var _Object$getOwnPropertyDescriptor = require("@babel/runtime-corejs3/core-js-stable/object/get-own-property-descriptor");
 var _interopRequireDefault = require("@babel/runtime-corejs3/helpers/interopRequireDefault");
 var _extends2 = _interopRequireDefault(require("@babel/runtime-corejs3/helpers/extends"));
-var _react = _interopRequireDefault(require("react"));
+var _react = _interopRequireWildcard(require("react"));
 var _EarthIcon = _interopRequireDefault(require("@pingux/mdi-react/EarthIcon"));
 var _react2 = require("@testing-library/react");
-var _testAxe = _interopRequireDefault(require("../../utils/testUtils/testAxe"));
+var _universalComponentTest = require("../../utils/testUtils/universalComponentTest");
 var _ = _interopRequireDefault(require("."));
 var _react3 = require("@emotion/react");
+function _getRequireWildcardCache(nodeInterop) { if (typeof _WeakMap !== "function") return null; var cacheBabelInterop = new _WeakMap(); var cacheNodeInterop = new _WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
+function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || _typeof(obj) !== "object" && typeof obj !== "function") { return { "default": obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = _Object$defineProperty && _Object$getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? _Object$getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { _Object$defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj["default"] = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
 var testId = 'test-icon';
 var defaultProps = {
   'data-testid': testId,
@@ -22,8 +28,26 @@ var getComponent = function getComponent() {
   return (0, _react2.render)((0, _react3.jsx)(_["default"], (0, _extends2["default"])({}, defaultProps, props)));
 };
 
-// Need to be added to each test file to test accessibility using axe.
-(0, _testAxe["default"])(getComponent);
+// The mdi-react and @pingux/mid-react libraries don't support ref forwarding
+// A simple implementation of an SVG component demonstrates ref forwarding
+var SVGTestComponent = /*#__PURE__*/(0, _react.forwardRef)(function (props, ref) {
+  return (0, _react3.jsx)("svg", (0, _extends2["default"])({
+    ref: ref
+  }, props, {
+    "aria-labelledby": "id"
+  }), (0, _react3.jsx)("title", {
+    id: "id"
+  }, "title"));
+});
+
+// Needs to be added to each components test file
+(0, _universalComponentTest.universalComponentTests)({
+  renderComponent: function renderComponent(props) {
+    return (0, _react3.jsx)(_["default"], (0, _extends2["default"])({
+      icon: SVGTestComponent
+    }, props));
+  }
+});
 test('default icon', function () {
   getComponent();
   var icon = _react2.screen.getByTestId(testId);

package/lib/cjs/utils/designUtils/figmaLinks.js

@@ -30,7 +30,8 @@ var FIGMA_LINKS = {
     isLast: 'https://www.figma.com/file/bpbEw54gTTcdIiZs4PKK1p/Astro-Specs?node-id=45365%3A7832&mode=dev'
   },
   breadcrumbs: {
-    "default": 'https://www.figma.com/file/bpbEw54gTTcdIiZs4PKK1p/Astro-Specs?type=design&node-id=1162-11924&t=8Wwd3tIBh3GEjCJB-0'
+    "default": 'https://www.figma.com/file/bpbEw54gTTcdIiZs4PKK1p/Astro-Specs?type=design&node-id=1162-11924&t=8Wwd3tIBh3GEjCJB-0',
+    overflowing: 'https://www.figma.com/file/bpbEw54gTTcdIiZs4PKK1p/Astro-Specs?type=design&node-id=45064-7148&mode=design&t=0RGDgLyKBzT6bOXO-0'
   },
   button: {
     "default": 'https://www.figma.com/file/bpbEw54gTTcdIiZs4PKK1p/Astro-Specs?type=design&node-id=0-1&t=8Wwd3tIBh3GEjCJB-0'

package/lib/components/Breadcrumbs/Breadcrumb.styles.js

@@ -9,34 +9,35 @@ import _Object$defineProperty from "@babel/runtime-corejs3/core-js-stable/object
 import _defineProperty from "@babel/runtime-corejs3/helpers/esm/defineProperty";
 function ownKeys(object, enumerableOnly) { var keys = _Object$keys(object); if (_Object$getOwnPropertySymbols) { var symbols = _Object$getOwnPropertySymbols(object); enumerableOnly && (symbols = _filterInstanceProperty(symbols).call(symbols, function (sym) { return _Object$getOwnPropertyDescriptor(object, sym).enumerable; })), keys.push.apply(keys, symbols); } return keys; }
 function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var _context, _context2; var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? _forEachInstanceProperty(_context = ownKeys(Object(source), !0)).call(_context, function (key) { _defineProperty(target, key, source[key]); }) : _Object$getOwnPropertyDescriptors ? _Object$defineProperties(target, _Object$getOwnPropertyDescriptors(source)) : _forEachInstanceProperty(_context2 = ownKeys(Object(source))).call(_context2, function (key) { _Object$defineProperty(target, key, _Object$getOwnPropertyDescriptor(source, key)); }); } return target; }
-import { base } from '../Button/Buttons.styles';
-var link = {
-  color: 'active',
-  fontFamily: 'standard',
-  fontSize: '15px',
-  textDecoration: 'none',
-  outline: 'none',
-  '&.is-hovered': {
-    textDecoration: 'underline'
-  },
-  '&.is-focused': {
-    textDecoration: 'underline'
-  },
-  '&.is-disabled': {
-    pointerEvents: 'none'
-  },
-  '&.is-current': _objectSpread(_objectSpread({
-    pointerEvents: 'none'
-  }, base), {}, {
-    fontWeight: 1
-  })
-};
-var containerLi = {
+import { link as buttonLink } from '../Button/Buttons.styles';
+import { text } from '../Text/Text.styles';
+var breadcrumbMinWidth = '40px';
+var link = _objectSpread(_objectSpread(_objectSpread({}, text.textEllipsis), buttonLink), {}, {
+  display: 'block',
+  minWidth: breadcrumbMinWidth,
+  '&.is-current': {
+    color: 'text.primary',
+    textDecoration: 'none',
+    cursor: 'default'
+  }
+});
+var containerOl = {
+  alignItems: 'center',
+  minHeight: 42,
+  paddingInlineStart: 'unset',
   'a.is-hovered.is-current ': {
     textDecoration: 'none'
   }
 };
+var containerLi = {
+  flex: '0 1000000 auto',
+  minWidth: breadcrumbMinWidth,
+  '&.is-current': {
+    flex: '0 1 auto'
+  }
+};
 export default {
-  link: link,
-  containerLi: containerLi
+  containerLi: containerLi,
+  containerOl: containerOl,
+  link: link
 };

package/lib/components/Breadcrumbs/BreadcrumbItem.js

@@ -17,8 +17,15 @@ import { mergeProps, useBreadcrumbItem } from 'react-aria';
 import { omit } from 'lodash/object';
 import PropTypes from 'prop-types';
 import { usePropWarning } from '../../hooks';
-import { Button, IconButton, Link, Text } from '../../index';
+import { Box, Button, IconButton, Link, Text } from '../../index';
 import { jsx as ___EmotionJSX } from "@emotion/react";
+export var ELEMENT_TYPE = {
+  BUTTON: 'Button',
+  ICON_BUTTON: 'IconButton',
+  TEXT: 'Text',
+  LINK: 'Link',
+  FRAGMENT: 'Fragment'
+};
 var BreadcrumbItem = /*#__PURE__*/forwardRef(function (props, ref) {
   var children = props.children,
     elementType = props.elementType,
@@ -36,15 +43,15 @@ var BreadcrumbItem = /*#__PURE__*/forwardRef(function (props, ref) {
     itemProps = _useBreadcrumbItem.itemProps;
   var ElementType = useMemo(function () {
     switch (elementType) {
-      case 'Button':
+      case ELEMENT_TYPE.BUTTON:
         return Button;
-      case 'IconButton':
+      case ELEMENT_TYPE.ICON_BUTTON:
         return IconButton;
-      case 'Text':
+      case ELEMENT_TYPE.TEXT:
         return Text;
-      case 'Link':
+      case ELEMENT_TYPE.LINK:
         return Link;
-      case 'Fragment':
+      case ELEMENT_TYPE.FRAGMENT:
         return Fragment;
       default:
         return elementType;
@@ -66,9 +73,15 @@ var BreadcrumbItem = /*#__PURE__*/forwardRef(function (props, ref) {
     }
     return omit(elementTypeProps, 'onClick', 'onKeyDown', 'onKeyUp');
   }, [elementType, itemProps, others, onPressHandler]);
-  return /*#__PURE__*/React.cloneElement(___EmotionJSX(ElementType, null), _objectSpread(_objectSpread({}, elementProps), {}, {
-    ref: ref
-  }), children);
+  var elementVariantProps = elementType !== ELEMENT_TYPE.FRAGMENT && _objectSpread({
+    variant: elementType === ELEMENT_TYPE.LINK ? 'variants.breadcrumb.link' : '',
+    ref: itemRef
+  }, elementProps);
+  return ___EmotionJSX(Box, {
+    as: "li",
+    className: isCurrent && 'is-current',
+    variant: "variants.breadcrumb.containerLi"
+  }, ___EmotionJSX(ElementType, elementVariantProps, children));
 });
 BreadcrumbItem.propTypes = {
   actionKey: PropTypes.string,

package/lib/components/Breadcrumbs/Breadcrumbs.js

@@ -31,37 +31,30 @@ var Breadcrumbs = /*#__PURE__*/forwardRef(function (props, ref) {
   });
   var createBreadcrumb = useCallback(function (child, idx) {
     var isCurrentItem = _Array$isArray(filteredChildren) && filteredChildren.length > 1 ? idx === children.length - 1 : true;
-    return ___EmotionJSX(Box, {
-      isRow: true,
-      sx: {
-        alignItems: 'center',
-        minHeight: 42
-      },
-      as: "li",
-      key: "li-".concat(child.key),
-      variant: "variants.breadcrumb.containerLi"
+    return ___EmotionJSX(React.Fragment, {
+      key: "li-".concat(child.key)
     }, ___EmotionJSX(BreadcrumbItem, _extends({
+      actionKey: child.key,
       "data-id": child['data-id'],
       isCurrent: isCurrentItem,
-      onAction: onAction,
-      actionKey: child.key,
-      variant: "variants.breadcrumb.link"
+      onAction: onAction
     }, child.props), child.props.children), icon && !isCurrentItem && ___EmotionJSX(Icon, _extends({
       "aria-hidden": "true",
       icon: icon,
       mx: 5,
-      size: "xs"
+      size: "xs",
+      title: {
+        name: 'Breadcrumb Separator'
+      }
     }, iconProps)));
   }, [children.length, filteredChildren, icon, iconProps, onAction]);
   return ___EmotionJSX("nav", {
     "aria-label": "Breadcrumb"
   }, ___EmotionJSX(Box, _extends({
-    ref: breadcrumbsRef,
+    as: "ol",
     isRow: true,
-    sx: {
-      paddingInlineStart: 'unset'
-    },
-    as: "ol"
+    ref: breadcrumbsRef,
+    variant: "variants.breadcrumb.containerOl"
   }, mergeProps(wrapperProps, others)), _Array$isArray(filteredChildren) ? _mapInstanceProperty(filteredChildren).call(filteredChildren, createBreadcrumb) : createBreadcrumb(children)));
 });
 Breadcrumbs.propTypes = {

package/lib/components/Breadcrumbs/Breadcrumbs.stories.js

@@ -5,7 +5,7 @@ import ChevronRightIcon from '@pingux/mdi-react/ChevronRightIcon';
 import { action } from '@storybook/addon-actions';
 import { withDesign } from 'storybook-addon-designs';
 import DocsLayout from '../../../.storybook/storybookDocsLayout';
-import { Breadcrumbs } from '../../index';
+import { Box, Breadcrumbs } from '../../index';
 import { FIGMA_LINKS } from '../../utils/designUtils/figmaLinks.ts';
 import BreadcrumbsReadme from './Breadcrumbs.mdx';
 import { jsx as ___EmotionJSX } from "@emotion/react";
@@ -35,9 +35,6 @@ export default {
         type: 'none'
       }
     }
-  },
-  args: {
-    icon: ChevronRightIcon
   }
 };
 export var Default = function Default(args) {
@@ -46,23 +43,22 @@ export var Default = function Default(args) {
   };
   return ___EmotionJSX(Breadcrumbs, _extends({
     onAction: onAction
-  }, args), ___EmotionJSX(Item, {
+  }, args, {
+    icon: ChevronRightIcon
+  }), ___EmotionJSX(Item, {
     "aria-label": "home",
     "data-id": "home",
     href: "https://www.pingidentity.com",
-    key: "home",
-    variant: "buttons.link"
+    key: "home"
   }, "Home"), ___EmotionJSX(Item, {
     "aria-label": "trendy",
     "data-id": "trendy",
     href: "https://www.pingidentity.com",
-    key: "trendy",
-    variant: "buttons.link"
+    key: "trendy"
   }, "Trendy"), ___EmotionJSX(Item, {
     "aria-label": "march-2020-assets",
     "data-id": "march",
-    key: "march 2020 assets",
-    variant: "buttons.link"
+    key: "march 2020 assets"
   }, "March 2020 Assets"));
 };
 Default.parameters = {
@@ -70,4 +66,49 @@ Default.parameters = {
     type: 'figma',
     url: FIGMA_LINKS.breadcrumbs["default"]
   }
+};
+export var Overflowing = function Overflowing() {
+  var reallyLongText = 'If This Text Were Really Very Much So Extremely Long';
+  return ___EmotionJSX(Box, {
+    gap: "md",
+    maxWidth: "385px"
+  }, ___EmotionJSX(Breadcrumbs, {
+    icon: ChevronRightIcon
+  }, ___EmotionJSX(Item, {
+    "aria-label": "breadcrumb_1",
+    "data-id": "breadcrumb_1",
+    href: "https://www.pingidentity.com",
+    key: "breadcrumb_1"
+  }, reallyLongText), ___EmotionJSX(Item, {
+    "aria-label": "breadcrumb_2",
+    "data-id": "breadcrumb_2",
+    href: "https://www.pingidentity.com",
+    key: "breadcrumb_2"
+  }, "Edit")), ___EmotionJSX(Breadcrumbs, {
+    icon: ChevronRightIcon
+  }, ___EmotionJSX(Item, {
+    "aria-label": "breadcrumb_1",
+    "data-id": "breadcrumb_1",
+    href: "https://www.pingidentity.com",
+    key: "breadcrumb_1"
+  }, reallyLongText), ___EmotionJSX(Item, {
+    "aria-label": "breadcrumb_1",
+    "data-id": "breadcrumb_2",
+    href: "https://www.pingidentity.com",
+    key: "breadcrumb_2"
+  }, reallyLongText)));
+};
+Overflowing.parameters = {
+  design: {
+    type: 'figma',
+    url: FIGMA_LINKS.breadcrumbs.overflowing
+  },
+  a11y: {
+    config: {
+      rules: [{
+        id: 'landmark-unique',
+        enabled: false
+      }]
+    }
+  }
 };

package/lib/components/Breadcrumbs/Breadcrumbs.test.js

@@ -8,6 +8,7 @@ import CreateIcon from '@pingux/mdi-react/CreateIcon';
 import userEvent from '@testing-library/user-event';
 import { render, screen } from '../../utils/testUtils/testWrapper';
 import { universalComponentTests } from '../../utils/testUtils/universalComponentTest';
+import { ELEMENT_TYPE } from './BreadcrumbItem';
 import Breadcrumbs from './Breadcrumbs';
 import { jsx as ___EmotionJSX } from "@emotion/react";
 var testId = 'test-breadcrumbs';
@@ -57,14 +58,14 @@ test('should render correct amount of icons', function () {
 });
 test('breadcrumbItem should render breadcrumbItem as a Text component when appropriate elementType passed', function () {
   getComponent({}, {
-    elementType: 'Text',
+    elementType: ELEMENT_TYPE.TEXT,
     'data-testid': testItemId
   });
   expect(screen.getAllByTestId(testItemId)[0]).toBeInstanceOf(HTMLSpanElement);
 });
 test('breadcrumbItem should render breadcrumbItem as a IconButton component when appropriate elementType passed', function () {
   getComponent({}, {
-    elementType: 'IconButton',
+    elementType: ELEMENT_TYPE.ICON_BUTTON,
     'data-testid': testItemId,
     icon: CreateIcon
   });
@@ -81,7 +82,7 @@ test('breadcrumbItem should render breadcrumbItem as a html tag when appropriate
 test('breadcrumbs will use onAction if provided', function () {
   var mockOnAction = jest.fn();
   getComponent({}, {
-    elementType: 'Link',
+    elementType: ELEMENT_TYPE.LINK,
     onAction: mockOnAction
   });
   userEvent.click(screen.getByText(testItemsArr[0]));

package/lib/components/Icon/Icon.test.js

@@ -1,8 +1,8 @@
 import _extends from "@babel/runtime-corejs3/helpers/esm/extends";
-import React from 'react';
+import React, { forwardRef } from 'react';
 import Earth from '@pingux/mdi-react/EarthIcon';
 import { render, screen } from '@testing-library/react';
-import axeTest from '../../utils/testUtils/testAxe';
+import { universalComponentTests } from '../../utils/testUtils/universalComponentTest';
 import Icon from '.';
 import { jsx as ___EmotionJSX } from "@emotion/react";
 var testId = 'test-icon';
@@ -19,8 +19,26 @@ var getComponent = function getComponent() {
   return render(___EmotionJSX(Icon, _extends({}, defaultProps, props)));
 };
 
-// Need to be added to each test file to test accessibility using axe.
-axeTest(getComponent);
+// The mdi-react and @pingux/mid-react libraries don't support ref forwarding
+// A simple implementation of an SVG component demonstrates ref forwarding
+var SVGTestComponent = /*#__PURE__*/forwardRef(function (props, ref) {
+  return ___EmotionJSX("svg", _extends({
+    ref: ref
+  }, props, {
+    "aria-labelledby": "id"
+  }), ___EmotionJSX("title", {
+    id: "id"
+  }, "title"));
+});
+
+// Needs to be added to each components test file
+universalComponentTests({
+  renderComponent: function renderComponent(props) {
+    return ___EmotionJSX(Icon, _extends({
+      icon: SVGTestComponent
+    }, props));
+  }
+});
 test('default icon', function () {
   getComponent();
   var icon = screen.getByTestId(testId);

package/lib/utils/designUtils/figmaLinks.js

@@ -23,7 +23,8 @@ export var FIGMA_LINKS = {
     isLast: 'https://www.figma.com/file/bpbEw54gTTcdIiZs4PKK1p/Astro-Specs?node-id=45365%3A7832&mode=dev'
   },
   breadcrumbs: {
-    "default": 'https://www.figma.com/file/bpbEw54gTTcdIiZs4PKK1p/Astro-Specs?type=design&node-id=1162-11924&t=8Wwd3tIBh3GEjCJB-0'
+    "default": 'https://www.figma.com/file/bpbEw54gTTcdIiZs4PKK1p/Astro-Specs?type=design&node-id=1162-11924&t=8Wwd3tIBh3GEjCJB-0',
+    overflowing: 'https://www.figma.com/file/bpbEw54gTTcdIiZs4PKK1p/Astro-Specs?type=design&node-id=45064-7148&mode=design&t=0RGDgLyKBzT6bOXO-0'
   },
   button: {
     "default": 'https://www.figma.com/file/bpbEw54gTTcdIiZs4PKK1p/Astro-Specs?type=design&node-id=0-1&t=8Wwd3tIBh3GEjCJB-0'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pingux/astro",
-  "version": "2.33.0",
+  "version": "2.34.0-alpha.1",
   "description": "React component library for Ping Identity's design system",
   "repository": {
     "type": "git",