@crisp-ui-kit/crisp 0.51.0 → 0.52.1

package/dist/esm/components/datatable/datatable.js

@@ -218,12 +218,61 @@ const CREATE_REFERENCE_VALUE = "__crisp_create_reference__";
 function dataReferenceOptionLabel(o) {
     return (_jsxs("span", { className: "crisp-datatable-refoption", children: [_jsx(Avatar, { size: "sm", className: "crisp-datatable-refoption-avatar", children: o.avatar ? (_jsx("img", { src: o.avatar, alt: "" })) : (_jsx(CircleUserRound, { size: 12, strokeWidth: 1.75, "aria-hidden": "true" })) }), o.label] }));
 }
+/** `Select` owns its own closed-by-default `open` state with no prop or
+ * imperative way to start it open (see select.tsx) -- the only thing that
+ * opens it is a real click on its trigger button. Attio's own picker opens
+ * the instant a `select`/`reference` cell enters edit mode (measured live
+ * against a GRC test company, 2026-09-24), not after a second click on an
+ * already-mounted, still-closed trigger, so drive that click ourselves.
+ * Guarded against firing twice (StrictMode double-invokes effects) --
+ * `Select`'s own trigger toggles, so a second click would close what the
+ * first one opened. */
+function useAutoOpenSelect() {
+    const ref = React.useRef(null);
+    const opened = React.useRef(false);
+    React.useEffect(() => {
+        if (opened.current)
+            return;
+        opened.current = true;
+        ref.current
+            ?.querySelector(".crisp-select-trigger")
+            ?.click();
+    }, []);
+    return ref;
+}
+/** `select`/`reference` open crisp's own `Select`, which closes its own
+ * popover on Escape (see select.tsx's "document-level fallback for Escape")
+ * but has no callback telling the CELL to leave editing mode -- and, same
+ * problem that fallback exists for, DOM focus doesn't reliably end up
+ * inside the newly-opened popover, so a `TableCell`-level `onKeyDown` can't
+ * be trusted to see the key at all (#453). Mirror Select's own fix for
+ * that: a raw `document` listener, not a focus-dependent React one. */
+function useEscapeExitsEditing(onCancel) {
+    React.useEffect(() => {
+        const onKey = (e) => {
+            if (e.key === "Escape")
+                onCancel();
+        };
+        document.addEventListener("keydown", onKey);
+        return () => document.removeEventListener("keydown", onKey);
+    }, [onCancel]);
+}
+/** `editor: "select"`'s picker, its own component (not inline in the
+ * row-render loop, like `DataReferenceEditor` below) because it needs the
+ * open-on-mount and escape-exits-editing effects above (#453). */
+function DataSelectEditor({ col, value, isPending, onCommit, onCancel, }) {
+    const ref = useAutoOpenSelect();
+    useEscapeExitsEditing(onCancel);
+    return (_jsx("span", { ref: ref, style: { display: "contents" }, children: _jsx(Select, { variant: "ghost", "aria-label": `Edit ${col.key}`, value: asText(value) || undefined, disabled: isPending, options: col.options ?? [], onValueChange: onCommit }) }));
+}
 /** `editor: "reference"`'s picker, its own component (not inline in the
  * row-render loop) because it needs its own `useState` for the search query
  * -- one instance per cell, same reasoning `select`/`switch` don't need a
  * component of their own but this does. */
