@eifi1/ui-kit 0.54.0 → 0.56.0

package/src/feedback/feedback-capture.ts

@@ -1,287 +0,0 @@
-/**
- * `captureAppScreenshot` — the picture of the app a feedback report attaches (0.27.0,
- * docs/feedback-harmonization.md §5).
- *
- * Lifted from keksdose `frontend/src/features/feedback/capture-screenshot.ts` (live #271),
- * which is the canon: Kurvenschmiede carried a second copy (`capture-page.ts`) with none of
- * its speed-ups, and kastlan had no capture at all. Each app wrote the same three awkward
- * decisions — which element, which library, what to do when it throws — and they drifted
- * the moment there were two copies (Kurvenschmiede feedback #128 says exactly that).
- *
- * WHY IT IS FAST (keksdose live #271: *"Capture screenshot — That takes ages in my mobile
- * phone"*). `modern-screenshot` deep-clones the subtree and inlines every node's computed
- * style onto the clone, because the final image is an SVG `foreignObject` that carries no
- * stylesheets. By default it walks the WHOLE computed style (~340 longhands in Chrome) per
- * element; keksdose's /transactions has ~4,800 elements under `#root` at phone width, so
- * that was ~1.6 million `getPropertyValue` calls before a pixel was drawn — 21–52 s on a
- * desktop CPU, 153–160 s with the CPU throttled 4×. Two levers fixed it, both kept here:
- *
- *   1. {@link CAPTURE_STYLE_PROPERTIES} — copy the ~140 longhands a Tailwind v4 app built
- *      on this kit actually uses. Alone: 21 s → 1.8 s.
- *   2. {@link inViewport} drops `display: none` subtrees — on a phone most of the
- *      document, since `DataTable` renders both layouts and hides one — and anything
- *      wholly outside the viewport. Together: 27–52 s → 0.4–1.1 s, the picture
- *      0.05% of pixels different (antialiasing).
- *
- * Plus WebP at 0.9 (222 KB → 46 KB, an upload a phone makes on a mobile connection) with
- * the file named from what the canvas ACTUALLY produced — a browser without WebP encodes
- * PNG, and a "screenshot.webp" that is a PNG is refused by the server on its type — and
- * `font: false`, since the app's faces are same-origin and already in the page.
- *
- * WHY THE PEER IS LOADED DYNAMICALLY. `modern-screenshot` is an OPTIONAL peer
- * (`peerDependenciesMeta.optional`, like `sonner`), and this module is re-exported by the
- * barrel: a static import would make every app that imports a Button install it. It is
- * imported inside the function, so an app pays nothing until somebody presses "Capture
- * screenshot", and an app without it gets a REJECTED promise — which the submit dialog
- * turns into the "Could not capture a screenshot" toast (`feedbackToast.captureFailed`).
- *
- * WHAT IS NOT HERE: Kurvenschmiede's `features: { restoreScrollPosition: true }` (its
- * feedback #153: a capture of a scrolled `<main>` came out as the TOP of the page). The
- * viewport filter already answers that case — what is scrolled out of view is dropped, so
- * what remains is what was on screen, give or take the part of the first row that was
- * scrolled past (checked in Chromium, 0.27.0: an inner `<main>` scrolled to row 20 captures
- * rows 20–26). Both together are worse than either: the clone is translated by the scroll
- * offset AFTER the rows above were dropped, and the picture came out blank below the
- * header (same check).
- */
-
-/**
- * The computed properties copied onto the clone (keksdose `capture-screenshot.ts:94`).
- *
- * LONGHANDS only: `getPropertyValue()` resolves a shorthand to `""` whenever its longhands
- * disagree, so `"border"` would silently drop every one-sided border. Chosen by what
- * Tailwind v4 emits rather than by what looks complete: `font-variant-numeric` because
- * money is `tabular-nums`, the SVG paint properties because every icon is a lucide `<svg>`
- * inheriting `stroke: currentColor`.
- */
-const CAPTURE_STYLE_PROPERTIES: readonly string[] = [
-  // box
-  "display",
-  "position",
-  "top",
-  "right",
-  "bottom",
-  "left",
-  "width",
-  "height",
-  "min-width",
-  "min-height",
-  "max-width",
-  "max-height",
-  "box-sizing",
-  "margin-top",
-  "margin-right",
-  "margin-bottom",
-  "margin-left",
-  "padding-top",
-  "padding-right",
-  "padding-bottom",
-  "padding-left",
-  "overflow-x",
-  "overflow-y",
-  "z-index",
-  "float",
-  "clear",
-  "aspect-ratio",
-  "inset-inline-start",
-  "inset-inline-end",
-  // flex + grid
-  "flex-direction",
-  "flex-wrap",
-  "flex-grow",
-  "flex-shrink",
-  "flex-basis",
-  "align-items",
-  "align-self",
-  "align-content",
-  "justify-content",
-  "justify-items",
-  "justify-self",
-  "row-gap",
-  "column-gap",
-  "order",
-  "grid-template-columns",
-  "grid-template-rows",
-  "grid-column-start",
-  "grid-column-end",
-  "grid-row-start",
-  "grid-row-end",
-  "grid-auto-flow",
-  "grid-auto-rows",
-  "grid-auto-columns",
-  // type
-  "color",
-  "font-family",
-  "font-size",
-  "font-weight",
-  "font-style",
-  "font-variant-numeric",
-  "font-feature-settings",
-  "line-height",
-  "letter-spacing",
-  "word-spacing",
-  "text-align",
-  "text-indent",
-  "text-transform",
-  "text-overflow",
-  "text-decoration-line",
-  "text-decoration-color",
-  "text-decoration-style",
-  "text-decoration-thickness",
-  "text-underline-offset",
-  "white-space",
-  "word-break",
-  "overflow-wrap",
-  "vertical-align",
-  "direction",
-  "list-style-type",
-  "list-style-position",
-  "-webkit-line-clamp",
-  "-webkit-box-orient",
-  "-webkit-text-fill-color",
-  // paint
-  "background-color",
-  "background-image",
-  "background-position",
-  "background-repeat",
-  "background-size",
-  "background-clip",
-  "background-origin",
-  "opacity",
-  "visibility",
-  "box-shadow",
-  "filter",
-  "backdrop-filter",
-  "mix-blend-mode",
-  "clip-path",
-  // borders + outline
-  "border-top-width",
-  "border-right-width",
-  "border-bottom-width",
-  "border-left-width",
-  "border-top-style",
-  "border-right-style",
-  "border-bottom-style",
-  "border-left-style",
-  "border-top-color",
-  "border-right-color",
-  "border-bottom-color",
-  "border-left-color",
-  "border-top-left-radius",
-  "border-top-right-radius",
-  "border-bottom-right-radius",
-  "border-bottom-left-radius",
-  "outline-width",
-  "outline-style",
-  "outline-color",
-  "outline-offset",
-  // transform
-  "transform",
-  "transform-origin",
-  "rotate",
-  "scale",
-  "translate",
-  // tables
-  "border-collapse",
-  "border-spacing",
-  "table-layout",
-  "caption-side",
-  // replaced content + svg
-  "object-fit",
-  "object-position",
-  "fill",
-  "fill-opacity",
-  "fill-rule",
-  "stroke",
-  "stroke-opacity",
-  "stroke-width",
-  "stroke-linecap",
-  "stroke-linejoin",
-  "stroke-dasharray",
-  "stroke-dashoffset",
-  "shape-rendering",
-  "paint-order",
-];
-
-/** The extension that belongs to what the canvas actually produced (keksdose
- *  `capture-screenshot.ts:247`). */
-const EXTENSIONS: Record<string, string> = {
-  "image/webp": "webp",
-  "image/jpeg": "jpg",
-  "image/png": "png",
-};
-
-/**
- * Keep a node only if it can paint inside the viewport (keksdose `capture-screenshot.ts:267`).
- *
- * - `display: none` goes, with its subtree — it paints nothing and no descendant can.
- *   `<style>` and `<link>` are computed `display: none` too but are NOT decoration: a
- *   stylesheet inside the captured subtree still applies inside the `foreignObject`.
- *   `visibility: hidden` is deliberately not checked: a descendant may set it back.
- * - A box wholly outside the viewport goes: the root is viewport-sized (the app scrolls an
- *   inner `<main>`), so it would have been clipped away unseen.
- * - A ZERO box is kept: `display: contents` reports 0×0 while its children paint, and the
- *   library's filter drops a rejected node's children too.
- */
-function inViewport(node: Node): boolean {
-  if (node.nodeType !== Node.ELEMENT_NODE) return true;
-  const el = node as Element;
-  const tag = el.tagName;
-  if (tag !== "STYLE" && tag !== "LINK") {
-    const display = el.ownerDocument?.defaultView?.getComputedStyle(el).display;
-    if (display === "none") return false;
-  }
-  const rect = el.getBoundingClientRect?.();
-  if (!rect) return true;
-  if (rect.width === 0 && rect.height === 0) return true;
-  return !(rect.bottom <= 0 || rect.right <= 0 || rect.top >= window.innerHeight || rect.left >= window.innerWidth);
-}
-
-/** What `captureAppScreenshot` takes. */
-export interface CaptureAppScreenshotOptions {
-  /**
-   * The id of the element to capture. Default `"root"` — the app's mount point, which is
-   * right on purpose: the kit's `Modal` portals to `document.body` as a SIBLING of `#root`,
-   * so capturing `#root` leaves the feedback dialog and its backdrop out of the picture
-   * without a single filter rule.
-   */
-  rootId?: string;
-}
-
-/**
- * Snapshot the app view behind the feedback dialog, as a `File` ready for the upload —
- * `screenshot.webp`, or `screenshot.png` where the browser cannot encode WebP.
- *
- * - Resolves `null` when there is no element with `rootId` — nothing to capture.
- * - **Rejects** when the optional peer `modern-screenshot` is not installed (the error
- *   says so), or when the capture itself fails. It never swallows: what to say about a
- *   failure is the caller's — `useFeedbackSubmit` toasts `feedbackToast.captureFailed`.
- *
- * Call it from the "Capture screenshot" button's own click and nowhere else (keksdose
- * checked: running it at dialog OPEN was suspected and was never the cause of the wait).
- */
-export async function captureAppScreenshot({
-  rootId = "root",
-}: CaptureAppScreenshotOptions = {}): Promise<File | null> {
-  let domToBlob: (typeof import("modern-screenshot"))["domToBlob"];
-  try {
-    ({ domToBlob } = await import("modern-screenshot"));
-  } catch (cause) {
-    throw new Error(
-      "@eifi1/ui-kit: captureAppScreenshot needs the optional peer `modern-screenshot` — install it (npm i modern-screenshot).",
-      { cause },
-    );
-  }
-  const el = document.getElementById(rootId);
-  if (!el) return null;
-  const blob = await domToBlob(el, {
-    // Falls back to PNG where WebP cannot be encoded; the File is named from the blob.
-    type: "image/webp",
-    quality: 0.9,
-    font: false,
-    includeStyleProperties: [...CAPTURE_STYLE_PROPERTIES],
-    filter: inViewport,
-  });
-  const type = blob.type || "image/png";
-  return new File([blob], `screenshot.${EXTENSIONS[type] ?? "png"}`, { type });
-}

