@eifi1/ui-kit 0.24.1 → 0.25.1

package/dist/components/tooltip.d.ts

@@ -8,6 +8,17 @@ export { CLIPS_ATTRIBUTE } from '../lib/clipping.js';
  *  for. `left` / `right` stay physical, for a bubble tied to something that does not
  *  mirror (a chart axis, a map). */
 type TooltipSide = "top" | "bottom" | "left" | "right" | "start" | "end";
+/** What a TAP — a touch or a pen press, never the mouse — on the trigger does to the
+ *  bubble. See "A tap is not a hover" on {@link Tooltip}.
+ *
+ *  - `"auto"` (the default): a tap that activates something shows nothing; a tap that
+ *    activates nothing — on a disabled or `aria-disabled` control (a write lock's
+ *    reason), or on a trigger with no control at all (a truncated name, a badge) —
+ *    toggles the bubble, since showing it is then the tap's only answer.
+ *  - `"toggle"`: every tap toggles it — for a control that exists only to explain, such
+ *    as a "?" whose click does nothing.
+ *  - `"ignore"`: a tap never shows it. */
+type TooltipTap = "auto" | "toggle" | "ignore";
 /** The placement the maths works in: a logical side resolved against the trigger. */
 type PhysicalSide = "top" | "bottom" | "left" | "right";
 /**
@@ -46,15 +57,22 @@ interface TooltipProps extends ComponentPropsWithoutRef<"span"> {
     lazy?: boolean;
     /** Tag the bubble `data-private`, for a label that repeats the user's own data. */
     redact?: boolean;
