@atlaskit/navigation-system 5.32.2 → 5.33.0

package/dist/es2019/ui/page-layout/top-nav/top-nav.js

@@ -8,13 +8,11 @@ import { useSkipLink } from '../../../context/skip-links/skip-links-context';
 import { useIsFhsEnabled } from '../../fhs-rollout/use-is-fhs-enabled';
 import { HasCustomThemeContext } from '../../top-nav-items/themed/has-custom-theme-context';
 import { useCustomTheme } from '../../top-nav-items/themed/use-custom-theme';
-import { bannerMountedVar, localSlotLayers, openLayerObserverTopNavEndNamespace, openLayerObserverTopNavMiddleNamespace, openLayerObserverTopNavStartNamespace, sideNavLiveWidthVar, topNavMountedVar, UNSAFE_topNavVar } from '../constants';
+import { bannerMountedVar, localSlotLayers, sideNavLiveWidthVar, topNavMountedVar, UNSAFE_topNavVar } from '../constants';
 import { DangerouslyHoistSlotSizes } from '../hoist-slot-sizes-context';
 import { DangerouslyHoistCssVarToDocumentRoot, HoistCssVarToLocalGrid } from '../hoist-utils';
 import { useLayoutId } from '../id-utils';
 import { useSideNavVisibility } from '../side-nav/use-side-nav-visibility';
-import { useHasOpenLayers } from '../use-open-layer-count';
-
 /**
  * Styles for the container for the top nav items.
  *
@@ -27,8 +25,7 @@ const styles = {
   root: "_nd5l8cbt _zulpu2gc _18zrutpp _179ria51 _1e0c11p5 _yv0e1mfv _4cvr1h6o _bfhkvuon _vchhusvi _4t3i1dgc _152t1nws _kqsw1if8 _1pbyegat _d6vu1bgi _1j8b18ax",
   fullHeightSidebar: "_18zrze3t _179rglyw _bfhkglyw _lcxvglyw _pdlmutpp",
   fullHeightSidebarExpanded: "_1rqt16a9 _jh1g18ax",
-  fullHeightSidebarWithLayeringFixes: "_1pbycs5v _lcxv1wug _bfhkvuon _aetrb3bt _18postnw _1gufidpf _1czdidpf _g0nf3tht _1beue4h9 _uaeunqa1 _1cte1l7x",
-  fullHeightSidebarWithLayeringFixesAndOpenLayer: "_1pby11wp"
+  fullHeightSidebarWithLayeringFixes: "_1pby11wp _lcxv1wug _bfhkvuon _aetrb3bt _18postnw _1gufidpf _1czdidpf _g0nf3tht _1beue4h9 _uaeunqa1 _1cte1l7x"
 };
 
 /**
@@ -41,14 +38,6 @@ const backgroundStyles = {
   sideNavExpanded: "_hyzqcs5v"
 };
 
-/**
- * Namespaces to check for open layers within the top nav.
- * When there is an open layer in the top nav, the top nav is given a higher z-index than the side nav.
- *
- * Placed outside the component for stability, as the list is used as an effect dependency.
- */
-const topNavOpenLayerNamespaces = [openLayerObserverTopNavStartNamespace, openLayerObserverTopNavMiddleNamespace, openLayerObserverTopNavEndNamespace];
-
 /**
  * The top nav layout area. It will display at the top of the screen, below the banner if one is present.
  */