package/src/feedback/feedback-comment-thread.tsx

@@ -1,353 +0,0 @@
-import { useContext } from "react";
-import type { ReactNode } from "react";
-import { EyeOff } from "lucide-react";
-import { Button } from "../components/button";
-import { useConfirm } from "../components/confirm-dialog";
-import type { CommitScope } from "../components/write-lock";
-import type { AuthedFetcher } from "../hooks/use-authed-src";
-import { useKitLabels, type LabelOverride } from "../i18n/kit-labels";
-import { FeedbackStatusBadge } from "./feedback-inbox";
-import type { FeedbackComment, FeedbackReply } from "./feedback-public-types";
-import { DEFAULT_FEEDBACK_PUBLIC_LABELS } from "./feedback-public-labels";
-import { FEEDBACK_COMMENT_MAX, attachmentName } from "./feedback-record";
-import { FeedbackEntryFile, useFeedbackEntryUpload, type FeedbackEntryUploadOptions } from "./feedback-entry-file";
-import { useFeedbackRepliesLabels } from "./feedback-replies";
-import { FeedbackRepliesDrawnAboveContext } from "./feedback-replies-context";
-import { FeedbackReworkEntryChip } from "./feedback-rework-entry-chip";
-import { ChatComposer, FeedbackThread } from "./feedback-thread";
-import type { FeedbackThreadMessage } from "./feedback-thread";
-import { DEFAULT_FEEDBACK_COMMENTS_LABELS } from "./feedback-thread-parts-labels";
-
-/**
- * A report's conversation as everyone who may read it sees it
- * (docs/feedback-harmonization.md §9.8): the admin's replies and other people's
- * comments, each its own row, oldest first — merged by time, on the
- * {@link FeedbackThread} look — and the box to add one.
- *
- * - **Replies** are drawn as staff posts with the status they set ("→ Done"); a reply
- *   without an author is "Admin" (§9.7).
- * - **Reworks** (0.40, §10.3, `kind: "rework"`) are the report's author's: the
- *   **Rework** chip, "→ status" when the entry moved one (a migrated one did not), the
- *   file when the view carries it; at the reading end for the author ({@link
- *   FeedbackCommentThreadProps.isAuthor}), and a `null` name is "A user", never "Admin".
- * - **Comments**: the viewer's own at the reading end ("You"); someone else's under the
- *   first name the server sent, `null` as "A user" (§9.5, `feedbackPublic.anonymous`).
- * - **Hidden** (§9.8): others see "Hidden by an admin" and no name, no text; the writer
- *   sees their text with that line; the admin sees it struck through, with **Unhide**.
- *   An admin has **Hide** on every visible comment; the writer **Delete** on their own,
- *   after a confirm (it is a hard delete). Nobody edits a comment.
- * - **The composer** (1–4 000 characters) only with `onComment`: left out for a reader
- *   who may not write (a demo, a CRASH report — the app knows). Since 0.47 (§12) with
- *   `onUpload` it takes one file as the rework note does — pasted into the box,
- *   captured or picked — uploaded first, its URL `onComment`'s second argument.
- * - **Files** (0.47, §12): a comment's or an entry's file under its text, a picture
- *   INLINE — a thumbnail through `fetcher` that opens full size — any other file a
- *   download ({@link FeedbackEntryFile}). A veiled comment shows none.
- *
- * Delete asks through `useConfirm`, so a thread with `onDelete` and a comment of the
- * viewer's needs the app's `ConfirmProvider` (each app mounts one already).
- */
-
-/* ── Labels ───────────────────────────────────────────────────────────────── */
-
-/** The `feedbackComments` namespace of `<UiKitProvider labels>`. A `null` name is
- *  `feedbackPublic.anonymous`, the public pages' word, so the two say the same. */
-export interface FeedbackCommentsLabels {
-  /** The thread's accessible name. de-CH: "Kommentare" */
-  comments: string;
-  /** Nothing yet. de-CH: "Noch keine Kommentare" */
-  empty: string;
-  /** In place of a hidden comment. de-CH: "Von einem Admin ausgeblendet" */
-  hidden: string;
-  /** The admin's button. de-CH: "Ausblenden" */
-  hide: string;
-  /** The admin's button on a hidden comment. de-CH: "Einblenden" */
-  unhide: string;
-  /** The writer's button. de-CH: "Löschen" */
-  delete: string;
-  /** The confirm's question. de-CH: "Diesen Kommentar löschen?" */
-  deleteTitle: string;
-  /** The confirm's line. de-CH: "Er wird für alle entfernt. Das kann nicht rückgängig
-   *  gemacht werden." */
-  deleteBody: string;
-  /** The box's accessible name. de-CH: "Kommentar schreiben" */
-  field: string;
-  /** de-CH: "Kommentar schreiben…" */
-  placeholder: string;
-  /** The send button. de-CH: "Kommentieren" */
-  send: string;
-}
-
-export { DEFAULT_FEEDBACK_COMMENTS_LABELS };
-
-/* ── The thread ───────────────────────────────────────────────────────────── */
-
-export interface FeedbackCommentThreadProps extends FeedbackEntryUploadOptions {
-  /** The report's replies (§9.7), oldest first. Left out or `[]`: none.
-   *
-   *  Inside `FeedbackRowDetail`'s `comments` they are dropped (0.38.1): the detail's
-   *  Replies section draws them already, and the thread there is the comments alone —
-   *  so passing them there cannot draw them twice. */
-  replies?: readonly FeedbackReply[];
-  /** `GET /feedback/{id}/comments`, oldest first. */
-  comments: readonly FeedbackComment[];
-  /** The viewer is the app's admin: Hide / Unhide, the hidden text struck through. */
-  isAdmin: boolean;
-  /** 0.40: the viewer filed the report (the record's `mine`) — its rework entries are
-   *  theirs, drawn at the reading end as "You". Default `false`. */
-  isAuthor?: boolean;
-  /** `POST /feedback/{id}/comments`. Return the promise: the box clears when it
-   *  resolves, and keeps the draft when it rejects. A failure is the app's to toast —
-   *  the server's `detail`, else `feedbackToast.commentFailed` (0.38.1). Left out, no
-   *  composer. 0.47 (§12): `attachmentUrl` is the one file uploaded with it (with
-   *  `onUpload`), `undefined` for none — `attachment_url` in the body (server-kit 0.18). */
-  onComment?: (body: string, attachmentUrl?: string) => void | Promise<unknown>;
-  /** 0.47: the app's authenticated GET (`AuthedImage`'s) for the files in the thread.
-   *  Left out, the browser's own fetch with same-origin cookies. */
-  fetcher?: AuthedFetcher | null;
-  /** `PATCH /feedback/{id}/comments/{comment_id} {hidden}` — the admin's. A rejection
-   *  is caught (0.38.1) and is the app's to toast (`feedbackToast.updateFailed`). Left
-   *  out, no Hide / Unhide. */
-  onHide?: (comment: FeedbackComment, hidden: boolean) => void | Promise<unknown>;
-  /** `DELETE /feedback/{id}/comments/{comment_id}` — the writer's own, after the
-   *  confirm. Left out, no Delete. */
-  onDelete?: (comment: FeedbackComment) => void | Promise<unknown>;
-  /** A comment is being sent. */
-  pending?: boolean;
-  /** The comments are loading. */
-  loading?: boolean;
-  /** Comment, Hide and Delete COMMIT — see Button's `commit`. */
-  commit?: CommitScope;
-  /** "Now" for the relative times — pin it in tests and stories. */
-  now?: Date | number;
-  labels?: LabelOverride<FeedbackCommentsLabels>;
-  className?: string;
-}
-
-type Entry = { at: number; order: number; message: FeedbackThreadMessage };
-
-/** The admin's Hide / Unhide as a promise — a sync throw or a `void` return included —
- *  so the click can catch it as Delete's does. */
-const hideComment = async (onHide: NonNullable<FeedbackCommentThreadProps["onHide"]>, comment: FeedbackComment) => {
-  await onHide(comment, !comment.hidden);
-};
-
-const instant = (iso: string) => {
-  const ms = Date.parse(iso);
-  return Number.isNaN(ms) ? 0 : ms;
-};
-
-export function FeedbackCommentThread({
-  replies: repliesProp = [],
-  comments,
-  isAdmin,
-  isAuthor = false,
-  onComment,
-  onHide,
-  onDelete,
-  pending = false,
-  loading = false,
-  commit,
-  now,
-  fetcher,
-  onUpload,
-  onCaptureScreenshot,
-  attachment,
-  labels: labelsProp,
-  className,
-}: FeedbackCommentThreadProps) {
-  const { composerAttachment, upload, uploading } = useFeedbackEntryUpload({
-    onUpload,
-    onCaptureScreenshot,
-    attachment,
-  });
-  const labels = useKitLabels("feedbackComments", DEFAULT_FEEDBACK_COMMENTS_LABELS, labelsProp);
-  const anonymous = useKitLabels("feedbackPublic", DEFAULT_FEEDBACK_PUBLIC_LABELS).anonymous;
-  const replyLabels = useFeedbackRepliesLabels();
-  const repliesDrawnAbove = useContext(FeedbackRepliesDrawnAboveContext);
-  const replies = repliesDrawnAbove ? [] : repliesProp;
-
-  const entries: Entry[] = [];
-  replies.forEach((reply, order) => {
-    if (reply.kind === "rework") {
-      entries.push({
-        at: instant(reply.created_at),
-        order,
-        message: {
-          id: `reply-${reply.id}`,
-          own: isAuthor,
-          author: isAuthor ? undefined : (reply.author_name ?? anonymous),
-          createdAt: reply.created_at,
-          body: reply.body,
-          attachments: reply.attachment_url
-            ? [{ name: attachmentName(reply.attachment_url), url: reply.attachment_url }]
-            : undefined,
-          badge: (
-            <span
-              className="inline-flex min-w-0 max-w-full flex-wrap items-center gap-1"
-              data-feedback-entry-kind="rework"
-            >
-              <FeedbackReworkEntryChip />
-              {reply.status_to && (
-                <span className="inline-flex items-center gap-1" data-feedback-reply-status={reply.status_to}>
-                  <span className="sr-only">{replyLabels.statusChanged}</span>
-                  <span aria-hidden>→</span>
-                  <FeedbackStatusBadge status={reply.status_to} />
-                </span>
-              )}
-            </span>
-          ),
-        },
-      });
-      return;
-    }
-    entries.push({
-      at: instant(reply.created_at),
-      order,
-      message: {
-        id: `reply-${reply.id}`,
-        author: reply.author_name ?? replyLabels.admin,
-        createdAt: reply.created_at,
-        body: reply.body,
-        // 0.47 (§12): an admin reply's file, where the view carries it.
-        attachments: reply.attachment_url
-          ? [{ name: attachmentName(reply.attachment_url), url: reply.attachment_url }]
-          : undefined,
-        staff: true,
-        badge: reply.status_to ? (
-          <span
-            className="inline-flex min-w-0 max-w-full flex-wrap items-center gap-1"
-            data-feedback-reply-status={reply.status_to}
-          >
-            <span className="sr-only">{replyLabels.statusChanged}</span>
-            <span aria-hidden>→</span>
-            <FeedbackStatusBadge status={reply.status_to} />
-          </span>
-        ) : undefined,
-      },
-    });
-  });
-  comments.forEach((comment, index) => {
-    // Shown as hidden — no name, no text — to anyone but its writer and an admin.
-    const veiled = comment.hidden && !comment.mine && !isAdmin;
-    const actions: ReactNode[] = [];
-    if (comment.hidden) {
-      actions.push(
-        <span key="hidden" className="inline-flex items-center gap-1" data-feedback-comment-hidden="">
-          <EyeOff className="size-3" aria-hidden />
-          {labels.hidden}
-        </span>,
-      );
-    }
-    if (isAdmin && onHide) {
-      actions.push(
-        <Button
-          key="hide"
-          type="button"
-          variant="link"
-          tone="muted"
-          size="sm"
-          commit={commit}
-          // Caught as Delete's is (0.38.1): a rejection is the app's to toast — its
-          // mutation's `onError` — and must not surface as an unhandled one.
-          onClick={() => void hideComment(onHide, comment).catch(() => {})}
-        >
-          {comment.hidden ? labels.unhide : labels.hide}
-        </Button>,
-      );
-    }
-    if (comment.mine && onDelete) {
-      actions.push(
-        <DeleteCommentButton key="delete" comment={comment} onDelete={onDelete} commit={commit} labels={labels} />,
-      );
-    }
-    entries.push({
-      at: instant(comment.created_at),
-      // Replies first on a tie, then the comments in their order.
-      order: replies.length + index,
-      message: {
-        id: `comment-${comment.id}`,
-        own: comment.mine,
-        author: veiled ? undefined : comment.mine ? undefined : (comment.author_name ?? anonymous),
-        createdAt: comment.created_at,
-        body: veiled ? "" : (comment.body ?? ""),
-        // 0.47 (§12): its one file; none when veiled (the server sends none either).
-        attachments:
-          !veiled && comment.attachment_url
-            ? [{ name: attachmentName(comment.attachment_url), url: comment.attachment_url }]
-            : undefined,
-        staff: !veiled && comment.staff,
-        struck: comment.hidden && isAdmin && !veiled,
-        footer: actions.length > 0 ? actions : undefined,
-      },
-    });
-  });
-  entries.sort((a, b) => a.at - b.at || a.order - b.order);
-
-  return (
-    <div className={className} data-feedback-comment-thread="">
-      <FeedbackThread
-        messages={entries.map((entry) => entry.message)}
-        loading={loading}
-        now={now}
-        renderAttachment={(file) => (file.url ? <FeedbackEntryFile url={file.url} fetcher={fetcher} /> : file.name)}
-        labels={{ thread: labels.comments, empty: labels.empty }}
-      />
-      {onComment && (
-        <ChatComposer
-          className="mt-3"
-          onSend={async (body, file) => {
-            const url = await upload(file);
-            // The second argument only with a file, so a text comment calls as before 0.47.
-            return url === undefined ? onComment(body) : onComment(body, url);
-          }}
-          pending={pending || uploading}
-          attachment={composerAttachment}
-          commit={commit}
-          maxLength={FEEDBACK_COMMENT_MAX}
-          showCount
-          labels={{ field: labels.field, placeholder: labels.placeholder, send: labels.send }}
-        />
-      )}
-    </div>
-  );
-}
-
-/** The writer's Delete: a confirm (danger tone, focus on Cancel), then `onDelete`. Its
- *  own component, so `useConfirm` — which needs a `ConfirmProvider` — is only called
- *  where there is something to delete. */
-function DeleteCommentButton({
-  comment,
-  onDelete,
-  commit,
-  labels,
-}: {
-  comment: FeedbackComment;
-  onDelete: (comment: FeedbackComment) => void | Promise<unknown>;
-  commit?: CommitScope;
-  labels: FeedbackCommentsLabels;
-}) {
-  const confirm = useConfirm();
-  const remove = async () => {
-    const ok = await confirm({
-      title: labels.deleteTitle,
-      body: labels.deleteBody,
-      confirmLabel: labels.delete,
-      tone: "danger",
-      commit,
-    });
-    if (ok) await onDelete(comment);
-  };
-  return (
-    <Button
-      type="button"
-      variant="link"
-      tone="danger"
-      size="sm"
-      commit={commit}
-      onClick={() => void remove().catch(() => {})}
-    >
-      {labels.delete}
-    </Button>
-  );
-}

package/src/feedback/feedback-context-labels.ts

@@ -1,11 +0,0 @@
-// The English words of ./feedback-context, in a module with no component code: the English
-// catalogue (src/i18n/defaults.ts) imports them without the component, so `en` loads as one
-// chunk like the other languages (0.34, KS U8). ./feedback-context re-exports them.
-
-import type { FeedbackContextLabels } from "./feedback-context";
-
-export const DEFAULT_FEEDBACK_CONTEXT_LABELS: FeedbackContextLabels = {
-  user: "User",
-  attachUrl: "Attach current page URL",
-  diagnostics: "Diagnostics sent with the report",
-};