aplance-ui 1.2.2 → 1.4.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "aplance-ui",
-  "version": "1.2.2",
+  "version": "1.4.0",
   "description": "The aplance-ui CLI - scaffold the design system and copy components into your project as source.",
   "keywords": [
     "react",

package/registry/components/primitives/modal/modal.tsx

@@ -22,6 +22,7 @@ import {
   type ButtonLayout,
 } from "../../ui/button/button";
 import { useFocusOutside } from "../../../hooks/useFocusOutside";
+import { useHasMounted } from "../../../hooks/useHasMounted";
 import { ModalContext, useModal } from "./modal-context";
 import s from "./modal.module.css";
 import { Icon } from "../icon/icon";
@@ -138,14 +139,14 @@ export type ModalProps = {
    */
   placement?: ModalPlacement;
   /**
-   * Placement override below the `--md` breakpoint (900px and below).
+   * Placement override below the `--md` breakpoint (≤ 900px).
    * Defaults to `bottom` so modals become bottom sheets on phones.
    * Pass the same value as `placement` (or any placement) to opt out.
    */
   mobilePlacement?: ModalPlacement;
   /**
    * Entrance/exit animation. Defaults based on `placement`:
-   * center -> scale, top/bottom/left/right -> slide from that edge, full -> fade.
+   * center→scale, top/bottom/left/right→slide from that edge, full→fade.
    */
   animation?: ModalAnimation;
   scrollBehavior?: ModalScrollBehavior;
@@ -201,6 +202,8 @@ export const Modal: FC<ModalProps> = ({
   // Always starts false so the element is in the DOM before the transition fires
   const [isVisible, setIsVisible] = useState(false);
 
+  const mounted = useHasMounted();
+
   const contentRef = useRef<HTMLDivElement>(null);
   const modalId = useId();
 
@@ -243,8 +246,8 @@ export const Modal: FC<ModalProps> = ({
   const close = useCallback(() => setIsOpen(false), [setIsOpen]);
   const toggle = useCallback(() => setIsOpen(!isOpen), [isOpen, setIsOpen]);
 
-  // Open: setShouldRender(true) -> node mounts -> useLayoutEffect below fires -> isVisible=true -> open transition.
-  // Close: setIsVisible(false) -> close transition plays -> setTimeout unmounts after duration.
+  // Open:  setShouldRender(true) → node mounts → useLayoutEffect below fires → isVisible=true → open transition
+  // Close: setIsVisible(false) immediately → close transition plays → setTimeout unmounts after duration
   useEffect(() => {
     if (isOpen) {
       setShouldRender(true);
@@ -263,12 +266,12 @@ export const Modal: FC<ModalProps> = ({
   // registers the closed CSS state before we flip to open - same trick
   // Popover uses with isMeasured + ResizeObserver.
   useLayoutEffect(() => {
-    if (shouldRender && contentRef.current) {
+    if (shouldRender && mounted && contentRef.current) {
       void contentRef.current.offsetHeight; // flush reflow
       setIsVisible(isOpen);
     }
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [shouldRender]); // only run when node mounts, not on every isOpen change
+  }, [shouldRender, mounted]); // only run when node mounts or mounted state changes
 
   useEffect(() => {
     if (isOpen) onOpenRef.current?.();
@@ -328,15 +331,14 @@ export const Modal: FC<ModalProps> = ({
     };
     el.addEventListener("keydown", trap);
     return () => el.removeEventListener("keydown", trap);
-  }, [isOpen]);
+  }, [isOpen, mounted]);
 
   // Close when a click lands outside the dialog panel - mirrors Popover's
   // outside-click pattern so the component family stays consistent.
   // `ignoreSelector` exempts portaled popover menus (CustomSelect, DatePicker,
   // RowActionsPopover, etc.): their menus portal to document.body so they're
   // outside `contentRef`, but a click on a dropdown option must NOT dismiss
-  // the dialog it was opened from.
-  // Backdrop / outside click - gated on being the topmost modal so a click
+  // the dialog it was opened from. Gated on being the topmost modal so a click
   // that dismisses a child modal doesn't bubble into closing its parent.
   useFocusOutside(
     [contentRef],
@@ -392,7 +394,11 @@ export const Modal: FC<ModalProps> = ({
   ) : null;
 
   const portal =
-    renderOnPortal && dialog ? createPortal(dialog, document.body) : dialog;
+    renderOnPortal && dialog
+      ? mounted
+        ? createPortal(dialog, document.body)
+        : null
+      : dialog;
 
   // Hoisted triggers sit outside the portal so they stay visible while closed;
   // dialog content lives inside the portal.

package/registry/components/primitives/popover/popover-context.ts

@@ -1,28 +1,33 @@
-import { createContext, useContext, type RefObject } from "react";
-import type { Coords, Position, Trigger } from "./popover";
-
-export type PopoverContextType = {
-  isOpen: boolean;
-  setIsOpen: (open: boolean) => void;
-  popoverRef: RefObject<HTMLDivElement | null>;
-  menuRef: RefObject<HTMLDivElement | null>;
-  coords: Coords;
-  isMeasured: boolean;
-  shouldRender: boolean;
-  animationDuration?: number;
-  triggerWidth: number;
-  trigger: Trigger;
-  position: Position;
-  hoverTimeoutRef: RefObject<ReturnType<typeof setTimeout> | null>;
-  renderOnPortal: boolean;
-};
-
-export const PopoverContext = createContext<PopoverContextType | undefined>(
-  undefined,
-);
-
-export const usePopover = () => {
-  const ctx = useContext(PopoverContext);
-  if (!ctx) throw new Error("usePopover must be used within Popover");
-  return ctx;
-};
+import { createContext, useContext, type RefObject } from "react";
+import type { Coords, Position, Trigger } from "./popover";
+
+export type PopoverContextType = {
+  isOpen: boolean;
+  setIsOpen: (open: boolean) => void;
+  popoverRef: RefObject<HTMLDivElement | null>;
+  menuRef: RefObject<HTMLDivElement | null>;
+  coords: Coords;
+  isMeasured: boolean;
+  shouldRender: boolean;
+  animationDuration?: number;
+  triggerWidth: number;
+  trigger: Trigger;
+  position: Position;
+  resolvedPosition: Position;
+  availableWidth?: number;
+  availableHeight?: number;
+  autoPosition: boolean;
+  hoverTimeoutRef: RefObject<ReturnType<typeof setTimeout> | null>;
+  renderOnPortal: boolean;
+  mounted: boolean;
+};
+
+export const PopoverContext = createContext<PopoverContextType | undefined>(
+  undefined,
+);
+
+export const usePopover = () => {
+  const ctx = useContext(PopoverContext);
+  if (!ctx) throw new Error("usePopover must be used within Popover");
+  return ctx;
+};

package/registry/components/primitives/popover/popover.tsx

@@ -14,6 +14,7 @@ import { createPortal } from "react-dom";
 import s from "./popover.module.css";
 import { PopoverContext, usePopover } from "./popover-context";
 import { useFocusOutside } from "../../../hooks/useFocusOutside";
+import { useHasMounted } from "../../../hooks/useHasMounted";
 
 // Types
 export type Position = "top" | "bottom" | "left" | "right";
@@ -25,6 +26,24 @@ export type Coords = {
   left: number;
 };
 
+const DEFAULT_COLLISION_PADDING = 8;
+
+const oppositePosition: Record<Position, Position> = {
+  top: "bottom",
+  bottom: "top",
+  left: "right",
+  right: "left",
+};
+
+const useIsomorphicLayoutEffect =
+  typeof window === "undefined" ? useEffect : useLayoutEffect;
+
+const isVerticalPosition = (position: Position): boolean =>
+  position === "top" || position === "bottom";
+
+const clamp = (value: number, min: number, max: number): number =>
+  Math.min(Math.max(value, min), Math.max(min, max));
+
 // Popover Component
 export type PopoverProps = {
   children?: ReactNode;
@@ -43,6 +62,10 @@ export type PopoverProps = {
   trigger?: Trigger;
   // Useful when using in isolated environments like Iframe
   renderOnPortal?: boolean;
+  /** Auto-position the menu so it stays within the viewport. Default: true. */
+  autoPosition?: boolean;
+  /** Viewport padding used by auto positioning. Default: 8. */
+  collisionPadding?: number;
 };
 
 export const Popover: FC<PopoverProps> = ({
@@ -61,6 +84,8 @@ export const Popover: FC<PopoverProps> = ({
   onClose,
   trigger = "click",
   renderOnPortal = true,
+  autoPosition,
+  collisionPadding = DEFAULT_COLLISION_PADDING,
 }) => {
   const readLocationKey = () =>
     `${window.location.pathname}${window.location.search}${window.location.hash}`;
@@ -82,10 +107,19 @@ export const Popover: FC<PopoverProps> = ({
   // For hover trigger
   const hoverTimeoutRef = useRef<ReturnType<typeof setTimeout> | null>(null);
 
+  const mounted = useHasMounted();
+  const shouldAutoPosition = autoPosition ?? true;
+
   const [coords, setCoords] = useState<Coords>({ top: 0, left: 0 });
   const [shouldRender, setShouldRender] = useState(false);
   const [isMeasured, setIsMeasured] = useState(false);
   const [triggerWidth, setTriggerWidth] = useState(0);
+  const [resolvedPosition, setResolvedPosition] =
+    useState<Position>(position);
+  const [availableSize, setAvailableSize] = useState<{
+    width?: number;
+    height?: number;
+  }>({});
 
   // Calculate Menu Position
   const calculatePosition = useCallback(() => {
@@ -94,21 +128,66 @@ export const Popover: FC<PopoverProps> = ({
     const triggerRect = popoverRef.current.getBoundingClientRect();
     const menuRect = menuRef.current.getBoundingClientRect();
     const { innerWidth, innerHeight } = window;
+    const padding = Number.isFinite(collisionPadding)
+      ? Math.max(0, collisionPadding)
+      : DEFAULT_COLLISION_PADDING;
 
     let top = 0;
     let left = 0;
+    let nextPosition = position;
+
+    const availableByPosition: Record<Position, number> = {
+      top: Math.max(0, triggerRect.top - padding - offset),
+      bottom: Math.max(0, innerHeight - triggerRect.bottom - padding - offset),
+      left: Math.max(0, triggerRect.left - padding - offset),
+      right: Math.max(0, innerWidth - triggerRect.right - padding - offset),
+    };
+
+    if (shouldAutoPosition) {
+      const preferredSpace = availableByPosition[position];
+      const opposite = oppositePosition[position];
+      const oppositeSpace = availableByPosition[opposite];
+      const requiredSpace = isVerticalPosition(position)
+        ? menuRect.height
+        : menuRect.width;
+
+      if (preferredSpace < requiredSpace && oppositeSpace > preferredSpace) {
+        nextPosition = opposite;
+      }
+    }
+
+    const viewportWidth = Math.max(0, innerWidth - padding * 2);
+    const viewportHeight = Math.max(0, innerHeight - padding * 2);
+    const availableWidth = shouldAutoPosition
+      ? isVerticalPosition(nextPosition)
+        ? viewportWidth
+        : availableByPosition[nextPosition]
+      : undefined;
+    const availableHeight = shouldAutoPosition
+      ? isVerticalPosition(nextPosition)
+        ? availableByPosition[nextPosition]
+        : viewportHeight
+      : undefined;
+    const measuredWidth =
+      availableWidth === undefined
+        ? menuRect.width
+        : Math.min(menuRect.width, availableWidth);
+    const measuredHeight =
+      availableHeight === undefined
+        ? menuRect.height
+        : Math.min(menuRect.height, availableHeight);
 
     // Main axis positioning (offset applies here)
-    switch (position) {
+    switch (nextPosition) {
       case "top":
-        top = triggerRect.top - menuRect.height - offset;
+        top = triggerRect.top - measuredHeight - offset;
 
         break;
       case "bottom":
         top = triggerRect.bottom + offset;
         break;
       case "left":
-        left = triggerRect.left - menuRect.width - offset;
+        left = triggerRect.left - measuredWidth - offset;
         break;
       case "right":
         left = triggerRect.right + offset;
@@ -116,37 +195,37 @@ export const Popover: FC<PopoverProps> = ({
     }
 
     // Cross axis positioning (align applies here)
-    switch (position) {
+    switch (nextPosition) {
       case "top":
       case "bottom":
         if (align === "start") left = triggerRect.left;
-        else if (align === "end") left = triggerRect.right - menuRect.width;
-        else left = triggerRect.left + (triggerRect.width - menuRect.width) / 2;
+        else if (align === "end") left = triggerRect.right - measuredWidth;
+        else left = triggerRect.left + (triggerRect.width - measuredWidth) / 2;
         break;
       case "left":
       case "right":
         if (align === "start") top = triggerRect.top;
-        else if (align === "end") top = triggerRect.bottom - menuRect.height;
+        else if (align === "end") top = triggerRect.bottom - measuredHeight;
         else
-          top = triggerRect.top + triggerRect.height / 2 - menuRect.height / 2;
+          top = triggerRect.top + triggerRect.height / 2 - measuredHeight / 2;
         break;
     }
 
-    // Collision detection
-    const PADDING = 8;
-    top = Math.max(
-      PADDING,
-      Math.min(top, innerHeight - menuRect.height - PADDING),
-    );
-    left = Math.max(
-      PADDING,
-      Math.min(left, innerWidth - menuRect.width - PADDING),
-    );
+    // Keep the resolved menu position inside the viewport padding.
+    if (shouldAutoPosition) {
+      top = clamp(top, padding, innerHeight - measuredHeight - padding);
+      left = clamp(left, padding, innerWidth - measuredWidth - padding);
+    }
 
     setCoords({ top, left });
     setTriggerWidth(triggerRect.width);
+    setResolvedPosition(nextPosition);
+    setAvailableSize({
+      width: availableWidth,
+      height: availableHeight,
+    });
     setIsMeasured(true);
-  }, [position, align, offset]);
+  }, [position, align, offset, shouldAutoPosition, collisionPadding]);
 
   // Open / Close Handling
   // Visibility (opacity / pointer-events) is derived from `isOpen && isMeasured`
@@ -167,18 +246,23 @@ export const Popover: FC<PopoverProps> = ({
 
   // Measure after menu is rendered. Cleanup resets isMeasured so the next mount
   // re-runs the entry animation from a known starting state.
-  useLayoutEffect(() => {
+  useIsomorphicLayoutEffect(() => {
     if (shouldRender && menuRef.current) {
       calculatePosition();
 
+      if (typeof ResizeObserver === "undefined") {
+        return () => setIsMeasured(false);
+      }
+
       const observer = new ResizeObserver(() => calculatePosition());
       observer.observe(menuRef.current);
+      if (popoverRef.current) observer.observe(popoverRef.current);
       return () => {
         observer.disconnect();
         setIsMeasured(false);
       };
     }
-  }, [shouldRender, calculatePosition]);
+  }, [shouldRender, mounted, calculatePosition]);
 
   // Clear any pending hover-close timer when the Popover unmounts so the
   // delayed setIsOpen never fires on a stale instance.
@@ -265,6 +349,11 @@ export const Popover: FC<PopoverProps> = ({
         hoverTimeoutRef,
         renderOnPortal,
         position,
+        resolvedPosition,
+        availableWidth: availableSize.width,
+        availableHeight: availableSize.height,
+        autoPosition: shouldAutoPosition,
+        mounted,
       }}
     >
       <div
@@ -368,6 +457,11 @@ export type PopoverMenuProps = {
   matchTriggerWidth?: boolean;
 };
 
+type PopoverMenuStyle = CSSProperties & {
+  "--popover-available-width"?: string;
+  "--popover-available-height"?: string;
+};
+
 export const PopoverMenu: FC<PopoverMenuProps> = ({
   children,
   className,
@@ -384,9 +478,13 @@ export const PopoverMenu: FC<PopoverMenuProps> = ({
     animationDuration,
     trigger,
     setIsOpen,
-    position,
+    resolvedPosition,
     hoverTimeoutRef,
     renderOnPortal,
+    mounted,
+    availableWidth,
+    availableHeight,
+    autoPosition,
   } = usePopover();
 
   const handleMouseEnter = () => {
@@ -434,14 +532,34 @@ export const PopoverMenu: FC<PopoverMenuProps> = ({
     return transforms[position];
   };
 
-  const menuStyles: CSSProperties = {
+  const constrainedMaxWidth =
+    autoPosition && availableWidth !== undefined
+      ? matchTriggerWidth && triggerWidth > 0
+        ? Math.min(availableWidth, triggerWidth)
+        : availableWidth
+      : matchTriggerWidth && triggerWidth > 0
+        ? triggerWidth
+        : undefined;
+
+  const menuStyles: PopoverMenuStyle = {
     position: "fixed",
     top: coords.top,
     left: coords.left,
     zIndex: "var(--z-index-popover)",
-    ...(matchTriggerWidth && { maxWidth: triggerWidth }),
+    ...(constrainedMaxWidth !== undefined && { maxWidth: constrainedMaxWidth }),
+    ...(autoPosition &&
+      availableHeight !== undefined && { maxHeight: availableHeight }),
+    ...(autoPosition &&
+      availableWidth !== undefined && {
+        "--popover-available-width": `${availableWidth}px`,
+      }),
+    ...(autoPosition &&
+      availableHeight !== undefined && {
+        "--popover-available-height": `${availableHeight}px`,
+      }),
+    ...(autoPosition && { overflow: "auto" }),
     opacity: isVisible ? 1 : 0,
-    transform: getTransform(position, isMeasured, isOpen),
+    transform: getTransform(resolvedPosition, isMeasured, isOpen),
     transition: animationDuration
       ? `opacity ${animationDuration}ms cubic-bezier(0.4, 0, 0.2, 1), transform ${animationDuration}ms cubic-bezier(0.4, 0, 0.2, 1)`
       : "opacity var(--transition-base), transform var(--transition-base)",
@@ -459,6 +577,7 @@ export const PopoverMenu: FC<PopoverMenuProps> = ({
       // `useFocusOutside({ ignoreSelector })` so clicking a menu option
       // doesn't dismiss the host.
       data-popover-menu=""
+      data-position={resolvedPosition}
       onMouseEnter={handleMouseEnter}
       onMouseLeave={handleMouseLeave}
       style={menuStyles}
@@ -467,5 +586,5 @@ export const PopoverMenu: FC<PopoverMenuProps> = ({
     </div>
   );
 
-  return renderOnPortal ? createPortal(menu, document.body) : menu;
+  return renderOnPortal ? (mounted ? createPortal(menu, document.body) : null) : menu;
 };

package/registry/components/primitives/slider/slider.module.css

@@ -4,11 +4,11 @@
    ─────────────────────────────────────────────────────────────────────────── */
 
 .container {
-  --slider-nav-size: var(--spacing-12);
+  --slider-nav-size: var(--spacing-11);
   --slider-nav-bg: color-mix(in srgb, var(--color-foreground) 56%, transparent);
   --slider-nav-bg-hover: color-mix(in srgb, var(--color-foreground) 78%, transparent);
   --slider-nav-fg: var(--color-background);
-  --slider-dot-size: var(--spacing-6);
+  --slider-dot-size: var(--spacing-5);
   --slider-dot-bg: color-mix(in srgb, var(--color-foreground) 28%, transparent);
   --slider-dot-bg-active: var(--color-primary);
 
@@ -39,6 +39,10 @@
   transition: height var(--transition-base);
 }
 
+.viewport_coverflow {
+  perspective: 72rem;
+}
+
 .track {
   display: flex;
   min-width: 0;
@@ -48,11 +52,21 @@
 }
 
 .track_fade,
-.track_scale {
+.track_scale,
+.track_flip {
   display: grid;
   transform: none;
 }
 
+.track_flip {
+  perspective: 72rem;
+  transform-style: preserve-3d;
+}
+
+.track_coverflow {
+  transform-style: preserve-3d;
+}
+
 .slide {
   min-width: 0;
   flex-grow: 0;
@@ -75,14 +89,45 @@
   }
 }
 
+.slide_cards,
+.slide_coverflow {
+  will-change: transform, opacity;
+  transition:
+    opacity var(--slider-transition-duration) var(--slider-transition-easing),
+    transform var(--slider-transition-duration) var(--slider-transition-easing);
+}
+
+.slide_cards {
+  transform:
+    translateY(var(--slider-slide-offset))
+    scale(var(--slider-slide-scale));
+}
+
+.slide_coverflow {
+  transform:
+    translateY(var(--slider-slide-offset))
+    translateZ(var(--slider-slide-depth))
+    rotateY(var(--slider-slide-rotate))
+    scale(var(--slider-slide-scale));
+  transform-style: preserve-3d;
+}
+
+.container_dragging,
+.container_no_transition {
+  .slide_cards,
+  .slide_coverflow {
+    transition: none;
+  }
+}
+
 .slide_overlay {
   grid-area: 1 / 1;
   width: 100%;
   opacity: 0;
   pointer-events: none;
   transition:
-    opacity var(--transition-base),
-    transform var(--transition-base);
+    opacity var(--slider-transition-duration) var(--slider-transition-easing),
+    transform var(--slider-transition-duration) var(--slider-transition-easing);
 }
 
 .track_fade .slide_overlay {
@@ -93,12 +138,22 @@
   transform: scale(0.96);
 }
 
+.track_flip .slide_overlay {
+  backface-visibility: hidden;
+  transform: rotateY(90deg) scale(0.96);
+  transform-style: preserve-3d;
+}
+
 .slide_active {
   opacity: 1;
   pointer-events: auto;
   transform: scale(1);
 }
 
+.slide_overlay.slide_active {
+  transform: scale(1);
+}
+
 /* ─── Built-in nav arrows ───────────────────────────────────────────────────
    `data-slider-no-drag` is set in TSX so `dragIgnoreSelector` does not start
    a track drag when the user clicks the chevron.
@@ -294,6 +349,8 @@
 @media (--motion-reduce) {
   .track,
   .slide_overlay,
+  .slide_cards,
+  .slide_coverflow,
   .progress_bar {
     transition: none;
   }