+    /**
+     * What a tap (touch or pen) on the trigger does — see {@link TooltipTap} and "A tap is
+     * not a hover" below. Default `"auto"`: a tap that activates the control shows nothing;
+     * one that activates nothing (a disabled control, plain text) toggles the bubble.
+     */
+    tap?: TooltipTap;
     children: ReactNode;
 }
 /**
  * Hover/focus label for a control.
  *
  * Two placements, and the choice matters more than it looks. In place, the bubble is
- * always mounted next to the trigger and fades in on `:hover`, which costs no state and
- * works in a plain render test. Portalled, the bubble is mounted in `document.body`
- * only while it is up, positioned by measurement.
+ * always mounted next to the trigger — so a plain render test finds it without a hover —
+ * and shown while it is up (`display: none` otherwise, see "never widens the page"
+ * below). Portalled, the bubble is mounted in `document.body` only while it is up,
+ * positioned by measurement.
  *
  * ⚠️ **A bubble that repeats a value has to be redactable.** The consuming app blurs
  * `[data-private]` under a `demo-mode` class on `<html>` — and the portalled bubble is
@@ -173,8 +191,8 @@ interface TooltipProps extends ComponentPropsWithoutRef<"span"> {
  * So now, when an in-place bubble goes up (default or `lazy`), a layout effect measures
  * it and, if it crosses the viewport edge minus the same margin the portalled bubble
  * keeps, slides it back along its CROSS axis only — sideways for `top` / `bottom`, up or
- * down for the four side placements — with an inline `transform`, which composes with the
- * placement classes' own `translate`. The main axis is left alone on purpose: sliding a
+ * down for the four side placements — with an inline margin (a `transform` until 0.25; see
+ * {@link shiftStyle} for why that widened the page). The main axis is left alone on purpose: sliding a
  * `start` bubble along the main axis would slide it over its own trigger, and turning it
  * round is a measured-placement decision this CSS-placed bubble does not make (pass
  * `portal` for that). The width is already capped to the viewport (see
@@ -183,15 +201,86 @@ interface TooltipProps extends ComponentPropsWithoutRef<"span"> {
  *
  * Why on by default, with no prop? Because it is a no-op for every bubble that already
  * fits — the shift is zero unless the bubble would overflow — so the only placements it
- * changes are ones that were broken. The maths is in physical viewport pixels, so it
- * needs no `dir`: an RTL row puts its long label at the RIGHT edge and gets slid left by
- * the same code. Under jsdom there is no layout — every rect is zero-sized — and a
- * zero-sized bubble is taken to be unmeasured, so tests see no transform at all. It is
- * measured on open (and when the label or side changes while open), not on every scroll:
- * an in-place bubble follows its trigger for free, and a page that scrolls sideways under
- * an open tooltip is not a case worth a listener per tooltip.
+ * changes are ones that were broken. The maths is in physical viewport pixels: an RTL
+ * row puts its long label at the RIGHT edge and gets slid left by the same code, and the
+ * reading direction only decides which edge is kept when a bubble is wider than the room
+ * (its start, as {@link placeTooltip} keeps). Under jsdom there is no layout — every rect
+ * is zero-sized — and a zero-sized bubble is taken to be unmeasured, so tests see no
+ * transform at all. It is measured on open (and when the label or side changes while
+ * open), not on every scroll: an in-place bubble follows its trigger for free, and a page
+ * that scrolls sideways under an open tooltip is not a case worth a listener per tooltip.
+ *
+ * ⚠️ **An in-place bubble never widens the page (keksdose run 72, live #381).** Two
+ * holes let it, and a page wider than the screen is worse than a bubble cut off: on a
+ * phone Chrome then lays every `position: fixed` layer out against the WIDER page while
+ * the glass still shows the old width. Measured on a 406px phone: the sync icon at x=254,
+ * its `bottom` bubble at 102–422, the page 426px wide, and a FloatingPanel 17px off the
+ * glass. First, the clamp compared against `window.innerWidth` — and on a phone that is
+ * the layout viewport, which GROWS to the page's width once something sticks out (430 on
+ * a 406px screen, measured in Chromium's mobile emulation), so a bubble already past the
+ * edge found itself "inside" and stayed there. It now measures against the root's client
+ * width, which on a phone is the screen (the initial containing block, the width `100vw`
+ * and media queries see) whatever overflows, and on a desktop also leaves out a classic
+ * scrollbar; the window is the fallback where there is no layout (jsdom). The portalled
+ * bubble takes the same width, since a `fixed` bubble placed against a grown layout
+ * viewport is off the glass for the same reason. Second, the always-mounted bubble was
+ * `opacity: 0` while closed — invisible, but still laid out where the CSS puts it, so a
+ * bubble nobody opened, centred on a trigger near the end edge, widened the page all by
+ * itself (dev#488's phantom scroll, at the scale of the page instead of a scroller, where
+ * the auto-portal cannot help: the page is not an ancestor it can portal out of). It is
+ * now `display: none` until it is up, so a closed bubble takes no room at all, and an open
+ * one is slid onto the glass in the same layout pass that shows it, before paint. jsdom
+ * computes no Tailwind, so tests still find the closed bubble without a hover.
+ *
+ * ⚠️ **A tap is not a hover (keksdose run 72, live #379).** The bubble used to show on
+ * `mouseenter` and on `focus`, and hide only on `mouseleave` and `blur`. A tap on a phone
+ * fires an emulated `mouseenter` and, on a button, focuses it — and the virtual mouse and
+ * the focus both stay on the button until the next tap somewhere else, so every
+ * IconButton's label (the top-bar icons, the sync indicator, a row's actions) stayed up
+ * over whatever the tap had just opened. Now a touch or pen press is told apart from the
+ * mouse: the emulated `mouseenter` that follows a tap is ignored, the press itself closes
+ * a bubble that was up, and a focus that arrives while the page's last input was a finger
+ * or a pen — the tap's own, or one a closing dialog RETURNS to the icon it was opened
+ * from — shows nothing. The next key press (a Bluetooth keyboard on a tablet) or mouse
+ * press makes focus count again, and a mouse entering the trigger hovers as ever, so a
+ * hybrid laptop gets both.
+ *
+ * Focus otherwise shows the bubble, as it always did — the keyboard, a script, a test's
+ * `.focus()` — with one more exception: the focus a mouse press on the trigger gives it.
+ * There it follows `:focus-visible`, so a text field still shows its bubble while you type
+ * and a clicked button does not hold it: a desktop click behaves as it always did while
+ * the pointer is over the button (the hover shows it) and lets it go when the pointer
+ * leaves, where the in-place bubble used to linger until the button lost focus — the same
+ * left-behind bubble on a smaller scale, and what the portalled bubble always did. A
+ * keyboard user's bubble, in turn, now stays up when a mouse passes over and leaves (the
+ * portalled one used to close). Any `pointerdown` outside the trigger closes an open
+ * bubble too — the Safari case, where a tap elsewhere moves no focus.
+ *
+ * What a tap DOES show is {@link TooltipTap} (`tap`). By default a tap that activates
+ * something shows nothing — the action is the answer — while a tap that activates nothing
+ * toggles the bubble, because there the bubble is the only answer: a write-locked or
+ * disabled control, whose bubble is the reason it did not respond, and a trigger with no
+ * control at all, a truncated payee or a badge, whose bubble is the rest of the text. A
+ * second tap, a tap anywhere else, Escape or the focus leaving closes it. "Activates" is
+ * read off the DOM — a link, a button, a field, a label, an element with a widget role
+ * (`button`, `link`, `checkbox`, `option`, …), or a DataTable row (`tr[tabindex]`) — from
+ * the tapped element outwards, past the tooltip, so a glyph inside a clickable row or a
+ * button counts as that row or button. What it cannot read is a click handler on an
+ * element with no role (give it `role="button"`, which it needs anyway, or pass
+ * `tap="ignore"`), and a button that exists only to explain, whose click does nothing —
+ * FieldHint's "?" — which says so with `tap="toggle"`.
+ *
+ * Why not show the label on a long press, as Android does for its own icons? Because the
+ * web gives no reliable long press on a control: browsers disagree on whether the release
+ * that ends one still clicks the button, and the button's own long press — the context
+ * menu, text selection, a row's drag — claims the gesture first on others. A gesture meant
+ * to ask "what does this do" must not risk doing it. Screen readers do not need it (the
+ * bubble is in `aria-describedby`, and an IconButton's label is its name), and a tap on
+ * anything that does nothing already shows it. No scroll listener either: every touch scroll begins with a `pointerdown`, which
+ * already closes the bubble, and closing on scroll would close a keyboard user's bubble
+ * the moment Tab scrolled its control into view.
  */
