aplance-ui 1.3.0 → 1.4.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "aplance-ui",
-  "version": "1.3.0",
+  "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/popover/popover-context.ts

@@ -1,29 +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;
-  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;
-};
+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

@@ -26,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;
@@ -44,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> = ({
@@ -62,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}`;
@@ -84,11 +108,18 @@ export const Popover: FC<PopoverProps> = ({
   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(() => {
@@ -97,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;
@@ -119,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`
@@ -170,12 +246,17 @@ 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);
@@ -268,6 +349,10 @@ export const Popover: FC<PopoverProps> = ({
         hoverTimeoutRef,
         renderOnPortal,
         position,
+        resolvedPosition,
+        availableWidth: availableSize.width,
+        availableHeight: availableSize.height,
+        autoPosition: shouldAutoPosition,
         mounted,
       }}
     >
@@ -372,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,
@@ -388,10 +478,13 @@ export const PopoverMenu: FC<PopoverMenuProps> = ({
     animationDuration,
     trigger,
     setIsOpen,
-    position,
+    resolvedPosition,
     hoverTimeoutRef,
     renderOnPortal,
     mounted,
+    availableWidth,
+    availableHeight,
+    autoPosition,
   } = usePopover();
 
   const handleMouseEnter = () => {
@@ -439,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)",
@@ -464,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}

package/registry/components/ui/copy-button/copy-button.tsx

@@ -18,6 +18,9 @@ export type CopyButtonProps = {
   layout?: ButtonLayout;
   preview?: CopyPreview;
   previewMaxLength?: number;
+  /** Called with the value after a successful copy (e.g. to toast, or clear a
+   *  one-time reveal). Not called if the copy fails. */
+  onCopy?: (value: string) => void;
 };
 
 const maskToken = (raw: string): string => {
@@ -44,6 +47,7 @@ export function CopyButton({
   layout,
   preview = "none",
   previewMaxLength = 24,
+  onCopy,
 }: CopyButtonProps) {
   const [copied, setCopied] = useState(false);
   const timeoutRef = useRef<number | null>(null);
@@ -82,6 +86,7 @@ export function CopyButton({
     const success = await copyWithFallback(value);
     if (!success) return;
 
+    onCopy?.(value);
     setCopied(true);
     if (timeoutRef.current !== null) {
       window.clearTimeout(timeoutRef.current);