@atlaskit/top-layer 1.10.1 → 3.0.0

package/CHANGELOG.md

@@ -1,5 +1,29 @@
 # @atlaskit/top-layer
 
+## 3.0.0
+
+### Major Changes
+
+- [`30abd36d3c8d6`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/30abd36d3c8d6) -
+  Removes the `onOpenChange` prop from Popover. This prop was not being relied on, and created a
+  divergence between the Dialog and Popover APIs.
+
+### Patch Changes
+
+- [`c0874b1b13309`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/c0874b1b13309) -
+  Refactor popover motion styles and rename `getPlacement` to `resolvePlacement`.
+
+## 2.0.0
+
+### Major Changes
+
+- [`3f6f3bd7da074`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/3f6f3bd7da074) -
+  Updates the Top Layer animation API:
+  - Renames `animate` to `shouldAnimate` and simplifies the type to just `boolean`, removing unused
+    config type
+  - Custom animation overrides should now be applied via the `enteringAnimationXcss` and
+    `exitingAnimationXcss` props.
+
 ## 1.10.1
 
 ### Patch Changes

package/README.md

@@ -12,7 +12,6 @@ placement — compose it with the `useAnchorPosition` hook when you need anchor-
 
 ```tsx
 import { useRef } from 'react';
-import { slideAndFade } from '@atlaskit/top-layer/animations';
 import { getAriaForTrigger } from '@atlaskit/top-layer/get-aria-for-trigger';
 import { Popover } from '@atlaskit/top-layer/popover';
 import { PopoverSurface } from '@atlaskit/top-layer/popover-surface';
@@ -45,7 +44,7 @@ function MyPopup({ isOpen, onClose }: { isOpen: boolean; onClose: () => void })
 				role="dialog"
 				label="Settings"
 				isOpen={isOpen}
-				animate={slideAndFade()}
+				shouldAnimate
 				onClose={onClose}
 			>
 				<PopoverSurface>Content here</PopoverSurface>
@@ -87,7 +86,7 @@ Compound component for modal dialogs using the native `<dialog>` element with `.
 
 ## Animations
 