-declare function Tooltip({ label, side, className, portal, lazy, redact, children, ...rest }: TooltipProps): react.JSX.Element;
+declare function Tooltip({ label, side, className, portal, lazy, redact, tap, children, ...rest }: TooltipProps): react.JSX.Element;
 interface TooltipSize {
     width: number;
     height: number;
@@ -230,4 +319,4 @@ interface TooltipPlacement {
  */
 declare function placeTooltip(r: AnchorRect, side: PhysicalSide, size: TooltipSize, viewport: TooltipViewport): TooltipPlacement;
 
-export { Tooltip, type TooltipPlacement, type TooltipProps, type TooltipSide, type TooltipSize, type TooltipViewport, placeTooltip };
+export { Tooltip, type TooltipPlacement, type TooltipProps, type TooltipSide, type TooltipSize, type TooltipTap, type TooltipViewport, placeTooltip };

package/dist/components/tooltip.js

@@ -3,6 +3,7 @@ import { Fragment, jsx, jsxs } from "react/jsx-runtime";
 import {
   cloneElement,
   isValidElement,
+  useEffect,
   useId,
   useLayoutEffect,
   useRef,
@@ -10,7 +11,7 @@ import {
 } from "react";
 import { createPortal } from "react-dom";
 import { cn } from "../lib/cn.js";
-import { useEscapeKey } from "../hooks/use-dismiss.js";
+import { useEscapeKey, useOutsideClick } from "../hooks/use-dismiss.js";
 import { useAnchoredRect } from "../hooks/use-anchored-rect.js";
 import { dirOf } from "../lib/direction.js";
 import { hasClippingAncestor } from "../lib/clipping.js";
@@ -38,6 +39,7 @@ function Tooltip({
   portal,
   lazy = false,
   redact = false,
+  tap = "auto",
   children,
   ...rest
 }) {
@@ -50,6 +52,7 @@ function Tooltip({
         side,
         className,
         redact,
+        tap,
         ...rest,
         children
       }
@@ -64,11 +67,152 @@ function Tooltip({
       redact,
       detect: portal === void 0,
       lazy,
+      tap,
       ...rest,
       children
     }
   );
 }
+function isTap(pointerType) {
+  return pointerType === "touch" || pointerType === "pen";
+}
+const ACTIVATES = [
+  "a[href]",
+  "button",
+  "input",
+  "select",
+  "textarea",
+  "label",
+  "summary",
+  '[contenteditable=""]',
+  '[contenteditable="true"]',
+  "tr[tabindex]",
+  ...[
+    "button",
+    "link",
+    "checkbox",
+    "radio",
+    "switch",
+    "tab",
+    "menuitem",
+    "menuitemcheckbox",
+    "menuitemradio",
+    "option",
+    "treeitem",
+    "slider",
+    "spinbutton",
+    "combobox"
+  ].map((role) => `[role="${role}"]`)
+].join(",");
+function tapShows(tap, target) {
+  if (tap !== "auto") return tap === "toggle";
+  if (!(target instanceof Element)) return true;
+  const control = target.closest(ACTIVATES);
+  return control === null || control.matches(":disabled") || control.closest('[aria-disabled="true"]') !== null;
+}
+let lastInput = null;
+let trackingInput = false;
+function trackInput() {
+  if (trackingInput || typeof document === "undefined") return;
+  trackingInput = true;
+  document.addEventListener(
+    "pointerdown",
+    (e) => {
+      lastInput = e.pointerType;
+    },
+    { capture: true, passive: true }
+  );
+  document.addEventListener(
+    "keydown",
+    (e) => {
+      if (e.key === "Unidentified" || e.isComposing || e.ctrlKey || e.metaKey || e.altKey) return;
+      lastInput = "keyboard";
+    },
+    { capture: true, passive: true }
+  );
+}
+function focusShows(target, pressedByMouse) {
+  if (lastInput !== null && isTap(lastInput)) return false;
+  if (!pressedByMouse || !(target instanceof Element)) return true;
+  try {
+    return target.matches(":focus-visible");
+  } catch {
+    return true;
+  }
+}
+function useTooltipTrigger(triggerRef, tap, passed, onArm) {
+  const [hovered, setHovered] = useState(false);
+  const [focused, setFocused] = useState(false);
+  const [tapped, setTapped] = useState(false);
+  const [dismissed, setDismissed] = useState(false);
+  const pressedByTouch = useRef(false);
+  const pressedByMouse = useRef(false);
+  const press = useRef(null);
+  useEffect(trackInput, []);
+  const open = (hovered || focused || tapped) && !dismissed;
+  const close = () => {
+    setHovered(false);
+    setFocused(false);
+    setTapped(false);
+  };
+  useEscapeKey(() => setDismissed(true), open);
+  useOutsideClick(triggerRef, close, open);
+  const arm = (el) => {
+    setDismissed(false);
+    onArm(el);
+  };
+  const handlers = {
+    onPointerEnter: (e) => {
+      if (e.pointerType === "touch") pressedByTouch.current = true;
+      else if (e.buttons === 0) pressedByTouch.current = false;
+      passed.onPointerEnter?.(e);
+    },
+    onPointerDown: (e) => {
+      if (isTap(e.pointerType)) {
+        pressedByTouch.current = true;
+        press.current = { wasOpen: open };
+        close();
+      } else {
+        pressedByTouch.current = false;
+        pressedByMouse.current = true;
+      }
+      passed.onPointerDown?.(e);
+    },
+    onPointerUp: (e) => {
+      pressedByMouse.current = false;
+      const began = press.current;
+      press.current = null;
+      if (began && !began.wasOpen && isTap(e.pointerType) && tapShows(tap, e.target)) {
+        setTapped(true);
+        arm(e.currentTarget);
+      }
+      passed.onPointerUp?.(e);
+    },
+    // The browser took the gesture over — a scroll, a pinch: not a tap.
+    onPointerCancel: (e) => {
+      pressedByMouse.current = false;
+      press.current = null;
+      passed.onPointerCancel?.(e);
+    },
+    onMouseEnter: (e) => {
+      if (pressedByTouch.current) return;
+      setHovered(true);
+      arm(e.currentTarget);
+    },
+    onMouseLeave: () => setHovered(false),
+    onFocus: (e) => {
+      if (!focusShows(e.target, pressedByMouse.current)) return;
+      setFocused(true);
+      arm(e.currentTarget);
+    },
+    onBlur: (e) => {
+      setFocused(false);
+      if (e.relatedTarget instanceof Node && e.currentTarget.contains(e.relatedTarget)) return;
+      setTapped(false);
+    }
+  };
+  return { open, dismissed, handlers };
+}
 function InPlaceTooltip({
   label,
   side,
@@ -76,57 +220,40 @@ function InPlaceTooltip({
   redact,
   detect,
   lazy,
+  tap,
   children,
   ...rest
 }) {
   const id = useId();
   const triggerRef = useRef(null);
   const [clipped, setClipped] = useState(false);
-  const [hovered, setHovered] = useState(false);
-  const [focused, setFocused] = useState(false);
-  const [dismissed, setDismissed] = useState(false);
   const [dir, setDir] = useState("ltr");
-  const open = (hovered || focused) && !dismissed;
-  useEscapeKey(() => setDismissed(true), open);
+  const { open, dismissed, handlers } = useTooltipTrigger(triggerRef, tap, rest, (el) => {
+    setDir(dirOf(el));
+    if (detect) setClipped(hasClippingAncestor(el));
+  });
   const bubbleRef = useRef(null);
-  const shift = useViewportClamp(bubbleRef, open && !clipped, side, label);
+  const { shift, measuring } = useViewportClamp(bubbleRef, open && !clipped, side, label, dir);
   useLayoutEffect(() => {
     if (detect) setClipped(hasClippingAncestor(triggerRef.current));
   }, [detect]);
-  const arm = (el) => {
-    setDismissed(false);
-    if (!detect) return;
-    setDir(dirOf(el));
-    setClipped(hasClippingAncestor(el));
-  };
   return (
-    /* eslint-disable-next-line jsx-a11y/no-static-element-interactions -- the handlers track
-       whether the bubble is up and activate nothing; the caller's child is the
-       interactive element, keeps its own handlers, and its focus shows the bubble too. */
+    // No role for this span: its handlers track whether the bubble is up and activate
+    // nothing; the caller's child is the interactive element, keeps its own handlers,
+    // and its focus shows the bubble too.
     /* @__PURE__ */ jsxs(
       "span",
       {
         ...rest,
         ref: triggerRef,
+        style: measuring ? { ...rest.style, overflow: "clip" } : rest.style,
         className: cn("relative inline-flex", !clipped && "group/tooltip", className),
-        onMouseEnter: (e) => {
-          setHovered(true);
-          arm(e.currentTarget);
-        },
-        onMouseLeave: () => setHovered(false),
-        onFocus: (e) => {
-          setFocused(true);
-          arm(e.currentTarget);
-        },
-        onBlur: () => setFocused(false),
+        ...handlers,
         children: [
           describedBy(children, clipped || lazy ? open ? id : void 0 : dismissed ? void 0 : id),
           clipped ? open && /* @__PURE__ */ jsx(PortalBubble, { triggerRef, id, label, side, dir, redact }) : lazy ? (
             // keksdose F6: the in-place slot and classes, the portalled lifetime. Mounted only
-            // while up, so it is visible whenever it exists — `opacity-100` outright rather
-            // than the `group-hover` switch, which would also be right but would make the
-            // bubble's visibility depend on two sources (the state that mounted it and the
-            // CSS that shows it) that can disagree for a frame after Escape.
+            // while up, so it is visible whenever it exists.
             open && /* @__PURE__ */ jsx(
               "span",
               {
@@ -150,7 +277,8 @@ function InPlaceTooltip({
               "data-private": redact ? "" : void 0,
               className: cn(
                 TOOLTIP_SURFACE,
-                "pointer-events-none absolute z-50 opacity-0 group-hover/tooltip:opacity-100 group-focus-within/tooltip:opacity-100",
+                "pointer-events-none absolute z-50",
+                open ? "opacity-100" : "hidden",
                 sidePositionClass[side]
               ),
               children: label
@@ -174,31 +302,46 @@ function isEmptyLabel(label) {
 const TOOLTIP_GAP = 4;
 const TOOLTIP_MARGIN = 4;
 const NO_SHIFT = { x: 0, y: 0 };
-function slideInto(low, high, extent) {
-  if (low < TOOLTIP_MARGIN) return TOOLTIP_MARGIN - low;
-  if (high > extent - TOOLTIP_MARGIN) return Math.max(extent - TOOLTIP_MARGIN - high, TOOLTIP_MARGIN - low);
+const UNPLACED = { shift: NO_SHIFT, for: null };
+function slideInto(low, high, extent, keepHigh = false) {
+  const min = TOOLTIP_MARGIN;
+  const max = extent - TOOLTIP_MARGIN;
+  if (high - low > max - min) return keepHigh ? max - high : min - low;
+  if (low < min) return min - low;
+  if (high > max) return max - high;
   return 0;
 }
-function useViewportClamp(bubbleRef, active, side, label) {
-  const [shift, setShift] = useState(NO_SHIFT);
+function viewportSize() {
+  return {
+    width: document.documentElement.clientWidth || window.innerWidth,
+    height: window.innerHeight
+  };
+}
+function useViewportClamp(bubbleRef, active, side, label, dir) {
+  const [placement, setPlacement] = useState(UNPLACED);
+  const measured = placement.for !== null && placement.for.side === side && placement.for.label === label && placement.for.dir === dir;
   useLayoutEffect(() => {
     const el = bubbleRef.current;
     if (!active || !el) {
-      setShift(NO_SHIFT);
+      setPlacement(UNPLACED);
       return;
     }
+    if (measured) return;
     const r = el.getBoundingClientRect();
-    if (r.width === 0 && r.height === 0) return;
+    const unlaid = r.width === 0 && r.height === 0;
     const horizontal = side === "top" || side === "bottom";
-    setShift((previous) => {
-      const next = horizontal ? { x: slideInto(r.left - previous.x, r.right - previous.x, window.innerWidth), y: 0 } : { x: 0, y: slideInto(r.top - previous.y, r.bottom - previous.y, window.innerHeight) };
-      return next.x === previous.x && next.y === previous.y ? previous : next;
+    const viewport = viewportSize();
+    setPlacement(({ shift: previous }) => {
+      const next = unlaid ? previous : horizontal ? { x: slideInto(r.left - previous.x, r.right - previous.x, viewport.width, dir === "rtl"), y: 0 } : { x: 0, y: slideInto(r.top - previous.y, r.bottom - previous.y, viewport.height) };
+      const shift = next.x === previous.x && next.y === previous.y ? previous : next;
+      return { shift, for: { side, label, dir } };
     });
-  }, [bubbleRef, active, side, label]);
-  return shift;
+  }, [bubbleRef, active, measured, side, label, dir]);
+  return { shift: placement.shift, measuring: active && !measured };
 }
 function shiftStyle(shift) {
-  return shift.x === 0 && shift.y === 0 ? void 0 : { transform: `translate(${shift.x}px, ${shift.y}px)` };
+  if (shift.x === 0 && shift.y === 0) return void 0;
+  return shift.x !== 0 ? { marginLeft: shift.x } : { marginTop: shift.y };
 }
 const portalTransformBySide = {
   right: "translate(0, -50%)",
@@ -257,18 +400,14 @@ function PortalTooltip({
   side,
   className,
   redact,
+  tap,
   children,
   ...rest
 }) {
   const triggerRef = useRef(null);
-  const [visible, setVisible] = useState(false);
   const [dir, setDir] = useState("ltr");
-  const show = (el) => {
-    setDir(dirOf(el));
-    setVisible(true);
-  };
   const id = useId();
-  useEscapeKey(() => setVisible(false), visible);
+  const { open, handlers } = useTooltipTrigger(triggerRef, tap, rest, (el) => setDir(dirOf(el)));
   return /* @__PURE__ */ jsxs(Fragment, { children: [
     /* @__PURE__ */ jsx(
       "span",
@@ -276,14 +415,11 @@ function PortalTooltip({
         ...rest,
         ref: triggerRef,
         className: cn("relative inline-flex", className),
-        onMouseEnter: (e) => show(e.currentTarget),
-        onMouseLeave: () => setVisible(false),
-        onFocus: (e) => show(e.currentTarget),
-        onBlur: () => setVisible(false),
-        children: describedBy(children, visible ? id : void 0)
+        ...handlers,
+        children: describedBy(children, open ? id : void 0)
       }
     ),
-    visible && /* @__PURE__ */ jsx(PortalBubble, { triggerRef, id, label, side, dir, redact })
+    open && /* @__PURE__ */ jsx(PortalBubble, { triggerRef, id, label, side, dir, redact })
   ] });
 }
 function PortalBubble({
@@ -303,7 +439,8 @@ function PortalBubble({
       if (!measured) return null;
       const next = {
         size: { width: measured.width, height: measured.height },
-        viewport: { width: window.innerWidth, height: window.innerHeight }
+        // The glass, not the window: see viewportSize (live #381).
+        viewport: viewportSize()
       };
       return previous && sameRoom(previous, next) ? previous : next;
     });