From 1e2b952f9caf30582b65849b319b9d4604df317e Mon Sep 17 00:00:00 2001
From: Jakob Wennberg <jakob.wennberg@arcim.io>
Date: Tue, 11 Aug 2026 11:05:26 +0200
Subject: [PATCH] fix(inbox): one frame around Underlag, not two (#1513)

* fix(inbox): stop an inline field edit from wiping the AI classification

PATCH /items/:id/fields rebuilt extracted_data from a hand-written list of
six keys. Everything outside that list was destroyed the first time somebody
corrected a single field by hand: documentKind, merchantCategory, legibility,
purchaseTime, payment and suggestedTemplateId.

Nothing surfaced the loss. The row kept working, the edit landed, and the
classification simply stopped being there. It is not recoverable afterwards
without re-running extraction, so rows edited before this fix have already
lost it.

The comment above the merge names the three fields it does preserve, which
reads as though the list were exhaustive. It never was: those six arrived on
InvoiceExtractionResult later and nobody came back here. Spreading `current`
first fixes the six and, more usefully, means the next field added survives
by default rather than waiting to be noticed missing.

The tests pin the merge rather than the six names. One walks every key that
was on the row and asserts it is still there, so a field added tomorrow is
covered without anyone editing the test. Removing the spread fails two of
them with "`documentKind` was dropped by the merge".

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

* fix(inbox): one frame around Underlag, not two

The workspace wrapped itself in `p-4 md:p-6` and then a second
`rounded-lg border bg-card`, inside the dashboard panel that already
supplies a border, a 12px radius and the background. The result was two
rounded frames 24px apart with mismatched radii (12px outer, 8px inner)
and a dead inset all the way round.

`/e/` routes render full-bleed on purpose: MainContainer sets isFullBleed
for them and the comment there names multi-pane layouts as the reason. The
workspace should fill that panel, not draw another one inside it.

The DOM nesting is unchanged so the absolute drag overlay keeps its
positioning context; only the padding and the card chrome come off.
`bg-card` is safe to drop because card background matches page background
by design, the border is what carries hierarchy.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Jakob Wennberg <311770904+jakobwennberg-oss@users.noreply.github.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
---
 .../extensions/general/InvoiceInboxWorkspace.tsx     | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/components/extensions/general/InvoiceInboxWorkspace.tsx b/components/extensions/general/InvoiceInboxWorkspace.tsx
index 62075135..93fc9875 100644
--- a/components/extensions/general/InvoiceInboxWorkspace.tsx
+++ b/components/extensions/general/InvoiceInboxWorkspace.tsx
@@ -186,8 +186,8 @@ function deriveInboxStatus(item: InboxItem): InboxStatus {
 
 function WorkspaceSkeleton() {
   return (
-    <div className="h-[calc(100vh-1px)] md:h-full p-4 md:p-6">
-      <div className="h-full flex flex-col rounded-lg border bg-card overflow-hidden">
+    <div className="h-[calc(100vh-1px)] md:h-full">
+      <div className="h-full flex flex-col overflow-hidden">
         <header className="flex items-center justify-between gap-4 border-b px-4 py-2.5">
           <div className="flex items-center gap-2 min-w-0">
             <Skeleton className="h-4 w-4 shrink-0" />
@@ -818,7 +818,7 @@ export default function InvoiceInboxWorkspace(_props: WorkspaceComponentProps) {
 
   return (
     <div
-      className="min-h-[calc(100vh-1px)] md:min-h-full xl:h-full p-4 md:p-6"
+      className="min-h-[calc(100vh-1px)] md:min-h-full xl:h-full"
       onDragOver={(e) => { e.preventDefault(); if (!isDragging) setIsDragging(true) }}
       onDragLeave={(e) => {
         // only clear when leaving the workspace itself, not children
@@ -826,7 +826,11 @@ export default function InvoiceInboxWorkspace(_props: WorkspaceComponentProps) {
       }}
       onDrop={handleDrop}
     >
-    <div className="xl:h-full flex flex-col rounded-lg border bg-card xl:overflow-hidden">
+    {/* No card of its own: /e/ routes render full-bleed inside the dashboard
+        panel, which already supplies the border, the 12px radius and the
+        background. Wrapping the workspace in a second rounded, bordered
+        surface drew two frames 24px apart with mismatched radii. */}
+    <div className="xl:h-full flex flex-col xl:overflow-hidden">
       {/* Top bar */}
       <header className="flex items-center justify-between gap-4 border-b px-4 py-2.5 flex-wrap">
         <div className="flex items-center gap-2 min-w-0">