@instructure/ui-breadcrumb 10.16.1-snapshot-0 → 10.16.1

package/lib/Breadcrumb/__new-tests__/Breadcrumb.test.js

@@ -1,14 +1,14 @@
 "use strict";
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault").default;
-var _react = _interopRequireDefault(require("react"));
-var _react2 = require("@testing-library/react");
+var _react = require("@testing-library/react");
 var _vitest = require("vitest");
 require("@testing-library/jest-dom");
 var _generateA11yTests = require("@instructure/ui-scripts/lib/test/generateA11yTests");
 var _runAxeCheck = require("@instructure/ui-axe-check/lib/runAxeCheck.js");
 var _index = require("../index");
 var _Breadcrumb8 = _interopRequireDefault(require("../__examples__/Breadcrumb.examples"));
+var _jsxRuntime = require("@emotion/react/jsx-runtime");
 var _Breadcrumb, _Breadcrumb2, _Breadcrumb3, _Breadcrumb4, _Breadcrumb5, _Breadcrumb6, _Breadcrumb7;
 /*
  * The MIT License (MIT)
@@ -51,11 +51,15 @@ describe('<Breadcrumb />', () => {
     consoleErrorMock.mockRestore();
   });
   it('should be accessible', async () => {
-    const _render = (0, _react2.render)(_Breadcrumb || (_Breadcrumb = /*#__PURE__*/_react.default.createElement(_index.Breadcrumb, {
-        label: TEST_LABEL
-      }, /*#__PURE__*/_react.default.createElement(_index.Breadcrumb.Link, {
-        href: TEST_LINK
-      }, TEST_TEXT_01), /*#__PURE__*/_react.default.createElement(_index.Breadcrumb.Link, null, TEST_TEXT_02)))),
+    const _render = (0, _react.render)(_Breadcrumb || (_Breadcrumb = (0, _jsxRuntime.jsxs)(_index.Breadcrumb, {
+        label: TEST_LABEL,
+        children: [(0, _jsxRuntime.jsx)(_index.Breadcrumb.Link, {
+          href: TEST_LINK,
+          children: TEST_TEXT_01
+        }), (0, _jsxRuntime.jsx)(_index.Breadcrumb.Link, {
+          children: TEST_TEXT_02
+        })]
+      }))),
       container = _render.container;
     const axeCheck = await (0, _runAxeCheck.runAxeCheck)(container);
     expect(axeCheck).toBe(true);
