@boostdev/design-system-components 1.1.0 → 1.1.2

package/src/components/interaction/Dialog/Dialog.spec.tsx

@@ -29,7 +29,26 @@ describe('Dialog', () => {
     expect(screen.getByRole('button', { name: /close dialog/i })).toBeInTheDocument();
   });
 
-  it('calls onClose when close button is clicked', async () => {
+  it('close button is type="button"', () => {
+    render(<Dialog isOpen><span>Content</span></Dialog>);
+    expect(screen.getByRole('button', { name: /close dialog/i })).toHaveAttribute('type', 'button');
+  });
+
+  it('close button always uses commandfor/command', () => {
+    render(<Dialog isOpen><span>Content</span></Dialog>);
+    const closeBtn = screen.getByRole('button', { name: /close dialog/i });
+    expect(closeBtn).toHaveAttribute('commandfor');
+    expect(closeBtn).toHaveAttribute('command', 'close');
+  });
+
+  it('close button commandfor matches dialog id', () => {
+    render(<Dialog id="my-dialog" isOpen><span>Content</span></Dialog>);
+    const closeBtn = screen.getByRole('button', { name: /close dialog/i });
+    expect(closeBtn).toHaveAttribute('commandfor', 'my-dialog');
+    expect(document.querySelector('dialog')).toHaveAttribute('id', 'my-dialog');
+  });
+
+  it('close button click via polyfill calls onClose', async () => {
     const user = userEvent.setup();
     const onClose = vi.fn();
     render(<Dialog isOpen onClose={onClose}><span>Content</span></Dialog>);
@@ -42,6 +61,12 @@ describe('Dialog', () => {
     expect(HTMLDialogElement.prototype.showModal).not.toHaveBeenCalled();
   });
 
+  it('does not call showModal again when dialog is already open', () => {
+    const { rerender } = render(<Dialog isOpen><span>Content</span></Dialog>);
+    rerender(<Dialog isOpen><span>Content</span></Dialog>);
+    expect(HTMLDialogElement.prototype.showModal).toHaveBeenCalledTimes(1);
+  });
+
   it('locks body scroll when open', () => {
     render(<Dialog isOpen><span>Content</span></Dialog>);
     expect(document.body.style.overflow).toBe('hidden');
@@ -77,4 +102,31 @@ describe('Dialog', () => {
 
     document.body.removeChild(trigger);
   });
+
+  describe('command event (Invoker Commands API)', () => {
+    function makeCommandEvent(command: string) {
+      const event = new Event('command');
+      Object.defineProperty(event, 'command', { value: command, configurable: true });
+      return event;
+    }
+
+    it('locks scroll and calls onOpen on show-modal command', () => {
+      const onOpen = vi.fn();
+      render(<Dialog onOpen={onOpen}><span>Content</span></Dialog>);
+      const dialog = document.querySelector('dialog')!;
+      dialog.dispatchEvent(makeCommandEvent('show-modal'));
+      expect(document.body.style.overflow).toBe('hidden');
+      expect(onOpen).toHaveBeenCalled();
+    });
+
+    it('restores scroll and calls onClose on close command', () => {
+      const onClose = vi.fn();
+      render(<Dialog isOpen onClose={onClose}><span>Content</span></Dialog>);
+      document.body.style.overflow = 'hidden';
+      const dialog = document.querySelector('dialog')!;
+      dialog.dispatchEvent(makeCommandEvent('close'));
+      expect(document.body.style.overflow).toBe('');
+      expect(onClose).toHaveBeenCalled();
+    });
+  });
 });

package/src/components/interaction/Dialog/Dialog.stories.tsx

@@ -33,3 +33,20 @@ export const Interactive: Story = {
     );
   },
 };
+
+// Open and close using the Invoker Commands API — no JS event handlers needed.
+// The browser wires up the buttons to the dialog natively via commandfor/command.
+export const InvokerCommands: Story = {
+  render: () => (
+    <>
+      <Button type="button" commandfor="invoker-dialog" command="show-modal">
+        Open Dialog (no JS)
+      </Button>
+      <Dialog id="invoker-dialog">
+        <h2>Invoker Dialog</h2>
+        <p>This dialog was opened with a declarative HTML invoker command — no onClick needed.</p>
+        <p>The close button (×) inside also uses an invoker command.</p>
+      </Dialog>
+    </>
+  ),
+};

package/src/components/interaction/Dialog/Dialog.tsx

@@ -1,4 +1,8 @@
-import { ReactNode, useEffect, useRef } from 'react';
+import { ReactNode, useEffect, useId, useRef } from 'react';
+import { installInvokerCommandsPolyfill } from '../../../polyfill-invoker-commands';
+
+// Install once at module load — no-op if native support is present.
+installInvokerCommandsPolyfill();
 
 const FOCUSABLE_SELECTOR = 'button, [href], input, select, textarea, [tabindex]:not([tabindex="-1"])';
 import css from './Dialog.module.css';
@@ -7,22 +11,28 @@ import type { WithClassName } from '../../../types';
 
 interface DialogProps extends WithClassName {
   children: ReactNode;
+  id?: string;
   isOpen?: boolean;
+  onOpen?: () => void;
   onClose?: () => void;
 }
 
-export function Dialog({ children, isOpen = false, className, onClose }: DialogProps) {
+export function Dialog({ children, id: idProp, isOpen = false, className, onOpen, onClose }: DialogProps) {
+  const generatedId = useId();
+  const id = idProp ?? generatedId;
+
   const dialogRef = useRef<HTMLDialogElement | null>(null);
   const triggerRef = useRef<Element | null>(null);
 
+  // Controlled mode: open/close in response to the isOpen prop.
   useEffect(() => {
     const dialog = dialogRef.current;
     if (!dialog) return;
-    if (isOpen) {
+    if (isOpen && !dialog.open) {
       triggerRef.current = document.activeElement;
       dialog.showModal();
       document.body.style.overflow = 'hidden';
-    } else if (dialog.open) {
+    } else if (!isOpen && dialog.open) {
       dialog.close();
       document.body.style.overflow = '';
       (triggerRef.current as HTMLElement | null)?.focus();
@@ -32,6 +42,30 @@ export function Dialog({ children, isOpen = false, className, onClose }: DialogP
     };
   }, [isOpen]);
 
+  // Invoker Commands API: sync state when the dialog is opened or closed via
+  // a <button commandfor="id" command="show-modal|close"> without JS.
+  useEffect(() => {
+    const dialog = dialogRef.current;
+    if (!dialog) return;
+    const handleCommand = (e: CommandEvent) => {
+      if (e.command === 'show-modal') {
+        triggerRef.current = document.activeElement;
+        document.body.style.overflow = 'hidden';
+        onOpen?.();
+      } else if (e.command === 'close') {
+        document.body.style.overflow = '';
+        // Defer focus restoration: the dialog is still open when the command
+        // event fires (the action runs after), so focusing outside would be
+        // blocked by the focus-trap. setTimeout defers until after close.
+        const trigger = triggerRef.current;
+        setTimeout(() => (trigger as HTMLElement | null)?.focus(), 0);
+        onClose?.();
+      }
+    };
+    dialog.addEventListener('command', handleCommand);
+    return () => dialog.removeEventListener('command', handleCommand);
+  }, [onOpen, onClose]);
+
   const handleBackdropClick = (e: React.MouseEvent<HTMLDialogElement>) => {
     if (e.target === dialogRef.current) onClose?.();
   };
@@ -59,24 +93,24 @@ export function Dialog({ children, isOpen = false, className, onClose }: DialogP
   return (
     <dialog
       ref={dialogRef}
+      id={id}
       className={cn(className, css.dialog)}
       aria-modal="true"
       onClick={handleBackdropClick}
       onCancel={handleCancel}
       onKeyDown={handleKeyDown}
     >
-      <form method="dialog" className={css.closeForm}>
-        <button
-          type="submit"
-          className={css.closeButton}
-          onClick={onClose}
-          aria-label="Close dialog"
-        >
-          <svg aria-hidden="true" viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="2">
-            <path strokeLinecap="round" strokeLinejoin="round" d="M18 6L6 18M6 6l12 12" />
-          </svg>
-        </button>
-      </form>
+      <button
+        type="button"
+        className={css.closeButton}
+        commandfor={id}
+        command="close"
+        aria-label="Close dialog"
+      >
+        <svg aria-hidden="true" viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="2">
+          <path strokeLinecap="round" strokeLinejoin="round" d="M18 6L6 18M6 6l12 12" />
+        </svg>
+      </button>
       <div className={css.dialogContent}>
         {children}
       </div>

package/src/components/interaction/Drawer/Drawer.module.css

@@ -4,10 +4,10 @@
     inset: 0;
     margin: 0;
     padding: 0;
-    width: 100%;
-    max-width: 100%;
-    height: 100%;
-    max-height: 100%;
+    inline-size: 100%;
+    max-inline-size: 100%;
+    block-size: 100%;
+    max-block-size: 100%;
     border: none;
     background: transparent;
     overflow: hidden;
@@ -23,8 +23,8 @@
     inset-block: 0;
     display: flex;
     flex-direction: column;
-    width: min(28rem, 90vw);
-    height: 100%;
+    inline-size: min(28rem, 90vw);
+    block-size: 100%;
     background-color: var(--drawer_color, var(--bds-color_bg));
     color: var(--drawer_on-color, var(--bds-color_on-bg));
     box-shadow: var(--bds-shadow_xl);
@@ -54,16 +54,16 @@
     display: flex;
     align-items: center;
     justify-content: center;
-    width: 2rem;
-    height: 2rem;
+    inline-size: 2rem;
+    block-size: 2rem;
     border-radius: 50%;
     cursor: pointer;
     transition: var(--bds-animation_transition);
   }
 
   .closeButton svg {
-    width: 1.25rem;
-    height: 1.25rem;
+    inline-size: 1.25rem;
+    block-size: 1.25rem;
   }
 
   .closeButton:focus-visible {
@@ -79,7 +79,7 @@
 
   .body {
     flex: 1;
-    overflow-y: auto;
+    overflow-block: auto;
     padding: var(--bds-space_l);
     display: flex;
     flex-direction: column;

package/src/components/interaction/Drawer/Drawer.spec.tsx

@@ -24,7 +24,21 @@ describe('Drawer', () => {
     expect(screen.getByText('Navigation items')).toBeInTheDocument();
   });
 
-  it('calls onClose when close button clicked', async () => {
+  it('close button uses commandfor/command', () => {
+    render(<Drawer isOpen title="Menu" onClose={() => {}}>Content</Drawer>);
+    const closeBtn = screen.getByRole('button', { name: 'Close drawer' });
+    expect(closeBtn).toHaveAttribute('commandfor');
+    expect(closeBtn).toHaveAttribute('command', 'close');
+  });
+
+  it('close button commandfor matches dialog id', () => {
+    render(<Drawer id="my-drawer" isOpen title="Menu" onClose={() => {}}>Content</Drawer>);
+    const closeBtn = screen.getByRole('button', { name: 'Close drawer' });
+    expect(closeBtn).toHaveAttribute('commandfor', 'my-drawer');
+    expect(document.querySelector('dialog')).toHaveAttribute('id', 'my-drawer');
+  });
+
+  it('close button click via polyfill calls onClose', async () => {
     const onClose = vi.fn();
     render(<Drawer isOpen title="Menu" onClose={onClose}>Content</Drawer>);
     await userEvent.click(screen.getByRole('button', { name: 'Close drawer' }));
@@ -42,6 +56,12 @@ describe('Drawer', () => {
     expect(HTMLDialogElement.prototype.close).toHaveBeenCalled();
   });
 
+  it('does not call showModal again when already open', () => {
+    const { rerender } = render(<Drawer isOpen title="Test" onClose={() => {}}>Body</Drawer>);
+    rerender(<Drawer isOpen title="Test" onClose={() => {}}>Body</Drawer>);
+    expect(HTMLDialogElement.prototype.showModal).toHaveBeenCalledTimes(1);
+  });
+
   it('locks body scroll when open', () => {
     render(<Drawer isOpen title="Settings" onClose={() => {}}>Content</Drawer>);
     expect(document.body.style.overflow).toBe('hidden');
@@ -88,4 +108,31 @@ describe('Drawer', () => {
 
     document.body.removeChild(trigger);
   });
+
+  describe('command event (Invoker Commands API)', () => {
+    function makeCommandEvent(command: string) {
+      const event = new Event('command');
+      Object.defineProperty(event, 'command', { value: command, configurable: true });
+      return event;
+    }
+
+    it('locks scroll and calls onOpen on show-modal command', () => {
+      const onOpen = vi.fn();
+      render(<Drawer title="Menu" isOpen={false} onOpen={onOpen} onClose={() => {}}>Content</Drawer>);
+      const dialog = document.querySelector('dialog')!;
+      dialog.dispatchEvent(makeCommandEvent('show-modal'));
+      expect(document.body.style.overflow).toBe('hidden');
+      expect(onOpen).toHaveBeenCalled();
+    });
+
+    it('restores scroll and calls onClose on close command', () => {
+      const onClose = vi.fn();
+      render(<Drawer title="Menu" isOpen onClose={onClose}>Content</Drawer>);
+      document.body.style.overflow = 'hidden';
+      const dialog = document.querySelector('dialog')!;
+      dialog.dispatchEvent(makeCommandEvent('close'));
+      expect(document.body.style.overflow).toBe('');
+      expect(onClose).toHaveBeenCalled();
+    });
+  });
 });

package/src/components/interaction/Drawer/Drawer.tsx

@@ -1,10 +1,17 @@
-import { ReactNode, useEffect, useRef } from 'react';
+import { ReactNode, useEffect, useId, useRef } from 'react';
+import { installInvokerCommandsPolyfill } from '../../../polyfill-invoker-commands';
+
+// Install once at module load — no-op if native support is present.
+installInvokerCommandsPolyfill();
+
 import css from './Drawer.module.css';
 import { cn } from '@boostdev/design-system-foundation';
 import type { WithClassName } from '../../../types';
 
 interface DrawerProps extends WithClassName {
+  id?: string;
   isOpen: boolean;
+  onOpen?: () => void;
   onClose: () => void;
   title: ReactNode;
   children: ReactNode;
@@ -13,7 +20,9 @@ interface DrawerProps extends WithClassName {
 }
 
 export function Drawer({
+  id: idProp,
   isOpen,
+  onOpen,
   onClose,
   title,
   children,
@@ -21,17 +30,20 @@ export function Drawer({
   ariaLabel,
   className,
 }: Readonly<DrawerProps>) {
+  const generatedId = useId();
+  const id = idProp ?? generatedId;
+
   const dialogRef = useRef<HTMLDialogElement>(null);
   const triggerRef = useRef<Element | null>(null);
 
   useEffect(() => {
     const dialog = dialogRef.current;
     if (!dialog) return;
-    if (isOpen) {
+    if (isOpen && !dialog.open) {
       triggerRef.current = document.activeElement;
       dialog.showModal();
       document.body.style.overflow = 'hidden';
-    } else if (dialog.open) {
+    } else if (!isOpen && dialog.open) {
       dialog.close();
       document.body.style.overflow = '';
       (triggerRef.current as HTMLElement | null)?.focus();
@@ -41,6 +53,26 @@ export function Drawer({
     };
   }, [isOpen]);
 
+  // Invoker Commands API: sync state when opened/closed via commandfor/command.
+  useEffect(() => {
+    const dialog = dialogRef.current;
+    if (!dialog) return;
+    const handleCommand = (e: CommandEvent) => {
+      if (e.command === 'show-modal') {
+        triggerRef.current = document.activeElement;
+        document.body.style.overflow = 'hidden';
+        onOpen?.();
+      } else if (e.command === 'close') {
+        document.body.style.overflow = '';
+        const trigger = triggerRef.current;
+        setTimeout(() => (trigger as HTMLElement | null)?.focus(), 0);
+        onClose();
+      }
+    };
+    dialog.addEventListener('command', handleCommand);
+    return () => dialog.removeEventListener('command', handleCommand);
+  }, [onOpen, onClose]);
+
   // Close on backdrop click
   const handleClick = (e: React.MouseEvent<HTMLDialogElement>) => {
     if (e.target === dialogRef.current) onClose();
@@ -55,6 +87,7 @@ export function Drawer({
   return (
     <dialog
       ref={dialogRef}
+      id={id}
       className={cn(css.drawer, css[`--side_${side}`], className)}
       aria-label={ariaLabel}
       aria-modal="true"
@@ -67,7 +100,8 @@ export function Drawer({
           <button
             type="button"
             className={css.closeButton}
-            onClick={onClose}
+            commandfor={id}
+            command="close"
             aria-label="Close drawer"
           >
             <svg aria-hidden="true" viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="2">

package/src/components/interaction/DropdownMenu/DropdownMenu.module.css

@@ -6,9 +6,9 @@
 
   .menu {
     position: absolute;
-    top: calc(100% + var(--bds-space_xxs));
+    inset-block-start: calc(100% + var(--bds-space_xxs));
     z-index: var(--bds-z-index_dropdown);
-    min-width: 10rem;
+    min-inline-size: 10rem;
     padding: var(--bds-space_xxs) 0;
     border: var(--dropdown_border, none);
     --bdc_color: currentcolor;
@@ -24,12 +24,12 @@
     margin: 0;
   }
 
-  .--placement_bottom-start { left: 0; }
-  .--placement_bottom-end   { right: 0; }
+  .--placement_bottom-start { inset-inline-start: 0; }
+  .--placement_bottom-end   { inset-inline-end: 0; }
 
   .separator {
     border: none;
-    border-top: 1px solid currentcolor;
+    border-block-start: 1px solid currentcolor;
     margin: var(--bds-space_xxs) 0;
     opacity: 0.15;
   }
@@ -39,7 +39,7 @@
     display: flex;
     align-items: center;
     gap: var(--bds-space_xs);
-    width: 100%;
+    inline-size: 100%;
     padding: var(--bds-space_xs) var(--bds-space_m);
     font-family: var(--bds-font_family--body);
     font-size: var(--bds-font_size--body);

package/src/components/interaction/Popover/Popover.module.css

@@ -1,18 +1,25 @@
 @layer component {
   .wrapper {
-    position: relative;
     display: inline-flex;
+
+    /* No position: relative — the panel is position: fixed so it doesn't need
+       the wrapper as a containing block. */
   }
 
   .panel {
-    /* Absolute positioning keeps the panel anchored to the wrapper/trigger and
-       scrolls with the page. CSS anchor positioning sets the exact placement;
-       @position-try fallbacks handle automatic repositioning on overflow. */
-    position: absolute;
+    /* Fixed positioning: containing block is the viewport.
+       - CSS anchor() values resolve in viewport space → polyfill applies them correctly.
+       - Ancestor overflow / stacking contexts cannot clip the panel. */
+    position: fixed;
+
+    /* Reset UA [popover] defaults (inset: 0, margin: auto, border, padding). */
+    inset: unset;
     margin: 0;
-    overflow-y: auto;
+    overflow-block: auto;
     z-index: var(--popover_z-index, 100);
-    min-width: 12rem;
+    min-inline-size: 12rem;
+    max-inline-size: var(--popover_max-width, 20rem);
+    max-block-size: var(--popover_max-height, 80dvh);
     padding: var(--bds-space_m);
     border: var(--popover_border, none);
     --bdc_color: currentcolor;
@@ -34,28 +41,28 @@
   .panel[data-placement='bottom'] {
     top: anchor(bottom);
     left: anchor(left);
-    margin-top: var(--bds-space_xs, 8px);
+    margin-block-start: var(--bds-space_xs, 8px);
     position-try-fallbacks: --popover-top, --popover-right, --popover-left;
   }
 
   .panel[data-placement='top'] {
     bottom: anchor(top);
     left: anchor(left);
-    margin-bottom: var(--bds-space_xs, 8px);
+    margin-block-end: var(--bds-space_xs, 8px);
     position-try-fallbacks: --popover-bottom, --popover-right, --popover-left;
   }
 
   .panel[data-placement='right'] {
     top: anchor(top);
     left: anchor(right);
-    margin-left: var(--bds-space_xs, 8px);
+    margin-inline-start: var(--bds-space_xs, 8px);
     position-try-fallbacks: --popover-left, --popover-bottom, --popover-top;
   }
 
   .panel[data-placement='left'] {
     top: anchor(top);
     right: anchor(left);
-    margin-right: var(--bds-space_xs, 8px);
+    margin-inline-end: var(--bds-space_xs, 8px);
     position-try-fallbacks: --popover-right, --popover-bottom, --popover-top;
   }
 
@@ -69,7 +76,7 @@
     bottom: unset;
     left: anchor(left);
     margin: 0;
-    margin-top: var(--bds-space_xs, 8px);
+    margin-block-start: var(--bds-space_xs, 8px);
   }
 
   @position-try --popover-top {
@@ -78,7 +85,7 @@
     bottom: anchor(top);
     left: anchor(left);
     margin: 0;
-    margin-bottom: var(--bds-space_xs, 8px);
+    margin-block-end: var(--bds-space_xs, 8px);
   }
 
   @position-try --popover-right {
@@ -87,7 +94,7 @@
     bottom: unset;
     left: anchor(right);
     margin: 0;
-    margin-left: var(--bds-space_xs, 8px);
+    margin-inline-start: var(--bds-space_xs, 8px);
   }
 
   @position-try --popover-left {
@@ -96,6 +103,58 @@
     bottom: unset;
     left: unset;
     margin: 0;
-    margin-right: var(--bds-space_xs, 8px);
+    margin-inline-end: var(--bds-space_xs, 8px);
+  }
+
+  /* Full-width fallback blocks for narrow screens.
+     Defined at layer level (not inside @media) because @position-try cannot
+     be nested inside @media. These preserve left:8px/right:8px so the
+     panel stays edge-to-edge even when a @position-try fallback fires. */
+
+  @position-try --popover-bottom-fullwidth {
+    top: anchor(bottom);
+    right: var(--bds-space_xs, 8px);
+    bottom: unset;
+    left: var(--bds-space_xs, 8px);
+    margin: 0;
+    margin-block-start: var(--bds-space_xs, 8px);
+  }
+
+  @position-try --popover-top-fullwidth {
+    top: unset;
+    right: var(--bds-space_xs, 8px);
+    bottom: anchor(top);
+    left: var(--bds-space_xs, 8px);
+    margin: 0;
+    margin-block-end: var(--bds-space_xs, 8px);
+  }
+
+  /* Narrow screens (≤420px): span full viewport width and only flip vertically.
+     max-width: none overrides the base 20rem cap; left/right edge constraints
+     handle the width instead. The panel still touches the anchor on the
+     vertical axis (top/bottom via anchor() is not overridden here). */
+  @media (width <= 420px) {
+    .panel[data-placement='bottom'],
+    .panel[data-placement='top'],
+    .panel[data-placement='left'],
+    .panel[data-placement='right'] {
+      left: var(--bds-space_xs, 8px);
+      right: var(--bds-space_xs, 8px);
+      inline-size: 100%;
+      max-inline-size: calc(100svw - 2 * var(--bds-space_xs, 8px));
+      position-try-fallbacks: --popover-top-fullwidth, --popover-bottom-fullwidth;
+    }
+  }
+
+  /* Wide + short screens (e.g. landscape mobile, split-screen): when the
+     viewport has more horizontal room than vertical, prefer side placements. */
+  @media (aspect-ratio >= 2/1) {
+    .panel[data-placement='bottom'] {
+      position-try-fallbacks: --popover-top, --popover-right, --popover-left;
+    }
+
+    .panel[data-placement='top'] {
+      position-try-fallbacks: --popover-bottom, --popover-right, --popover-left;
+    }
   }
 }