nexus-shared 2.0.0 → 3.1.0

package/dist/Components/Forms/Crud.js

@@ -1,15 +1,16 @@
 "use client";
-import { jsx as _jsx, Fragment as _Fragment, jsxs as _jsxs } from "react/jsx-runtime";
-import { iconArchive, iconArchiveOff, iconEye, iconFlag, iconFlagOff, iconList, iconLock, iconNotes, iconPencil, iconPin, iconPinnedOff, iconPlus, iconRestore, iconTrash, iconTrashX, InlineIcon, } from "nexus-icons";
+import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
+import { iconArrowBackUp, iconEye, iconFlag, iconFlagOff, iconInfoCircle, iconList, iconLoader2, iconLock, iconNotes, iconPencil, iconPin, iconPinnedOff, iconPlus, iconTrash, iconTrashX, InlineIcon, } from "nexus-icons";
 import { useCallback, useEffect, useImperativeHandle, useMemo, useRef, useState, useSyncExternalStore } from "react";
 import { abortError } from "../../Helpers/ApiFormHelpers.js";
-import { unwrapApiResult } from "../../Helpers/ApiResponses.js";
+import { isAbortError, unwrapApiResult } from "../../Helpers/ApiResponses.js";
 import { getCrudBackend } from "../../Helpers/CrudBackend.js";
-import { actionAllowed, availableModes, CRUD_CACHE_LIFETIME, CRUD_MODES, CRUD_ROW_ACTIONS, CrudCache, crudListEndpoint, crudMassActions, crudMessageAction, crudRowActions, crudRowEndpoint, getCrudSettings, leavesList, massActionAllowed, pinRoom, } from "../../Helpers/CrudHelpers.js";
+import { actionAllowed, availableModes, CRUD_CACHE_LIFETIME, CRUD_MODES, CRUD_ROW_ACTIONS, CrudCache, crudCacheMode, crudListEndpoint, crudMassActions, crudMessageAction, crudRowActions, crudRowEndpoint, getCrudSettings, leavesList, massActionAllowed, patchCachedList, pinRoom, } from "../../Helpers/CrudHelpers.js";
 import { cx } from "../../Helpers/ClassHelpers.js";
 import { FormStore } from "../../Helpers/FormStore.js";
 import { pluralize } from "../../Helpers/MessageBuilder.js";
 import { getPermissions, permissionsVersion, subscribePermissions } from "../../Helpers/Permissions.js";
+import { filterRequest } from "../../Helpers/TableFilters.js";
 import { columnValue, formatCellText, readPath } from "../../Helpers/TableHelpers.js";
 import { api as browserApi } from "../../Services/BrowserApi.js";
 import { Button } from "../Documents/Button.js";
@@ -22,23 +23,23 @@ import { CrudDetails } from "./CrudParts.js";
 import { confirmDiscard, FormButtons } from "./SubmitForm.js";
 /*
  * Crud: a module's records in one component. It takes a controller's endpoints, the table's columns, and the form's
- * inputs, and handles the rest: the lists (all, flagged, archived, trash, deleted) the user may see, the actions they
+ * inputs, and handles the rest: the lists (all, flagged, trash, deleted) the user may see, the actions they
  * may take, the records the user pinned in a section above the list, Add and Edit popups built on ApiForm, a details
  * popup, actions on many rows, a cache of lists while the page is open, and rows updated from each answer instead of
  * reloading.
  */
-const ALL_ACTIONS = ["create", "view", "edit", "flag", "pin", "archive", "restore", "trash", "recover", "delete", "trashFromDelete", "description"];
+const ALL_ACTIONS = ["create", "view", "edit", "flag", "pin", "trash", "recover", "delete", "trashFromDelete", "description"];
 const MODE_INFO = {
-    normal: { label: "All", icon: iconList, empty: plural => `No ${plural} yet.` },
+    // "Active", not "All": the list leaves out what is in the trash and what is deleted, and "All" said otherwise.
+    normal: { label: "Active", icon: iconList, empty: plural => `No ${plural} yet.` },
     flag: { label: "Flagged", icon: iconFlag, empty: plural => `No flagged ${plural}.` },
-    archive: { label: "Archived", icon: iconArchive, empty: () => "The archive is empty." },
     trash: { label: "Trash", icon: iconTrash, empty: () => "The trash is empty." },
     deleted: { label: "Deleted", icon: iconTrashX, empty: plural => `No deleted ${plural}.` },
 };
-/** Actions that take a record out of reach are asked about first; flags, pins, restore, and recover are undone by a click. */
-const ASKED = new Set(["archive", "trash", "delete", "trashFromDelete"]);
+/** Actions that move a record between the lists are asked about first; flags and pins are undone by a click. */
+const ASKED = new Set(["trash", "recover", "delete", "trashFromDelete"]);
 /** Actions that take a record out of the active lists, so it is no longer among the pinned records either. */
-const LEFT_ACTIVE = new Set(["archive", "trash", "delete", "trashFromDelete"]);
+const LEFT_ACTIVE = new Set(["trash", "delete", "trashFromDelete"]);
 /** The words of an action that flags some records and unflags others (a mass toggle of a mixed selection). */
 const TOGGLED_FLAG = { verb: "change the flags of {object}", done: "updated", doing: "Changing the flags of {object}", object: "the records" };
 const capitalize = (text) => (text ? text[0].toUpperCase() + text.slice(1) : text);