-Both `Popover` and `Dialog` support CSS-based entry/exit animations via the `animate` prop.
+Both `Popover` and `Dialog` support CSS-based entry/exit animations via the `shouldAnimate` prop.
 Animations use [`@starting-style`](https://developer.mozilla.org/en-US/docs/Web/CSS/@starting-style)
 for entry and
 [`allow-discrete`](https://developer.mozilla.org/en-US/docs/Web/CSS/transition-behavior) on
@@ -109,15 +108,12 @@ import {
 
 ### Popover animation
 
-Pass a preset to `Popover`:
+Pass `shouldAnimate` to enable the default popover entry and exit animation:
 
 ```tsx
-import { slideAndFade } from '@atlaskit/top-layer/animations';
 import { Popover } from '@atlaskit/top-layer/popover';
 
-const animation = slideAndFade();
-
-<Popover animate={animation} role="dialog" label="My popover">
+<Popover shouldAnimate role="dialog" label="My popover" isOpen={isOpen}>
 	Content
 </Popover>;
 ```
@@ -125,31 +121,23 @@ const animation = slideAndFade();
 For overlay styling (background, shadow, border-radius), wrap children in `PopoverSurface` from
 `@atlaskit/top-layer/popover-surface`.
 
-Available popover presets:
-
-- `slideAndFade({ distance? })` — directional slide + opacity (default for tooltip)
-- `fade` — simple opacity transition
-- `scaleAndFade` — scale from 0.95 + opacity (suitable for menus/dropdowns)
+Use `enteringAnimationXcss` and `exitingAnimationXcss` when a composed component needs to override
+the phase-specific animation styles.
 
 ### Dialog animation
 
-Pass a preset to `Dialog.Content`:
+Pass `shouldAnimate` to enable the default dialog entry and exit animation:
 
 ```tsx
-import { dialogSlideUpAndFade } from '@atlaskit/top-layer/animations';
-
-const animation = dialogSlideUpAndFade();
+import { Dialog } from '@atlaskit/top-layer/dialog';
 
-<Dialog.Content animate={animation}>
-	<Dialog.Surface>...</Dialog.Surface>
-</Dialog.Content>;
+<Dialog shouldAnimate isOpen={isOpen} onClose={onClose} label="My dialog">
+	Content
+</Dialog>;
 ```
 
-Available dialog presets:
-
-- `dialogSlideUpAndFade({ distance? })` — slide up + opacity with backdrop fade (matches legacy
-  modal entrance)
-- `dialogFade` — simple opacity + backdrop fade
+Use `enteringAnimationXcss` and `exitingAnimationXcss` when a composed component needs to override
+the phase-specific animation styles.
 
 ### Progressive enhancement
 

package/dist/cjs/dialog/dialog-content.compiled.css

@@ -1,30 +1,21 @@
 ._19itglyw{border:none}
-._1iu47kvq[open]{animation:var(--ds-modal-enter,.25s cubic-bezier(.4,0,0,1) ScaleIn95to100)}
 ._1m5e1op4::-ms-backdrop{animation:var(--ds-blanket-exit,.2s cubic-bezier(.6,0,.8,.6) FadeOut100to0)}
 ._1m5e1op4::backdrop{animation:var(--ds-blanket-exit,.2s cubic-bezier(.6,0,.8,.6) FadeOut100to0)}
-._vhgsxbuz[open]::-ms-backdrop{animation:var(--ds-blanket-enter,.25s cubic-bezier(.4,0,0,1) FadeIn0to100)}
-._vhgsxbuz[open]::backdrop{animation:var(--ds-blanket-enter,.25s cubic-bezier(.4,0,0,1) FadeIn0to100)}
+._1m5exbuz::-ms-backdrop{animation:var(--ds-blanket-enter,.25s cubic-bezier(.4,0,0,1) FadeIn0to100)}
+._1m5exbuz::backdrop{animation:var(--ds-blanket-enter,.25s cubic-bezier(.4,0,0,1) FadeIn0to100)}
+._y44v7kvq{animation:var(--ds-modal-enter,.25s cubic-bezier(.4,0,0,1) ScaleIn95to100)}
 ._y44vgeqi{animation:var(--ds-modal-exit,.2s cubic-bezier(.6,0,.8,.6) ScaleOut100to95)}._18u01wug{margin-left:auto}
 ._19pk1wug{margin-top:auto}
-._1fsl1mzy[open]{transition-duration:var(--ds-duration-long,.25s)}
-._1fyf1a5r::-ms-backdrop{transition-behavior:allow-discrete}
-._1fyf1a5r::backdrop{transition-behavior:allow-discrete}
-._1i0neoah[open]{animation-fill-mode:backwards}
 ._1i4q4jg8{overflow-wrap:normal}
-._1jhdq7pw::-ms-backdrop{animation-fill-mode:forwards}
-._1jhdq7pw::backdrop{animation-fill-mode:forwards}
 ._1ltk1i5c::-ms-backdrop{background-color:var(--ds-blanket,#050c1f75)}
 ._1ltk1i5c::backdrop{background-color:var(--ds-blanket,#050c1f75)}
 ._1ltk1j28::-ms-backdrop{background-color:transparent}
 ._1ltk1j28::backdrop{background-color:transparent}
 ._1nmz4jg8{word-break:normal}
-._1o51q7pw{animation-fill-mode:forwards}
+._1o511hrf{animation-fill-mode:both}
 ._1oeci9qs{transition-duration:var(--ds-duration-medium,.2s)}
-._1oil1d08[open]{transition-timing-function:var(--ds-easing-inout-bold,cubic-bezier(.4,0,0,1))}
 ._1p1dglyw{text-transform:none}
 ._1q51ze3t{padding-block-start:var(--ds-space-0,0)}
-._1rsueoah[open]::-ms-backdrop{animation-fill-mode:backwards}
-._1rsueoah[open]::backdrop{animation-fill-mode:backwards}
 ._1s9z1a5r{transition-behavior:allow-discrete}
 ._2hwx1wug{margin-right:auto}
 ._85i5ze3t{padding-block-end:var(--ds-space-0,0)}
@@ -33,13 +24,9 @@
 ._bozgze3t{padding-inline-start:var(--ds-space-0,0)}
 ._c71lglyw{max-height:none}
 ._k8m011k7{transition-property:overlay,display}
-._njwk11k7::-ms-backdrop{-ms-transition-property:overlay,display;transition-property:overlay,display}
-._njwk11k7::backdrop{transition-property:overlay,display}
 ._o5724jg8{white-space:normal}
 ._otyr1wug{margin-bottom:auto}
 ._p12fglyw{max-width:none}
 ._y3gnv2br{text-align:start}
 ._y4tize3t{padding-inline-end:var(--ds-space-0,0)}
-._yhpqi9qs::-ms-backdrop{transition-duration:var(--ds-duration-medium,.2s)}
-._yhpqi9qs::backdrop{transition-duration:var(--ds-duration-medium,.2s)}
-@media (prefers-reduced-motion:reduce){._bl8aglyw{animation-name:none}._156y147i{transition-duration:var(--ds-duration-instant,0ms)}._10aiglyw[open]{animation-name:none}._aeld147i[open]{transition-duration:var(--ds-duration-instant,0ms)}._l57nglyw::-ms-backdrop{animation-name:none}._l57nglyw::backdrop{animation-name:none}._1sbq147i::-ms-backdrop{transition-duration:var(--ds-duration-instant,0ms)}._1sbq147i::backdrop{transition-duration:var(--ds-duration-instant,0ms)}._19awglyw[open]::-ms-backdrop{animation-name:none}._19awglyw[open]::backdrop{animation-name:none}._1i6z147i[open]::-ms-backdrop{transition-duration:var(--ds-duration-instant,0ms)}._1i6z147i[open]::backdrop{transition-duration:var(--ds-duration-instant,0ms)}}
+@media (prefers-reduced-motion:reduce){._bl8aglyw{animation-name:none}._156y147i{transition-duration:var(--ds-duration-instant,0ms)}}

package/dist/cjs/dialog/dialog-content.js

@@ -1,4 +1,4 @@
-/* dialog-content.tsx generated by @compiled/babel-plugin v0.40.0 */
+/* dialog-content.tsx generated by @compiled/babel-plugin v2.0.0 */
 "use strict";
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
@@ -10,6 +10,7 @@ exports.Dialog = void 0;
 require("./dialog-content.compiled.css");
 var _runtime = require("@compiled/react/runtime");
 var _react = _interopRequireWildcard(require("react"));
+var _react2 = require("@compiled/react");
 var _bindEventListener = require("bind-event-listener");
 var _mergeRefs = _interopRequireDefault(require("@atlaskit/ds-lib/merge-refs"));
 var _noop = _interopRequireDefault(require("@atlaskit/ds-lib/noop"));
@@ -18,15 +19,6 @@ var _useAnimatedVisibility = require("../internal/use-animated-visibility");
 var _useFocusWrap = require("../internal/use-focus-wrap");
 var _useSafariEscapeFix2 = require("../internal/use-safari-escape-fix");
 function _interopRequireWildcard(e, t) { if ("function" == typeof WeakMap) var r = new WeakMap(), n = new WeakMap(); return (_interopRequireWildcard = function _interopRequireWildcard(e, t) { if (!t && e && e.__esModule) return e; var o, i, f = { __proto__: null, default: e }; if (null === e || "object" != _typeof(e) && "function" != typeof e) return f; if (o = t ? n : r) { if (o.has(e)) return o.get(e); o.set(e, f); } for (var _t in e) "default" !== _t && {}.hasOwnProperty.call(e, _t) && ((i = (o = Object.defineProperty) && Object.getOwnPropertyDescriptor(e, _t)) && (i.get || i.set) ? o(f, _t, i) : f[_t] = e[_t]); return f; })(e, t); }
-// These animation styles use the non-strict Compiled cssMap because they rely
-// on `@starting-style`, `[open]`, and `allow-discrete`. Keep them in this
-// component so the Compiled transform can statically extract every referenced
-// style.
-var dialogAnimationStyles = {
-  root: "_k8m011k7 _1oeci9qs _1s9z1a5r _1fsl1mzy _1oil1d08",
-  motion: "_y44vgeqi _1o51q7pw _1iu47kvq _1i0neoah"
-};
-
 // Surface reset — see the rationale on `surfaceResetStyles` in `popover/popover.tsx`.
 // Neutralises inherited text-layout properties (e.g. `white-space: nowrap`) that leak
 // into the top-layer surface. Excludes `color`/`font` (theming) and
@@ -39,13 +31,17 @@ var dialogAnimationStyles = {
 var surfaceResetStyles = {
   root: "_o5724jg8 _1nmz4jg8 _1i4q4jg8 _y3gnv2br _azhwidpf _1p1dglyw"
 };
-var styles = {
-  dialog: "_19itglyw _19pk1wug _2hwx1wug _otyr1wug _18u01wug _1q51ze3t _y4tize3t _85i5ze3t _bozgze3t _p12fglyw _c71lglyw _bfhk1j28 _bl8aglyw _156y147i _10aiglyw _aeld147i"
+var dialogStyles = {
+  root: "_19itglyw _19pk1wug _2hwx1wug _otyr1wug _18u01wug _1q51ze3t _y4tize3t _85i5ze3t _bozgze3t _p12fglyw _c71lglyw _bfhk1j28",
+  motion: "_k8m011k7 _1oeci9qs _1s9z1a5r _1o511hrf _bl8aglyw _156y147i",
+  enter: "_y44v7kvq",
+  exit: "_y44vgeqi"
 };
 var backdropStyles = {
   root: "_1ltk1i5c",
   hidden: "_1ltk1j28",
-  motion: "_1m5e1op4 _1jhdq7pw _njwk11k7 _yhpqi9qs _1fyf1a5r _vhgsxbuz _1rsueoah _l57nglyw _1sbq147i _19awglyw _1i6z147i"
+  enter: "_1m5exbuz",
+  exit: "_1m5e1op4"
 };
 
 /**
@@ -71,9 +67,11 @@ var Dialog = exports.Dialog = /*#__PURE__*/(0, _react.forwardRef)(function Dialo
     onClose = _ref.onClose,
     onEnterFinish = _ref.onEnterFinish,
     onExitFinish = _ref.onExitFinish,
-    _ref$animate = _ref.animate,
-    animate = _ref$animate === void 0 ? false : _ref$animate,
+    _ref$shouldAnimate = _ref.shouldAnimate,
+    shouldAnimate = _ref$shouldAnimate === void 0 ? false : _ref$shouldAnimate,
     consumerXcss = _ref.xcss,
+    enteringAnimationXcss = _ref.enteringAnimationXcss,
+    exitingAnimationXcss = _ref.exitingAnimationXcss,
     style = _ref.style,
     testId = _ref.testId,
     providedId = _ref.id,
@@ -87,13 +85,12 @@ var Dialog = exports.Dialog = /*#__PURE__*/(0, _react.forwardRef)(function Dialo
   var _useAnimatedVisibilit = (0, _useAnimatedVisibility.useAnimatedVisibility)({
       isOpen: isOpen,
       animationKind: 'dialog',
-      animate: animate,
+      shouldAnimate: shouldAnimate,
       elementRef: ownRef,
       onEnterFinish: onEnterFinish,
       onExitFinish: onExitFinish
     }),
-    phase = _useAnimatedVisibilit.phase,
-    preset = _useAnimatedVisibilit.preset;
+    phase = _useAnimatedVisibilit.phase;
 
   // True while the host element is mounted (any phase except `closed`).
   // Used as a dep so listener-rebind effects re-attach after a remount.
@@ -198,6 +195,7 @@ var Dialog = exports.Dialog = /*#__PURE__*/(0, _react.forwardRef)(function Dialo
   if (!isVisible) {
     return null;
   }
+  var shouldAnimateBackdrop = !shouldHideBackdrop && shouldAnimate;
   return /*#__PURE__*/_react.default.createElement("dialog", {
     ref: combinedRef,
     id: dialogId
@@ -211,7 +209,11 @@ var Dialog = exports.Dialog = /*#__PURE__*/(0, _react.forwardRef)(function Dialo
     "aria-labelledby": label ? undefined : labelledBy,
     style: style,
     onCancel: handleCancel,
-    "data-testid": testId,
-    className: (0, _runtime.ax)([styles.dialog, surfaceResetStyles.root, shouldHideBackdrop ? backdropStyles.hidden : backdropStyles.root, preset && backdropStyles.motion, preset && dialogAnimationStyles.root, preset && dialogAnimationStyles.motion, consumerXcss])
+    "data-testid": testId
+    // eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop, @atlaskit/ui-styling-standard/local-cx-xcss, @compiled/local-cx-xcss
+    ,
+    className: (0, _runtime.ax)([
+    // Dialog styles
+    dialogStyles.root, surfaceResetStyles.root, shouldAnimate && phase === 'entering' && dialogStyles.enter, shouldAnimate && phase === 'exiting' && dialogStyles.exit, shouldAnimate && dialogStyles.motion, shouldHideBackdrop ? backdropStyles.hidden : backdropStyles.root, shouldAnimateBackdrop && phase === 'entering' && backdropStyles.enter, shouldAnimateBackdrop && phase === 'exiting' && backdropStyles.exit, (0, _react2.cx)(consumerXcss, shouldAnimate && phase === 'entering' && enteringAnimationXcss, shouldAnimate && phase === 'exiting' && exitingAnimationXcss)])
   }, children);
 });

package/dist/cjs/internal/placement-to-position-area.js

@@ -4,7 +4,7 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.placementToPositionArea = placementToPositionArea;
-var _resolvePlacement = require("./resolve-placement");
+var _resolvePlacement2 = require("./resolve-placement");
 /**
  * Resolves placement options to CSS `position-area` syntax.
  *
@@ -24,12 +24,12 @@ var _resolvePlacement = require("./resolve-placement");
  */
 function placementToPositionArea(_ref) {
   var placement = _ref.placement;
-  var _getPlacement = (0, _resolvePlacement.getPlacement)({
+  var _resolvePlacement = (0, _resolvePlacement2.resolvePlacement)({
       placement: placement
     }),
-    axis = _getPlacement.axis,
-    edge = _getPlacement.edge,
-    align = _getPlacement.align;
+    axis = _resolvePlacement.axis,
+    edge = _resolvePlacement.edge,
+    align = _resolvePlacement.align;
   var edgeValue = "".concat(axis, "-").concat(edge);
   if (align === 'center') {
     return edgeValue;

package/dist/cjs/internal/placement-to-try-fallbacks.js

@@ -49,7 +49,7 @@ function placementToTryFallbacks(_ref3) {
    * The browser tries them _in order_ and uses the _first_ one that fits.
    */
 
-  var resolved = (0, _resolvePlacement.getPlacement)({
+  var resolved = (0, _resolvePlacement.resolvePlacement)({
     placement: placement
   });
   var axis = resolved.axis,

package/dist/cjs/internal/resolve-css-length-to-pixels.js

@@ -64,7 +64,7 @@ function resolveCssLengthToPixels(_ref3) {
   }
 
   // Fast path: bare `${n}px` strings come from the consumer normalisation
-  // in `getPlacement`. Avoid a reflow when we can.
+  // in `resolvePlacement`. Avoid a reflow when we can.
   var pxMatch = value.match(/^(-?\d+(?:\.\d+)?)px$/);
   if (pxMatch) {
     return Number(pxMatch[1]);

package/dist/cjs/internal/resolve-placement.js

@@ -3,7 +3,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.getPlacement = getPlacement;
+exports.resolvePlacement = resolvePlacement;
 var _resolveCssLength = require("./resolve-css-length");
 /**
  * Default gap between trigger and popover. Resolves to a design
@@ -19,7 +19,7 @@ var DEFAULT_GAP = "var(--ds-space-100, 8px)";
  */
 
 /**
- * Fully-resolved placement (internal). Use `getPlacement` for partial input.
+ * Fully-resolved placement (internal). Use `resolvePlacement` for partial input.
  * `offset.gap` and `offset.crossAxisShift.value` are always CSS
  * length strings.
  */
@@ -42,7 +42,7 @@ var DEFAULT_GAP = "var(--ds-space-100, 8px)";
  * `offset.gap: token('space.100', '8px')`,
  * `offset.crossAxisShift: { value: '0px', direction: 'forwards' }`.
  */
-function getPlacement(_ref) {
+function resolvePlacement(_ref) {
   var _placement$offset, _placement$axis, _placement$edge, _placement$align, _placement$offset$cro, _placement$offset2, _placement$offset$cro2, _placement$offset3;
   var placement = _ref.placement;
   var consumerGap = (_placement$offset = placement.offset) === null || _placement$offset === void 0 ? void 0 : _placement$offset.gap;

package/dist/cjs/internal/use-anchor-position.js

@@ -33,7 +33,7 @@ var _setStyle = require("./set-style");
  * is never painted at the wrong location.
  */
 
-// `getPlacement` is intentionally NOT re-exported here - import it directly
+// `resolvePlacement` is intentionally NOT re-exported here - import it directly
 // from `./resolve-placement` so search-and-jump lands at the source of truth.
 
 /**
@@ -157,7 +157,7 @@ function crossAxisShiftMargin(_ref2) {
  * (or undefined fields vs explicit defaults) does not produce a fresh result.
  */
 function useStablePlacement(placement) {
-  var resolved = (0, _resolvePlacement.getPlacement)({
+  var resolved = (0, _resolvePlacement.resolvePlacement)({
     placement: placement
   });
   var axis = resolved.axis;

package/dist/cjs/internal/use-animated-visibility.js

@@ -67,13 +67,6 @@ function runOnTransitionEndOrTimeout(_ref) {
   return teardown;
 }
 
-/**
- * Animation config for entry/exit transitions.
- *
- * Currently no configuration is supported via this prop,
- * and this is just a placeholder.
- */
-
 /**
  * Visibility lifecycle phase shared between `useAnimatedVisibility`,
  * `useFocusWrap`, `useInitialFocus`, and the `Popover` / `Dialog`
@@ -138,17 +131,11 @@ function runOnTransitionEndOrTimeout(_ref) {
 function useAnimatedVisibility(_ref2) {
   var isOpen = _ref2.isOpen,
     animationKind = _ref2.animationKind,
-    animate = _ref2.animate,
+    shouldAnimate = _ref2.shouldAnimate,
     elementRef = _ref2.elementRef,
     onEnterFinish = _ref2.onEnterFinish,
     onExitFinish = _ref2.onExitFinish;
-  // Styles are applied via the `css` prop on the host element (see `Popover`
-  // and `Dialog`), so there is no longer any CSS injection step. Normalize a
-  // falsy `animate` to `null`.
-  var preset = animate || null;
-
-  // False when there is no preset or `prefers-reduced-motion` is set.
-  var willAnimate = Boolean(animate) && !(0, _reducedMotion.prefersReducedMotion)();
+  var willAnimate = shouldAnimate && !(0, _reducedMotion.prefersReducedMotion)();
 
   // Promote `closed → entering`/`closed → open` synchronously during
   // render so the host element mounts in the same commit as the open
@@ -317,7 +304,6 @@ function useAnimatedVisibility(_ref2) {
     }
   }, [phase, willAnimate]);
   return {
-    phase: phase,
-    preset: preset
+    phase: phase
   };
 }

package/dist/cjs/popover/popover.compiled.css

@@ -1,32 +1,26 @@
-._15ns179n:popover-open{animation:var(--ds-popover-motion-enter)}
 ._19itglyw{border:none}
 ._1r041wug{inset:auto}
-._y44vof1f{animation:var(--ds-popover-motion-exit)}._12t71ign:popover-open{transition-duration:var(--ds-duration-short,.15s)}
-._18m9ewfl{overflow-y:visible}
+._y44v179n{animation:var(--ds-popover-motion-enter)}
+._y44vof1f{animation:var(--ds-popover-motion-exit)}._18m9ewfl{overflow-y:visible}
 ._18u0idpf{margin-left:0}
 ._19bvidpf{padding-left:0}
 ._19pkidpf{margin-top:0}
 ._1i4q4jg8{overflow-wrap:normal}
 ._1nmz4jg8{word-break:normal}
-._1o51q7pw{animation-fill-mode:forwards}
+._1o511hrf{animation-fill-mode:both}
 ._1oecdn4k{transition-duration:var(--ds-duration-xshort,.1s)}
 ._1p1dglyw{text-transform:none}
-._1pgl1lbj{animation-timing-function:var(--ds-easing-in-practical,cubic-bezier(.6,0,.8,.6))}
-._1psg1ign:popover-open{animation-duration:var(--ds-duration-short,.15s)}
 ._1reoewfl{overflow-x:visible}
 ._1s9z1a5r{transition-behavior:allow-discrete}
 ._2hwxidpf{margin-right:0}
 ._4t3i1wug{height:auto}
-._5sagdn4k{animation-duration:var(--ds-duration-xshort,.1s)}
 ._azhwidpf{text-indent:0}
 ._bfhk1j28{background-color:transparent}
 ._ca0qidpf{padding-top:0}
 ._k8m011k7{transition-property:overlay,display}
 ._n3tdidpf{padding-bottom:0}
-._n70peoah:popover-open{animation-fill-mode:backwards}
 ._o5724jg8{white-space:normal}
-._o8i7indk:popover-open{animation-timing-function:var(--ds-easing-out-practical,cubic-bezier(.4,1,.6,1))}
 ._otyridpf{margin-bottom:0}
 ._u5f3idpf{padding-right:0}
 ._y3gnv2br{text-align:start}
-@media (prefers-reduced-motion:reduce){._bl8aglyw{animation-name:none}._156yru3m{transition-duration:0s}._8rmvglyw:popover-open{animation-name:none}._2qm2ru3m:popover-open{transition-duration:0s}}
+@media (prefers-reduced-motion:reduce){._bl8aglyw{animation-name:none}._156y147i{transition-duration:var(--ds-duration-instant,0ms)}}

package/dist/cjs/popover/popover.js

@@ -1,4 +1,4 @@
-/* popover.tsx generated by @compiled/babel-plugin v0.40.0 */
+/* popover.tsx generated by @compiled/babel-plugin v2.0.0 */
 "use strict";
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
@@ -10,16 +10,16 @@ exports.Popover = void 0;
 require("./popover.compiled.css");
 var _runtime = require("@compiled/react/runtime");
 var _react = _interopRequireWildcard(require("react"));
+var _react2 = require("@compiled/react");
 var _bindEventListener = require("bind-event-listener");
 var _mergeRefs = _interopRequireDefault(require("@atlaskit/ds-lib/merge-refs"));
 var _once = _interopRequireDefault(require("@atlaskit/ds-lib/once"));
 var _useNotifyOpenLayerObserver = require("@atlaskit/layering/use-notify-open-layer-observer");
+var _resolvePlacement2 = require("../internal/resolve-placement");
 var _roleTypes = require("../internal/role-types");
-var _setStyle = require("../internal/set-style");
 var _useAnimatedVisibility = require("../internal/use-animated-visibility");
 var _useFocusWrap = require("../internal/use-focus-wrap");
 var _useInitialFocus = require("../internal/use-initial-focus");
-var _animations = require("./animations");
 function _interopRequireWildcard(e, t) { if ("function" == typeof WeakMap) var r = new WeakMap(), n = new WeakMap(); return (_interopRequireWildcard = function _interopRequireWildcard(e, t) { if (!t && e && e.__esModule) return e; var o, i, f = { __proto__: null, default: e }; if (null === e || "object" != _typeof(e) && "function" != typeof e) return f; if (o = t ? n : r) { if (o.has(e)) return o.get(e); o.set(e, f); } for (var _t in e) "default" !== _t && {}.hasOwnProperty.call(e, _t) && ((i = (o = Object.defineProperty) && Object.getOwnPropertyDescriptor(e, _t)) && (i.get || i.set) ? o(f, _t, i) : f[_t] = e[_t]); return f; })(e, t); }
 /**
  * Detects `popover="hint"` support via DOM reflection. SSR-safe, cached.
@@ -34,14 +34,6 @@ var supportsPopoverHint = (0, _once.default)(function () {
   return element.popover === 'hint';
 });
 
-// These animation styles use the non-strict Compiled cssMap because they rely
-// on `@starting-style`, `:popover-open`, `[dir='rtl']`, and `allow-discrete`.
-// Keep them in this component so the Compiled transform can statically extract
-// every referenced style.
-var popoverAnimationStyles = {
-  root: "_y44vof1f _k8m011k7 _1oecdn4k _1s9z1a5r _1o51q7pw _5sagdn4k _1pgl1lbj _15ns179n _12t71ign _n70peoah _1psg1ign _o8i7indk _bl8aglyw _156yru3m _8rmvglyw _2qm2ru3m"
-};
-
 // Surface reset — neutralises inherited text-layout properties (e.g.
 // `white-space: nowrap` from an `@atlaskit/select` option) that leak into a
 // top-layer element because, for CSS inheritance, it is still a DOM child of its
@@ -57,7 +49,35 @@ var surfaceResetStyles = {
   root: "_o5724jg8 _1nmz4jg8 _1i4q4jg8 _y3gnv2br _azhwidpf _1p1dglyw"
 };
 var styles = {
-  root: "_19itglyw _ca0qidpf _u5f3idpf _n3tdidpf _19bvidpf _19pkidpf _2hwxidpf _otyridpf _18u0idpf _1r041wug _1reoewfl _18m9ewfl _bfhk1j28 _4t3i1wug"
+  root: "_19itglyw _ca0qidpf _u5f3idpf _n3tdidpf _19bvidpf _19pkidpf _2hwxidpf _otyridpf _18u0idpf _1r041wug _1reoewfl _18m9ewfl _bfhk1j28 _4t3i1wug",
+  motion: "_k8m011k7 _1oecdn4k _1s9z1a5r _1o511hrf _bl8aglyw _156y147i",
+  enter: "_y44v179n",
+  exit: "_y44vof1f"
+};
+
+// Intentionally using `Record<string, string>` instead of `React.CSSProperties`
+// to avoid type errors about the CSS variables not being valid property names
+var popupMotionStyles = {
+  block: {
+    start: {
+      '--ds-popover-motion-enter': "var(--ds-popup-enter-top, 150ms cubic-bezier(0.4, 1, 0.6, 1) SlideInTop8px, 150ms cubic-bezier(0.4, 1, 0.6, 1) FadeIn0to100)",
+      '--ds-popover-motion-exit': "var(--ds-popup-exit-top, 100ms cubic-bezier(0.6, 0, 0.8, 0.6) SlideOutTop8px, 100ms cubic-bezier(0.6, 0, 0.8, 0.6) FadeOut100to0)"
+    },
+    end: {
+      '--ds-popover-motion-enter': "var(--ds-popup-enter-bottom, 150ms cubic-bezier(0.4, 1, 0.6, 1) SlideInBottom8px, 150ms cubic-bezier(0.4, 1, 0.6, 1) FadeIn0to100)",
+      '--ds-popover-motion-exit': "var(--ds-popup-exit-bottom, 100ms cubic-bezier(0.6, 0, 0.8, 0.6) SlideOutBottom8px, 100ms cubic-bezier(0.6, 0, 0.8, 0.6) FadeOut100to0)"
+    }
+  },
+  inline: {
+    start: {
+      '--ds-popover-motion-enter': "var(--ds-popup-enter-left, 150ms cubic-bezier(0.4, 1, 0.6, 1) SlideInLeft8px, 150ms cubic-bezier(0.4, 1, 0.6, 1) FadeIn0to100)",
+      '--ds-popover-motion-exit': "var(--ds-popup-exit-left, 100ms cubic-bezier(0.6, 0, 0.8, 0.6) SlideOutLeft8px, 100ms cubic-bezier(0.6, 0, 0.8, 0.6) FadeOut100to0)"
+    },
+    end: {
+      '--ds-popover-motion-enter': "var(--ds-popup-enter-right, 150ms cubic-bezier(0.4, 1, 0.6, 1) SlideInRight8px, 150ms cubic-bezier(0.4, 1, 0.6, 1) FadeIn0to100)",
+      '--ds-popover-motion-exit': "var(--ds-popup-exit-right, 100ms cubic-bezier(0.6, 0, 0.8, 0.6) SlideOutRight8px, 100ms cubic-bezier(0.6, 0, 0.8, 0.6) FadeOut100to0)"
+    }
+  }
 };
 /**
  * Roles registered with the open layer observer as `popup`, so `closeLayers()`
@@ -102,15 +122,15 @@ var Popover = exports.Popover = /*#__PURE__*/(0, _react.forwardRef)(function Pop
     _ref$mode = _ref.mode,
     modeProp = _ref$mode === void 0 ? 'auto' : _ref$mode,
     onClose = _ref.onClose,
-    onOpenChange = _ref.onOpenChange,
     onEnterFinish = _ref.onEnterFinish,
     onExitFinish = _ref.onExitFinish,
-    _ref$animate = _ref.animate,
-    animate = _ref$animate === void 0 ? false : _ref$animate,
+    _ref$shouldAnimate = _ref.shouldAnimate,
+    shouldAnimate = _ref$shouldAnimate === void 0 ? false : _ref$shouldAnimate,
     placement = _ref.placement,
     testId = _ref.testId,
     isOpen = _ref.isOpen,
-    consumerXcss = _ref.xcss,
+    enteringAnimationXcss = _ref.enteringAnimationXcss,
+    exitingAnimationXcss = _ref.exitingAnimationXcss,
     role = _ref.role,
     label = _ref.label,
     labelledBy = _ref.labelledBy,
@@ -123,13 +143,12 @@ var Popover = exports.Popover = /*#__PURE__*/(0, _react.forwardRef)(function Pop
   var _useAnimatedVisibilit = (0, _useAnimatedVisibility.useAnimatedVisibility)({
       isOpen: isOpen,
       animationKind: 'popover',
-      animate: animate,
+      shouldAnimate: shouldAnimate,
       elementRef: ownRef,
       onEnterFinish: onEnterFinish,
       onExitFinish: onExitFinish
     }),
-    phase = _useAnimatedVisibilit.phase,
-    preset = _useAnimatedVisibilit.preset;
+    phase = _useAnimatedVisibilit.phase;
 
   // True while the host element is mounted (any phase except `closed`).
   // Used as a dep so listener-rebind effects re-attach after a remount.
@@ -170,6 +189,17 @@ var Popover = exports.Popover = /*#__PURE__*/(0, _react.forwardRef)(function Pop
   // will close other `auto` popovers when opened. Acceptable trade-off given
   // shrinking browser set without `hint` support.
   var mode = modeProp === 'hint' && !supportsPopoverHint() ? 'auto' : modeProp;
+  var popupMotionStyle = (0, _react.useMemo)(function () {
+    if (!placement || !shouldAnimate) {
+      return undefined;
+    }
+    var _resolvePlacement = (0, _resolvePlacement2.resolvePlacement)({
+        placement: placement
+      }),
+      axis = _resolvePlacement.axis,
+      edge = _resolvePlacement.edge;
+    return popupMotionStyles[axis][edge];
+  }, [placement, shouldAnimate]);
 
   // Prevents toggle handler from calling onClose for our own hidePopover() calls.
   var programmaticCloseRef = (0, _react.useRef)(false);
@@ -191,21 +221,10 @@ var Popover = exports.Popover = /*#__PURE__*/(0, _react.forwardRef)(function Pop
   var handleToggle = (0, _react.useCallback)(function (event) {
     var element = ownRef.current;
     if (event.newState === 'open') {
-      if (element) {
-        onOpenChange === null || onOpenChange === void 0 || onOpenChange({
-          isOpen: true,
-          element: element
-        });
-      }
       return;
     }
     if (event.newState === 'closed') {
       if (element) {
-        onOpenChange === null || onOpenChange === void 0 || onOpenChange({
-          isOpen: false,
-          element: element
-        });
-
         // Nested-popover focus restoration fallback (browser only
         // restores outermost; Firefox skips nested entirely).
         // Restore only when the role moved focus in on open AND the
@@ -242,7 +261,7 @@ var Popover = exports.Popover = /*#__PURE__*/(0, _react.forwardRef)(function Pop
         reason: reason
       });
     }
-  }, [onClose, onOpenChange, role]);
+  }, [onClose, role]);
 
   // Bind via `useLayoutEffect`, and BEFORE the show/hide layout effect
   // below, so listeners are attached when `showPopover()` synchronously
@@ -284,23 +303,6 @@ var Popover = exports.Popover = /*#__PURE__*/(0, _react.forwardRef)(function Pop
     };
   }, [handleToggle, handleBeforeToggle, isVisible]);
 
-  // Placement-dependent animation CSS vars. Kept separate from show/hide
-  // so preset reference changes do not re-trigger visibility. The cleanup
-  // restores the previous inline values so a preset change does not leave
-  // stale custom properties on the element.
-  (0, _react.useLayoutEffect)(function () {
-    var element = ownRef.current;
-    if (!element || !placement) {
-      return;
-    }
-    return (0, _setStyle.setStyle)({
-      element: element,
-      styles: (0, _animations.getPopupMotionStyles)({
-        placement: placement
-      })
-    });
-  }, [preset, placement, isVisible]);
-
   // Show/hide based on isOpen. `showPopover`/`hidePopover` are no-ops when
   // already in the target state. Try/catch guards `InvalidStateError` on
   // disconnected elements (StrictMode/concurrent/unmount edge cases).
@@ -347,7 +349,11 @@ var Popover = exports.Popover = /*#__PURE__*/(0, _react.forwardRef)(function Pop
     role: role,
     "aria-label": label,
     "aria-labelledby": labelledBy,
-    "data-testid": testId,
-    className: (0, _runtime.ax)([styles.root, surfaceResetStyles.root, preset && popoverAnimationStyles.root, consumerXcss])
+    "data-testid": testId
+    // eslint-disable-next-line @atlaskit/ui-styling-standard/enforce-style-prop -- Motion styles are selected at runtime from the placement.
+    ,
+    style: popupMotionStyle,
+    // eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop, @atlaskit/ui-styling-standard/local-cx-xcss, @compiled/local-cx-xcss
+    className: (0, _runtime.ax)([styles.root, surfaceResetStyles.root, shouldAnimate && phase === 'entering' && styles.enter, shouldAnimate && phase === 'exiting' && styles.exit, shouldAnimate && styles.motion, (0, _react2.cx)(shouldAnimate && phase === 'entering' && enteringAnimationXcss, shouldAnimate && phase === 'exiting' && exitingAnimationXcss)])
   }, children);
 });

package/dist/cjs/popover-surface/popover-surface.js

@@ -1,4 +1,4 @@
-/* popover-surface.tsx generated by @compiled/babel-plugin v0.40.0 */
+/* popover-surface.tsx generated by @compiled/babel-plugin v2.0.0 */
 "use strict";
 
 var _typeof = require("@babel/runtime/helpers/typeof");