@atlaskit/side-navigation 10.1.0 → 11.0.0

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # @atlaskit/side-navigation
 
+## 11.0.0
+
+### Major Changes
+
+- [#157531](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/pull-requests/157531)
+  [`dd1068010784d`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/dd1068010784d) -
+  Migrated `SkeletonItem` and `SkeletonHeadingItem` components from @emotion/react to
+  @compiled/react in order toimprove performance, align with the rest of the Atlaskit techstack, and
+  support React 18 StreamingSSR.
+
+  Removed the `cssFn` prop from the `SkeletonItem` and `SkeletonHeadingItem` components.
+  Customization can now be applied via the new `xcss` prop. This allows for safe/bounded style
+  overrides.
+
+  Removed the following types related to the `cssFn` prop: `CSSFn`, `StatelessCSSFn`, `ItemState`.
+
+### Patch Changes
+
+- Updated dependencies
+
 ## 10.1.0
 
 ### Minor Changes

package/dist/cjs/components/Item/skeleton-item.js

@@ -6,13 +6,9 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 var _extends2 = _interopRequireDefault(require("@babel/runtime/helpers/extends"));
-var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _react = _interopRequireDefault(require("react"));
 var _menu = require("@atlaskit/menu");
-var _styles = require("../../common/styles");
 var _context = require("../NestableNavigationContent/context");
-function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(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 t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 /**
  * __Skeleton item__
  *
@@ -27,19 +23,8 @@ var SkeletonItem = function SkeletonItem(props) {
   if (!shouldRender) {
     return null;
   }
-  return /*#__PURE__*/_react.default.createElement(_menu.SkeletonItem
-  // eslint-disable-next-line @atlaskit/design-system/no-deprecated-apis, @repo/internal/react/no-unsafe-overrides
-  , (0, _extends2.default)({
-    cssFn: function cssFn() {
-      return _objectSpread(_objectSpread({}, (0, _styles.sectionHeaderSpacingStyles)()), {}, {
-        // This doubles up & to get a higher specificity as well as to not overwite the base styles.
-        '&&::before': {
-          height: '1.5rem',
-          marginRight: "var(--ds-space-200, 16px)",
-          width: '1.5rem'
-        }
-      });
-    }
+  return /*#__PURE__*/_react.default.createElement(_menu.SkeletonItem, (0, _extends2.default)({
+    isSideNavSkeleton: true
     // eslint-disable-next-line @repo/internal/react/no-unsafe-spread-props
   }, props));
 };

package/dist/cjs/components/Section/heading-item.js

@@ -11,10 +11,8 @@ require("./heading-item.compiled.css");
 var React = _interopRequireWildcard(require("react"));
 var _runtime = require("@compiled/react/runtime");
 var _extends2 = _interopRequireDefault(require("@babel/runtime/helpers/extends"));
-var _objectWithoutProperties2 = _interopRequireDefault(require("@babel/runtime/helpers/objectWithoutProperties"));
 var _menu = require("@atlaskit/menu");
 var _context = require("../NestableNavigationContent/context");
-var _excluded = ["className"];
 function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(e) { return e ? t : r; })(e); }
 function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != _typeof(e) && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && {}.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
 var styles = {
@@ -27,14 +25,16 @@ var styles = {
  * Available for advanced use cases, for most situations providing a `title` to `section` should be enough.
  *
  */
-var HeadingItem = function HeadingItem(_ref) {
-  var UNSAFE_className = _ref.className,
-    props = (0, _objectWithoutProperties2.default)(_ref, _excluded);
+var HeadingItem = function HeadingItem(props) {
   var _useShouldNestedEleme = (0, _context.useShouldNestedElementRender)(),
     shouldRender = _useShouldNestedEleme.shouldRender;
   if (!shouldRender) {
     return null;
   }
+
+  // @ts-expect-error - Although this isn't defined on props it is available because we've used
+  // spread props below and on the jsx element in MenuHeadingItem.
+  var UNSAFE_className = props.className;
   return /*#__PURE__*/React.createElement(_menu.HeadingItem
   // eslint-disable-next-line @repo/internal/react/no-unsafe-spread-props, @repo/internal/react/no-unsafe-overrides
   , (0, _extends2.default)({}, props, {

package/dist/cjs/components/Section/skeleton-heading-item.compiled.css

@@ -0,0 +1 @@
+._18zru2gc{padding-inline:var(--ds-space-100,8px)}

package/dist/cjs/components/Section/skeleton-heading-item.js

@@ -1,3 +1,4 @@
+/* skeleton-heading-item.tsx generated by @compiled/babel-plugin v0.36.1 */
 "use strict";
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
@@ -5,14 +6,16 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = void 0;
+require("./skeleton-heading-item.compiled.css");
+var _runtime = require("@compiled/react/runtime");
 var _extends2 = _interopRequireDefault(require("@babel/runtime/helpers/extends"));
-var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _react = _interopRequireDefault(require("react"));
 var _menu = require("@atlaskit/menu");
-var _styles = require("../../common/styles");
 var _context = require("../NestableNavigationContent/context");
-function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(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 t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
+var styles = {
+  root: "_18zru2gc"
+};
+
 /**
  * __Skeleton heading item__
  *
@@ -26,12 +29,8 @@ var SkeletonHeadingItem = function SkeletonHeadingItem(props) {
   if (!shouldRender) {
     return null;
   }
-  return /*#__PURE__*/_react.default.createElement(_menu.SkeletonHeadingItem
-  // eslint-disable-next-line @atlaskit/design-system/no-deprecated-apis, @repo/internal/react/no-unsafe-overrides
-  , (0, _extends2.default)({
-    cssFn: function cssFn() {
-      return _objectSpread({}, (0, _styles.sectionHeaderSpacingStyles)());
-    }
+  return /*#__PURE__*/_react.default.createElement(_menu.SkeletonHeadingItem, (0, _extends2.default)({
+    xcss: styles.root
     // eslint-disable-next-line @repo/internal/react/no-unsafe-spread-props
   }, props));
 };

package/dist/es2019/components/Item/skeleton-item.js

@@ -1,7 +1,6 @@
 import _extends from "@babel/runtime/helpers/extends";
 import React from 'react';
 import { SkeletonItem as SkelItem } from '@atlaskit/menu';
-import { sectionHeaderSpacingStyles } from '../../common/styles';
 import { useShouldNestedElementRender } from '../NestableNavigationContent/context';
 /**
  * __Skeleton item__
@@ -18,18 +17,8 @@ const SkeletonItem = props => {
   if (!shouldRender) {
     return null;
   }
-  return /*#__PURE__*/React.createElement(SkelItem
-  // eslint-disable-next-line @atlaskit/design-system/no-deprecated-apis, @repo/internal/react/no-unsafe-overrides
-  , _extends({
-    cssFn: () => ({
-      ...sectionHeaderSpacingStyles(),
-      // This doubles up & to get a higher specificity as well as to not overwite the base styles.
-      '&&::before': {
-        height: '1.5rem',
-        marginRight: "var(--ds-space-200, 16px)",
-        width: '1.5rem'
-      }
-    })
+  return /*#__PURE__*/React.createElement(SkelItem, _extends({
+    isSideNavSkeleton: true
     // eslint-disable-next-line @repo/internal/react/no-unsafe-spread-props
   }, props));
 };

package/dist/es2019/components/Section/heading-item.js

@@ -15,19 +15,17 @@ const styles = {
  * Available for advanced use cases, for most situations providing a `title` to `section` should be enough.
  *
  */
-const HeadingItem = ({
-  // @ts-expect-error
-  // Although this isn't defined on props it is available because we've used
-  // Spread props below and on the jsx element in MenuHeadingItem.
-  className: UNSAFE_className,
-  ...props
-}) => {
+const HeadingItem = props => {
   const {
     shouldRender
   } = useShouldNestedElementRender();
   if (!shouldRender) {
     return null;
   }
+
+  // @ts-expect-error - Although this isn't defined on props it is available because we've used
+  // spread props below and on the jsx element in MenuHeadingItem.
+  const UNSAFE_className = props.className;
   return /*#__PURE__*/React.createElement(MenuHeadingItem
   // eslint-disable-next-line @repo/internal/react/no-unsafe-spread-props, @repo/internal/react/no-unsafe-overrides
   , _extends({}, props, {

package/dist/es2019/components/Section/skeleton-heading-item.compiled.css

@@ -0,0 +1 @@
+._18zru2gc{padding-inline:var(--ds-space-100,8px)}

package/dist/es2019/components/Section/skeleton-heading-item.js

@@ -1,8 +1,14 @@
+/* skeleton-heading-item.tsx generated by @compiled/babel-plugin v0.36.1 */
 import _extends from "@babel/runtime/helpers/extends";
+import "./skeleton-heading-item.compiled.css";
+import { ax, ix } from "@compiled/react/runtime";
 import React from 'react';
 import { SkeletonHeadingItem as MenuSkeletonHeadingItem } from '@atlaskit/menu';
-import { sectionHeaderSpacingStyles } from '../../common/styles';
 import { useShouldNestedElementRender } from '../NestableNavigationContent/context';
+const styles = {
+  root: "_18zru2gc"
+};
+
 /**
  * __Skeleton heading item__
  *
@@ -17,12 +23,8 @@ const SkeletonHeadingItem = props => {
   if (!shouldRender) {
     return null;
   }
-  return /*#__PURE__*/React.createElement(MenuSkeletonHeadingItem
-  // eslint-disable-next-line @atlaskit/design-system/no-deprecated-apis, @repo/internal/react/no-unsafe-overrides
-  , _extends({
-    cssFn: () => ({
-      ...sectionHeaderSpacingStyles()
-    })
+  return /*#__PURE__*/React.createElement(MenuSkeletonHeadingItem, _extends({
+    xcss: styles.root
     // eslint-disable-next-line @repo/internal/react/no-unsafe-spread-props
   }, props));
 };

package/dist/esm/components/Item/skeleton-item.js

@@ -1,10 +1,6 @@
 import _extends from "@babel/runtime/helpers/extends";
-import _defineProperty from "@babel/runtime/helpers/defineProperty";
-function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(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 t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 import React from 'react';
 import { SkeletonItem as SkelItem } from '@atlaskit/menu';
-import { sectionHeaderSpacingStyles } from '../../common/styles';
 import { useShouldNestedElementRender } from '../NestableNavigationContent/context';
 /**
  * __Skeleton item__
@@ -20,19 +16,8 @@ var SkeletonItem = function SkeletonItem(props) {
   if (!shouldRender) {
     return null;
   }
-  return /*#__PURE__*/React.createElement(SkelItem
-  // eslint-disable-next-line @atlaskit/design-system/no-deprecated-apis, @repo/internal/react/no-unsafe-overrides
-  , _extends({
-    cssFn: function cssFn() {
-      return _objectSpread(_objectSpread({}, sectionHeaderSpacingStyles()), {}, {
-        // This doubles up & to get a higher specificity as well as to not overwite the base styles.
-        '&&::before': {
-          height: '1.5rem',
-          marginRight: "var(--ds-space-200, 16px)",
-          width: '1.5rem'
-        }
-      });
-    }
+  return /*#__PURE__*/React.createElement(SkelItem, _extends({
+    isSideNavSkeleton: true
     // eslint-disable-next-line @repo/internal/react/no-unsafe-spread-props
   }, props));
 };

package/dist/esm/components/Section/heading-item.js

@@ -1,7 +1,5 @@
 /* heading-item.tsx generated by @compiled/babel-plugin v0.36.1 */
 import _extends from "@babel/runtime/helpers/extends";
-import _objectWithoutProperties from "@babel/runtime/helpers/objectWithoutProperties";
-var _excluded = ["className"];
 import "./heading-item.compiled.css";
 import * as React from 'react';
 import { ax, ix } from "@compiled/react/runtime";
@@ -17,14 +15,16 @@ var styles = {
  * Available for advanced use cases, for most situations providing a `title` to `section` should be enough.
  *
  */
-var HeadingItem = function HeadingItem(_ref) {
-  var UNSAFE_className = _ref.className,
-    props = _objectWithoutProperties(_ref, _excluded);
+var HeadingItem = function HeadingItem(props) {
   var _useShouldNestedEleme = useShouldNestedElementRender(),
     shouldRender = _useShouldNestedEleme.shouldRender;
   if (!shouldRender) {
     return null;
   }
+
+  // @ts-expect-error - Although this isn't defined on props it is available because we've used
+  // spread props below and on the jsx element in MenuHeadingItem.
+  var UNSAFE_className = props.className;
   return /*#__PURE__*/React.createElement(MenuHeadingItem
   // eslint-disable-next-line @repo/internal/react/no-unsafe-spread-props, @repo/internal/react/no-unsafe-overrides
   , _extends({}, props, {

package/dist/esm/components/Section/skeleton-heading-item.compiled.css

@@ -0,0 +1 @@
+._18zru2gc{padding-inline:var(--ds-space-100,8px)}

package/dist/esm/components/Section/skeleton-heading-item.js

@@ -1,11 +1,14 @@
+/* skeleton-heading-item.tsx generated by @compiled/babel-plugin v0.36.1 */
 import _extends from "@babel/runtime/helpers/extends";
-import _defineProperty from "@babel/runtime/helpers/defineProperty";
-function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(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 t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
+import "./skeleton-heading-item.compiled.css";
+import { ax, ix } from "@compiled/react/runtime";
 import React from 'react';
 import { SkeletonHeadingItem as MenuSkeletonHeadingItem } from '@atlaskit/menu';
-import { sectionHeaderSpacingStyles } from '../../common/styles';
 import { useShouldNestedElementRender } from '../NestableNavigationContent/context';
+var styles = {
+  root: "_18zru2gc"
+};
+
 /**
  * __Skeleton heading item__
  *
@@ -19,12 +22,8 @@ var SkeletonHeadingItem = function SkeletonHeadingItem(props) {
   if (!shouldRender) {
     return null;
   }
-  return /*#__PURE__*/React.createElement(MenuSkeletonHeadingItem
-  // eslint-disable-next-line @atlaskit/design-system/no-deprecated-apis, @repo/internal/react/no-unsafe-overrides
-  , _extends({
-    cssFn: function cssFn() {
-      return _objectSpread({}, sectionHeaderSpacingStyles());
-    }
+  return /*#__PURE__*/React.createElement(MenuSkeletonHeadingItem, _extends({
+    xcss: styles.root
     // eslint-disable-next-line @repo/internal/react/no-unsafe-spread-props
   }, props));
 };

package/dist/types/components/Section/heading-item.d.ts

@@ -11,5 +11,5 @@ export type { HeadingItemProps } from '@atlaskit/menu';
  * Available for advanced use cases, for most situations providing a `title` to `section` should be enough.
  *
  */
-declare const HeadingItem: ({ className: UNSAFE_className, ...props }: HeadingItemProps) => JSX.Element | null;
+declare const HeadingItem: (props: HeadingItemProps) => JSX.Element | null;
 export default HeadingItem;

package/dist/types-ts4.5/components/Section/heading-item.d.ts

@@ -11,5 +11,5 @@ export type { HeadingItemProps } from '@atlaskit/menu';
  * Available for advanced use cases, for most situations providing a `title` to `section` should be enough.
  *
  */
-declare const HeadingItem: ({ className: UNSAFE_className, ...props }: HeadingItemProps) => JSX.Element | null;
+declare const HeadingItem: (props: HeadingItemProps) => JSX.Element | null;
 export default HeadingItem;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/side-navigation",
-	"version": "10.1.0",
+	"version": "11.0.0",
 	"description": "A highly composable side navigation component that supports nested views.",
 	"publishConfig": {
 		"registry": "https://registry.npmjs.org/"
@@ -35,7 +35,7 @@
 		"@atlaskit/css": "^0.10.0",
 		"@atlaskit/ds-lib": "^4.0.0",
 		"@atlaskit/icon": "^26.1.0",
-		"@atlaskit/menu": "^7.1.0",
+		"@atlaskit/menu": "^8.0.0",
 		"@atlaskit/motion": "^5.1.0",
 		"@atlaskit/platform-feature-flags": "^1.1.0",
 		"@atlaskit/primitives": "^14.8.0",

package/dist/cjs/common/styles.js

@@ -1,26 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.sectionHeaderSpacingStyles = exports.overrideStyleFunction = void 0;
-/**
- * Allows chaining of style functions on top of base style functions
- * @param baseStyle the base custom cssFn
- * @param newStyle a new cssFn to be applied after the base style
- *
- * @deprecated Please avoid using this prop as we intend to remove the prop completely in a future release. See DSP-2682 for more information.
- */
-var overrideStyleFunction = exports.overrideStyleFunction = function overrideStyleFunction(baseStyle) {
-  var newStyle = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : function () {
-    return {};
-  };
-  return function (state) {
-    return [baseStyle(state), newStyle(state)];
-  };
-};
-var sectionHeaderSpacingStyles = exports.sectionHeaderSpacingStyles = function sectionHeaderSpacingStyles() {
-  return {
-    paddingInline: "var(--ds-space-100, 8px)"
-  };
-};

package/dist/es2019/common/styles.js

@@ -1,17 +0,0 @@
-/**
- * Allows chaining of style functions on top of base style functions
- * @param baseStyle the base custom cssFn
- * @param newStyle a new cssFn to be applied after the base style
- *
- * @deprecated Please avoid using this prop as we intend to remove the prop completely in a future release. See DSP-2682 for more information.
- */
-export const overrideStyleFunction = (baseStyle, newStyle = () => ({})) => {
-  return state => {
-    return [baseStyle(state), newStyle(state)];
-  };
-};
-export const sectionHeaderSpacingStyles = () => {
-  return {
-    paddingInline: "var(--ds-space-100, 8px)"
-  };
-};

package/dist/esm/common/styles.js

@@ -1,20 +0,0 @@
-/**
- * Allows chaining of style functions on top of base style functions
- * @param baseStyle the base custom cssFn
- * @param newStyle a new cssFn to be applied after the base style
- *
- * @deprecated Please avoid using this prop as we intend to remove the prop completely in a future release. See DSP-2682 for more information.
- */
-export var overrideStyleFunction = function overrideStyleFunction(baseStyle) {
-  var newStyle = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : function () {
-    return {};
-  };
-  return function (state) {
-    return [baseStyle(state), newStyle(state)];
-  };
-};
-export var sectionHeaderSpacingStyles = function sectionHeaderSpacingStyles() {
-  return {
-    paddingInline: "var(--ds-space-100, 8px)"
-  };
-};

package/dist/types/common/styles.d.ts

@@ -1,10 +0,0 @@
-import { type CSSFn, type StatelessCSSFn } from '@atlaskit/menu';
-/**
- * Allows chaining of style functions on top of base style functions
- * @param baseStyle the base custom cssFn
- * @param newStyle a new cssFn to be applied after the base style
- *
- * @deprecated Please avoid using this prop as we intend to remove the prop completely in a future release. See DSP-2682 for more information.
- */
-export declare const overrideStyleFunction: <TState>(baseStyle: CSSFn<TState>, newStyle?: CSSFn<TState> | undefined) => CSSFn<TState>;
-export declare const sectionHeaderSpacingStyles: StatelessCSSFn;

package/dist/types-ts4.5/common/styles.d.ts

@@ -1,10 +0,0 @@
-import { type CSSFn, type StatelessCSSFn } from '@atlaskit/menu';
-/**
- * Allows chaining of style functions on top of base style functions
- * @param baseStyle the base custom cssFn
- * @param newStyle a new cssFn to be applied after the base style
- *
- * @deprecated Please avoid using this prop as we intend to remove the prop completely in a future release. See DSP-2682 for more information.
- */
-export declare const overrideStyleFunction: <TState>(baseStyle: CSSFn<TState>, newStyle?: CSSFn<TState> | undefined) => CSSFn<TState>;
-export declare const sectionHeaderSpacingStyles: StatelessCSSFn;