@@ -51,6 +52,80 @@ function endpointSubject(api) {
     const endpoints = [api.create, api.update, api.details, ...Object.values(api.lists ?? {}), ...Object.values(api.pages ?? {})];
     return endpoints.find(endpoint => endpoint?.subject)?.subject;
 }
+/**
+ * The body of an Add, Edit or details popup. Without an `aside` it is the form (or the details) and nothing else, so
+ * a popup that wants no panel carries no wrapper at all. With one, the two become columns that scroll on their own,
+ * and the form is first in the DOM — which is also the order they stack in when there is no room for two. A details
+ * popup is marked, because there the panel is what the popup is for and takes the larger share.
+ */
+function CrudFormBody({ aside, details, children }) {
+    if (!aside)
+        return children;
+    return (_jsxs("div", { className: cx("nx-crud-aside", details && "nx-crud-aside--details"), children: [_jsx("div", { className: "nx-crud-aside__main", children: children }), _jsx("aside", { className: "nx-crud-aside__side", children: aside })] }));
+}
+/** Whether the details endpoint's record says anything the list's row did not. Plain JSON, compared field by field. */
+function addsToRow(row, full) {
+    return Object.entries(full).some(([name, value]) => JSON.stringify(value) !== JSON.stringify(row[name]));
+}
+/**
+ * A details record that lands sooner than this says nothing: the demo answers in about 0.3s and most backends sooner,
+ * and a word shown for a frame or two is the flicker this avoids. Longer than the popup's 160ms fade, so the word
+ * never lands on a half-faded popup.
+ */
+const DETAILS_STATUS_DELAY = 400;
+/** Once "Loading…" shows, it stays at least this long, so a record that lands just after the delay does not blink it. */
+const DETAILS_STATUS_HOLD = 300;
+/** One details popup's load state, shared by the details (which load) and the footer (which says so), as a form popup shares its FormStore with FormButtons. */
+function detailsLoad(initial) {
+    let state = initial;
+    const listeners = new Set();
+    return {
+        get: () => state,
+        set(next) {
+            if (next === state)
+                return;
+            state = next;
+            for (const listener of listeners)
+                listener();
+        },
+        subscribe(listener) {
+            listeners.add(listener);
+            return () => {
+                listeners.delete(listener);
+            };
+        },
+    };
+}
+/**
+ * A details popup's footer: a form's bar, its status line on the left and Edit and Close on the right. The line is
+ * always there, so what it says never moves anything: "Loading…" only for a slow record, "Not all fields loaded." for
+ * one that failed.
+ */
+function DetailsFooter({ load, children }) {
+    const state = useSyncExternalStore(load.subscribe, load.get, load.get);
+    const [said, setSaid] = useState(null);
+    const since = useRef(0);
+    useEffect(() => {
+        if (state === "loading") {
+            if (said === "loading")
+                return;
+            const timer = window.setTimeout(() => {
+                since.current = performance.now();
+                setSaid("loading");
+            }, DETAILS_STATUS_DELAY);
+            return () => window.clearTimeout(timer);
+        }
+        const next = state === "failed" ? "failed" : null;
+        const left = said === "loading" ? since.current + DETAILS_STATUS_HOLD - performance.now() : 0;
+        if (left <= 0) {
+            setSaid(next);
+            return;
+        }
+        const timer = window.setTimeout(() => setSaid(next), left);
+        return () => window.clearTimeout(timer);
+    }, [state, said]);
+    return (_jsxs("div", { className: "nx-form-buttons nx-form-buttons--end nx-crud-details__foot", children: [_jsxs("p", { className: "nx-form-status nx-form-status--muted", "aria-live": "polite", children: [said === "loading" && (_jsxs(_Fragment, { children: [_jsx(InlineIcon, { icon: iconLoader2, className: "nx-spin" }), _jsx("span", { children: "Loading\u2026" })] })), said === "failed" && (_jsxs(_Fragment, { children: [_jsx(InlineIcon, { icon: iconInfoCircle }), _jsx("span", { children: "Not all fields loaded." })] }))] }), _jsx("div", { className: "nx-form-buttons__actions", children: children })] }));
+}
 /**
  * A CRUD page in one component: a data table of a controller's records with its lists, Add and Edit popups, details,
  * and actions on one record or many, each shown only when its endpoint exists and the user may call it.
@@ -89,7 +164,7 @@ export function Crud(props) {
     }, [ready, api, wantedActions, can]);
     const massAllowed = useMemo(() => {
         const map = {};
-        for (const action of ["flag", "pin", "archive", "restore", "trash", "recover", "delete", "trashFromDelete", "description"])
+        for (const action of ["flag", "pin", "trash", "recover", "delete", "trashFromDelete", "description"])
             map[action] = ready && massActionAllowed(action, api, wantedActions, can);
         return map;
     }, [ready, api, wantedActions, can]);
@@ -187,13 +262,14 @@ export function Crud(props) {
             return new Promise(() => undefined);
         const endpoint = crudListEndpoint(api, mode, paging);
         const sortColumn = query.sort ? latest.current.columns.find(column => column.key === query.sort.key) : undefined;
-        const key = paging === "server" ? `${mode}|${query.search}|${query.sort ? `${query.sort.key}:${query.sort.direction}` : ""}|${query.page}|${query.pageSize}` : mode;
+        const filters = query.filters ? JSON.stringify(query.filters) : "";
+        const key = paging === "server" ? `${mode}|${query.search}|${query.sort ? `${query.sort.key}:${query.sort.direction}` : ""}|${query.page}|${query.pageSize}|${filters}` : mode;
         const kept = cache.get(key);
         if (kept)
             return kept;
         const options = { signal, toast: false, subject: plural };
         if (paging === "server")
-            options.body = backend.pageRequest(query, sortColumn?.sortKey ?? sortColumn?.key);
+            options.body = backend.pageRequest(query, sortColumn?.sortKey ?? sortColumn?.key, filterRequest(query.filters, latest.current.table?.filterInputs ?? [], latest.current.columns));
         return browserApi.send(endpoint, options).then(answer => {
             if (answer.aborted)
                 throw abortError();
@@ -204,27 +280,62 @@ export function Crud(props) {
             return result;
         });
     }, [mode, api, paging, cache, backend, plural]);
-    /** After a change: the list shown keeps its rows in the cache, the other lists load again when shown. */
-    function afterChange(action, rows) {
+    /** Which list a record belongs in, as this backend reads the record. */
+    const member = (row, of) => backend.listMembership(row, of);
+    /**
+     * The lists kept in the cache, after a change: **every one of them is patched**, from the record the answer
+     * carried. A record flagged joins the flagged list, a record trashed leaves the active list and joins the trash,
+     * and coming back to any of them costs no request. Only a list the record cannot answer for (`changes` empty, or
+     * a backend with no `listMembership`) is dropped, and that one loads when it is shown.
+     *
+     * A server page is not patched when a record joined it or left it: its order, its last row and its counts are the
+     * server's, so that page is dropped and the next one loads. One whose rows only changed is written in place.
+     */
+    function syncCaches(changes, written) {
         const shown = modeRef.current;
-        if (paging === "client" && shown) {
-            cache.drop(key => key !== shown);
-            cache.update(shown, table.getRows());
-        }
+        if (changes.length)
+            cache.map((key, value) => patchCachedList(value, crudCacheMode(key), changes, row => table.keyOf(row), member));
         else
             cache.clear();
+        // The list on screen is the table's own, every row of it, so it is written back whole rather than patched - when
+        // the change was written into it. When it was not (see `holds`), the table is read again from the lists just
+        // patched, or loaded when it has none, and a load still out for it is stopped: it may have been asked before the
+        // change, and would land without it.
+        if (!written)
+            void table.reload({ quiet: true });
+        else if (paging === "client" && shown)
+            cache.update(shown, table.getRows());
+    }
+    /**
+     * Whether the table holds the list an action was asked from, every row of it, so the answer can be written into it.
+     * Not when the user moved to another list while a slow answer was out - the table holds that list then - and not
+     * while a list is loading: a switch keeps the old list's rows until the new ones land, and writing those back would
+     * file one list under another's name.
+     */
+    function holds(from) {
+        return (from === undefined || from === modeRef.current) && table.getState().status === "ready";
+    }
+    function afterChange(action, rows, changes, written) {
+        syncCaches(changes, written);
+        const shown = modeRef.current;
         if (shown)
             latest.current.onChange?.({ action, rows, mode: shown });
     }
-    /* Changing rows after an answer */
-    function applyChange(action, rows, result, markOn, values) {
+    function applyChange(action, rows, options = {}) {
+        const { result, mass = false, markOn, values, from } = options;
         const shown = modeRef.current;
         if (!shown)
             return;
+        // Every table write below is for the list the action was asked from; when the table holds another, none is made.
+        const write = holds(from);
         const answered = rows.length > 1 ? backend.answeredKeys(result, rowKey) : null;
         const done = answered ? rows.filter(row => answered.has(table.keyOf(row))) : rows;
+        const changes = backend.answeredChanges(result, rowKey);
+        // What the answer said about the one record acted on: the record, or the fields that changed - a mark's new
+        // status. It is what lets the other lists be patched rather than thrown away.
+        const one = mass ? null : backend.answeredRow(result);
+        const moved = [];
         if (action === "flag") {
-            const changes = backend.answeredChanges(result, rowKey);
             const leaving = [];
             const updated = [];
             for (const row of done) {
@@ -235,19 +346,23 @@ export function Crud(props) {
                 const change = changes.get(key);
                 if (change)
                     next = { ...next, ...change };
-                if (leavesList(action, shown, backend.isFlagged(next)))
-                    leaving.push(key);
-                else
-                    table.upsert(next, { flash: false });
+                if (one)
+                    next = { ...next, ...one };
+                if (write) {
+                    if (leavesList(action, shown, backend.isFlagged(next)))
+                        leaving.push(key);
+                    else
+                        table.upsert(next, { flash: false });
+                }
                 updated.push(next);
+                moved.push({ key, row: next, before: current });
             }
             table.remove(leaving);
             patchPins(updated);
-            afterChange(action, updated);
+            afterChange(action, updated, moved, write);
             return;
         }
         if (action === "pin") {
-            const changes = backend.answeredChanges(result, rowKey);
             const updated = [];
             for (const row of done) {
                 const key = table.keyOf(row);
@@ -257,29 +372,97 @@ export function Crud(props) {
                 const change = changes.get(key);
                 if (change)
                     next = { ...next, ...change };
+                if (one)
+                    next = { ...next, ...one };
                 // A pin changes the section above the list, never the list itself: the record stays where it is, on its page.
-                if (table.getRow(key))
+                if (write && table.getRow(key))
                     table.upsert(next, { flash: false });
                 updated.push(next);
+                moved.push({ key, row: next, before: current });
             }
             applyPins(updated);
-            afterChange(action, updated);
+            afterChange(action, updated, moved, write);
             return;
         }
         if (action === "description") {
-            const updated = done.map(row => ({ ...(table.getRow(table.keyOf(row)) ?? row), ...values }));
-            for (const row of updated)
-                table.upsert(row);
+            const updated = [];
+            for (const row of done) {
+                const key = table.keyOf(row);
+                const current = table.getRow(key) ?? row;
+                const next = { ...current, ...values };
+                if (write)
+                    table.upsert(next);
+                updated.push(next);
+                moved.push({ key, row: next, before: current });
+            }
             patchPins(updated);
-            afterChange(action, updated);
+            afterChange(action, updated, moved, write);
             return;
         }
+        // Trash, recover, delete and the move back to the trash take the record to another band, and only the answer
+        // says which. Every record has to say so for the other lists to be patched; one that does not leaves `moved`
+        // empty, and they load again, which is what they all did before.
         const keys = done.map(row => table.keyOf(row));
-        if (leavesList(action, shown))
+        const updated = [];
+        for (const row of done) {
+            const key = table.keyOf(row);
+            const current = table.getRow(key) ?? row;
+            const change = changes.get(key) ?? one;
+            const next = change ? { ...current, ...change } : current;
+            if (change)
+                moved.push({ key, row: next, before: current });
+            updated.push(next);
+        }
+        if (moved.length !== done.length)
+            moved.length = 0;
+        if (write && leavesList(action, shown))
             table.remove(keys);
         if (LEFT_ACTIVE.has(action))
             dropPins(keys);
-        afterChange(action, done);
+        afterChange(action, updated, moved, write);
+    }
+    /**
+     * Records a page's own action changed, written exactly where the built-in ones write theirs. The handle's `apply`.
+     */
+    function applyRows(rows, gone = false) {
+        const shown = modeRef.current;
+        if (!shown || rows.length === 0)
+            return;
+        const moved = [];
+        const updated = [];
+        const leaving = [];
+        let pinned = false;
+        // The page asked from whichever list it did; the record's band decides, so only a table still loading is not
+        // written. A record that joined the list on screen - asked from another list, answered after a switch - is not in
+        // the table to be updated, so the table is read again from the patched lists instead.
+        let write = holds(undefined);
+        for (const given of rows) {
+            const next = { ...(table.getRow(table.keyOf(given)) ?? {}), ...given };
+            const key = table.keyOf(next);
+            if (!key)
+                continue;
+            const before = table.getRow(key);
+            // The record decides whether it is still in the list on screen; a backend that cannot say keeps it there.
+            if (gone || member(next, shown) === false)
+                leaving.push(key);
+            else if (!before && member(next, shown))
+                write = false;
+            else if (before && write)
+                table.upsert(next, { flash: false });
+            if (before && backend.isPinned(next) !== backend.isPinned(before))
+                pinned = true;
+            updated.push(next);
+            moved.push({ key, row: next, before, gone });
+        }
+        if (write)
+            table.remove(leaving);
+        if (gone)
+            dropPins(leaving);
+        else if (pinned)
+            applyPins(updated);
+        else
+            patchPins(updated);
+        syncCaches(moved, write);
     }
     /* Row actions */
     async function runRowAction(action, row) {
@@ -287,6 +470,7 @@ export function Crud(props) {
         if (!endpoint)
             return;
         const key = rawKey(row);
+        const from = modeRef.current;
         const markOn = action === "flag" ? !backend.isFlagged(row) : action === "pin" ? !backend.isPinned(row) : undefined;
         if (action === "pin" && markOn && pinRoom(pinned?.length) < 1) {
             toast.warning(pinsFull());
@@ -295,7 +479,7 @@ export function Crud(props) {
         const options = { params: { id: key, [rowKey]: key }, subject: labelOf(row), action: crudMessageAction(action, markOn) };
         const answer = ASKED.has(action) ? await browserApi.confirm(endpoint, options) : await browserApi.send(endpoint, options);
         if (answer?.ok)
-            applyChange(action, [row], answer.result, markOn);
+            applyChange(action, [row], { result: answer.result, markOn, from });
     }
     async function runMassAction(action, rows) {
         if (action === "description") {
@@ -329,10 +513,11 @@ export function Crud(props) {
         }
         const messageAction = action === "flag" && markOn === undefined ? TOGGLED_FLAG : crudMessageAction(action, markOn);
         const options = { body: backend.massRequest(action, sent.map(rawKey)), subject, plural: props.subjectPlural, count: sent.length, action: messageAction };
+        const from = modeRef.current;
         const answer = ASKED.has(action) ? await browserApi.confirm(endpoint, options) : await browserApi.send(endpoint, options);
         if (!answer?.ok)
             return;
-        applyChange(action, sent, answer.result, markOn);
+        applyChange(action, sent, { result: answer.result, mass: true, markOn, from });
         table.clearSelection();
         if (over > 0)
             toast.warning(`${sent.length} of ${sent.length + over} ${plural} pinned: you can pin up to ${maxPins}.`);
@@ -384,10 +569,10 @@ export function Crud(props) {
                     form.dropDraft();
                 return discard;
             },
-            content: popup => (_jsx(ApiForm, { form: form, inputs: inputs, buttons: false, defaultValue: (action === "create" ? { ...newValues, ...values } : (row ?? undefined)), loadValue: loadValue, api: endpoint, transform: options.transform ? formValues => options.transform(formValues, { action, row }) : undefined, validate: options.validate, onFieldChange: options.onFieldChange, design: options.design, draftKey: options.draftKey ? `${options.draftKey}:${name}:${action === "create" ? "new" : key}` : undefined, "aria-label": action === "create" ? `Add ${subject}` : `Edit ${label}`, onSuccess: (result, saved) => {
-                    onSaved(action, row, result, saved);
-                    popup.close();
-                } })),
+            content: popup => (_jsx(CrudFormBody, { aside: typeof options.aside === "function" ? options.aside({ form, action, row }) : options.aside, children: _jsx(ApiForm, { form: form, inputs: inputs, buttons: false, defaultValue: (action === "create" ? { ...newValues, ...values } : (row ?? undefined)), loadValue: loadValue, api: endpoint, transform: options.transform ? formValues => options.transform(formValues, { action, row }) : undefined, validate: options.validate, onFieldChange: options.onFieldChange, design: options.design, draftKey: options.draftKey ? `${options.draftKey}:${name}:${action === "create" ? "new" : key}` : undefined, "aria-label": action === "create" ? `Add ${subject}` : `Edit ${label}`, onSuccess: (result, saved) => {
+                        onSaved(action, row, result, saved);
+                        popup.close();
+                    } }) })),
             footer: popup => _jsx(FormButtons, { form: form, submitLabel: action === "create" ? `Add ${subject}` : "Save", onCancel: () => popup.close() }),
         });
         if (action === "create") {
@@ -399,23 +584,58 @@ export function Crud(props) {
             });
         }
     }
+    /**
+     * The record a `Create` answered only the key of, read back on its own: the row goes in at once with what the form
+     * sent, and the fields the server owns - its status, its times, anything it worked out - land a moment later. One
+     * small request, where loading the list again was a request for every record on it.
+     */
+    async function fillSaved(key, record) {
+        const endpoint = api.details;
+        if (!endpoint)
+            return;
+        const id = readPath(record, rowKey);
+        const answer = await browserApi.get(endpoint, { params: { id, [rowKey]: id }, toast: false });
+        const full = unwrapApiResult(answer);
+        if (!isRecord(full))
+            return;
+        const before = table.getRow(key);
+        const next = { ...(before ?? record), ...full };
+        const write = holds(undefined);
+        if (write && before)
+            table.upsert(next, { flash: false });
+        syncCaches([{ key, row: next, before }], write);
+    }
     function onSaved(action, row, result, values) {
-        const record = { ...row, ...values, ...(isRecord(result) ? result : {}) };
+        // The record as the server now has it: what was sent, over the row it came from, with whatever the answer added.
+        // A backend that answers the record it wrote leaves nothing to load at all.
+        const answered = backend.answeredRow(result);
+        const record = { ...row, ...values, ...answered };
         const shown = modeRef.current;
         if (action === "edit") {
-            table.upsert(record);
+            const key = table.keyOf(record);
+            const before = table.getRow(key);
+            const write = holds(undefined);
+            if (write)
+                table.upsert(record);
             patchPins([record]);
-            afterChange("edit", [record]);
+            afterChange("edit", [record], key ? [{ key, row: record, before }] : [], write);
             return;
         }
-        // A new record belongs to the list of every record; without its key from the server, the list loads again.
-        if (shown === "normal") {
-            if (table.keyOf(record))
-                table.upsert(record);
-            else
-                void table.reload({ quiet: true });
+        // A new record belongs to the list of every record. An answer that is only the new key still gives it one, so
+        // the row goes in and the record is read back on its own; without even a key the list has to load again.
+        if (!answered && !readPath(record, rowKey)) {
+            const given = backend.answeredKey(result);
+            if (given !== null && !rowKey.includes("."))
+                record[rowKey] = given;
         }
-        afterChange("create", [record]);
+        const key = table.keyOf(record);
+        // Without a key the lists are dropped and the one on screen loads again (`syncCaches`).
+        const write = Boolean(key) && holds(undefined);
+        if (write && shown === "normal")
+            table.upsert(record);
+        afterChange("create", [record], key ? [{ key, row: record, before: undefined }] : [], write);
+        if (key && !answered && api.details)
+            void fillSaved(key, record);
     }
     function openDetails(row) {
         const details = latest.current.details;
@@ -424,15 +644,44 @@ export function Crud(props) {
         const key = table.keyOf(row);
         const canEdit = allowed.edit && modeRef.current !== undefined && CRUD_ROW_ACTIONS[modeRef.current].includes("edit");
         const endpoint = api.details;
+        const status = detailsLoad(endpoint ? "loading" : "done");
+        // The panel's record: the row at once, so it has something to show while the full one loads, and that one after.
+        const shown = typeof details?.aside === "function" ? new FormStore() : null;
+        shown?.load(row, { draft: false });
+        const aside = typeof details?.aside === "function" ? details.aside({ form: shown, row }) : details?.aside;
         openPopup({
             key: `${name}-view-${key}`,
             reopen: "restore",
-            title: `${capitalize(subject)} details`,
+            title: `${capitalize(subject)} ${details?.label ?? "details"}`,
             description: labelOf(row),
             icon: _jsx(InlineIcon, { icon: iconEye }),
             size: details?.size ?? "md",
-            content: (_jsx(CrudDetails, { row: row, columns: details?.columns ?? latest.current.columns, render: details?.render, load: endpoint ? signal => browserApi.get(endpoint, { params: { id: rawKey(row), [rowKey]: rawKey(row) }, signal, toast: false }).then(unwrapApiResult) : undefined })),
-            footer: popup => (_jsxs(_Fragment, { children: [canEdit && (_jsx(Button, { icon: _jsx(InlineIcon, { icon: iconPencil }), onClick: () => {
+            // Focus goes where it goes without a panel, the footer, rather than to the first button the panel happens to
+            // draw: a panel that renders at once (a Copy button) would otherwise take it, and one that loads would not.
+            initialFocus: aside ? ".nx-popup__foot button" : undefined,
+            content: (_jsx(CrudFormBody, { aside: aside, details: true, children: _jsx(CrudDetails, { row: row, columns: details?.columns ?? latest.current.columns, render: details?.render, load: endpoint
+                        ? signal => {
+                            const loading = browserApi
+                                .get(endpoint, { params: { id: rawKey(row), [rowKey]: rawKey(row) }, signal, toast: false })
+                                .then(unwrapApiResult)
+                                .then(full => {
+                                // Only when it adds something: a panel that renders from the record would otherwise ask again
+                                // for exactly what it is already showing.
+                                if (shown && isRecord(full) && addsToRow(row, full))
+                                    shown.load({ ...row, ...full }, { draft: false });
+                                return full;
+                            });
+                            // The footer follows the very promise the details do, with the same test for a popup closed meanwhile.
+                            void loading.then(() => status.set("done"), error => {
+                                if (!signal.aborted && !isAbortError(error))
+                                    status.set("failed");
+                            });
+                            return loading;
+                        }
+                        : undefined }) })),
+            footer: popup => (_jsxs(DetailsFooter, { load: status, children: [details?.actions
+                        ?.filter(action => (!action.endpoint || (ready && can(action.endpoint))) && !action.hidden?.(row))
+                        .map(action => (_jsx(Button, { icon: action.icon, "data-action": action.id, onClick: () => action.onSelect(table.getRow(key) ?? row, popup), children: action.label }, action.id))), canEdit && (_jsx(Button, { icon: _jsx(InlineIcon, { icon: iconPencil }), onClick: () => {
                             popup.close();
                             void openForm("edit", table.getRow(key) ?? row);
                         }, children: "Edit" })), _jsx(Button, { variant: "primary", onClick: () => popup.close(), children: "Close" })] })),
@@ -453,7 +702,7 @@ export function Crud(props) {
             closeOnBackdrop: false,
             beforeClose: () => confirmDiscard(form),
             content: popup => (_jsx(ApiForm, { form: form, buttons: false, inputs: [{ type: "textarea", name: "description", label: "Description", maxLength: 500, rows: 4, showCount: true }], api: endpoint, transform: values => backend.massRequest("description", rows.map(rawKey), values), "aria-label": `Description of ${count} ${count === 1 ? subject : plural}`, onSuccess: (result, values) => {
-                    applyChange("description", rows, result, undefined, values);
+                    applyChange("description", rows, { result, mass: true, values });
                     table.clearSelection();
                     popup.close();
                 } })),
@@ -461,8 +710,8 @@ export function Crud(props) {
         });
     }
     // Row actions and the handle call these through a ref, so the table's callbacks stay the same between renders.
-    const run = useRef({ runRowAction, runMassAction, openForm, openDetails });
-    run.current = { runRowAction, runMassAction, openForm, openDetails };
+    const run = useRef({ runRowAction, runMassAction, openForm, openDetails, applyRows });
+    run.current = { runRowAction, runMassAction, openForm, openDetails, applyRows };
     /* The table's callbacks */
     const rowActions = useCallback((row) => {
         if (!mode)
@@ -472,16 +721,14 @@ export function Crud(props) {
         const flagged = backend.isFlagged(row);
         const isPinned = backend.isPinned(row);
         const info = {
-            view: { label: "View details", icon: _jsx(InlineIcon, { icon: iconEye }) },
+            view: { label: `View ${(latest.current.details || undefined)?.label ?? "details"}`, icon: _jsx(InlineIcon, { icon: iconEye }) },
             edit: { label: "Edit", icon: _jsx(InlineIcon, { icon: iconPencil }), inline: true },
             flag: flagged ? { label: "Unflag", icon: _jsx(InlineIcon, { icon: iconFlagOff }) } : { label: "Flag", icon: _jsx(InlineIcon, { icon: iconFlag }) },
             pin: isPinned ? { label: "Unpin", icon: _jsx(InlineIcon, { icon: iconPinnedOff }) } : { label: "Pin", icon: _jsx(InlineIcon, { icon: iconPin }) },
-            archive: { label: "Archive", icon: _jsx(InlineIcon, { icon: iconArchive }) },
-            restore: { label: "Restore", description: "Back to the active list", icon: _jsx(InlineIcon, { icon: iconArchiveOff }), inline: true },
             trash: { label: "Move to trash", icon: _jsx(InlineIcon, { icon: iconTrash }), tone: "danger" },
-            recover: { label: "Recover", description: "Back to the active list", icon: _jsx(InlineIcon, { icon: iconRestore }), inline: true },
+            recover: { label: "Recover", description: "Back to the active list", icon: _jsx(InlineIcon, { icon: iconArrowBackUp }), inline: true },
             delete: { label: "Delete permanently", icon: _jsx(InlineIcon, { icon: iconTrashX }), tone: "danger" },
-            trashFromDelete: { label: "Move back to trash", icon: _jsx(InlineIcon, { icon: iconRestore }), inline: true },
+            trashFromDelete: { label: "Move back to trash", icon: _jsx(InlineIcon, { icon: iconArrowBackUp }), inline: true },
         };
         for (const action of crudRowActions(mode, candidate => allowed[candidate], trashless)) {
             if (action === "view" && latest.current.details === false)
@@ -504,12 +751,10 @@ export function Crud(props) {
             const info = {
                 flag: flagged === rows.length ? { label: "Unflag", icon: _jsx(InlineIcon, { icon: iconFlagOff }) } : { label: flagged ? "Flag / unflag" : "Flag", icon: _jsx(InlineIcon, { icon: iconFlag }) },
                 pin: unpinned === 0 ? { label: "Unpin", icon: _jsx(InlineIcon, { icon: iconPinnedOff }) } : { label: "Pin", icon: _jsx(InlineIcon, { icon: iconPin }) },
-                archive: { label: "Archive", icon: _jsx(InlineIcon, { icon: iconArchive }) },
-                restore: { label: "Restore", icon: _jsx(InlineIcon, { icon: iconArchiveOff }) },
                 trash: { label: "Move to trash", icon: _jsx(InlineIcon, { icon: iconTrash }), tone: "danger" },
-                recover: { label: "Recover", icon: _jsx(InlineIcon, { icon: iconRestore }) },
+                recover: { label: "Recover", icon: _jsx(InlineIcon, { icon: iconArrowBackUp }) },
                 delete: { label: "Delete permanently", icon: _jsx(InlineIcon, { icon: iconTrashX }), tone: "danger" },
-                trashFromDelete: { label: "Move back to trash", icon: _jsx(InlineIcon, { icon: iconRestore }) },
+                trashFromDelete: { label: "Move back to trash", icon: _jsx(InlineIcon, { icon: iconArrowBackUp }) },
                 description: { label: "Description", icon: _jsx(InlineIcon, { icon: iconNotes }) },
             };
             list.push({ ...info[action], id: action, onSelect });
@@ -534,6 +779,7 @@ export function Crud(props) {
             reloadPins();
             return table.reload({ quiet: true });
         },
+        apply: (rows, options) => run.current.applyRows(rows, options?.gone ?? false),
         create: values => void run.current.openForm("create", null, values),
         edit: row => void run.current.openForm("edit", row),
         view: row => run.current.openDetails(row),
@@ -547,10 +793,35 @@ export function Crud(props) {
         return (_jsxs("div", { id: id, className: cx("nx-crud", "nx-crud--denied", className), style: style, role: "status", children: [_jsx(InlineIcon, { icon: iconLock, className: "nx-crud__denied-icon" }), _jsxs("p", { className: "nx-crud__denied-title", children: ["You do not have permission to see ", plural, "."] }), _jsx("p", { className: "nx-crud__denied-detail", children: "Ask an administrator for access." })] }));
     }
     const modeOptions = modes.map(value => ({ value, label: MODE_INFO[value].label, icon: _jsx(InlineIcon, { icon: MODE_INFO[value].icon }) }));
-    const addButton = allowed.create ? (_jsxs(Button, { variant: "primary", size: "sm", icon: _jsx(InlineIcon, { icon: iconPlus }), onClick: () => void run.current.openForm("create", null), children: ["Add ", subject] })) : null;
+    const add = allowed.create ? { label: `Add ${subject}`, onClick: () => void run.current.openForm("create", null) } : undefined;
+    // In the main list, select again by pin and by flag, the way the mass actions act on them.
+    const selectBy = mode === "normal"
+        ? [
+            ...(massAllowed.pin
+                ? [
+                    { label: "Pinned", icon: _jsx(InlineIcon, { icon: iconPin }), test: (row) => backend.isPinned(row) },
+                    { label: "Not pinned", icon: _jsx(InlineIcon, { icon: iconPinnedOff }), test: (row) => !backend.isPinned(row) },
+                ]
+                : []),
+            ...(massAllowed.flag
+                ? [
+                    { label: "Flagged", icon: _jsx(InlineIcon, { icon: iconFlag }), test: (row) => backend.isFlagged(row) },
+                    { label: "Not flagged", icon: _jsx(InlineIcon, { icon: iconFlagOff }), test: (row) => !backend.isFlagged(row) },
+                ]
+                : []),
+        ]
+        : [];
     const selectable = props.massActions !== false && Boolean(mode) && (crudMassActions(mode, action => massAllowed[action], massTrashless).length > 0 || Boolean(props.massActions));
-    return (_jsx("div", { id: id, className: cx("nx-crud", className), style: style, "data-mode": mode, children: _jsx(DataTable, { table: table, columns: columns, load: load, loadKey: mode ?? "waiting", paging: paging, rowKey: rowKey, rowLabel: labelOf, "aria-label": tableLabel, "aria-labelledby": props["aria-labelledby"], toolbar: modes.length > 1 && mode ? (_jsx(TabButtons, { size: "sm", labels: "selected", options: modeOptions, value: mode, onChange: setChosenMode, "aria-label": `${capitalize(plural)}: lists` })) : undefined, actions: _jsxs(_Fragment, { children: [props.toolbar, addButton] }), selectable: selectable, massActions: massActionsFor, rowActions: rowActions, pinnedRows: mode === "normal" && pinsOn ? (pinned ?? undefined) : undefined, pinnedLoading: pinsLoading, pinnedLabel: `Pinned ${plural}`, pinnedMax: maxPins, onRowOpen: onRowOpen, onReload: () => {
+    return (_jsx("div", { id: id, className: cx("nx-crud", className), style: style, "data-mode": mode, children: _jsx(DataTable, { table: table, columns: columns, load: load, loadKey: mode ?? "waiting", paging: paging, rowKey: rowKey, rowLabel: labelOf, "aria-label": tableLabel, "aria-labelledby": props["aria-labelledby"], toolbar: modes.length > 1 && mode ? (_jsx(TabButtons, { size: "sm", labels: "auto", options: modeOptions, value: mode, onChange: setChosenMode, "aria-label": `${capitalize(plural)}: lists` })) : undefined, actions: props.toolbar, add: add, selectable: selectable, massActions: massActionsFor, selectBy: selectBy, rowActions: rowActions, pinnedRows: mode === "normal" && pinsOn ? (pinned ?? undefined) : undefined, pinnedLoading: pinsLoading, pinnedLabel: `Pinned ${plural}`, pinnedMax: maxPins, onRowOpen: onRowOpen, openOnClick: tableOptions?.openOnClick, onReload: () => {
                 cache.clear();
                 reloadPins();
-            }, emptyMessage: tableOptions?.emptyMessage ?? (mode ? MODE_INFO[mode].empty(plural) : undefined), emptyAction: mode === "normal" ? addButton : undefined, searchPlaceholder: tableOptions?.searchPlaceholder ?? `Search ${plural}…`, stateKey: tableOptions?.stateKey ?? `crud:${name}`, pageSize: tableOptions?.pageSize, pageSizes: tableOptions?.pageSizes, defaultSort: tableOptions?.defaultSort, numbered: tableOptions?.numbered, maxHeight: tableOptions?.maxHeight, size: tableOptions?.size, maxSelection: tableOptions?.maxSelection, columnsMenu: tableOptions?.columnsMenu, searchDelay: tableOptions?.searchDelay, reorderable: tableOptions?.reorderable, resizable: tableOptions?.resizable, pinnable: tableOptions?.pinnable, moreActions: tableOptions?.moreActions, exportable: tableOptions?.exportable, printable: tableOptions?.printable, exportName: tableOptions?.exportName, onImport: tableOptions?.onImport }) }));
+            }, emptyMessage: tableOptions?.emptyMessage ?? (mode ? MODE_INFO[mode].empty(plural) : undefined), emptyAction: mode === "normal" ? undefined : null, searchPlaceholder: tableOptions?.searchPlaceholder ?? `Search ${plural}…`, stateKey: tableOptions?.stateKey ?? `crud:${name}`, pageSize: tableOptions?.pageSize, pageSizes: tableOptions?.pageSizes, defaultSort: tableOptions?.defaultSort, numbered: tableOptions?.numbered, maxHeight: tableOptions?.maxHeight, fill: tableOptions?.fill, size: tableOptions?.size, maxSelection: tableOptions?.maxSelection, columnsMenu: tableOptions?.columnsMenu, searchDelay: tableOptions?.searchDelay, filterInputs: tableOptions?.filterInputs, reorderable: tableOptions?.reorderable, resizable: tableOptions?.resizable, pinnable: tableOptions?.pinnable, moreActions: tableOptions?.moreActions, exportable: tableOptions?.exportable, printable: tableOptions?.printable, exportName: tableOptions?.exportName, onImport: tableOptions?.onImport, excel: tableOptions?.excel && {
+                ...tableOptions.excel,
+                // A spreadsheet wrote records straight into the database: every list this CRUD kept is out of date.
+                onImported: () => {
+                    cache.clear();
+                    reloadPins();
+                    tableOptions.excel?.onImported?.();
+                },
+            } }) }));
 }