@uniflowed/ui 0.0.0-alpha.12 → 0.0.0-alpha.14

package/internal/anchor.js

@@ -108,6 +108,19 @@ import { directionOf } from "./roving-focus.js";
  */
 export type Side = "top" | "right" | "bottom" | "left";
 
+/**
+ * A side a caller may ask for, which is the four above plus the two that mean
+ * "the way the reader reads".
+ *
+ * The WAI-ARIA menu pattern puts a submenu on the inline end - to the right of a
+ * left-to-right menu and to the left of a right-to-left one - and `menu.js`
+ * already spells the *keys* that way, so a physical-only `side` would have left
+ * one half of that pattern mirrored and the other half not. `Placement` and
+ * `placeOverlay` stay physical: a logical side is resolved once, against the
+ * trigger's own direction, before any arithmetic sees it.
+ */
+export type LogicalSide = Side | "inline-start" | "inline-end";
+
 /**
  * Where the overlay sits along the trigger's other axis.
  *
@@ -170,10 +183,29 @@ export type Anchored = {|
 /** What `useAnchor` is told, on top of the geometry `placeOverlay` needs. */
 export type AnchorRequest = {|
   readonly anchorRef: { current: HTMLElement | null },
+  /**
+   * A box to place against instead of the anchor element's own.
+   *
+   * A context menu opens *at a point* rather than against an element: the
+   * pointer coordinates the reader right-clicked at, which is a zero-sized
+   * rectangle no element in the document has. The element is still needed —
+   * it is what the writing direction is read from, what a `ResizeObserver`
+   * watches, and what focus returns to — so this replaces the *measurement*
+   * and nothing else.
+   *
+   * `null` (and absent) means "measure the element", which is every other
+   * overlay in this package.
+   *
+   * It has to be stable between renders for the same position, because it is
+   * one of the things the placement effect re-runs for; a fresh object each
+   * render would re-measure on every render of the page around it.
+   */
+  readonly anchorRect?: Rect | null,
   readonly overlayRef: { current: HTMLElement | null },
   /** Nothing is measured while it is closed: there is nothing to measure. */
   readonly open: boolean,
-  readonly side: Side,
+  /** Resolved against the trigger's writing direction; see `LogicalSide`. */
+  readonly side: LogicalSide,
   readonly align: Align,
   readonly sideOffset: number,
   readonly alignOffset: number,
@@ -189,6 +221,28 @@ const OPPOSITE: { readonly [Side]: Side } = {
   right: "left",
 };
 
+/**
+ * The side on the screen that `side` names for a reader reading `direction`.
+ *
+ * The four physical ones pass through unchanged, because a design that puts a
+ * popover to the right of a toolbar means the right of the toolbar in Arabic
+ * too; `Side`'s own documentation says why that is the useful default and why
+ * alignment is the axis that mirrors.
+ */
+export function physicalSide(side: LogicalSide, direction: Direction): Side {
+  // Written out rather than left to a wildcard, so that a sixth side added to
+  // `LogicalSide` one day is a checker error here instead of a value that falls
+  // through unresolved.
+  return match (side) {
+    "inline-start" => direction === "rtl" ? "right" : "left",
+    "inline-end" => direction === "rtl" ? "left" : "right",
+    "top" => "top",
+    "right" => "right",
+    "bottom" => "bottom",
+    "left" => "left",
+  };
+}
+
 /** Whether a side stacks the overlay above the trigger or beside it. */
 function isVertical(side: Side): boolean {
   return side === "top" || side === "bottom";
@@ -388,6 +442,7 @@ export hook useAnchor(request: AnchorRequest): Anchored {
   const {
     align,
     alignOffset,
+    anchorRect,
     anchorRef,
     avoidCollisions,
     collisionPadding,
@@ -396,7 +451,15 @@ export hook useAnchor(request: AnchorRequest): Anchored {
     side,
     sideOffset,
   } = request;
-  const [settled, setSettled] = useState<Anchored>({ align, side });
+  // Absent and `null` are one answer here — "measure the element" — so the two
+  // spellings become one value before anything depends on it.
+  const virtual = anchorRect ?? null;
+  // The left-to-right reading of a logical side, which is what `Anchored`
+  // reports until something has been measured. In a right-to-left page a
+  // submenu's `inline-end` is the *left*, and the first `reflow` says so - one
+  // commit later, exactly as a flip does, and for the same reason: the direction
+  // is a fact about the document, and a render may not read one.
+  const [settled, setSettled] = useState<Anchored>({ align, side: physicalSide(side, "ltr") });
 
   const reflow = useStableCallback(() => {
     const anchor = anchorRef.current;
@@ -405,7 +468,7 @@ export hook useAnchor(request: AnchorRequest): Anchored {
     if (anchor == null || overlay == null || view == null) {
       return;
     }
-    const box = rectOf(anchor);
+    const box = virtual ?? rectOf(anchor);
     const placement = placeOverlay({
       align,
       alignOffset,
@@ -414,7 +477,7 @@ export hook useAnchor(request: AnchorRequest): Anchored {
       collisionPadding,
       direction: directionOf(anchor),
       overlay: rectOf(overlay),
-      side,
+      side: physicalSide(side, directionOf(anchor)),
       sideOffset,
       // The viewport of a fixed element, which is the whole of it: a fixed box
       // is positioned against the viewport rather than against whatever is
@@ -443,8 +506,9 @@ export hook useAnchor(request: AnchorRequest): Anchored {
       // drawn from `data-side` pointing the wrong way, after a reopen that
       // follows a flip.
       if (!open) {
+        const asked = physicalSide(side, anchor == null ? "ltr" : directionOf(anchor));
         setSettled((current) =>
-          current.side === side && current.align === align ? current : { align, side },
+          current.side === asked && current.align === align ? current : { align, side: asked },
         );
       }
       return;
@@ -482,6 +546,7 @@ export hook useAnchor(request: AnchorRequest): Anchored {
   }, [
     align,
     alignOffset,
+    anchorRect,
     anchorRef,
     avoidCollisions,
     collisionPadding,
@@ -496,5 +561,5 @@ export hook useAnchor(request: AnchorRequest): Anchored {
   // first commit — and the server's markup, which measures nothing at all —
   // says the requested side rather than the last one some other opening
   // happened to settle on.
-  return open ? settled : { align, side };
+  return open ? settled : { align, side: physicalSide(side, "ltr") };
 }

package/internal/date-grid.js

@@ -0,0 +1,260 @@
+// @flow
+//
+// The month a calendar shows, and the keyboard that walks it.
+//
+// # Why this is not `roving-focus.js`
+//
+// A date grid is a roving tab stop — one stop in the page's tab order, arrow
+// keys inside it — so most of `internal/roving-focus.js` applies and the
+// calendar uses it: `directionOf` for a right-to-left week, `isEnabled` nowhere,
+// and the same rule about reading the document rather than a registry. What does
+// not fit is the part that does the moving, and it does not fit for four
+// reasons rather than one:
+//
+//   * **The movement is arithmetic on a date, not an index in a NodeList.**
+//     `moveTo` walks the items it was handed. `ArrowDown` in a calendar is "a
+//     week later", and a week later is often a cell that is not in the grid at
+//     all yet — so the answer cannot be found among the elements, and a
+//     `movementFor` grown to two axes would still return "next" for a key whose
+//     real meaning is `+7 days`.
+//   * **Running off the end changes what is rendered.** `ArrowRight` on the 31st
+//     shows the next month *and* leaves focus on the 1st, which is a cell that
+//     did not exist when the key was pressed. That is a `pendingFocus` problem,
+//     the same shape `Menu.Body` solves for a menu opening onto its last item,
+//     and it is why the movement is computed as a value the component can act on
+//     over two renders rather than as a `.focus()` inside a helper.
+//   * **An unavailable date stays focusable.** `moveTo` skips anything
+//     `isEnabled` rejects, which is right for a menu item and wrong here: a
+//     reader arrowing through October has to be able to pass over the days that
+//     cannot be booked, each `aria-disabled="true"` and each still reachable. A
+//     grid that skipped them would present a month with holes in it and no way
+//     to find out what is in the holes.
+//   * **There is no wrap, and no ends.** A list has a first and a last item.
+//     A calendar has neither: every direction leads to another month.
+//
+// So the two primitives stay apart, and the boundary is that one owns *focus
+// among elements that exist* and this one owns *which date the keyboard means*.
+// Everything below is a pure function over `PlainDate` values: no element, no
+// React, no document. That is deliberate for the same reason
+// `internal/anchor.js` keeps `placeOverlay` pure — the month-boundary cases are
+// the ones worth testing exhaustively, and testing them through a rendered grid
+// would test the renderer instead.
+//
+// # Dates come from Temporal, and from `@uniflowed/core/temporal` in particular
+//
+// Not `Date`. A package that ships a temporal library and then computes a month
+// length with `new Date(y, m + 1, 0)` is the opposite of what "build uf with uf"
+// asks for, and `Date`'s month-is-zero-based, mutates-in-place, local-timezone
+// arithmetic is where calendar bugs come from in the first place.
+//
+// The specifier is `@uniflowed/core/temporal` rather than `@uniflowed/temporal`,
+// which is the same object — `packages/temporal/index.js` is a re-export and
+// says so. It has to be that one: `@uniflowed/ui` is published to npm,
+// `@uniflowed/temporal` is not yet (its calendar surface waits on the native
+// runtime), and `tools/ci/publishable.sh` refuses a published package that
+// depends on an unpublished one because `npm install` would answer `ETARGET`.
+// When the name is published this import can move, and nothing else changes.
+
+import type { PlainDate } from "@uniflowed/core/temporal";
+import { Temporal } from "@uniflowed/core/temporal";
+
+import type { Direction } from "./roving-focus.js";
+
+/**
+ * ISO 8601's first day of the week, which is Monday.
+ *
+ * The fallback when nothing better is known, and deliberately not Sunday: ISO is
+ * the standard the rest of this package's date handling follows, and a default
+ * that matched one large locale would be a guess dressed as a convention.
+ */
+export const ISO_WEEK_START: number = 1;
+
+/** Days in a week, which is the width of every grid here. */
+export const DAYS_IN_WEEK: number = 7;
+
+/**
+ * What a key asks the focused date to become.
+ *
+ * A value rather than a new date, because the component that acts on it has to
+ * do two things with the answer — move the focus and possibly re-render a
+ * different month — and because "PageDown means a month" is the part worth
+ * asserting on its own.
+ */
+export type DateMovement =
+  | {| readonly kind: "days", readonly by: number |}
+  | {| readonly kind: "months", readonly by: number |}
+  | {| readonly kind: "years", readonly by: number |}
+  | {| readonly kind: "week-edge", readonly to: "start" | "end" |};
+
+/** The part of a key event a grid reads. */
+export type DateKeyPress = {
+  readonly key: string,
+  readonly shiftKey?: boolean,
+  ...
+};
+
+/**
+ * The movement a key asks for, or null when the key is not the grid's.
+ *
+ * The unhandled keys matter as much as the handled ones, for the reason
+ * `movementFor` gives: `Tab` belongs to the page and `Escape` belongs to
+ * whatever the calendar is inside, and a grid that swallowed either would be a
+ * place a reader could not leave.
+ *
+ * `direction` mirrors the horizontal pair and nothing else. `ArrowDown` is a
+ * week later in an Arabic calendar exactly as in an English one — a
+ * right-to-left page still runs top to bottom — and `Home` and `End` name the
+ * first and last day of the week in *reading* order, which is what
+ * `weekEdge` walks.
+ *
+ * `Shift` turns the two page keys into years, which is the one keyboard
+ * convention here that a reader cannot discover by trying: it is in the
+ * WAI-ARIA date-picker pattern, every native date field has it, and a year is
+ * otherwise twelve `PageDown` presses.
+ */
+export function movementForDateKey(event: DateKeyPress, direction: Direction): DateMovement | null {
+  const forward = direction === "rtl" ? -1 : 1;
+  const pages = event.shiftKey === true ? 12 : 1;
+  return match (event.key) {
+    "ArrowRight" => { kind: "days", by: forward },
+    "ArrowLeft" => { kind: "days", by: -forward },
+    "ArrowDown" => { kind: "days", by: DAYS_IN_WEEK },
+    "ArrowUp" => { kind: "days", by: -DAYS_IN_WEEK },
+    "Home" => { kind: "week-edge", to: "start" },
+    "End" => { kind: "week-edge", to: "end" },
+    // A year is twelve months rather than `{ years: 1 }`, so that the day is
+    // clamped once by the same rule the month keys use: the 29th of February
+    // plus a year is the 28th, and adding a year to a month-clamped date and
+    // adding twelve months have to agree or `Shift+PageDown` twice would not
+    // equal `PageDown` twenty-four times.
+    "PageUp" => { kind: "months", by: -pages },
+    "PageDown" => { kind: "months", by: pages },
+    _ => null,
+  };
+}
+
+/**
+ * How far into its week `date` sits, counting from `weekStartsOn`.
+ *
+ * Zero for the first column, six for the last, in whichever order the week is
+ * laid out. Both `weekEdge` and the grid's leading blanks are this number.
+ */
+export function columnOf(date: PlainDate, weekStartsOn: number): number {
+  return (date.dayOfWeek - weekStartsOn + DAYS_IN_WEEK) % DAYS_IN_WEEK;
+}
+
+/** The first or the last day of `date`'s week, as this locale lays a week out. */
+export function weekEdge(date: PlainDate, weekStartsOn: number, to: "start" | "end"): PlainDate {
+  const into = columnOf(date, weekStartsOn);
+  return to === "start"
+    ? date.subtract({ days: into })
+    : date.add({ days: DAYS_IN_WEEK - 1 - into });
+}
+
+/**
+ * The date `movement` reaches from `from`.
+ *
+ * Nothing here refuses: a movement onto an unavailable date, into a month with
+ * nothing selectable in it, or past whatever range the caller allows still
+ * returns that date. Refusing is the component's decision and it makes a
+ * different one — the module header says why a reader has to be able to walk
+ * over an unavailable day rather than around it.
+ *
+ * The month and year movements clamp the day, because `PlainDate.add` does: the
+ * 31st of January plus a month is the 28th of February rather than the 3rd of
+ * March, which is Temporal's `constrain` overflow and what a person means by
+ * "next month".
+ */
+export function moveDate(from: PlainDate, movement: DateMovement, weekStartsOn: number): PlainDate {
+  return match (movement) {
+    {kind: "days", by: const by} => from.add({ days: by }),
+    {kind: "months", by: const by} => from.add({ months: by }),
+    {kind: "years", by: const by} => from.add({ years: by }),
+    {kind: "week-edge", to: const to} => weekEdge(from, weekStartsOn, to),
+  };
+}
+
+/**
+ * The rows of one month, seven cells wide, with `null` where no day falls.
+ *
+ * The blanks are blanks rather than the neighbouring months' days, and that is
+ * the decision the rest of the keyboard behaviour rests on. A grid that showed
+ * the 1st of November inside October's would have two cells that mean the same
+ * date whenever a reader moved between them, and the WAI-ARIA pattern's
+ * promise — that `ArrowRight` off the end of the month *changes the month* — has
+ * nowhere to happen. Every blank is a `<td>` with no `gridcell` role, so the
+ * rows stay rectangular for a screen reader counting columns.
+ */
+export function weeksOf(
+  year: number,
+  month: number,
+  weekStartsOn: number,
+): $ReadOnlyArray<$ReadOnlyArray<PlainDate | null>> {
+  const first = Temporal.PlainDate.from({ day: 1, month, year });
+  const blanks = columnOf(first, weekStartsOn);
+  const days = first.daysInMonth;
+  const rows = Math.ceil((blanks + days) / DAYS_IN_WEEK);
+
+  const weeks = [];
+  for (let row = 0; row < rows; row += 1) {
+    const week = [];
+    for (let column = 0; column < DAYS_IN_WEEK; column += 1) {
+      const day = row * DAYS_IN_WEEK + column - blanks + 1;
+      week.push(day >= 1 && day <= days ? first.add({ days: day - 1 }) : null);
+    }
+    weeks.push(week);
+  }
+  return weeks;
+}
+
+/**
+ * Seven dates, one per column, for naming the columns.
+ *
+ * Dates rather than strings, because the names are the locale's and
+ * `PlainDate.toLocaleString` is where a locale's day names live. Any week would
+ * do; this one starts on a Monday so that `weekStartsOn` indexes it directly.
+ */
+export function weekdaysFrom(weekStartsOn: number): $ReadOnlyArray<PlainDate> {
+  const monday = Temporal.PlainDate.from("2024-01-01");
+  const days = [];
+  for (let column = 0; column < DAYS_IN_WEEK; column += 1) {
+    days.push(monday.add({ days: weekStartsOn - ISO_WEEK_START + column }));
+  }
+  return days;
+}
+
+/** Whether two dates are in the same month of the same year. */
+export function sameMonth(a: PlainDate, b: PlainDate): boolean {
+  return a.year === b.year && a.month === b.month;
+}
+
+/**
+ * The day `locale` starts its weeks on, in ISO numbering.
+ *
+ * `Intl.Locale.prototype.getWeekInfo` is the answer the platform has, and its
+ * `firstDay` is already ISO-numbered, so no translation is needed. Everything
+ * around the call is defence rather than logic: Flow's vendored `intl.js`
+ * declares no `Locale` at all, the method is newer than the class on some hosts,
+ * and a locale tag that is merely *malformed* throws where an unknown one does
+ * not. Each of those ends at ISO Monday, which is a defensible week rather than
+ * a broken one.
+ *
+ * The cast is the narrowest available: one property read off `Intl`, for a class
+ * the checker has never heard of. Widening it to the return value would hide
+ * whether `firstDay` was a number at all, which is why that is asked separately.
+ */
+export function firstDayOfWeekFor(locale: string | void): number {
+  const factory = (Intl as $FlowFixMe).Locale;
+  if (typeof factory !== "function") {
+    return ISO_WEEK_START;
+  }
+  try {
+    const info = new factory(locale ?? "und").getWeekInfo?.();
+    const first = info?.firstDay;
+    return typeof first === "number" && first >= 1 && first <= DAYS_IN_WEEK
+      ? first
+      : ISO_WEEK_START;
+  } catch {
+    return ISO_WEEK_START;
+  }
+}

package/internal/disclosure.js

@@ -48,6 +48,58 @@
 // or re-adds the attribute and this upgrades it again. Nothing here writes an
 // attribute React believes it owns while React believes it.
 //
+// # The height a closed panel would have
+//
+// The third rule, and the one that took a piece of work rather than a line.
+// `height: 0 → var(--uf-collapsible-height)` is the whole of animating a
+// disclosure, and the number in that property is the one thing a stylesheet
+// cannot compute: it is the height the content *would* have, wanted at the
+// moment the panel is still closed, because a transition has to know its
+// destination before it starts.
+//
+// Every obvious way of getting it answers zero. A closed panel is `hidden`, so
+// it has no box: `ResizeObserver` reports `0`, `getBoundingClientRect()` is
+// empty, `scrollHeight` is `0`, and `useElementSize` from
+// `@uniflowed/hooks/dom` measures a hidden element and reports zero — which is
+// exactly the moment the number is wanted. Measuring after the panel opens
+// gives the right number one frame late, which is the jank this removes.
+//
+// So `useMeasuredHeight` lays the panel out without painting it: inline
+// `display`, `position: absolute` and `visibility: hidden`, read, restore. Four
+// things about that are not obvious:
+//
+//   * It overrides `display` inline rather than removing `hidden`. The
+//     attribute is `useUntilFound`'s and React's; an effect that took it away
+//     and put it back would be fighting both of them for one frame, and would
+//     lose whichever ran last. An inline declaration beats the user-agent
+//     stylesheet's `[hidden] { display: none }` and touches nothing anybody
+//     else believes they own.
+//   * It sets `content-visibility: visible` in the same pass, because
+//     `hidden="until-found"` is `content-visibility: hidden`, which does not lay
+//     its subtree out either. Defeating one of the two and not the other
+//     measures zero on exactly the panels this package ships.
+//   * It sets `height: auto` in the same pass, for the same reason and against
+//     the very rule this property exists for. A closed panel is `height: 0` —
+//     that is the half of `height: 0 → var(--uf-collapsible-height)` that is
+//     always on — so a panel laid out with the stylesheet still applying
+//     measures zero and writes `0px` back into the property it was asked to
+//     fill. Defeating `display` and not `height` is the same mistake as
+//     defeating `display` and not `content-visibility`, one declaration along.
+//   * And it pins the width, because taking a box out of flow makes it
+//     shrink-to-fit against its containing block — the nearest positioned
+//     ancestor, which on most pages is the viewport. Text that wraps to four
+//     lines where the panel lives measures one line there. The width it would
+//     have in flow is its parent's content box; when that cannot be read the
+//     pass leaves the width alone rather than inventing one.
+//
+// It is opt-in, and that is the honest answer to "it should cost nothing on a
+// page that never animates". Whether a stylesheet reads the property is not
+// something the component can ask — `getComputedStyle` on a `display: none`
+// element answers about the declaration, not about whether anybody transitions
+// on it — so the alternative to a prop is a forced layout per panel per render
+// on every page that has a collapsible, animated or not. A forty-section FAQ is
+// forty synchronous layouts nobody asked for.
+//
 // # Why this is `internal/` and not a subpath
 //
 // The same reason `roving-focus.js` gives. These are rules about markup this
@@ -95,3 +147,152 @@ export hook useUntilFound(ref: { current: HTMLElement | null }, open: boolean):
     element.setAttribute("hidden", "until-found");
   }, [ref, open]);
 }
+
+/** The custom property a stylesheet transitions a disclosure's height to. */
+export const HEIGHT_PROPERTY: string = "--uf-collapsible-height";
+
+/**
+ * The width `element` would have in flow, as a CSS length, or nothing.
+ *
+ * Taking the panel out of flow to measure it costs shrink-to-fit: an absolutely
+ * positioned box with `width: auto` is as wide as its *content* wants to be,
+ * bounded by its containing block — which is the nearest positioned ancestor
+ * and, on most pages, is the viewport rather than the panel's parent. A
+ * paragraph that wraps to four lines in a sidebar measures one line there, and
+ * the number written into the property is then a height the panel never has.
+ *
+ * The width it would have in flow is its parent's content box, which is what
+ * `getComputedStyle` reports for `width` on a laid-out element. Nothing is
+ * returned when the answer is not a length — a parent that is itself
+ * `display: none`, or a DOM with no layout to report — and the pass then does
+ * what it did before rather than pinning a width it had to guess.
+ */
+function widthInFlow(element: HTMLElement): string | null {
+  const parent = element.parentElement;
+  const view: $FlowFixMe = element.ownerDocument?.defaultView;
+  if (parent == null || view == null || typeof view.getComputedStyle !== "function") {
+    return null;
+  }
+  const width: mixed = view.getComputedStyle(parent).width;
+  return typeof width === "string" && width.endsWith("px") ? width : null;
+}
+
+/**
+ * The height `element` has, or would have if it were not hidden.
+ *
+ * The measuring pass, and the reason this module has a header section about it.
+ * An open panel is measured where it stands; a closed one is briefly laid out
+ * and not painted. Either way this reads layout, which is a synchronous reflow
+ * — the caller is the one that decides it is worth paying.
+ *
+ * # The two things a closed panel is not, and has to be made
+ *
+ * Laying it out is necessary and is not sufficient, because the stylesheet this
+ * property exists for is `height: 0` on the closed panel and
+ * `height: var(--uf-collapsible-height)` on the open one. A panel laid out with
+ * that rule still applying measures **zero**, the property is written back as
+ * `0px`, and the transition has a destination of nothing — the bug the whole
+ * hook was written to avoid, arriving through the rule it was written for. So
+ * `height: auto` is set inline for the pass, exactly as `display` is: not
+ * because the panel wants an inline height but because the author declaration
+ * has to be defeated for one synchronous read and put back.
+ *
+ * `width` is the same argument for the other axis and is `widthInFlow`'s. Both
+ * are restored with everything else; a stylesheet is never left fighting an
+ * inline declaration this wrote.
+ */
+function heightOf(element: HTMLElement): number {
+  if (!element.hasAttribute("hidden")) {
+    return element.getBoundingClientRect().height;
+  }
+  const style = element.style;
+  const before = {
+    boxSizing: style.boxSizing,
+    contentVisibility: style.getPropertyValue("content-visibility"),
+    display: style.display,
+    height: style.height,
+    position: style.position,
+    visibility: style.visibility,
+    width: style.width,
+  };
+  // Out of flow and unpainted, so nothing below the panel moves and no frame
+  // shows it. The order is not significant; this is all one style
+  // recalculation, paid for by the single read below.
+  style.display = "block";
+  style.setProperty("content-visibility", "visible");
+  style.position = "absolute";
+  style.visibility = "hidden";
+  style.height = "auto";
+  const width = widthInFlow(element);
+  if (width != null) {
+    // `border-box`, because what fills the parent's content width in flow is
+    // the panel's margin box rather than its content box: measuring a padded
+    // panel content-box wide would make it wider than it will ever be and its
+    // text shorter than it will ever wrap to.
+    style.boxSizing = "border-box";
+    style.width = width;
+  }
+  const height = element.getBoundingClientRect().height;
+  style.boxSizing = before.boxSizing;
+  style.display = before.display;
+  style.setProperty("content-visibility", before.contentVisibility);
+  style.height = before.height;
+  style.position = before.position;
+  style.visibility = before.visibility;
+  style.width = before.width;
+  return height;
+}
+
+/**
+ * Keep `HEIGHT_PROPERTY` on a disclosure's panel equal to its content's height.
+ *
+ * `enabled` is the caller's opt-in; see the module header for why there is one.
+ * When it is off this writes nothing and measures nothing, which is what makes
+ * a page with no animation pay nothing.
+ *
+ * Two effects rather than one, because they answer different questions. The
+ * first measures on every render, with no dependency array on purpose: what the
+ * panel would be worth changes when the *caller* renders different children
+ * into it, and a dependency list here would be a claim about when that happens
+ * that only the caller could keep — `useFirstItem` in `roving-focus.js` makes
+ * the same argument for the same reason. The second watches for a size change
+ * no render caused: an image that finished loading, a font that swapped. It
+ * only ever fires while the panel is open, because a `display: none` element
+ * has no box for a `ResizeObserver` to report on — which is the whole problem
+ * this hook exists for, arriving one more time.
+ */
+export hook useMeasuredHeight(ref: { current: HTMLElement | null }, enabled: boolean): void {
+  useEffect(() => {
+    const element = ref.current;
+    if (!enabled || element == null) {
+      return;
+    }
+    const measured = `${String(heightOf(element))}px`;
+    // Compared before writing, so a render that changed nothing does not dirty
+    // the element's style and invite another style recalculation.
+    if (element.style.getPropertyValue(HEIGHT_PROPERTY) !== measured) {
+      element.style.setProperty(HEIGHT_PROPERTY, measured);
+    }
+  });
+
+  useEffect(() => {
+    const element = ref.current;
+    const view = element?.ownerDocument?.defaultView;
+    if (!enabled || element == null || view == null) {
+      return;
+    }
+    // Read off the window rather than through a local, for the reason
+    // `internal/anchor.js` gives where it does the same: a capitalised name
+    // holding a constructor is read as a React component by `uf lint`, and the
+    // window's own property is the thing being asked about anyway.
+    const host: $FlowFixMe = view;
+    if (typeof host.ResizeObserver !== "function") {
+      return;
+    }
+    const sizes = new host.ResizeObserver(() => {
+      element.style.setProperty(HEIGHT_PROPERTY, `${String(heightOf(element))}px`);
+    });
+    sizes.observe(element);
+    return () => sizes.disconnect();
+  }, [ref, enabled]);
+}