@@ -102,10 +91,6 @@ export function TopNav({
   const {
     isExpandedOnDesktop
   } = useSideNavVisibility();
-  const hasOpenPopup = useHasOpenLayers({
-    namespaces: topNavOpenLayerNamespaces,
-    type: 'popup'
-  });
   return /*#__PURE__*/React.createElement(HasCustomThemeContext.Provider, {
     value: customTheme.isEnabled
   }, isFhsEnabled && !fg('platform-dst-side-nav-layering-fixes') && /*#__PURE__*/React.createElement("div", {
@@ -118,7 +103,7 @@ export function TopNav({
   }), /*#__PURE__*/React.createElement("header", {
     id: id,
     "data-layout-slot": true,
-    className: ax([styles.root, isFhsEnabled && styles.fullHeightSidebar, isFhsEnabled && fg('platform-dst-side-nav-layering-fixes') && styles.fullHeightSidebarWithLayeringFixes, hasOpenPopup && isFhsEnabled && fg('platform-dst-side-nav-layering-fixes') && styles.fullHeightSidebarWithLayeringFixesAndOpenLayer, isExpandedOnDesktop && isFhsEnabled && styles.fullHeightSidebarExpanded, xcss]),
+    className: ax([styles.root, isFhsEnabled && styles.fullHeightSidebar, isFhsEnabled && fg('platform-dst-side-nav-layering-fixes') && styles.fullHeightSidebarWithLayeringFixes, isExpandedOnDesktop && isFhsEnabled && styles.fullHeightSidebarExpanded, xcss]),
     "data-testid": testId
     // eslint-disable-next-line @atlaskit/ui-styling-standard/enforce-style-prop
     ,

package/dist/esm/ui/page-layout/banner.compiled.css

@@ -1,7 +1,7 @@
 
 ._nd5ldkfm{grid-area:banner}._152tidpf{inset-block-start:0}
 ._18m915vq{overflow-y:hidden}
-._1pbycs5v{z-index:2}
+._1pby11wp{z-index:3}
 ._1pbyegat{z-index:4}
 ._1reo15vq{overflow-x:hidden}
 ._4t3iutvi{height:var(--n_bnrM)}

package/dist/esm/ui/page-layout/banner.js

@@ -12,7 +12,7 @@ import { DangerouslyHoistCssVarToDocumentRoot, HoistCssVarToLocalGrid } from './
 import { useLayoutId } from './id-utils';
 var styles = {
   root: "_nd5ldkfm _1reo15vq _18m915vq _4t3iutvi _152tidpf _kqsw1if8 _1pbyegat",
-  fullHeightSidebarWithLayeringFixes: "_1pbycs5v"
+  fullHeightSidebarWithLayeringFixes: "_1pby11wp"
 };
 
 /**

package/dist/esm/ui/page-layout/constants.js

@@ -48,14 +48,15 @@ export var UNSAFE_MAIN_INLINE_END_FOR_LEGACY_PAGES_ONLY = "calc(var(".concat(UNS
  * rely on accessing them through global means.
  */
 export var localSlotLayers = {
+  // The side nav panel splitter is layered above the top nav when FHS and 'platform-dst-side-nav-layering-fixes' is enabled.
+  // It has the same z-index value, but is rendered after the top nav in the DOM so is stacked above.
+  sideNavPanelSplitterFHS: 4,
   topBar: 4,
   banner: 4,
-  // With the FHS layering refactors, the banner and top nav have a lower z-index to allow layers from the side nav to overlay them.
-  // When they all have equal z-index values, the DOM order determines the layering - meaning the side nav will be layered above the rest.
-  // But, when the top bar contains an open layer, it needs to be layered above the side nav, so has a higher value.
-  topNavFHSWithOpenLayer: 3,
-  bannerFHS: 2,
-  topNavFHS: 2,
+  // When FHS and 'platform-dst-side-nav-layering-fixes' is enabled, the side nav is layered below the top nav,
+  // but above the panel
+  bannerFHS: 3,
+  topNavFHS: 3,
   sideNav: 2,
   panelSmallViewports: 1
 };

package/dist/esm/ui/page-layout/panel-splitter/panel-splitter.compiled.css

@@ -2,6 +2,7 @@
 ._12y31o36{outline-width:medium}
 ._152tidpf{inset-block-start:0}
 ._1bsb1l7b{width:3px}
+._1bsb1ris{width:max-content}
 ._1bsbl52n{width:17px}
 ._1e021v6z{inset-inline-start:7px}
 ._1e02zeo2{inset-inline-start:-9px}

package/dist/esm/ui/page-layout/panel-splitter/panel-splitter.js

@@ -48,7 +48,8 @@ var lineStyles = {
   root: "_kqswstnw _1e0c1ule _1bsb1l7b _4t3i1osq _syaz1kw7 _bfhk1r31 _1e021v6z"
 };
 var tooltipStyles = {
-  root: "_ahbq196n"
+  root: "_ahbq196n",
+  fullHeightSidebarWithLayeringFixes: "_1bsb1ris"
 };
 var panelSplitterDragDataSymbol = Symbol('panel-splitter-drag-data');
 function signPanelSplitterDragData(data) {
@@ -86,7 +87,7 @@ var PanelSplitterTooltip = /*#__PURE__*/forwardRef(function (_ref, ref) {
     // Must be statically passed
     // eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop, @atlaskit/design-system/no-unsafe-style-overrides
     ,
-    className: ax([tooltipStyles.root, className])
+    className: ax([tooltipStyles.root, fg('platform-dst-side-nav-layering-fixes') && tooltipStyles.fullHeightSidebarWithLayeringFixes, className])
     // eslint-disable-next-line @atlaskit/design-system/no-unsafe-style-overrides
     ,
 

package/dist/esm/ui/page-layout/panel-splitter/side-nav-panel-splitter.js

@@ -1,10 +1,11 @@
-import React, { useContext } from 'react';
+import _slicedToArray from "@babel/runtime/helpers/slicedToArray";
+import React, { useContext, useEffect, useState } from 'react';
 import invariant from 'tiny-invariant';
+import { useOpenLayerObserver } from '@atlaskit/layering/experimental/open-layer-observer';
 import { fg } from '@atlaskit/platform-feature-flags';
 import { useIsFhsEnabled } from '../../fhs-rollout/use-is-fhs-enabled';
 import { openLayerObserverSideNavNamespace, openLayerObserverTopNavEndNamespace, openLayerObserverTopNavMiddleNamespace, openLayerObserverTopNavStartNamespace, sideNavPanelSplitterId } from '../constants';
 import { useToggleSideNav } from '../side-nav/use-toggle-side-nav';
-import { useHasOpenLayers } from '../use-open-layer-count';
 import { OnDoubleClickContext, PanelSplitterContext } from './context';
 import { PanelSplitter } from './panel-splitter';
 
@@ -14,14 +15,56 @@ import { PanelSplitter } from './panel-splitter';
  * Placed outside the component for stability, as the list is used as an effect dependency.
  */
 var openLayerNamespacesToCheck = [
-// We don't technically need to check the side nav for open layers, as they wouldn't overlay the
-// panel splitter, as it sits within the same stacking context as the side nav. For consistency however,
-// we check it as well.
+// The side nav panel splitter is layered above the side nav, so needs to be disabled when there are open popups in the side nav.
 openLayerObserverSideNavNamespace,
-// When there is an open layer in the top nav, the top nav is given a higher z-index than the side nav.
-// This means the part of the side nav panel splitter that was sitting above the top nav will no longer
-// be interactive (as it is now behind the top nav). So, we need to disable the entire panel splitter.
+// The side nav panel splitter is layered above the top nav, so needs to be disabled when there are open popups in the top nav.
 openLayerObserverTopNavStartNamespace, openLayerObserverTopNavMiddleNamespace, openLayerObserverTopNavEndNamespace];
+
+/**
+ * Returns whether there are any open popups in the side nav or top nav
+ */
+function useHasOpenPopupsInSideNavOrTopNav() {
+  var isFhsEnabled = useIsFhsEnabled();
+  var openLayerObserver = useOpenLayerObserver();
+  // Setting the initial state to false, as it is unlikely that there would be any open popups when the app starts.
+  var _useState = useState(false),
+    _useState2 = _slicedToArray(_useState, 2),
+    hasOpenPopups = _useState2[0],
+    setHasOpenPopups = _useState2[1];
+  useEffect(function () {
+    if (!openLayerObserver || !isFhsEnabled || !fg('platform-dst-side-nav-layering-fixes')) {
+      return;
+    }
+    function updateState() {
+      if (!openLayerObserver) {
+        return;
+      }
+      var hasAnyOpenLayers = openLayerNamespacesToCheck.some(function (namespace) {
+        return openLayerObserver.getCount({
+          namespace: namespace,
+          type: 'popup'
+        }) > 0;
+      });
+      setHasOpenPopups(hasAnyOpenLayers);
+    }
+
+    // Initial check. We do this _in case_ a popup is already open when the component mounts.
+    updateState();
+
+    // Subscribe to each namespace
+    var cleanups = openLayerNamespacesToCheck.map(function (namespace) {
+      return openLayerObserver.onChange(updateState, {
+        namespace: namespace
+      });
+    });
+    return function cleanupHook() {
+      cleanups.forEach(function (cleanup) {
+        return cleanup();
+      });
+    };
+  }, [isFhsEnabled, openLayerObserver]);
+  return hasOpenPopups;
+}
 /**
  * _SideNavPanelSplitter_
  *
@@ -53,11 +96,8 @@ export var SideNavPanelSplitter = function SideNavPanelSplitter(_ref) {
   // The logic and state for disabling the panel splitter when there are open popups
   // in the side nav or top nav is being placed here, instead of in `SideNav`, to prevent
   // re-rendering the side nav anytime the number of open popups changes.
-  var hasOpenLayers = useHasOpenLayers({
-    namespaces: openLayerNamespacesToCheck,
-    type: 'popup'
-  });
-  if (hasOpenLayers && isFhsEnabled && fg('platform-dst-side-nav-layering-fixes')) {
+  var hasOpenLayersInSideNavOrTopNav = useHasOpenPopupsInSideNavOrTopNav();
+  if (hasOpenLayersInSideNavOrTopNav && isFhsEnabled && fg('platform-dst-side-nav-layering-fixes')) {
     return null;
   }
   return /*#__PURE__*/React.createElement(OnDoubleClickContext.Provider, {

package/dist/esm/ui/page-layout/side-nav/side-nav.compiled.css

@@ -9,19 +9,20 @@
 ._1e0c1txw{display:flex}
 ._1e0cglyw{display:none}
 ._1pbycs5v{z-index:2}
+._1pbyegat{z-index:4}
 ._2lx21bp4{flex-direction:column}
 ._4t3i1osq{height:100%}
 ._4t3ieqxy{height:calc(100vh - var(--n_bnrM, 0px) - var(--n_tNvM, 0px))}
 ._bfhk1bhr{background-color:var(--ds-surface-overlay,#fff)}
 ._kqsw1if8{position:sticky}
-._kqswstnw{position:absolute}
-._rjxpidpf{inset-inline-end:0}
+._kqsw1n9t{position:fixed}
+._t9ectl5p{transform:translateX(calc(var(--n_snvRsz, var(--n_snvW, 0px))))}
 ._u7coidpf{inset-block-end:0}
 ._vchhusvi{box-sizing:border-box}
 [dir=rtl] ._65m41mrw{--animation-direction:-1}
 @media (prefers-reduced-motion:no-preference){._10t88iot{transition-property:transform,display}._1vrh1a5r{transition-behavior:allow-discrete}._xrrpfnf5{transition-duration:.2s}._1lh81gzg{grid-area:main}._1xq51mm8{transition-timing-function:cubic-bezier(0,.4,0,1)}@starting-style{._1nu51p9u{transform:translateX(calc(-100%*var(--animation-direction)))}}._1xq51ku9{transition-timing-function:cubic-bezier(.6,0,0,1)}}
 @media (min-width:48rem){._14b5hc79{width:var(--n_snvRsz,var(--n_sNvw))}}
-@media (min-width:64rem){._165t56xv{height:calc(100vh - var(--n_bnrM, 0px))}._180k1wjm{inset-block-start:calc(var(--n_bnrM, 0px))}._26vxoned{padding-block-start:calc(var(--n_tNvM, 0px))}._1mt19dtb{margin-block-start:calc(var(--n_tNvM, 0px)*-1)}._dm2518uv{display:initial}._dm25glyw{display:none}._qiln1gzg{grid-area:main}._p5clglyw{border-inline-end:none}._4ap31bhr{background-color:var(--ds-surface-overlay,#fff)}._scbp130s{box-shadow:var(--ds-shadow-overlay,0 8px 9pt #1e1f2126,0 0 1px #1e1f214f)}._qilnk0mc{grid-area:side-nav}._p5clia51{border-inline-end:var(--ds-border-width,1px) solid var(--ds-border,#0b120e24)}._4ap3vuon{background-color:var(--ds-surface,#fff)}._scbpglyw{box-shadow:none}}
+@media (min-width:64rem){._165t56xv{height:calc(100vh - var(--n_bnrM, 0px))}._180k1wjm{inset-block-start:calc(var(--n_bnrM, 0px))}._26vxoned{padding-block-start:calc(var(--n_tNvM, 0px))}._1mt19dtb{margin-block-start:calc(var(--n_tNvM, 0px)*-1)}._p0az1elq{transform:translateX(calc(var(--n_snvRsz, var(--n_snvW, 0px)) - var(--ds-border-width, 1px)))}._dm2518uv{display:initial}._dm25glyw{display:none}._qiln1gzg{grid-area:main}._p5clglyw{border-inline-end:none}._4ap31bhr{background-color:var(--ds-surface-overlay,#fff)}._scbp130s{box-shadow:var(--ds-shadow-overlay,0 8px 9pt #1e1f2126,0 0 1px #1e1f214f)}._qilnk0mc{grid-area:side-nav}._p5clia51{border-inline-end:var(--ds-border-width,1px) solid var(--ds-border,#0b120e24)}._4ap3vuon{background-color:var(--ds-surface,#fff)}._scbpglyw{box-shadow:none}}
 @media (prefers-reduced-motion:no-preference) and (min-width:64rem){._17ly8iot{transition-property:transform,display}._177m1a5r{transition-behavior:allow-discrete}._1sg81gzg{grid-area:main}._vgub1mm8{transition-timing-function:cubic-bezier(0,.4,0,1)}._hh1u1p9u{transform:translateX(calc(-100%*var(--animation-direction)))}@starting-style{._s2eg1p9u{transform:translateX(calc(-100%*var(--animation-direction)))}}._1sg8k0mc{grid-area:side-nav}._vgub1ku9{transition-timing-function:cubic-bezier(.6,0,0,1)}._hjoifnf5{transition-duration:.2s}}
 @media (prefers-reduced-motion:no-preference) and (not (min-width:64rem)){._aadi1p9u{transform:translateX(calc(-100%*var(--animation-direction)))}}
 @supports not (-moz-appearance:none){@media (prefers-reduced-motion:no-preference){._139f8iot{transition-property:transform,display}._1tpvfnf5{transition-duration:.2s}._sylc1a5r{transition-behavior:allow-discrete}._1uwsjq3t{transform:translateX(-100%)}}@media (prefers-reduced-motion:no-preference){@starting-style{._oyeijq3t{transform:translateX(-100%)}}._139f8iot{transition-property:transform,display}._1tpvfnf5{transition-duration:.2s}._sylc1a5r{transition-behavior:allow-discrete}}}

package/dist/esm/ui/page-layout/side-nav/side-nav.js

@@ -19,7 +19,7 @@ import { media } from '@atlaskit/primitives/responsive';
 import { useSkipLinkInternal } from '../../../context/skip-links/skip-links-context';
 import { TopNavStartElement } from '../../../context/top-nav-start/top-nav-start-context';
 import { useIsFhsEnabled } from '../../fhs-rollout/use-is-fhs-enabled';
-import { bannerMountedVar, contentHeightWhenFixed, contentInsetBlockStart, localSlotLayers, openLayerObserverSideNavNamespace, sideNavLiveWidthVar, sideNavPanelSplitterId, sideNavVar, topNavMountedVar, UNSAFE_sideNavLayoutVar } from '../constants';
+import { bannerMountedVar, contentHeightWhenFixed, contentInsetBlockStart, localSlotLayers, openLayerObserverSideNavNamespace, openLayerObserverTopNavStartNamespace, sideNavLiveWidthVar, sideNavPanelSplitterId, sideNavVar, topNavMountedVar, UNSAFE_sideNavLayoutVar } from '../constants';
 import { DangerouslyHoistSlotSizes } from '../hoist-slot-sizes-context';
 import { DangerouslyHoistCssVarToDocumentRoot } from '../hoist-utils';
 import { useLayoutId } from '../id-utils';
@@ -38,6 +38,9 @@ import { useSideNavVisibilityCallbacks } from './use-side-nav-visibility-callbac
 import { useToggleSideNav } from './use-toggle-side-nav';
 import { SetSideNavVisibilityState, SideNavVisibilityState } from './visibility-context';
 var panelSplitterResizingVar = '--n_snvRsz';
+// Used to share the side nav width with the panel splitter, which is rendered outside the side nav element
+// but positioned to stay at its right edge.
+var sideNavClampedWidthVar = '--n_snvW';
 var widthResizeBounds = {
   min: '240px',
   max: '50vw'
@@ -53,7 +56,7 @@ function getResizeBounds() {
  */
 var isFirefox = typeof navigator !== 'undefined' && navigator.userAgent.toLowerCase().indexOf('firefox') > -1;
 var panelSplitterPortalTargetStyles = {
-  root: "_kqswstnw _rjxpidpf _u7coidpf _4t3i1osq _165t56xv"
+  root: "_kqsw1n9t _1pbyegat _u7coidpf _t9ectl5p _4t3ieqxy _165t56xv _p0az1elq"
 };
 var styles = {
   root: "_nd5l1b6c _191wglyw _t51zglyw _bfhk1bhr _16qs130s _vchhusvi _4t3ieqxy _152timx3 _kqsw1if8 _1bsb1ego _1pbycs5v _14b5hc79 _qilnk0mc _p5clia51 _4ap3vuon _scbpglyw",
@@ -212,7 +215,7 @@ function SideNavInternal(_ref) {
     function open() {
       // Prevent the flyout from being opened if there are any open layers in the top nav start
       if (openLayerObserver && openLayerObserver.getCount({
-        namespace: 'top-nav-start',
+        namespace: openLayerObserverTopNavStartNamespace,
         type: 'popup'
       }) > 0 && fg('platform_dst_nav4_side_nav_resize_tooltip_feedback')) {
         return;
@@ -740,7 +743,7 @@ function SideNavInternal(_ref) {
    * Avoiding nesting the `@supports` at-rule inside of `@media` means Compiled can remove duplicate styles from the generated CSS.
    */
   !isFirefox;
-  return /*#__PURE__*/React.createElement("nav", _extends({
+  return /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement("nav", _extends({
     id: id
   }, devTimeOnlyAttributes, {
     "data-layout-slot": true,
@@ -781,9 +784,11 @@ function SideNavInternal(_ref) {
     shortcut: isShortcutEnabled ? sideNavToggleTooltipKeyboardShortcut : undefined
   }, /*#__PURE__*/React.createElement("div", {
     className: ax([styles.flexContainer])
-  }, children)), isFhsEnabled && fg('platform-dst-side-nav-layering-fixes') && /*#__PURE__*/React.createElement("div", {
+  }, children))), isFhsEnabled && fg('platform-dst-side-nav-layering-fixes') && /*#__PURE__*/React.createElement("div", {
     ref: panelSplitterPortalTargetRef,
-    className: ax([panelSplitterPortalTargetStyles.root])
+    "data-layout-slot": true,
+    style: _defineProperty({}, sideNavClampedWidthVar, clampedWidth),
+    className: ax([panelSplitterPortalTargetStyles.root, isExpandedOnDesktop && !isExpandedOnMobile && !isFlyoutVisible && styles.hiddenMobileOnly, !isExpandedOnDesktop && isExpandedOnMobile && !isFlyoutVisible && styles.hiddenDesktopOnly, !isExpandedOnDesktop && !isExpandedOnMobile && !isFlyoutVisible && styles.hiddenMobileAndDesktop])
   }));
 }
 
@@ -795,21 +800,21 @@ function SideNavInternal(_ref) {
  *
  * You can optionally render a `PanelSplitter` as a child to make the side navigation slot resizable.
  */
-export function SideNav(_ref8) {
-  var children = _ref8.children,
-    defaultCollapsed = _ref8.defaultCollapsed,
-    _ref8$defaultWidth = _ref8.defaultWidth,
-    defaultWidth = _ref8$defaultWidth === void 0 ? 320 : _ref8$defaultWidth,
-    testId = _ref8.testId,
-    label = _ref8.label,
-    _ref8$skipLinkLabel = _ref8.skipLinkLabel,
-    skipLinkLabel = _ref8$skipLinkLabel === void 0 ? label : _ref8$skipLinkLabel,
-    onExpand = _ref8.onExpand,
-    onCollapse = _ref8.onCollapse,
-    onPeekStart = _ref8.onPeekStart,
-    onPeekEnd = _ref8.onPeekEnd,
-    canToggleWithShortcut = _ref8.canToggleWithShortcut,
-    id = _ref8.id;
+export function SideNav(_ref9) {
+  var children = _ref9.children,
+    defaultCollapsed = _ref9.defaultCollapsed,
+    _ref9$defaultWidth = _ref9.defaultWidth,
+    defaultWidth = _ref9$defaultWidth === void 0 ? 320 : _ref9$defaultWidth,
+    testId = _ref9.testId,
+    label = _ref9.label,
+    _ref9$skipLinkLabel = _ref9.skipLinkLabel,
+    skipLinkLabel = _ref9$skipLinkLabel === void 0 ? label : _ref9$skipLinkLabel,
+    onExpand = _ref9.onExpand,
+    onCollapse = _ref9.onCollapse,
+    onPeekStart = _ref9.onPeekStart,
+    onPeekEnd = _ref9.onPeekEnd,
+    canToggleWithShortcut = _ref9.canToggleWithShortcut,
+    id = _ref9.id;
   return /*#__PURE__*/React.createElement(OpenLayerObserverNamespaceProvider, {
     namespace: openLayerObserverSideNavNamespace
   }, /*#__PURE__*/React.createElement(SideNavInternal, {

package/dist/esm/ui/page-layout/top-nav/top-nav.compiled.css

@@ -13,7 +13,6 @@
 ._1e0c11p5{display:grid}
 ._1gufidpf:after{inset-block-end:0}
 ._1pby11wp{z-index:3}
-._1pbycs5v{z-index:2}
 ._1pbyegat{z-index:4}
 ._4cvr1h6o{align-items:center}
 ._4t3i1dgc{height:var(--n_tNvM)}

package/dist/esm/ui/page-layout/top-nav/top-nav.js

@@ -10,13 +10,11 @@ import { useSkipLink } from '../../../context/skip-links/skip-links-context';
 import { useIsFhsEnabled } from '../../fhs-rollout/use-is-fhs-enabled';
 import { HasCustomThemeContext } from '../../top-nav-items/themed/has-custom-theme-context';
 import { useCustomTheme } from '../../top-nav-items/themed/use-custom-theme';
-import { bannerMountedVar, localSlotLayers, openLayerObserverTopNavEndNamespace, openLayerObserverTopNavMiddleNamespace, openLayerObserverTopNavStartNamespace, sideNavLiveWidthVar, topNavMountedVar, UNSAFE_topNavVar } from '../constants';
+import { bannerMountedVar, localSlotLayers, sideNavLiveWidthVar, topNavMountedVar, UNSAFE_topNavVar } from '../constants';
 import { DangerouslyHoistSlotSizes } from '../hoist-slot-sizes-context';
 import { DangerouslyHoistCssVarToDocumentRoot, HoistCssVarToLocalGrid } from '../hoist-utils';
 import { useLayoutId } from '../id-utils';
 import { useSideNavVisibility } from '../side-nav/use-side-nav-visibility';
-import { useHasOpenLayers } from '../use-open-layer-count';
-
 /**
  * Styles for the container for the top nav items.
  *
@@ -29,8 +27,7 @@ var styles = {
   root: "_nd5l8cbt _zulpu2gc _18zrutpp _179ria51 _1e0c11p5 _yv0e1mfv _4cvr1h6o _bfhkvuon _vchhusvi _4t3i1dgc _152t1nws _kqsw1if8 _1pbyegat _d6vu1bgi _1j8b18ax",
   fullHeightSidebar: "_18zrze3t _179rglyw _bfhkglyw _lcxvglyw _pdlmutpp",
   fullHeightSidebarExpanded: "_1rqt16a9 _jh1g18ax",
-  fullHeightSidebarWithLayeringFixes: "_1pbycs5v _lcxv1wug _bfhkvuon _aetrb3bt _18postnw _1gufidpf _1czdidpf _g0nf3tht _1beue4h9 _uaeunqa1 _1cte1l7x",
-  fullHeightSidebarWithLayeringFixesAndOpenLayer: "_1pby11wp"
+  fullHeightSidebarWithLayeringFixes: "_1pby11wp _lcxv1wug _bfhkvuon _aetrb3bt _18postnw _1gufidpf _1czdidpf _g0nf3tht _1beue4h9 _uaeunqa1 _1cte1l7x"
 };
 
 /**
@@ -43,14 +40,6 @@ var backgroundStyles = {
   sideNavExpanded: "_hyzqcs5v"
 };
 
-/**
- * Namespaces to check for open layers within the top nav.
- * When there is an open layer in the top nav, the top nav is given a higher z-index than the side nav.
- *
- * Placed outside the component for stability, as the list is used as an effect dependency.
- */
-var topNavOpenLayerNamespaces = [openLayerObserverTopNavStartNamespace, openLayerObserverTopNavMiddleNamespace, openLayerObserverTopNavEndNamespace];
-
 /**
  * The top nav layout area. It will display at the top of the screen, below the banner if one is present.
  */
@@ -102,10 +91,6 @@ export function TopNav(_ref) {
     foregroundStyle = _useMemo.foregroundStyle;
   var _useSideNavVisibility = useSideNavVisibility(),
     isExpandedOnDesktop = _useSideNavVisibility.isExpandedOnDesktop;
-  var hasOpenPopup = useHasOpenLayers({
-    namespaces: topNavOpenLayerNamespaces,
-    type: 'popup'
-  });
   return /*#__PURE__*/React.createElement(HasCustomThemeContext.Provider, {
     value: customTheme.isEnabled
   }, isFhsEnabled && !fg('platform-dst-side-nav-layering-fixes') && /*#__PURE__*/React.createElement("div", {
@@ -118,7 +103,7 @@ export function TopNav(_ref) {
   }), /*#__PURE__*/React.createElement("header", {
     id: id,
     "data-layout-slot": true,
-    className: ax([styles.root, isFhsEnabled && styles.fullHeightSidebar, isFhsEnabled && fg('platform-dst-side-nav-layering-fixes') && styles.fullHeightSidebarWithLayeringFixes, hasOpenPopup && isFhsEnabled && fg('platform-dst-side-nav-layering-fixes') && styles.fullHeightSidebarWithLayeringFixesAndOpenLayer, isExpandedOnDesktop && isFhsEnabled && styles.fullHeightSidebarExpanded, xcss]),
+    className: ax([styles.root, isFhsEnabled && styles.fullHeightSidebar, isFhsEnabled && fg('platform-dst-side-nav-layering-fixes') && styles.fullHeightSidebarWithLayeringFixes, isExpandedOnDesktop && isFhsEnabled && styles.fullHeightSidebarExpanded, xcss]),
     "data-testid": testId
     // eslint-disable-next-line @atlaskit/ui-styling-standard/enforce-style-prop
     ,

package/dist/types/ui/page-layout/constants.d.ts

@@ -33,9 +33,9 @@ export declare const UNSAFE_MAIN_INLINE_END_FOR_LEGACY_PAGES_ONLY = "calc(var(--
  * rely on accessing them through global means.
  */
 export declare const localSlotLayers: {
+    sideNavPanelSplitterFHS: number;
     topBar: number;
     banner: number;
-    topNavFHSWithOpenLayer: number;
     bannerFHS: number;
     topNavFHS: number;
     sideNav: number;

package/dist/types-ts4.5/ui/page-layout/constants.d.ts

@@ -33,9 +33,9 @@ export declare const UNSAFE_MAIN_INLINE_END_FOR_LEGACY_PAGES_ONLY = "calc(var(--
  * rely on accessing them through global means.
  */
 export declare const localSlotLayers: {
+    sideNavPanelSplitterFHS: number;
     topBar: number;
     banner: number;
-    topNavFHSWithOpenLayer: number;
     bannerFHS: number;
     topNavFHS: number;
     sideNav: number;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/navigation-system",
-	"version": "5.32.2",
+	"version": "5.33.0",
 	"description": "The latest navigation system for Atlassian apps.",
 	"repository": "https://bitbucket.org/atlassian/atlassian-frontend-mirror",
 	"author": "Atlassian Pty Ltd",
@@ -78,7 +78,7 @@
 		"@atlaskit/popup": "^4.13.0",
 		"@atlaskit/pragmatic-drag-and-drop": "^1.7.0",
 		"@atlaskit/primitives": "^18.0.0",
-		"@atlaskit/side-nav-items": "^1.4.0",
+		"@atlaskit/side-nav-items": "^1.5.0",
 		"@atlaskit/tokens": "^11.0.0",
 		"@atlaskit/tooltip": "^20.14.0",
 		"@atlaskit/visually-hidden": "^3.0.0",
@@ -102,7 +102,7 @@
 		"@atlaskit/breadcrumbs": "^15.3.0",
 		"@atlaskit/dropdown-menu": "^16.4.0",
 		"@atlaskit/form": "^15.3.0",
-		"@atlaskit/heading": "^5.2.0",
+		"@atlaskit/heading": "^5.3.0",
 		"@atlaskit/link": "^3.3.0",
 		"@atlaskit/lozenge": "^13.4.0",
 		"@atlaskit/menu": "^8.4.0",
@@ -115,7 +115,7 @@
 		"@atlaskit/skeleton": "^2.1.0",
 		"@atlaskit/textfield": "^8.2.0",
 		"@atlassian/feature-flags-test-utils": "^1.0.0",
-		"@atlassian/gemini": "^1.31.0",
+		"@atlassian/gemini": "^1.32.0",
 		"@atlassian/search-dialog": "^9.13.0",
 		"@atlassian/ssr-tests": "workspace:^",
 		"@atlassian/test-utils": "^1.0.0",

package/dist/cjs/ui/page-layout/use-open-layer-count.js

@@ -1,59 +0,0 @@
-"use strict";
-
-var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.useHasOpenLayers = useHasOpenLayers;
-var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
-var _react = require("react");
-var _openLayerObserver = require("@atlaskit/layering/experimental/open-layer-observer");
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
-var _useIsFhsEnabled = require("../fhs-rollout/use-is-fhs-enabled");
-/**
- * Returns whether there are any open layers across the specified namespaces, optionally filtered by type.
- */
-function useHasOpenLayers(_ref) {
-  var namespaces = _ref.namespaces,
-    type = _ref.type;
-  var isFhsEnabled = (0, _useIsFhsEnabled.useIsFhsEnabled)();
-  var openLayerObserver = (0, _openLayerObserver.useOpenLayerObserver)();
-  // Setting the initial state to false, as it is unlikely that there would be any open layers when the app starts.
-  var _useState = (0, _react.useState)(false),
-    _useState2 = (0, _slicedToArray2.default)(_useState, 2),
-    hasOpenLayers = _useState2[0],
-    setHasOpenLayers = _useState2[1];
-  (0, _react.useEffect)(function () {
-    if (!openLayerObserver || !isFhsEnabled || !(0, _platformFeatureFlags.fg)('platform-dst-side-nav-layering-fixes')) {
-      return;
-    }
-    function updateState() {
-      if (!openLayerObserver) {
-        return;
-      }
-      var hasAnyOpenLayers = namespaces.some(function (namespace) {
-        return openLayerObserver.getCount({
-          namespace: namespace,
-          type: type
-        }) > 0;
-      });
-      setHasOpenLayers(hasAnyOpenLayers);
-    }
-
-    // Initial check
-    updateState();
-
-    // Subscribe to each namespace
-    var cleanups = namespaces.map(function (namespace) {
-      return openLayerObserver.onChange(updateState, {
-        namespace: namespace
-      });
-    });
-    return function cleanupHook() {
-      cleanups.forEach(function (cleanup) {
-        return cleanup();
-      });
-    };
-  }, [isFhsEnabled, openLayerObserver, namespaces, type]);
-  return hasOpenLayers;
-}

package/dist/es2019/ui/page-layout/use-open-layer-count.js

@@ -1,44 +0,0 @@
-import { useEffect, useState } from 'react';
-import { useOpenLayerObserver } from '@atlaskit/layering/experimental/open-layer-observer';
-import { fg } from '@atlaskit/platform-feature-flags';
-import { useIsFhsEnabled } from '../fhs-rollout/use-is-fhs-enabled';
-
-/**
- * Returns whether there are any open layers across the specified namespaces, optionally filtered by type.
- */
-export function useHasOpenLayers({
-  namespaces,
-  type
-}) {
-  const isFhsEnabled = useIsFhsEnabled();
-  const openLayerObserver = useOpenLayerObserver();
-  // Setting the initial state to false, as it is unlikely that there would be any open layers when the app starts.
-  const [hasOpenLayers, setHasOpenLayers] = useState(false);
-  useEffect(() => {
-    if (!openLayerObserver || !isFhsEnabled || !fg('platform-dst-side-nav-layering-fixes')) {
-      return;
-    }
-    function updateState() {
-      if (!openLayerObserver) {
-        return;
-      }
-      const hasAnyOpenLayers = namespaces.some(namespace => openLayerObserver.getCount({
-        namespace,
-        type
-      }) > 0);
-      setHasOpenLayers(hasAnyOpenLayers);
-    }
-
-    // Initial check
-    updateState();
-
-    // Subscribe to each namespace
-    const cleanups = namespaces.map(namespace => openLayerObserver.onChange(updateState, {
-      namespace
-    }));
-    return function cleanupHook() {
-      cleanups.forEach(cleanup => cleanup());
-    };
-  }, [isFhsEnabled, openLayerObserver, namespaces, type]);
-  return hasOpenLayers;
-}

package/dist/esm/ui/page-layout/use-open-layer-count.js

@@ -1,53 +0,0 @@
-import _slicedToArray from "@babel/runtime/helpers/slicedToArray";
-import { useEffect, useState } from 'react';
-import { useOpenLayerObserver } from '@atlaskit/layering/experimental/open-layer-observer';
-import { fg } from '@atlaskit/platform-feature-flags';
-import { useIsFhsEnabled } from '../fhs-rollout/use-is-fhs-enabled';
-
-/**
- * Returns whether there are any open layers across the specified namespaces, optionally filtered by type.
- */
-export function useHasOpenLayers(_ref) {
-  var namespaces = _ref.namespaces,
-    type = _ref.type;
-  var isFhsEnabled = useIsFhsEnabled();
-  var openLayerObserver = useOpenLayerObserver();
-  // Setting the initial state to false, as it is unlikely that there would be any open layers when the app starts.
-  var _useState = useState(false),
-    _useState2 = _slicedToArray(_useState, 2),
-    hasOpenLayers = _useState2[0],
-    setHasOpenLayers = _useState2[1];
-  useEffect(function () {
-    if (!openLayerObserver || !isFhsEnabled || !fg('platform-dst-side-nav-layering-fixes')) {
-      return;
-    }
-    function updateState() {
-      if (!openLayerObserver) {
-        return;
-      }
-      var hasAnyOpenLayers = namespaces.some(function (namespace) {
-        return openLayerObserver.getCount({
-          namespace: namespace,
-          type: type
-        }) > 0;
-      });
-      setHasOpenLayers(hasAnyOpenLayers);
-    }
-
-    // Initial check
-    updateState();
-
-    // Subscribe to each namespace
-    var cleanups = namespaces.map(function (namespace) {
-      return openLayerObserver.onChange(updateState, {
-        namespace: namespace
-      });
-    });
-    return function cleanupHook() {
-      cleanups.forEach(function (cleanup) {
-        return cleanup();
-      });
-    };
-  }, [isFhsEnabled, openLayerObserver, namespaces, type]);
-  return hasOpenLayers;
-}

package/dist/types/ui/page-layout/use-open-layer-count.d.ts

@@ -1,8 +0,0 @@
-import { type LayerType } from '@atlaskit/layering/experimental/open-layer-observer';
-/**
- * Returns whether there are any open layers across the specified namespaces, optionally filtered by type.
- */
-export declare function useHasOpenLayers({ namespaces, type, }: {
-    namespaces: string[];
-    type?: LayerType;
-}): boolean;