-function DataReferenceEditor({ col, value, isPending, onCommit, }) {
+function DataReferenceEditor({ col, value, isPending, onCommit, onCancel, }) {
     const [query, setQuery] = React.useState("");
+    const ref = useAutoOpenSelect();
+    useEscapeExitsEditing(onCancel);
     const options = (col.refOptions ?? []).map((o) => ({
         value: o.value,
         label: dataReferenceOptionLabel(o),
@@ -238,12 +287,12 @@ function DataReferenceEditor({ col, value, isPending, onCommit, }) {
             keywords: query,
         });
     }
-    return (_jsx(Select, { variant: "ghost", searchable: true, searchPlaceholder: col.refSearchPlaceholder, onSearchChange: setQuery, "aria-label": `Edit ${col.key}`, value: asText(value) || undefined, disabled: isPending, options: options, onValueChange: (v) => {
-            if (v === CREATE_REFERENCE_VALUE)
-                col.onCreateReference?.(query);
-            else
-                onCommit(v);
-        } }));
+    return (_jsx("span", { ref: ref, style: { display: "contents" }, children: _jsx(Select, { variant: "ghost", searchable: true, searchPlaceholder: col.refSearchPlaceholder, onSearchChange: setQuery, "aria-label": `Edit ${col.key}`, value: asText(value) || undefined, disabled: isPending, options: options, onValueChange: (v) => {
+                if (v === CREATE_REFERENCE_VALUE)
+                    col.onCreateReference?.(query);
+                else
+                    onCommit(v);
+            } }) }));
 }
 export function DataTable({ columns: columnsProp, data, selectable = false, selectedIds, onSelectionChange, selectAllLabel = "Select all rows", selectRowLabel = (row) => `Select row ${row.id}`, selectionAnnouncement = (count) => count === 0 ? "Selection cleared" : `${count} selected`, columnMenu = false, onColumnsChange, groupBy, onRowsReorder, error, sort: sortProp, defaultSort, onSortChange, filterable = false, searchPlaceholder = "Search…", searchLabel = "Search table", onEdit, stickyFirst = true, stickyLast = false, locale: localeProp, avatarTone = "brand", surface, interactive = false, onRowClick, onAddRow, addRowLabel = "New record", empty = "No results.", showCount = true, countLabel = (shown, total) => shown === total
     ? `${total} ${total === 1 ? "row" : "rows"}`
@@ -466,10 +515,11 @@ export function DataTable({ columns: columnsProp, data, selectable = false, sele
      *                     throughout, and the browser's own picker already
      *                     carries both a calendar and a time control.
      *   text/entity       a plain text field
-     * `status`, `reference` and `tags` are pickers in RecordsTable, not
-     * double-click editors, so they are not editable here either -- except
-     * `editor: "reference"` (#432), which is a picker for the same reason
-     * `select`/`switch` are: always live, no double-click gate.
+     * `editor: "select"`/`"reference"` open their own picker instead of this
+     * native-input family, but through the SAME double-click gate (#453) --
+     * see the row-render loop's `isEditing` branch for those, this function
+     * only covers the native-input-backed types. `editor: "switch"` is the
+     * one editor with no double-click gate at all (always live).
      */
     function editorFor(col) {
         switch (col.type) {
@@ -523,11 +573,12 @@ export function DataTable({ columns: columnsProp, data, selectable = false, sele
             return;
         }
         else if (e.key === "Enter") {
-            // select/switch columns have no separate editing mode to enter --
-            // they're already their own live widget (#385); Enter there is a
-            // no-op here, same as any other non-editable cell that isn't wired to
-            // onRowClick.
-            if (columns[c]?.editable && !columns[c]?.editor) {
+            // #453: `switch` has no separate editing mode to enter -- it's
+            // already its own live widget (#385); Enter there is a no-op, same
+            // as any other non-editable cell that isn't wired to onRowClick.
+            // `select`/`reference` DO have one now (like every other editor),
+            // so Enter opens them too.
+            if (columns[c]?.editable && columns[c]?.editor !== "switch") {
                 e.preventDefault();
                 openEditing(selCell.id, selCell.key);
             }
@@ -758,14 +809,37 @@ export function DataTable({ columns: columnsProp, data, selectable = false, sele
                                             const errorHere = editError?.id === row.id && editError?.key === col.key
                                                 ? editError.message
                                                 : null;
-                                            // A select/switch editable column is ALWAYS its own
-                                            // live widget -- no double-click gate, no separate
-                                            // edit/view mode, the same convention RecordsTable's
-                                            // status/reference pickers already use: choosing a
-                                            // value or flipping a toggle IS the commit gesture
-                                            // (#385).
-                                            if (col.editable && col.editor) {
-                                                return (_jsxs(TableCell, { align: alignFor(col), ...sticky, className: "crisp-datatable-td--picker", "data-pending": isPending ? "" : undefined, children: [_jsxs("span", { className: "crisp-datatable-edit-picker", children: [col.editor === "select" ? (_jsx(Select, { variant: "ghost", "aria-label": `Edit ${col.key}`, value: asText(value) || undefined, disabled: isPending, options: col.options ?? [], onValueChange: (v) => commitEdit(row.id, col.key, v) })) : col.editor === "reference" ? (_jsx(DataReferenceEditor, { col: col, value: value, isPending: isPending, onCommit: (v) => commitEdit(row.id, col.key, v) })) : (_jsx(Switch, { "aria-label": `Edit ${col.key}`, checked: value === true || value === "true", disabled: isPending, onCheckedChange: (checked) => commitEdit(row.id, col.key, String(checked)) })), isPending && (_jsx(Spinner, { size: 14, className: "crisp-datatable-edit-pending" }))] }), errorHere && (_jsx("span", { className: "crisp-datatable-edit-error", children: errorHere }))] }, col.key));
+                                            // #453: a `switch` editable column IS always its own
+                                            // live widget -- flipping the toggle IS the commit
+                                            // gesture, there's no meaningfully different "rest"
+                                            // display for a boolean. `select`/`reference` are
+                                            // NOT this: at rest they must show the column's own
+                                            // render/format like every other editable type
+                                            // (#412 empty handling included), and only open on
+                                            // double-click (#411) -- this branch previously
+                                            // rendered them unconditionally, replacing every
+                                            // row's own render with a picker trigger, which is
+                                            // the regression #453 reported (also #432/#449,
+                                            // which never had a "rest" state of their own to get
+                                            // right). See the `isEditing` block below for their
+                                            // actual editor.
+                                            if (col.editable && col.editor === "switch") {
+                                                return (_jsxs(TableCell, { align: alignFor(col), ...sticky, className: "crisp-datatable-td--picker", "data-pending": isPending ? "" : undefined, children: [_jsxs("span", { className: "crisp-datatable-edit-picker", children: [_jsx(Switch, { "aria-label": `Edit ${col.key}`, checked: value === true || value === "true", disabled: isPending, onCheckedChange: (checked) => commitEdit(row.id, col.key, String(checked)) }), isPending && (_jsx(Spinner, { size: 14, className: "crisp-datatable-edit-pending" }))] }), errorHere && (_jsx("span", { className: "crisp-datatable-edit-error", children: errorHere }))] }, col.key));
+                                            }
+                                            if (isEditing &&
+                                                (col.editor === "select" || col.editor === "reference")) {
+                                                return (_jsxs(TableCell, { align: alignFor(col), ...sticky, className: "crisp-datatable-td--picker", "data-pending": isPending ? "" : undefined, 
+                                                    // Same as the plain text editor's own cell:
+                                                    // opening/using the picker must not also fire
+                                                    // the row's onRowClick. Escape is handled by
+                                                    // `DataSelectEditor`/`DataReferenceEditor`
+                                                    // themselves (a document-level listener, not a
+                                                    // `TableCell`-level `onKeyDown`) -- opening the
+                                                    // picker moves DOM focus into a floating,
+                                                    // portaled popover, so a handler here can't
+                                                    // rely on the key event ever bubbling through
+                                                    // this element (#453).
+                                                    onClick: (e) => e.stopPropagation(), children: [_jsxs("span", { className: "crisp-datatable-edit-picker", children: [col.editor === "select" ? (_jsx(DataSelectEditor, { col: col, value: value, isPending: isPending, onCommit: (v) => commitEdit(row.id, col.key, v), onCancel: () => setEditing(null) })) : (_jsx(DataReferenceEditor, { col: col, value: value, isPending: isPending, onCommit: (v) => commitEdit(row.id, col.key, v), onCancel: () => setEditing(null) })), isPending && (_jsx(Spinner, { size: 14, className: "crisp-datatable-edit-pending" }))] }), errorHere && (_jsx("span", { className: "crisp-datatable-edit-error", children: errorHere }))] }, col.key));
                                             }
                                             if (isEditing) {
                                                 return (_jsxs(TableCell, { align: alignFor(col), ...sticky, onClick: (e) => e.stopPropagation(), children: [_jsx("input", { className: "crisp-datatable-edit", 

package/dist/esm/core/caret-rect.js

@@ -0,0 +1,70 @@
+/**
+ * The pixel position of a caret index inside a `<textarea>`/`<input>`, in
+ * viewport coordinates — where a mention picker anchors.
+ *
+ * Neither element exposes caret geometry directly, so this uses the standard
+ * mirror-div technique: an off-screen div replicating every computed style
+ * that affects text layout (font, padding, border, wrapping), holding the
+ * same text up to the caret plus a marker span. The marker's own
+ * `getBoundingClientRect()`, translated back onto the real field's box, is
+ * the caret's actual rendered position — nothing here guesses at line
+ * height or character width itself.
+ */
+export function caretRectInField(field, index) {
+    const div = document.createElement("div");
+    const style = window.getComputedStyle(field);
+    const mirrored = [
+        "boxSizing",
+        "width",
+        "paddingTop",
+        "paddingRight",
+        "paddingBottom",
+        "paddingLeft",
+        "borderTopWidth",
+        "borderRightWidth",
+        "borderBottomWidth",
+        "borderLeftWidth",
+        "fontFamily",
+        "fontSize",
+        "fontWeight",
+        "fontStyle",
+        "letterSpacing",
+        "lineHeight",
+        "textTransform",
+        "textIndent",
+        "textAlign",
+        "wordSpacing",
+        "tabSize",
+    ];
+    for (const prop of mirrored)
+        div.style.setProperty(prop, style[prop]);
+    div.style.position = "absolute";
+    div.style.visibility = "hidden";
+    div.style.top = "0";
+    div.style.left = "-9999px";
+    div.style.whiteSpace =
+        field instanceof HTMLTextAreaElement ? "pre-wrap" : "pre";
+    div.style.wordWrap = "break-word";
+    div.style.overflow = "hidden";
+    div.style.height = "auto";
+    const value = field.value;
+    div.textContent = value.slice(0, index);
+    const marker = document.createElement("span");
+    // a trailing empty span collapses to zero width and reports the wrong x —
+    // give it a hair of content so its box sits exactly at the caret
+    marker.textContent = value.slice(index, index + 1) || ".";
+    div.appendChild(marker);
+    document.body.appendChild(div);
+    div.scrollTop = field.scrollTop;
+    div.scrollLeft = field.scrollLeft;
+    const fieldRect = field.getBoundingClientRect();
+    const divRect = div.getBoundingClientRect();
+    const markerRect = marker.getBoundingClientRect();
+    const x = fieldRect.left + (markerRect.left - divRect.left) - div.scrollLeft;
+    const y = fieldRect.top +
+        (markerRect.top - divRect.top) -
+        div.scrollTop +
+        markerRect.height;
+    document.body.removeChild(div);
+    return { x, y };
+}

package/dist/esm/core/index.js

@@ -1,3 +1,4 @@
+export { caretRectInField } from "./caret-rect.js";
 export { cellDatePrecision, formatCellDate, parseCellDate, } from "./cell-date.js";
 export { cx } from "./cx.js";
 export { defineRecipe } from "./define.js";
@@ -6,6 +7,7 @@ export { isRtlElement } from "./direction.js";
 export { FloatingPortal } from "./floating-portal.js";
 export { hasOpenFloatingLayer, isInsideLaterFloatingLayer, registerFloatingLayer, } from "./floating-stack.js";
 export { formValueStyle } from "./form-value.js";
+export { matchMentionTrigger, } from "./mention.js";
 export { placeFloating, } from "./place-floating.js";
 export { Primitive } from "./primitive.js";
 export { resolveRecipe } from "./resolve.js";
@@ -14,4 +16,5 @@ export { defineSpec, specToRecipe } from "./spec.js";
 export { useFocusAfterRemoval } from "./use-focus-after-removal.js";
 export { useFormReset } from "./use-form-reset.js";
 export { useLabelAssociation } from "./use-label-association.js";
+export { useMentionTrigger, } from "./use-mention-trigger.js";
 export { usePrefersReducedMotion } from "./use-prefers-reduced-motion.js";

package/dist/esm/core/mention.js

@@ -0,0 +1,29 @@
+/**
+ * Is there an active "@query" mention trigger ending at the caret?
+ *
+ * Matches the rule every "@mention" implementation (GitHub, Slack, Linear)
+ * actually uses: the trigger character must start a word — the very first
+ * character, or preceded by whitespace — and nothing between it and the
+ * caret may contain whitespace. `foo@bar` is an email/handle stuck to a word,
+ * not a trigger; `@jo next` has already moved past the mention it typed.
+ *
+ * Takes plain text rather than a live DOM node so it is one pure function
+ * usable against a `<textarea>`'s `value` and a `contentEditable` text
+ * node's content alike, and is the one thing here worth unit-testing on its
+ * own — everything downstream of it (caret geometry, DOM insertion) needs a
+ * real browser to verify.
+ */
+export function matchMentionTrigger(textBeforeCaret, trigger = "@") {
+    if (!trigger)
+        return null;
+    const start = textBeforeCaret.lastIndexOf(trigger);
+    if (start === -1)
+        return null;
+    const before = start > 0 ? textBeforeCaret[start - 1] : undefined;
+    if (before !== undefined && !/\s/.test(before))
+        return null;
+    const query = textBeforeCaret.slice(start + trigger.length);
+    if (/\s/.test(query))
+        return null;
+    return { query, start };
+}

package/dist/esm/core/use-mention-trigger.js

@@ -0,0 +1,88 @@
+import * as React from "react";
+import { caretRectInField } from "./caret-rect.js";
+import { matchMentionTrigger, } from "./mention.js";
+/**
+ * Watches a `<textarea>`/`<input>` for an "@query" mention trigger and
+ * reports where to anchor a consumer-supplied picker (#372).
+ *
+ * crisp draws no picker of its own here — GitHub/Slack/Linear-style @mention
+ * UI varies too much by app (avatars, groups, recent-first ordering, a
+ * "create new person" row) to have one measured shape, and this repo's rule
+ * is measure-first, never invented polish. What is shared and worth owning
+ * centrally is the fiddly, easy-to-get-wrong part: deciding when "@" is
+ * actually a trigger rather than part of a word or an email address (see
+ * `matchMentionTrigger`), and where its picker anchors on screen (see
+ * `caretRectInField`, since neither element exposes caret geometry itself).
+ *
+ * Does not close on blur: a picker built from this needs to survive the
+ * field losing focus to a `mousedown` on one of its own rows, the same
+ * problem every floating surface in this repo solves for itself (see
+ * `RichTextEditor`'s toolbar `keepSel`, or `Popover`'s own outside-pointer-
+ * down listener) — so closing on outside interaction is left to the
+ * consumer's picker, which already knows what counts as "outside" for its
+ * own markup.
+ */
+export function useMentionTrigger(ref, { trigger = "@" } = {}) {
+    const [state, setState] = React.useState({
+        open: false,
+        query: "",
+        anchor: null,
+    });
+    const matchRef = React.useRef(null);
+    const recompute = React.useCallback(() => {
+        const el = ref.current;
+        if (!el || el.selectionStart == null) {
+            matchRef.current = null;
+            setState({ open: false, query: "", anchor: null });
+            return;
+        }
+        const caret = el.selectionStart;
+        const match = matchMentionTrigger(el.value.slice(0, caret), trigger);
+        matchRef.current = match;
+        if (!match) {
+            setState({ open: false, query: "", anchor: null });
+            return;
+        }
+        setState({
+            open: true,
+            query: match.query,
+            anchor: caretRectInField(el, caret),
+        });
+    }, [ref, trigger]);
+    React.useEffect(() => {
+        const el = ref.current;
+        if (!el)
+            return;
+        // "input" covers typing; "click"/"keyup" cover the caret moving without
+        // the text changing (arrow keys, clicking elsewhere on the same line).
+        el.addEventListener("input", recompute);
+        el.addEventListener("click", recompute);
+        el.addEventListener("keyup", recompute);
+        return () => {
+            el.removeEventListener("input", recompute);
+            el.removeEventListener("click", recompute);
+            el.removeEventListener("keyup", recompute);
+        };
+    }, [ref, recompute]);
+    const close = React.useCallback(() => {
+        matchRef.current = null;
+        setState({ open: false, query: "", anchor: null });
+    }, []);
+    const insert = React.useCallback((text) => {
+        const el = ref.current;
+        const match = matchRef.current;
+        if (!el || !match)
+            return;
+        const caret = el.selectionStart ?? el.value.length;
+        const before = el.value.slice(0, match.start);
+        const after = el.value.slice(caret);
+        el.value = before + text + after;
+        const cursor = before.length + text.length;
+        el.focus();
+        el.setSelectionRange(cursor, cursor);
+        // a controlled consumer listens on "input", the same event typing fires
+        el.dispatchEvent(new Event("input", { bubbles: true }));
+        close();
+    }, [ref, close]);
+    return { ...state, close, insert };
+}

package/dist/esm/product/recordpanel/index.js

@@ -1,2 +1,2 @@
-export { RecordPanel } from "./recordpanel.js";
+export { RecordPanel, } from "./recordpanel.js";
 export { recordpanelRecipe } from "./recordpanel.recipe.js";

package/dist/esm/product/recordpanel/recordpanel.js

@@ -1,38 +1,21 @@
-import { Fragment as _Fragment, jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
-import { X } from "lucide-react";
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+import { ArrowUpRight, ChevronDown, ChevronUp, X } from "lucide-react";
 import * as React from "react";
 import { Avatar } from "../../components/avatar/index.js";
-import { Select } from "../../components/select/index.js";
-import { Spinner } from "../../components/spinner/index.js";
-import { Switch } from "../../components/switch/index.js";
 import { hasOpenFloatingLayer } from "../../core/index.js";
-import { editableInputType, tagTone, useEditableField, } from "../recordstable/index.js";
 import { recordpanelRecipe } from "./recordpanel.recipe.js";
-function FieldValue({ col, row, emptyLabel, }) {
-    const v = row[col.key];
-    if (v == null || v === "" || (Array.isArray(v) && !v.length)) {
-        return _jsx("span", { className: "crisp-recordpanel-empty", children: emptyLabel });
-    }
-    switch (col.type) {
-        case "tags":
-            return (_jsx(_Fragment, { children: v.map((t) => (_jsx("span", { className: "crisp-recordstable-tag", "data-tone": tagTone(t), children: t }, t))) }));
-        case "number":
-            return _jsx(_Fragment, { children: typeof v === "number" ? v.toLocaleString() : v });
-        case "url":
-            return (_jsx("a", { className: "crisp-recordpanel-link", href: `https://${v}`, target: "_blank", rel: "noreferrer", children: v }));
-        default:
-            return _jsx(_Fragment, { children: String(v) });
-    }
-}
 /**
- * Right-side record peek panel — shows one record's fields, opened from a
- * RecordsTable row (via `onOpenRecord`). Render it inside a `position:
- * relative` container (e.g. the app main area); it slides in from the right and
- * overlays the content.
+ * Right-side record peek panel — Attio's own peek shows a Highlights card
+ * area and an Activity feed (measured live: hover a Companies row, click its
+ * "Open preview" trigger), never an editable field list (#448 — a "Details"
+ * list was this component's own invention before, and #431's editable API
+ * had nothing real in this panel to attach to; that API stays on
+ * `RecordPage`'s own Details column, where Attio does let you edit in
+ * place). Render it inside a `position: relative` container (e.g. the app
+ * main area); it slides in from the right and overlays the content.
  */
-export function RecordPanel({ record, columns, onClose, className, closeLabel = "Close panel", title, detailsLabel = "Details", emptyLabel = "Empty", footer, onEdit, ...rest }) {
+export function RecordPanel({ record, onClose, className, closeLabel = "Close panel", onOpenRecord, openRecordLabel = "Open record", onPrevious, previousRecordLabel = "Previous record", onNext, nextRecordLabel = "Next record", title, avatar, avatarTone = "brand", highlightsLabel = "Highlights", highlights, activityLabel = "Activity", viewAllActivityLabel = "View all", onViewAllActivity, activity, footer, ...rest }) {
     const titleId = React.useId();
-    const { editingKey, pendingKey, error, openEditing, closeEditing, commit } = useEditableField(onEdit);
     // close on Escape while open -- but not when a Menu/Select/Popover/
     // ContextMenu opened from inside the panel (e.g. a footer "More actions"
     // menu) is itself open (#361): that Escape belongs to the nested layer,
@@ -57,67 +40,16 @@ export function RecordPanel({ record, columns, onClose, className, closeLabel =
         if (!record)
             return;
         const onKey = (e) => {
-            if (e.key !== "Escape")
-                return;
-            // #431: Escape while a field is open cancels JUST that field, the
-            // same way DataTable's own editable cells treat it -- not the whole
-            // panel. Checked ahead of `hasOpenFloatingLayer()` (capture phase, so
-            // this runs before the field's own bubble-phase onKeyDown would ever
-            // get a chance to stopPropagation -- same reasoning as the
-            // floating-layer check below).
-            if (editingKey) {
-                closeEditing();
-                return;
-            }
-            if (hasOpenFloatingLayer())
+            if (e.key !== "Escape" || hasOpenFloatingLayer())
                 return;
             onClose();
         };
         document.addEventListener("keydown", onKey, true);
         return () => document.removeEventListener("keydown", onKey, true);
-    }, [record, onClose, editingKey, closeEditing]);
+    }, [record, onClose]);
     if (!record)
         return null;
-    const entityCol = columns.find((c) => c.type === "entity");
-    const entity = entityCol
-        ? record[entityCol.key]
-        : undefined;
-    const fields = columns.filter((c) => c.type !== "entity");
-    const heading = title ?? entity?.name ?? "Record";
-    return (_jsxs("div", { className: recordpanelRecipe({ className }), ...rest, role: "dialog", "aria-labelledby": titleId, children: [_jsxs("div", { className: "crisp-recordpanel-header", children: [entity?.avatar ? (_jsx("img", { className: "crisp-recordpanel-fav", src: entity.avatar, alt: "" })) : entity ? (_jsx(Avatar, { size: "md", shape: "rounded", tone: "brand", children: entity.name.charAt(0) })) : null, _jsx("span", { id: titleId, className: "crisp-recordpanel-title", children: heading }), _jsx("button", { type: "button", className: "crisp-recordpanel-iconbtn", "aria-label": closeLabel, onClick: onClose, children: _jsx(X, { size: 18, strokeWidth: 1.75 }) })] }), _jsxs("div", { className: "crisp-recordpanel-body", children: [_jsx("div", { className: "crisp-recordpanel-sectionlabel", children: detailsLabel }), fields.map((col) => {
-                        // #431: a field only actually opens when there's an `onEdit` to
-                        // commit through -- `editable` with nothing to call would be a
-                        // dead click.
-                        const canEdit = Boolean(col.editable && onEdit);
-                        const isPending = pendingKey === col.key;
-                        const fieldError = error?.key === col.key ? error.message : null;
-                        // select/switch are always their own live widget -- no click-to-
-                        // open, no separate edit/view mode, the same convention
-                        // DataTable's identical `editor` columns use (#385): choosing a
-                        // value or flipping a toggle IS the commit gesture.
-                        if (canEdit && col.editor) {
-                            const value = record[col.key];
-                            return (_jsxs("div", { className: "crisp-recordpanel-field", children: [_jsxs("span", { className: "crisp-recordpanel-field-label", children: [col.icon ? _jsx(col.icon, { size: 15, strokeWidth: 1.75 }) : null, col.label] }), _jsxs("span", { className: "crisp-recordpanel-field-value", children: [_jsxs("span", { className: "crisp-recordpanel-edit-picker", children: [col.editor === "select" ? (_jsx(Select, { variant: "ghost", "aria-label": `Edit ${col.label}`, value: value != null ? String(value) : undefined, disabled: isPending, options: col.editOptions ?? [], onValueChange: (v) => commit(record.id, col.key, v) })) : (_jsx(Switch, { "aria-label": `Edit ${col.label}`, checked: value === true || value === "true", disabled: isPending, onCheckedChange: (checked) => commit(record.id, col.key, String(checked)) })), isPending && (_jsx(Spinner, { size: 14, className: "crisp-recordpanel-edit-pending" }))] }), fieldError && (_jsx("span", { className: "crisp-recordpanel-edit-error", children: fieldError }))] })] }, col.key));
-                        }
-                        if (canEdit && editingKey === col.key) {
-                            const value = record[col.key];
-                            const editor = editableInputType(col.type);
-                            return (_jsxs("div", { className: "crisp-recordpanel-field", children: [_jsxs("span", { className: "crisp-recordpanel-field-label", children: [col.icon ? _jsx(col.icon, { size: 15, strokeWidth: 1.75 }) : null, col.label] }), _jsxs("span", { className: "crisp-recordpanel-field-value", children: [_jsx("input", { className: "crisp-recordpanel-edit", 
-                                                // biome-ignore lint/a11y/noAutofocus: focus the field the moment it opens
-                                                autoFocus: true, ...editor, disabled: isPending, defaultValue: value == null ? "" : String(value), "aria-label": `Edit ${col.label}`, onKeyDown: (e) => {
-                                                    if (isPending)
-                                                        return;
-                                                    if (e.key === "Enter")
-                                                        commit(record.id, col.key, e.currentTarget.value);
-                                                    else if (e.key === "Escape")
-                                                        closeEditing();
-                                                }, onBlur: (e) => {
-                                                    if (isPending)
-                                                        return;
-                                                    commit(record.id, col.key, e.currentTarget.value);
-                                                } }), isPending && (_jsx(Spinner, { size: 14, className: "crisp-recordpanel-edit-pending" })), fieldError && (_jsx("span", { className: "crisp-recordpanel-edit-error", children: fieldError }))] })] }, col.key));
-                        }
-                        return (_jsxs("div", { className: "crisp-recordpanel-field", children: [_jsxs("span", { className: "crisp-recordpanel-field-label", children: [col.icon ? _jsx(col.icon, { size: 15, strokeWidth: 1.75 }) : null, col.label] }), _jsxs("span", { className: "crisp-recordpanel-field-value", "data-editable": canEdit ? "" : undefined, children: [canEdit ? (_jsx("button", { type: "button", className: "crisp-recordpanel-edit-trigger", onClick: () => openEditing(col.key), children: _jsx(FieldValue, { col: col, row: record, emptyLabel: emptyLabel }) })) : (_jsx(FieldValue, { col: col, row: record, emptyLabel: emptyLabel })), fieldError && (_jsx("span", { className: "crisp-recordpanel-edit-error", children: fieldError }))] })] }, col.key));
-                    })] }), footer != null && (_jsx("div", { className: "crisp-recordpanel-footer", children: footer }))] }));
+    const heading = title ?? record.name ?? "Record";
+    return (_jsxs("div", { className: recordpanelRecipe({ className }), ...rest, role: "dialog", "aria-labelledby": titleId, children: [_jsxs("div", { className: "crisp-recordpanel-toolbar", children: [_jsx("button", { type: "button", className: "crisp-recordpanel-iconbtn", "aria-label": closeLabel, onClick: onClose, children: _jsx(X, { size: 16, strokeWidth: 1.75 }) }), (onPrevious || onNext) && (_jsxs("span", { className: "crisp-recordpanel-pager", children: [_jsx("button", { type: "button", className: "crisp-recordpanel-navbtn", "aria-label": previousRecordLabel, disabled: !onPrevious, onClick: onPrevious, children: _jsx(ChevronUp, { size: 16, strokeWidth: 1.75 }) }), _jsx("button", { type: "button", className: "crisp-recordpanel-navbtn", "aria-label": nextRecordLabel, disabled: !onNext, onClick: onNext, children: _jsx(ChevronDown, { size: 16, strokeWidth: 1.75 }) })] })), _jsx("span", { className: "crisp-recordpanel-toolbar-spacer" }), onOpenRecord && (_jsx("button", { type: "button", className: "crisp-recordpanel-navbtn", "aria-label": openRecordLabel, onClick: onOpenRecord, children: _jsx(ArrowUpRight, { "data-rtl-flip": "", size: 16, strokeWidth: 1.75 }) }))] }), _jsxs("div", { className: "crisp-recordpanel-header", children: [avatar ? (_jsx("img", { className: "crisp-recordpanel-fav", src: avatar, alt: "" })) : (_jsx(Avatar, { size: "md", shape: "rounded", tone: avatarTone, children: (record.name ?? "?").charAt(0) })), _jsx("span", { id: titleId, className: "crisp-recordpanel-title", children: heading })] }), _jsxs("div", { className: "crisp-recordpanel-body", children: [highlights && highlights.length > 0 && (_jsxs("div", { className: "crisp-recordpanel-section", children: [_jsx("div", { className: "crisp-recordpanel-section-head", children: _jsx("span", { className: "crisp-recordpanel-section-label", children: highlightsLabel }) }), _jsx("div", { className: "crisp-recordpanel-highlights", children: highlights.map((h) => (_jsxs("div", { className: "crisp-recordpanel-highlight", children: [_jsxs("span", { className: "crisp-recordpanel-highlight-label", children: [_jsx("span", { children: h.label }), h.icon ? _jsx(h.icon, { size: 16, strokeWidth: 1.75 }) : null] }), _jsx("span", { className: "crisp-recordpanel-highlight-value", children: h.value })] }, h.key))) })] })), activity != null && (_jsxs("div", { className: "crisp-recordpanel-section", children: [_jsxs("div", { className: "crisp-recordpanel-section-head", children: [_jsx("span", { className: "crisp-recordpanel-section-label", children: activityLabel }), onViewAllActivity && (_jsx("button", { type: "button", className: "crisp-recordpanel-section-action", onClick: onViewAllActivity, children: viewAllActivityLabel }))] }), _jsx("div", { className: "crisp-recordpanel-activity", children: activity })] }))] }), footer != null && (_jsx("div", { className: "crisp-recordpanel-footer", children: footer }))] }));
 }
 RecordPanel.displayName = "RecordPanel";