@@ -65,37 +69,48 @@ describe('<Breadcrumb />', () => {
     it.each(generatedComponents)('should be accessible with example: $description', async ({
       content
     }) => {
-      const _render2 = (0, _react2.render)(content),
+      const _render2 = (0, _react.render)(content),
         container = _render2.container;
       const axeCheck = await (0, _runAxeCheck.runAxeCheck)(container);
       expect(axeCheck).toBe(true);
     });
   });
   it('should render the label as an aria-label attribute', () => {
-    (0, _react2.render)(_Breadcrumb2 || (_Breadcrumb2 = /*#__PURE__*/_react.default.createElement(_index.Breadcrumb, {
-      label: TEST_LABEL
-    }, /*#__PURE__*/_react.default.createElement(_index.Breadcrumb.Link, null, TEST_TEXT_01))));
-    const label = _react2.screen.getByLabelText(TEST_LABEL);
+    (0, _react.render)(_Breadcrumb2 || (_Breadcrumb2 = (0, _jsxRuntime.jsx)(_index.Breadcrumb, {
+      label: TEST_LABEL,
+      children: (0, _jsxRuntime.jsx)(_index.Breadcrumb.Link, {
+        children: TEST_TEXT_01
+      })
+    })));
+    const label = _react.screen.getByLabelText(TEST_LABEL);
     expect(label).toBeInTheDocument();
     expect(label).toHaveAttribute('aria-label', TEST_LABEL);
   });
   it('should render an icon as a separator', () => {
-    const _render3 = (0, _react2.render)(_Breadcrumb3 || (_Breadcrumb3 = /*#__PURE__*/_react.default.createElement(_index.Breadcrumb, {
-        label: TEST_LABEL
-      }, /*#__PURE__*/_react.default.createElement(_index.Breadcrumb.Link, {
-        href: TEST_LINK
-      }, TEST_TEXT_01), /*#__PURE__*/_react.default.createElement(_index.Breadcrumb.Link, null, TEST_TEXT_02)))),
+    const _render3 = (0, _react.render)(_Breadcrumb3 || (_Breadcrumb3 = (0, _jsxRuntime.jsxs)(_index.Breadcrumb, {
+        label: TEST_LABEL,
+        children: [(0, _jsxRuntime.jsx)(_index.Breadcrumb.Link, {
+          href: TEST_LINK,
+          children: TEST_TEXT_01
+        }), (0, _jsxRuntime.jsx)(_index.Breadcrumb.Link, {
+          children: TEST_TEXT_02
+        })]
+      }))),
       container = _render3.container;
     const icon = container.querySelector('svg');
     expect(icon).toBeInTheDocument();
     expect(icon).toHaveAttribute('aria-hidden', 'true');
   });
   it('should add aria-current="page" to the last element by default', () => {
-    const _render4 = (0, _react2.render)(_Breadcrumb4 || (_Breadcrumb4 = /*#__PURE__*/_react.default.createElement(_index.Breadcrumb, {
-        label: TEST_LABEL
-      }, /*#__PURE__*/_react.default.createElement(_index.Breadcrumb.Link, {
-        href: TEST_LINK
-      }, TEST_TEXT_01), /*#__PURE__*/_react.default.createElement(_index.Breadcrumb.Link, null, TEST_TEXT_02)))),
+    const _render4 = (0, _react.render)(_Breadcrumb4 || (_Breadcrumb4 = (0, _jsxRuntime.jsxs)(_index.Breadcrumb, {
+        label: TEST_LABEL,
+        children: [(0, _jsxRuntime.jsx)(_index.Breadcrumb.Link, {
+          href: TEST_LINK,
+          children: TEST_TEXT_01
+        }), (0, _jsxRuntime.jsx)(_index.Breadcrumb.Link, {
+          children: TEST_TEXT_02
+        })]
+      }))),
       container = _render4.container;
     const links = container.querySelectorAll('[class$="--block-link"]');
     const firstLink = links[0];
@@ -104,12 +119,16 @@ describe('<Breadcrumb />', () => {
     expect(lastLink).toHaveAttribute('aria-current', 'page');
   });
   it('should add aria-current="page" to the element if isCurrent is true', () => {
-    const _render5 = (0, _react2.render)(_Breadcrumb5 || (_Breadcrumb5 = /*#__PURE__*/_react.default.createElement(_index.Breadcrumb, {
-        label: TEST_LABEL
-      }, /*#__PURE__*/_react.default.createElement(_index.Breadcrumb.Link, {
-        isCurrentPage: true,
-        href: TEST_LINK
-      }, TEST_TEXT_01), /*#__PURE__*/_react.default.createElement(_index.Breadcrumb.Link, null, TEST_TEXT_02)))),
+    const _render5 = (0, _react.render)(_Breadcrumb5 || (_Breadcrumb5 = (0, _jsxRuntime.jsxs)(_index.Breadcrumb, {
+        label: TEST_LABEL,
+        children: [(0, _jsxRuntime.jsx)(_index.Breadcrumb.Link, {
+          isCurrentPage: true,
+          href: TEST_LINK,
+          children: TEST_TEXT_01
+        }), (0, _jsxRuntime.jsx)(_index.Breadcrumb.Link, {
+          children: TEST_TEXT_02
+        })]
+      }))),
       container = _render5.container;
     const links = container.querySelectorAll('[class$="--block-link"]');
     const firstLink = links[0];
@@ -117,25 +136,31 @@ describe('<Breadcrumb />', () => {
     expect(firstLink).toHaveAttribute('aria-current', 'page');
     expect(lastLink).not.toHaveAttribute('aria-current', 'page');
   });
-  it('should throw a warning when multiple elements have isCurrent set to true ', () => {
-    (0, _react2.render)(_Breadcrumb6 || (_Breadcrumb6 = /*#__PURE__*/_react.default.createElement(_index.Breadcrumb, {
-      label: TEST_LABEL
-    }, /*#__PURE__*/_react.default.createElement(_index.Breadcrumb.Link, {
-      isCurrentPage: true,
-      href: TEST_LINK
-    }, TEST_TEXT_01), /*#__PURE__*/_react.default.createElement(_index.Breadcrumb.Link, {
-      isCurrentPage: true
-    }, TEST_TEXT_02))));
+  it('should throw a warning when multiple elements have isCurrent set to true', () => {
+    (0, _react.render)(_Breadcrumb6 || (_Breadcrumb6 = (0, _jsxRuntime.jsxs)(_index.Breadcrumb, {
+      label: TEST_LABEL,
+      children: [(0, _jsxRuntime.jsx)(_index.Breadcrumb.Link, {
+        isCurrentPage: true,
+        href: TEST_LINK,
+        children: TEST_TEXT_01
+      }), (0, _jsxRuntime.jsx)(_index.Breadcrumb.Link, {
+        isCurrentPage: true,
+        children: TEST_TEXT_02
+      })]
+    })));
     expect(consoleWarningMock).toHaveBeenCalledWith(expect.stringContaining('Warning: Multiple elements with isCurrentPage=true found. Only one element should be set to current.'));
   });
   it('should not add aria-current="page" to the last element if it set to false', () => {
-    const _render6 = (0, _react2.render)(_Breadcrumb7 || (_Breadcrumb7 = /*#__PURE__*/_react.default.createElement(_index.Breadcrumb, {
-        label: TEST_LABEL
-      }, /*#__PURE__*/_react.default.createElement(_index.Breadcrumb.Link, {
-        href: TEST_LINK
-      }, TEST_TEXT_01), /*#__PURE__*/_react.default.createElement(_index.Breadcrumb.Link, {
-        isCurrentPage: false
-      }, TEST_TEXT_02)))),
+    const _render6 = (0, _react.render)(_Breadcrumb7 || (_Breadcrumb7 = (0, _jsxRuntime.jsxs)(_index.Breadcrumb, {
+        label: TEST_LABEL,
+        children: [(0, _jsxRuntime.jsx)(_index.Breadcrumb.Link, {
+          href: TEST_LINK,
+          children: TEST_TEXT_01
+        }), (0, _jsxRuntime.jsx)(_index.Breadcrumb.Link, {
+          isCurrentPage: false,
+          children: TEST_TEXT_02
+        })]
+      }))),
       container = _render6.container;
     const links = container.querySelectorAll('[class$="--block-link"]');
     const firstLink = links[0];

package/lib/Breadcrumb/index.js

@@ -1,6 +1,5 @@
 "use strict";
 
-var _interopRequireWildcard = require("@babel/runtime/helpers/interopRequireWildcard").default;
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault").default;
 Object.defineProperty(exports, "__esModule", {
   value: true
@@ -13,7 +12,7 @@ Object.defineProperty(exports, "BreadcrumbLink", {
   }
 });
 exports.default = void 0;
-var _react = _interopRequireWildcard(require("react"));
+var _react = require("react");
 var _View = require("@instructure/ui-view/lib/View");
 var _testable = require("@instructure/ui-testable/lib/testable.js");
 var _emotion = require("@instructure/emotion");
@@ -22,6 +21,7 @@ var _BreadcrumbLink = require("./BreadcrumbLink");
 var _styles = _interopRequireDefault(require("./styles"));
 var _theme = _interopRequireDefault(require("./theme"));
 var _props = require("./props");
+var _jsxRuntime = require("@emotion/react/jsx-runtime");
 var _dec, _dec2, _class, _Breadcrumb;
 /*
  * The MIT License (MIT)
@@ -46,7 +46,6 @@ var _dec, _dec2, _class, _Breadcrumb;
  * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
  * SOFTWARE.
  */
-/** @jsx jsx */
 /**
 ---
 category: components
@@ -60,7 +59,7 @@ let Breadcrumb = exports.Breadcrumb = (_dec = (0, _emotion.withStyle)(_styles.de
       this.ref = el;
     };
     this.addAriaCurrent = child => {
-      const updatedChild = /*#__PURE__*/_react.default.cloneElement(child, {
+      const updatedChild = /*#__PURE__*/(0, _react.cloneElement)(child, {
         'aria-current': 'page'
       });
       return updatedChild;
@@ -78,40 +77,43 @@ let Breadcrumb = exports.Breadcrumb = (_dec = (0, _emotion.withStyle)(_styles.de
     const _this$props3 = this.props,
       styles = _this$props3.styles,
       children = _this$props3.children;
-    const numChildren = _react.default.Children.count(children);
+    const numChildren = _react.Children.count(children);
     const inlineStyle = {
       maxWidth: `${Math.floor(100 / numChildren)}%`
     };
     let isAriaCurrentSet = false;
-    return _react.default.Children.map(children, (child, index) => {
+    return _react.Children.map(children, (child, index) => {
       const isLastElement = index === numChildren - 1;
-      if (/*#__PURE__*/_react.default.isValidElement(child)) {
+      if (/*#__PURE__*/(0, _react.isValidElement)(child)) {
         const isCurrentPage = child.props.isCurrentPage || false;
         if (isAriaCurrentSet && isCurrentPage) {}
         if (isCurrentPage) {
           isAriaCurrentSet = true;
         }
       }
-      return (0, _emotion.jsx)("li", {
+      return (0, _jsxRuntime.jsxs)("li", {
         css: styles === null || styles === void 0 ? void 0 : styles.crumb,
-        style: inlineStyle
-      }, !isAriaCurrentSet && isLastElement && child.props.isCurrentPage !== false ? this.addAriaCurrent(child) : child, index < numChildren - 1 && (0, _emotion.jsx)(_IconArrowOpenEndSolid.IconArrowOpenEndSolid, {
-        color: "auto",
-        css: styles === null || styles === void 0 ? void 0 : styles.separator
-      }));
+        style: inlineStyle,
+        children: [!isAriaCurrentSet && isLastElement && child.props.isCurrentPage !== false ? this.addAriaCurrent(child) : child, index < numChildren - 1 && (0, _jsxRuntime.jsx)(_IconArrowOpenEndSolid.IconArrowOpenEndSolid, {
+          color: "auto",
+          css: styles === null || styles === void 0 ? void 0 : styles.separator
+        })]
+      });
     });
   }
   render() {
     const styles = this.props.styles;
-    return (0, _emotion.jsx)(_View.View, {
+    return (0, _jsxRuntime.jsx)(_View.View, {
       role: "navigation",
       as: "div",
       margin: this.props.margin,
       "aria-label": this.props.label,
-      elementRef: this.handleRef
-    }, (0, _emotion.jsx)("ol", {
-      css: styles === null || styles === void 0 ? void 0 : styles.breadcrumb
-    }, this.renderChildren()));
+      elementRef: this.handleRef,
+      children: (0, _jsxRuntime.jsx)("ol", {
+        css: styles === null || styles === void 0 ? void 0 : styles.breadcrumb,
+        children: this.renderChildren()
+      })
+    });
   }
 }, _Breadcrumb.displayName = "Breadcrumb", _Breadcrumb.componentId = 'Breadcrumb', _Breadcrumb.propTypes = _props.propTypes, _Breadcrumb.allowedProps = _props.allowedProps, _Breadcrumb.defaultProps = {
   size: 'medium',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@instructure/ui-breadcrumb",
-  "version": "10.16.1-snapshot-0",
+  "version": "10.16.1",
   "description": "A breadcrumb component",
   "author": "Instructure, Inc. Engineering and Product Design",
   "module": "./es/index.js",
@@ -23,27 +23,27 @@
   },
   "license": "MIT",
   "devDependencies": {
-    "@instructure/ui-axe-check": "10.16.1-snapshot-0",
-    "@instructure/ui-babel-preset": "10.16.1-snapshot-0",
-    "@instructure/ui-scripts": "10.16.1-snapshot-0",
-    "@instructure/ui-themes": "10.16.1-snapshot-0",
+    "@instructure/ui-axe-check": "10.16.1",
+    "@instructure/ui-babel-preset": "10.16.1",
+    "@instructure/ui-scripts": "10.16.1",
+    "@instructure/ui-themes": "10.16.1",
     "@testing-library/jest-dom": "^6.6.3",
     "@testing-library/react": "^16.0.1",
     "vitest": "^2.1.8"
   },
   "dependencies": {
     "@babel/runtime": "^7.26.0",
-    "@instructure/emotion": "10.16.1-snapshot-0",
-    "@instructure/shared-types": "10.16.1-snapshot-0",
-    "@instructure/ui-icons": "10.16.1-snapshot-0",
-    "@instructure/ui-link": "10.16.1-snapshot-0",
-    "@instructure/ui-prop-types": "10.16.1-snapshot-0",
-    "@instructure/ui-react-utils": "10.16.1-snapshot-0",
-    "@instructure/ui-testable": "10.16.1-snapshot-0",
-    "@instructure/ui-tooltip": "10.16.1-snapshot-0",
-    "@instructure/ui-truncate-text": "10.16.1-snapshot-0",
-    "@instructure/ui-utils": "10.16.1-snapshot-0",
-    "@instructure/ui-view": "10.16.1-snapshot-0",
+    "@instructure/emotion": "10.16.1",
+    "@instructure/shared-types": "10.16.1",
+    "@instructure/ui-icons": "10.16.1",
+    "@instructure/ui-link": "10.16.1",
+    "@instructure/ui-prop-types": "10.16.1",
+    "@instructure/ui-react-utils": "10.16.1",
+    "@instructure/ui-testable": "10.16.1",
+    "@instructure/ui-tooltip": "10.16.1",
+    "@instructure/ui-truncate-text": "10.16.1",
+    "@instructure/ui-utils": "10.16.1",
+    "@instructure/ui-view": "10.16.1",
     "prop-types": "^15.8.1"
   },
   "peerDependencies": {

package/src/Breadcrumb/BreadcrumbLink/__new-tests__/BreadcrumbLink.test.tsx

@@ -22,7 +22,6 @@
  * SOFTWARE.
  */
 
-import React from 'react'
 import { fireEvent, render, screen } from '@testing-library/react'
 import { vi } from 'vitest'
 import '@testing-library/jest-dom'

package/src/Breadcrumb/BreadcrumbLink/index.tsx

@@ -22,7 +22,7 @@
  * SOFTWARE.
  */
 
-import React, { Component } from 'react'
+import { Component } from 'react'
 
 import { TruncateText } from '@instructure/ui-truncate-text'
 import { Link } from '@instructure/ui-link'

package/src/Breadcrumb/__new-tests__/Breadcrumb.test.tsx

@@ -22,7 +22,6 @@
  * SOFTWARE.
  */
 
-import React from 'react'
 import { render, screen } from '@testing-library/react'
 import { vi } from 'vitest'
 import type { MockInstance } from 'vitest'
@@ -144,7 +143,7 @@ describe('<Breadcrumb />', () => {
     expect(lastLink).not.toHaveAttribute('aria-current', 'page')
   })
 
-  it('should throw a warning when multiple elements have isCurrent set to true ', () => {
+  it('should throw a warning when multiple elements have isCurrent set to true', () => {
     render(
       <Breadcrumb label={TEST_LABEL}>
         <Breadcrumb.Link isCurrentPage href={TEST_LINK}>

package/src/Breadcrumb/index.tsx

@@ -22,13 +22,12 @@
  * SOFTWARE.
  */
 
-/** @jsx jsx */
-import React, { Component } from 'react'
+import { isValidElement, cloneElement, Children, Component } from 'react'
 
 import { View } from '@instructure/ui-view'
 import { testable } from '@instructure/ui-testable'
 
-import { withStyle, jsx } from '@instructure/emotion'
+import { withStyle } from '@instructure/emotion'
 import { IconArrowOpenEndSolid } from '@instructure/ui-icons'
 import { BreadcrumbLink } from './BreadcrumbLink'
 
@@ -63,7 +62,7 @@ class Breadcrumb extends Component<BreadcrumbProps> {
   }
 
   addAriaCurrent = (child: React.ReactNode) => {
-    const updatedChild = React.cloneElement(
+    const updatedChild = cloneElement(
       child as React.ReactElement<{ 'aria-current'?: string }>,
       {
         'aria-current': 'page'
@@ -83,15 +82,15 @@ class Breadcrumb extends Component<BreadcrumbProps> {
 
   renderChildren() {
     const { styles, children } = this.props
-    const numChildren = React.Children.count(children)
+    const numChildren = Children.count(children)
     const inlineStyle = {
       maxWidth: `${Math.floor(100 / numChildren)}%`
     }
     let isAriaCurrentSet = false
 
-    return React.Children.map(children, (child, index) => {
+    return Children.map(children, (child, index) => {
       const isLastElement = index === numChildren - 1
-      if (React.isValidElement(child)) {
+      if (isValidElement(child)) {
         const isCurrentPage = child.props.isCurrentPage || false
         if (isAriaCurrentSet && isCurrentPage) {
           console.warn(