From ae445a78aecbe484c57318a8a03b78bf89407f31 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 16:01:21 +0200
Subject: [PATCH 01/46] feat(documents): extend quick-upload with optional
 batch metadata part

- Add DocumentBatchMetadataDTO (titles, senderId, receiverIds, documentDate, location, tags, metadataComplete)
- Add BATCH_TOO_LARGE to ErrorCode
- Extend quickUpload to accept optional @RequestPart("metadata"); dispatches to storeDocumentWithBatchMetadata when present
- Cap batch at 50 files/request; reject 400 when titles.size > files.size
- Add DocumentService.storeDocumentWithBatchMetadata applying shared fields + index-based titles to both created and updated docs
- Raise max-request-size to 500MB (10-file chunk at max per-file size)
- Add structured SLF4J logging for every quickUpload call

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../controller/DocumentController.java        |  22 ++-
 .../dto/DocumentBatchMetadataDTO.java         |  18 +++
 .../familienarchiv/exception/ErrorCode.java   |   2 +
 .../service/DocumentService.java              |  38 +++++
 backend/src/main/resources/application.yaml   |   3 +-
 .../controller/DocumentControllerTest.java    | 133 +++++++++++++++++-
 6 files changed, 212 insertions(+), 4 deletions(-)
 create mode 100644 backend/src/main/java/org/raddatz/familienarchiv/dto/DocumentBatchMetadataDTO.java

diff --git a/backend/src/main/java/org/raddatz/familienarchiv/controller/DocumentController.java b/backend/src/main/java/org/raddatz/familienarchiv/controller/DocumentController.java
index 8b17a0ef..cd03a22f 100644
--- a/backend/src/main/java/org/raddatz/familienarchiv/controller/DocumentController.java
+++ b/backend/src/main/java/org/raddatz/familienarchiv/controller/DocumentController.java
@@ -18,6 +18,7 @@ import jakarta.validation.constraints.Min;
 import org.springframework.data.domain.PageRequest;
 import org.springframework.data.domain.Pageable;
 import org.springframework.validation.annotation.Validated;
+import org.raddatz.familienarchiv.dto.DocumentBatchMetadataDTO;
 import org.raddatz.familienarchiv.dto.DocumentSearchResult;
 import org.raddatz.familienarchiv.dto.DocumentUpdateDTO;
 import org.raddatz.familienarchiv.dto.TagOperator;
@@ -193,6 +194,7 @@ public class DocumentController {
     @RequirePermission(Permission.WRITE_ALL)
     public QuickUploadResult quickUpload(
             @RequestPart(value = "files", required = false) List<MultipartFile> files,
+            @RequestPart(value = "metadata", required = false) DocumentBatchMetadataDTO metadata,
             Authentication authentication) {
         List<Document> created = new ArrayList<>();
         List<Document> updated = new ArrayList<>();
@@ -202,14 +204,26 @@ public class DocumentController {
             return new QuickUploadResult(created, updated, errors);
         }
 
+        if (files.size() > 50) {
+            throw DomainException.badRequest(ErrorCode.BATCH_TOO_LARGE, "Batch exceeds maximum of 50 files per request");
+        }
+        if (metadata != null && metadata.getTitles() != null && metadata.getTitles().size() > files.size()) {
+            throw DomainException.badRequest(ErrorCode.VALIDATION_ERROR, "titles count must not exceed files count");
+        }
+
         UUID actorId = requireUserId(authentication);
-        for (MultipartFile file : files) {
+        long totalBytes = files.stream().mapToLong(MultipartFile::getSize).sum();
+
+        for (int i = 0; i < files.size(); i++) {
+            MultipartFile file = files.get(i);
             if (!ALLOWED_CONTENT_TYPES.contains(file.getContentType())) {
                 errors.add(new UploadError(file.getOriginalFilename(), "UNSUPPORTED_FILE_TYPE"));
                 continue;
             }
             try {
-                DocumentService.StoreResult result = documentService.storeDocument(file, actorId);
+                DocumentService.StoreResult result = metadata != null
+                        ? documentService.storeDocumentWithBatchMetadata(file, metadata, i, actorId)
+                        : documentService.storeDocument(file, actorId);
                 if (result.isNew()) {
                     created.add(result.document());
                 } else {
@@ -221,6 +235,10 @@ public class DocumentController {
             }
         }
 
+        log.info("quickUpload actor={} files={} totalBytes={} withMetadata={} created={} updated={} errors={}",
+                actorId, files.size(), totalBytes, metadata != null,
+                created.size(), updated.size(), errors.size());
+
         return new QuickUploadResult(created, updated, errors);
     }
 
diff --git a/backend/src/main/java/org/raddatz/familienarchiv/dto/DocumentBatchMetadataDTO.java b/backend/src/main/java/org/raddatz/familienarchiv/dto/DocumentBatchMetadataDTO.java
new file mode 100644
index 00000000..93e17fbf
--- /dev/null
+++ b/backend/src/main/java/org/raddatz/familienarchiv/dto/DocumentBatchMetadataDTO.java
@@ -0,0 +1,18 @@
+package org.raddatz.familienarchiv.dto;
+
+import lombok.Data;
+
+import java.time.LocalDate;
+import java.util.List;
+import java.util.UUID;
+
+@Data
+public class DocumentBatchMetadataDTO {
+    private List<String> titles;
+    private UUID senderId;
+    private List<UUID> receiverIds;
+    private LocalDate documentDate;
+    private String location;
+    private String tags;
+    private Boolean metadataComplete;
+}
diff --git a/backend/src/main/java/org/raddatz/familienarchiv/exception/ErrorCode.java b/backend/src/main/java/org/raddatz/familienarchiv/exception/ErrorCode.java
index 85cd7d2c..686c8627 100644
--- a/backend/src/main/java/org/raddatz/familienarchiv/exception/ErrorCode.java
+++ b/backend/src/main/java/org/raddatz/familienarchiv/exception/ErrorCode.java
@@ -109,6 +109,8 @@ public enum ErrorCode {
     // --- Generic ---
     /** Request validation failed (missing or malformed fields). 400 */
     VALIDATION_ERROR,
+    /** Batch upload exceeds the maximum allowed file count per request. 400 */
+    BATCH_TOO_LARGE,
     /** An unexpected server-side error occurred. 500 */
     INTERNAL_ERROR,
 }
diff --git a/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java b/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java
index fbdec954..36575f28 100644
--- a/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java
+++ b/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java
@@ -7,6 +7,7 @@ import org.raddatz.familienarchiv.audit.ActivityActorDTO;
 import org.raddatz.familienarchiv.audit.AuditKind;
 import org.raddatz.familienarchiv.audit.AuditLogQueryService;
 import org.raddatz.familienarchiv.audit.AuditService;
+import org.raddatz.familienarchiv.dto.DocumentBatchMetadataDTO;
 import org.raddatz.familienarchiv.dto.DocumentSearchItem;
 import org.raddatz.familienarchiv.dto.DocumentSearchResult;
 import org.raddatz.familienarchiv.dto.DocumentSort;
@@ -132,6 +133,43 @@ public class DocumentService {
         return new StoreResult(saved, isNew);
     }
 
+    @Transactional
+    public StoreResult storeDocumentWithBatchMetadata(
+            MultipartFile file, DocumentBatchMetadataDTO metadata, int fileIndex, UUID actorId) throws IOException {
+        StoreResult base = storeDocument(file, actorId);
+        Document doc = base.document();
+
+        if (metadata.getTitles() != null && fileIndex < metadata.getTitles().size()) {
+            doc.setTitle(metadata.getTitles().get(fileIndex));
+        }
+        if (metadata.getSenderId() != null) {
+            doc.setSender(personService.getById(metadata.getSenderId()));
+        }
+        if (metadata.getReceiverIds() != null && !metadata.getReceiverIds().isEmpty()) {
+            doc.setReceivers(new HashSet<>(personService.getAllById(metadata.getReceiverIds())));
+        }
+        if (metadata.getDocumentDate() != null) {
+            doc.setDocumentDate(metadata.getDocumentDate());
+        }
+        if (metadata.getLocation() != null) {
+            doc.setLocation(metadata.getLocation());
+        }
+        if (metadata.getMetadataComplete() != null) {
+            doc.setMetadataComplete(metadata.getMetadataComplete());
+        }
+        if (metadata.getTags() != null && !metadata.getTags().isBlank()) {
+            List<String> tagNames = Arrays.stream(metadata.getTags().split(","))
+                    .map(String::trim).filter(s -> !s.isEmpty()).toList();
+            UUID docId = doc.getId();
+            updateDocumentTags(docId, tagNames);
+            doc = documentRepository.findById(docId)
+                    .orElseThrow(() -> DomainException.notFound(ErrorCode.DOCUMENT_NOT_FOUND, "Not found after batch metadata: " + docId));
+        }
+
+        Document saved = documentRepository.save(doc);
+        return new StoreResult(saved, base.isNew());
+    }
+
     @Transactional
     public Document createDocument(DocumentUpdateDTO dto, MultipartFile file) throws IOException {
         String filename = (file != null && !file.isEmpty())
diff --git a/backend/src/main/resources/application.yaml b/backend/src/main/resources/application.yaml
index d9bbe9d0..1cdd7673 100644
--- a/backend/src/main/resources/application.yaml
+++ b/backend/src/main/resources/application.yaml
@@ -23,7 +23,8 @@ spring:
   servlet:
     multipart:
       max-file-size: 50MB
-      max-request-size: 50MB
+      max-request-size: 500MB   # supports 10-file chunk at max per-file size; see #317
+      file-size-threshold: 2KB
 
   mail:
     host: ${MAIL_HOST:}
diff --git a/backend/src/test/java/org/raddatz/familienarchiv/controller/DocumentControllerTest.java b/backend/src/test/java/org/raddatz/familienarchiv/controller/DocumentControllerTest.java
index eb4c6873..1981b1b2 100644
--- a/backend/src/test/java/org/raddatz/familienarchiv/controller/DocumentControllerTest.java
+++ b/backend/src/test/java/org/raddatz/familienarchiv/controller/DocumentControllerTest.java
@@ -1,15 +1,17 @@
 package org.raddatz.familienarchiv.controller;
 
 import org.junit.jupiter.api.Test;
+import org.raddatz.familienarchiv.dto.DocumentBatchMetadataDTO;
 import org.raddatz.familienarchiv.dto.DocumentSearchResult;
 import org.raddatz.familienarchiv.dto.DocumentVersionSummary;
 import org.raddatz.familienarchiv.exception.DomainException;
 import org.raddatz.familienarchiv.exception.ErrorCode;
+import org.raddatz.familienarchiv.model.AppUser;
 import org.raddatz.familienarchiv.model.Document;
 import org.raddatz.familienarchiv.model.DocumentStatus;
 import org.raddatz.familienarchiv.model.DocumentVersion;
+import org.raddatz.familienarchiv.model.Person;
 import org.raddatz.familienarchiv.security.PermissionAspect;
-import org.raddatz.familienarchiv.model.AppUser;
 import org.raddatz.familienarchiv.service.CustomUserDetailsService;
 import org.raddatz.familienarchiv.service.DocumentService;
 import org.raddatz.familienarchiv.service.DocumentVersionService;
@@ -766,4 +768,133 @@ class DocumentControllerTest {
                 .andExpect(status().isOk())
                 .andExpect(jsonPath("$.editorName").value("Otto"));
     }
+
+    // ─── POST /api/documents/quick-upload — metadata part ────────────────────
+
+    @Test
+    @WithMockUser(authorities = "WRITE_ALL")
+    void quickUpload_withMetadata_appliesSharedFieldsToAllCreatedDocuments() throws Exception {
+        UUID senderId = UUID.randomUUID();
+        Person sender = Person.builder().id(senderId).lastName("Müller").build();
+
+        Document doc1 = Document.builder().id(UUID.randomUUID()).title("Brief 1").originalFilename("a.pdf").sender(sender).build();
+        Document doc2 = Document.builder().id(UUID.randomUUID()).title("Brief 2").originalFilename("b.pdf").sender(sender).build();
+        Document doc3 = Document.builder().id(UUID.randomUUID()).title("Brief 3").originalFilename("c.pdf").sender(sender).build();
+
+        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
+        when(documentService.storeDocumentWithBatchMetadata(any(), any(), eq(0), any()))
+                .thenReturn(new DocumentService.StoreResult(doc1, true));
+        when(documentService.storeDocumentWithBatchMetadata(any(), any(), eq(1), any()))
+                .thenReturn(new DocumentService.StoreResult(doc2, true));
+        when(documentService.storeDocumentWithBatchMetadata(any(), any(), eq(2), any()))
+                .thenReturn(new DocumentService.StoreResult(doc3, true));
+
+        org.springframework.mock.web.MockMultipartFile f1 =
+                new org.springframework.mock.web.MockMultipartFile("files", "a.pdf", "application/pdf", new byte[]{1});
+        org.springframework.mock.web.MockMultipartFile f2 =
+                new org.springframework.mock.web.MockMultipartFile("files", "b.pdf", "application/pdf", new byte[]{1});
+        org.springframework.mock.web.MockMultipartFile f3 =
+                new org.springframework.mock.web.MockMultipartFile("files", "c.pdf", "application/pdf", new byte[]{1});
+        org.springframework.mock.web.MockMultipartFile metadata =
+                new org.springframework.mock.web.MockMultipartFile("metadata", "metadata", "application/json",
+                        ("{\"senderId\":\"" + senderId + "\"}").getBytes());
+
+        mockMvc.perform(multipart("/api/documents/quick-upload").file(f1).file(f2).file(f3).file(metadata))
+                .andExpect(status().isOk())
+                .andExpect(jsonPath("$.created.length()").value(3))
+                .andExpect(jsonPath("$.created[0].sender.id").value(senderId.toString()))
+                .andExpect(jsonPath("$.created[1].sender.id").value(senderId.toString()))
+                .andExpect(jsonPath("$.created[2].sender.id").value(senderId.toString()))
+                .andExpect(jsonPath("$.updated").isEmpty())
+                .andExpect(jsonPath("$.errors").isEmpty());
+    }
+
+    @Test
+    @WithMockUser(authorities = "WRITE_ALL")
+    void quickUpload_withMetadata_appliesSharedFieldsToUpdatedDocuments() throws Exception {
+        UUID senderId = UUID.randomUUID();
+        Person sender = Person.builder().id(senderId).lastName("Müller").build();
+        Document existing = Document.builder().id(UUID.randomUUID()).title("Alt").originalFilename("alt.pdf").sender(sender).build();
+
+        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
+        when(documentService.storeDocumentWithBatchMetadata(any(), any(), eq(0), any()))
+                .thenReturn(new DocumentService.StoreResult(existing, false));
+
+        org.springframework.mock.web.MockMultipartFile file =
+                new org.springframework.mock.web.MockMultipartFile("files", "alt.pdf", "application/pdf", new byte[]{1});
+        org.springframework.mock.web.MockMultipartFile metadata =
+                new org.springframework.mock.web.MockMultipartFile("metadata", "metadata", "application/json",
+                        ("{\"senderId\":\"" + senderId + "\"}").getBytes());
+
+        mockMvc.perform(multipart("/api/documents/quick-upload").file(file).file(metadata))
+                .andExpect(status().isOk())
+                .andExpect(jsonPath("$.created").isEmpty())
+                .andExpect(jsonPath("$.updated[0].sender.id").value(senderId.toString()))
+                .andExpect(jsonPath("$.errors").isEmpty());
+    }
+
+    @Test
+    @WithMockUser(authorities = "WRITE_ALL")
+    void quickUpload_withMetadata_mapsTitlesByIndex() throws Exception {
+        Document docA = Document.builder().id(UUID.randomUUID()).title("Alpha").originalFilename("a.pdf").build();
+        Document docB = Document.builder().id(UUID.randomUUID()).title("Beta").originalFilename("b.pdf").build();
+        Document docC = Document.builder().id(UUID.randomUUID()).title("Gamma").originalFilename("c.pdf").build();
+
+        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
+        when(documentService.storeDocumentWithBatchMetadata(any(), any(), eq(0), any()))
+                .thenReturn(new DocumentService.StoreResult(docA, true));
+        when(documentService.storeDocumentWithBatchMetadata(any(), any(), eq(1), any()))
+                .thenReturn(new DocumentService.StoreResult(docB, true));
+        when(documentService.storeDocumentWithBatchMetadata(any(), any(), eq(2), any()))
+                .thenReturn(new DocumentService.StoreResult(docC, true));
+
+        org.springframework.mock.web.MockMultipartFile f1 =
+                new org.springframework.mock.web.MockMultipartFile("files", "a.pdf", "application/pdf", new byte[]{1});
+        org.springframework.mock.web.MockMultipartFile f2 =
+                new org.springframework.mock.web.MockMultipartFile("files", "b.pdf", "application/pdf", new byte[]{1});
+        org.springframework.mock.web.MockMultipartFile f3 =
+                new org.springframework.mock.web.MockMultipartFile("files", "c.pdf", "application/pdf", new byte[]{1});
+        org.springframework.mock.web.MockMultipartFile metadata =
+                new org.springframework.mock.web.MockMultipartFile("metadata", "metadata", "application/json",
+                        "{\"titles\":[\"Alpha\",\"Beta\",\"Gamma\"]}".getBytes());
+
+        mockMvc.perform(multipart("/api/documents/quick-upload").file(f1).file(f2).file(f3).file(metadata))
+                .andExpect(status().isOk())
+                .andExpect(jsonPath("$.created[0].title").value("Alpha"))
+                .andExpect(jsonPath("$.created[1].title").value("Beta"))
+                .andExpect(jsonPath("$.created[2].title").value("Gamma"));
+    }
+
+    @Test
+    @WithMockUser(authorities = "WRITE_ALL")
+    void quickUpload_withMetadata_rejects400_whenTitlesSizeExceedsFilesSize() throws Exception {
+        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
+
+        org.springframework.mock.web.MockMultipartFile f1 =
+                new org.springframework.mock.web.MockMultipartFile("files", "a.pdf", "application/pdf", new byte[]{1});
+        org.springframework.mock.web.MockMultipartFile f2 =
+                new org.springframework.mock.web.MockMultipartFile("files", "b.pdf", "application/pdf", new byte[]{1});
+        org.springframework.mock.web.MockMultipartFile metadata =
+                new org.springframework.mock.web.MockMultipartFile("metadata", "metadata", "application/json",
+                        "{\"titles\":[\"A\",\"B\",\"C\"]}".getBytes());
+
+        mockMvc.perform(multipart("/api/documents/quick-upload").file(f1).file(f2).file(metadata))
+                .andExpect(status().isBadRequest());
+    }
+
+    @Test
+    @WithMockUser(authorities = "WRITE_ALL")
+    void quickUpload_returns400_whenBatchExceedsCap() throws Exception {
+        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
+
+        var builder = multipart("/api/documents/quick-upload");
+        for (int i = 0; i < 51; i++) {
+            builder.file(new org.springframework.mock.web.MockMultipartFile(
+                    "files", "f" + i + ".pdf", "application/pdf", new byte[]{1}));
+        }
+
+        mockMvc.perform(builder)
+                .andExpect(status().isBadRequest())
+                .andExpect(jsonPath("$.code").value("BATCH_TOO_LARGE"));
+    }
 }
-- 
2.49.1


From f86105a1bea78e209817c148b6c3792a2a9ee902 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 16:18:51 +0200
Subject: [PATCH 02/46] feat(i18n): add BATCH_TOO_LARGE error code + 16
 bulk-upload Paraglide keys

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/messages/de.json  | 19 ++++++++++++++++++-
 frontend/messages/en.json  | 19 ++++++++++++++++++-
 frontend/messages/es.json  | 19 ++++++++++++++++++-
 frontend/src/lib/errors.ts |  3 +++
 4 files changed, 57 insertions(+), 3 deletions(-)

diff --git a/frontend/messages/de.json b/frontend/messages/de.json
index 1b90d913..8bd62c50 100644
--- a/frontend/messages/de.json
+++ b/frontend/messages/de.json
@@ -811,6 +811,7 @@
 	"pagination_next": "Weiter",
 	"pagination_page_of": "Seite {page} von {total}",
 	"pagination_nav_label": "Seitennavigation",
+<<<<<<< HEAD
 
 	"common_opens_new_tab": "(öffnet in neuem Tab)",
 
@@ -850,5 +851,21 @@
 	"richtlinien_klaer_umbrueche": "Originale Zeilenumbrüche",
 	"richtlinien_klaer_caps": "Alte Groß-/Kleinschreibung",
 	"richtlinien_closing_title": "Fehlt eine Regel?",
-	"richtlinien_closing_body": "Stolpern Sie beim Transkribieren über eine Situation, die hier nicht steht — schreiben Sie einen Kommentar beim betreffenden Block. Wir sammeln sie und besprechen sie beim nächsten Familientreffen."
+	"richtlinien_closing_body": "Stolpern Sie beim Transkribieren über eine Situation, die hier nicht steht — schreiben Sie einen Kommentar beim betreffenden Block. Wir sammeln sie und besprechen sie beim nächsten Familientreffen.",
+	"error_batch_too_large": "Zu viele Dateien auf einmal — bitte in Blöcken hochladen.",
+	"bulk_drop_hint": "Eine oder mehrere Dateien ablegen",
+	"bulk_drop_sub": "PDF, JPEG, PNG oder TIFF · bis zu 50 MB pro Datei",
+	"bulk_count_pill": "{count} werden erstellt",
+	"bulk_save_cta": "{count, plural, one {Speichern →} other {{count} speichern →}}",
+	"bulk_discard_all": "Alle verwerfen",
+	"bulk_add_more": "Weitere hinzufügen",
+	"bulk_scope_per_file_label": "Nur diese Datei",
+	"bulk_scope_shared_label": "Gilt für alle {count}",
+	"bulk_title_suggested_hint": "Vorschlag aus Dateiname — zum Bearbeiten anklicken",
+	"bulk_switcher_prev": "Vorherige Datei",
+	"bulk_switcher_next": "Nächste Datei",
+	"bulk_file_error_chip_label": "Fehler beim Hochladen",
+	"bulk_upload_progress": "{done} von {total} hochgeladen",
+	"bulk_partial_success": "{created} erstellt, {failed} fehlgeschlagen",
+	"bulk_all_failed": "Alle Uploads fehlgeschlagen"
 }
diff --git a/frontend/messages/en.json b/frontend/messages/en.json
index f5862f0f..afc3ed73 100644
--- a/frontend/messages/en.json
+++ b/frontend/messages/en.json
@@ -811,6 +811,7 @@
 	"pagination_next": "Next",
 	"pagination_page_of": "Page {page} of {total}",
 	"pagination_nav_label": "Pagination",
+<<<<<<< HEAD
 
 	"common_opens_new_tab": "(opens in new tab)",
 
@@ -850,5 +851,21 @@
 	"richtlinien_klaer_umbrueche": "Original line breaks",
 	"richtlinien_klaer_caps": "Old capitalisation",
 	"richtlinien_closing_title": "Missing a rule?",
-	"richtlinien_closing_body": "If you hit a situation while transcribing that isn't listed here — leave a comment on the relevant block. We collect them and discuss them at the next family gathering."
+	"richtlinien_closing_body": "If you hit a situation while transcribing that isn't listed here — leave a comment on the relevant block. We collect them and discuss them at the next family gathering.",
+	"error_batch_too_large": "Too many files at once — please upload in smaller batches.",
+	"bulk_drop_hint": "Drop one or more files here",
+	"bulk_drop_sub": "PDF, JPEG, PNG or TIFF · up to 50 MB per file",
+	"bulk_count_pill": "{count} will be created",
+	"bulk_save_cta": "{count, plural, one {Save →} other {Save {count} →}}",
+	"bulk_discard_all": "Discard all",
+	"bulk_add_more": "Add more",
+	"bulk_scope_per_file_label": "This file only",
+	"bulk_scope_shared_label": "Applies to all {count}",
+	"bulk_title_suggested_hint": "Suggested from filename — click to edit",
+	"bulk_switcher_prev": "Previous file",
+	"bulk_switcher_next": "Next file",
+	"bulk_file_error_chip_label": "Upload failed",
+	"bulk_upload_progress": "{done} of {total} uploaded",
+	"bulk_partial_success": "{created} created, {failed} failed",
+	"bulk_all_failed": "All uploads failed"
 }
diff --git a/frontend/messages/es.json b/frontend/messages/es.json
index c949ce2c..cf1f27e4 100644
--- a/frontend/messages/es.json
+++ b/frontend/messages/es.json
@@ -811,6 +811,7 @@
 	"pagination_next": "Siguiente",
 	"pagination_page_of": "Página {page} de {total}",
 	"pagination_nav_label": "Paginación",
+<<<<<<< HEAD
 
 	"common_opens_new_tab": "(abre en pestaña nueva)",
 
@@ -850,5 +851,21 @@
 	"richtlinien_klaer_umbrueche": "Saltos de línea originales",
 	"richtlinien_klaer_caps": "Mayúsculas antiguas",
 	"richtlinien_closing_title": "¿Falta una regla?",
-	"richtlinien_closing_body": "Si al transcribir encuentras una situación que no está aquí — deja un comentario en el bloque. Las recogemos y las discutimos en la próxima reunión familiar."
+	"richtlinien_closing_body": "Si al transcribir encuentras una situación que no está aquí — deja un comentario en el bloque. Las recogemos y las discutimos en la próxima reunión familiar.",
+	"error_batch_too_large": "Demasiados archivos a la vez — sube en lotes más pequeños.",
+	"bulk_drop_hint": "Suelta uno o varios archivos aquí",
+	"bulk_drop_sub": "PDF, JPEG, PNG o TIFF · hasta 50 MB por archivo",
+	"bulk_count_pill": "Se crearán {count}",
+	"bulk_save_cta": "{count, plural, one {Guardar →} other {Guardar {count} →}}",
+	"bulk_discard_all": "Descartar todo",
+	"bulk_add_more": "Añadir más",
+	"bulk_scope_per_file_label": "Solo este archivo",
+	"bulk_scope_shared_label": "Para todos los {count}",
+	"bulk_title_suggested_hint": "Sugerencia del nombre de archivo — haz clic para editar",
+	"bulk_switcher_prev": "Archivo anterior",
+	"bulk_switcher_next": "Archivo siguiente",
+	"bulk_file_error_chip_label": "Error al subir",
+	"bulk_upload_progress": "{done} de {total} subidos",
+	"bulk_partial_success": "{created} creados, {failed} fallidos",
+	"bulk_all_failed": "Todos los uploads fallaron"
 }
diff --git a/frontend/src/lib/errors.ts b/frontend/src/lib/errors.ts
index d9ec385d..1a1d1551 100644
--- a/frontend/src/lib/errors.ts
+++ b/frontend/src/lib/errors.ts
@@ -41,6 +41,7 @@ export type ErrorCode =
 	| 'UNAUTHORIZED'
 	| 'FORBIDDEN'
 	| 'VALIDATION_ERROR'
+	| 'BATCH_TOO_LARGE'
 	| 'INTERNAL_ERROR';
 
 export interface BackendError {
@@ -139,6 +140,8 @@ export function getErrorMessage(code: ErrorCode | string | undefined): string {
 			return m.error_forbidden();
 		case 'VALIDATION_ERROR':
 			return m.error_validation_error();
+		case 'BATCH_TOO_LARGE':
+			return m.error_batch_too_large();
 		default:
 			return m.error_internal_error();
 	}
-- 
2.49.1


From 4248d8af72fec850d5c022b427c19fd98323ce22 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 16:30:12 +0200
Subject: [PATCH 03/46] feat(bulk-upload): add bulkTitleFromFilename utility

Converts a raw filename into a human-readable title candidate by
stripping the extension and replacing underscore/hyphen runs with spaces.
Reuses the existing stripExtension() helper.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/src/lib/utils/filename.spec.ts | 20 +++++++++++++++++++-
 frontend/src/lib/utils/filename.ts      |  4 ++++
 2 files changed, 23 insertions(+), 1 deletion(-)

diff --git a/frontend/src/lib/utils/filename.spec.ts b/frontend/src/lib/utils/filename.spec.ts
index 297c8a1e..77b3fbdd 100644
--- a/frontend/src/lib/utils/filename.spec.ts
+++ b/frontend/src/lib/utils/filename.spec.ts
@@ -1,5 +1,5 @@
 import { describe, it, expect } from 'vitest';
-import { parseFilename, stripExtension } from './filename';
+import { parseFilename, stripExtension, bulkTitleFromFilename } from './filename';
 
 describe('parseFilename', () => {
 	describe('date-first patterns', () => {
@@ -86,6 +86,24 @@ describe('parseFilename', () => {
 	});
 });
 
+describe('bulkTitleFromFilename', () => {
+	it('replaces underscores with spaces', () => {
+		expect(bulkTitleFromFilename('hello_world.pdf')).toBe('hello world');
+	});
+
+	it('replaces hyphens with spaces', () => {
+		expect(bulkTitleFromFilename('2024-01-01_Max.pdf')).toBe('2024 01 01 Max');
+	});
+
+	it('collapses multiple separators', () => {
+		expect(bulkTitleFromFilename('foo__bar--baz.pdf')).toBe('foo bar baz');
+	});
+
+	it('strips extension', () => {
+		expect(bulkTitleFromFilename('document.pdf')).toBe('document');
+	});
+});
+
 describe('stripExtension', () => {
 	it('removes the extension', () => {
 		expect(stripExtension('document.pdf')).toBe('document');
diff --git a/frontend/src/lib/utils/filename.ts b/frontend/src/lib/utils/filename.ts
index 101fdd1c..582060aa 100644
--- a/frontend/src/lib/utils/filename.ts
+++ b/frontend/src/lib/utils/filename.ts
@@ -81,3 +81,7 @@ export function parseFilename(filename: string): FilenameParseResult {
 export function stripExtension(filename: string): string {
 	return filename.replace(/\.[^/.]+$/, '');
 }
+
+export function bulkTitleFromFilename(filename: string): string {
+	return stripExtension(filename).replace(/[_-]+/g, ' ').trim();
+}
-- 
2.49.1


From 22bba5cfcd30e973b9471263c3bbc2e5507b981b Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 16:40:04 +0200
Subject: [PATCH 04/46] feat(bulk-upload): add BulkDropZone component

Full-panel drop target that supports multi-file selection via drag-and-drop
or file picker. Fires onFilesAdded callback with the full File array.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../components/document/BulkDropZone.svelte   | 64 +++++++++++++++++++
 .../document/BulkDropZone.svelte.spec.ts      | 39 +++++++++++
 2 files changed, 103 insertions(+)
 create mode 100644 frontend/src/lib/components/document/BulkDropZone.svelte
 create mode 100644 frontend/src/lib/components/document/BulkDropZone.svelte.spec.ts

diff --git a/frontend/src/lib/components/document/BulkDropZone.svelte b/frontend/src/lib/components/document/BulkDropZone.svelte
new file mode 100644
index 00000000..065159e3
--- /dev/null
+++ b/frontend/src/lib/components/document/BulkDropZone.svelte
@@ -0,0 +1,64 @@
+<script lang="ts">
+let {
+	onFilesAdded
+}: {
+	onFilesAdded: (files: File[]) => void;
+} = $props();
+
+let isDragging = $state(false);
+</script>
+
+<div class="flex min-h-[300px] flex-1 flex-col items-center justify-center bg-pdf-bg">
+	<div
+		role="region"
+		aria-label="Dateien ablegen"
+		data-testid="bulk-drop-zone"
+		class="flex flex-col items-center gap-4 rounded-sm border border-dashed p-12 text-center transition-colors
+			{isDragging ? 'border-brand-mint ring-2 ring-brand-mint' : 'border-white/20'}"
+		ondragover={(e) => {
+			e.preventDefault();
+			isDragging = true;
+		}}
+		ondragleave={() => (isDragging = false)}
+		ondrop={(e) => {
+			e.preventDefault();
+			isDragging = false;
+			if (e.dataTransfer && e.dataTransfer.files.length > 0) {
+				onFilesAdded(Array.from(e.dataTransfer.files));
+			}
+		}}
+	>
+		<div class="flex h-8 w-8 items-center justify-center rounded-full bg-white/10 text-white/40">
+			<svg
+				width="16"
+				height="16"
+				viewBox="0 0 32 32"
+				xmlns="http://www.w3.org/2000/svg"
+				aria-hidden="true"
+			>
+				<polygon
+					fill="currentColor"
+					points="6 12.5 16 2 26 12.5 24.5714286 14 16.999 6.049 17 30 15 30 14.999 6.051 7.42857143 14"
+				/>
+			</svg>
+		</div>
+		<p class="text-sm font-medium text-white/60">PDF-Dateien hier ablegen</p>
+		<p class="text-xs text-white/30">oder</p>
+		<label
+			class="flex min-h-[44px] cursor-pointer items-center rounded-sm bg-brand-navy px-4 py-1.5 text-xs font-bold tracking-widest text-white/90 uppercase"
+			aria-label="Dateien auswählen"
+		>
+			Dateien auswählen
+			<input
+				type="file"
+				multiple
+				accept="application/pdf,image/jpeg,image/png,image/tiff"
+				class="sr-only"
+				onchange={(e) => {
+					const files = Array.from(e.currentTarget.files ?? []);
+					if (files.length > 0) onFilesAdded(files);
+				}}
+			/>
+		</label>
+	</div>
+</div>
diff --git a/frontend/src/lib/components/document/BulkDropZone.svelte.spec.ts b/frontend/src/lib/components/document/BulkDropZone.svelte.spec.ts
new file mode 100644
index 00000000..139933b6
--- /dev/null
+++ b/frontend/src/lib/components/document/BulkDropZone.svelte.spec.ts
@@ -0,0 +1,39 @@
+import { describe, it, expect, vi, afterEach } from 'vitest';
+import { cleanup, render } from 'vitest-browser-svelte';
+import { page, userEvent } from 'vitest/browser';
+import BulkDropZone from './BulkDropZone.svelte';
+
+afterEach(cleanup);
+
+describe('BulkDropZone', () => {
+	it('file input has multiple attribute', async () => {
+		const { container } = render(BulkDropZone, { onFilesAdded: vi.fn() });
+		const input = container.querySelector('input[type="file"]');
+		expect(input).not.toBeNull();
+		expect(input?.hasAttribute('multiple')).toBe(true);
+	});
+
+	it('fires onFilesAdded with selected files when 3 files are picked via input', async () => {
+		const onFilesAdded = vi.fn();
+		render(BulkDropZone, { onFilesAdded });
+
+		const files = [
+			new File(['a'], 'a.pdf', { type: 'application/pdf' }),
+			new File(['b'], 'b.pdf', { type: 'application/pdf' }),
+			new File(['c'], 'c.pdf', { type: 'application/pdf' })
+		];
+
+		const input = page.getByRole('button', { name: /Dateien auswählen/i });
+		await userEvent.upload(input, files);
+
+		expect(onFilesAdded).toHaveBeenCalledOnce();
+		const received: File[] = onFilesAdded.mock.calls[0][0];
+		expect(received).toHaveLength(3);
+		expect(received.map((f) => f.name)).toEqual(['a.pdf', 'b.pdf', 'c.pdf']);
+	});
+
+	it('shows drop hint text', async () => {
+		render(BulkDropZone, { onFilesAdded: vi.fn() });
+		await expect.element(page.getByText(/hier ablegen/i)).toBeInTheDocument();
+	});
+});
-- 
2.49.1


From 1f1b7aeab5a9af4383e472a6cb4b3f0a082dcc89 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 17:40:33 +0200
Subject: [PATCH 05/46] feat(bulk-upload): add FileSwitcherStrip component

Horizontal chip strip for switching between files in a bulk upload session.
Supports keyboard navigation (arrow keys cycle within the strip), error state
chips, and onSelect/onRemove callbacks.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../document/FileSwitcherStrip.svelte         | 91 +++++++++++++++++
 .../document/FileSwitcherStrip.svelte.spec.ts | 97 +++++++++++++++++++
 2 files changed, 188 insertions(+)
 create mode 100644 frontend/src/lib/components/document/FileSwitcherStrip.svelte
 create mode 100644 frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts

diff --git a/frontend/src/lib/components/document/FileSwitcherStrip.svelte b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
new file mode 100644
index 00000000..e7badb4d
--- /dev/null
+++ b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
@@ -0,0 +1,91 @@
+<script lang="ts">
+export interface FileEntry {
+	id: string;
+	file: File;
+	title: string;
+	status: 'idle' | 'error';
+}
+
+let {
+	files,
+	activeId,
+	onSelect,
+	onRemove
+}: {
+	files: FileEntry[];
+	activeId: string;
+	onSelect: (id: string) => void;
+	onRemove: (id: string) => void;
+} = $props();
+
+let ulEl = $state<HTMLUListElement | null>(null);
+
+$effect(() => {
+	if (!ulEl) return;
+	const node = ulEl;
+
+	function handleKeyDown(event: KeyboardEvent) {
+		const buttons = Array.from(node.querySelectorAll<HTMLElement>('[role="button"]'));
+		if (buttons.length === 0) return;
+
+		const focusedIndex = buttons.indexOf(document.activeElement as HTMLElement);
+		if (focusedIndex === -1) return;
+
+		if (event.key === 'ArrowRight' || event.key === 'ArrowDown') {
+			event.preventDefault();
+			const nextIndex = (focusedIndex + 1) % buttons.length;
+			buttons[nextIndex].focus();
+		} else if (event.key === 'ArrowLeft' || event.key === 'ArrowUp') {
+			event.preventDefault();
+			const prevIndex = (focusedIndex - 1 + buttons.length) % buttons.length;
+			buttons[prevIndex].focus();
+		}
+	}
+
+	node.addEventListener('keydown', handleKeyDown);
+	return () => {
+		node.removeEventListener('keydown', handleKeyDown);
+	};
+});
+</script>
+
+<ul
+	bind:this={ulEl}
+	data-testid="file-switcher-strip"
+	aria-live="polite"
+	role="list"
+	class="flex flex-row gap-2 overflow-x-auto px-1 py-2"
+>
+	{#each files as entry (entry.id)}
+		<li role="listitem" class="inline-flex items-center gap-0.5">
+			<button
+				type="button"
+				role="button"
+				tabindex="0"
+				aria-current={entry.id === activeId ? 'true' : undefined}
+				data-status={entry.status}
+				data-chip-id={entry.id}
+				onclick={() => onSelect(entry.id)}
+				class={[
+					'inline-flex cursor-pointer items-center gap-1 rounded-sm border px-3 py-1.5 text-xs font-medium transition-colors',
+					entry.id === activeId
+						? 'border-brand-mint bg-brand-mint/10 font-bold text-brand-navy'
+						: 'border-brand-sand bg-white text-brand-navy',
+					entry.status === 'error'
+						? 'border-dashed border-red-300 bg-red-50 text-red-700'
+						: ''
+				].join(' ')}
+				>{entry.title}</button
+			>
+			<button
+				type="button"
+				aria-label="Entfernen"
+				data-remove-id={entry.id}
+				onclick={() => onRemove(entry.id)}
+				class="ml-1 text-xs text-gray-400 hover:text-brand-navy"
+			>
+				×
+			</button>
+		</li>
+	{/each}
+</ul>
diff --git a/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts b/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts
new file mode 100644
index 00000000..86968e24
--- /dev/null
+++ b/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts
@@ -0,0 +1,97 @@
+import { describe, it, expect, vi, afterEach } from 'vitest';
+import { cleanup, render } from 'vitest-browser-svelte';
+import { page, userEvent } from 'vitest/browser';
+import FileSwitcherStrip from './FileSwitcherStrip.svelte';
+
+afterEach(cleanup);
+
+export interface FileEntry {
+	id: string;
+	file: File;
+	title: string;
+	status: 'idle' | 'error';
+}
+
+function makeFiles(n: number): FileEntry[] {
+	return Array.from({ length: n }, (_, i) => ({
+		id: `id-${i}`,
+		file: new File([''], `file${i}.pdf`),
+		title: `File ${i}`,
+		status: 'idle' as const
+	}));
+}
+
+describe('FileSwitcherStrip', () => {
+	it('renders N chips for N files', async () => {
+		const files = makeFiles(4);
+		render(FileSwitcherStrip, {
+			files,
+			activeId: files[0].id,
+			onSelect: vi.fn(),
+			onRemove: vi.fn()
+		});
+		const chips = page.getByRole('listitem');
+		await expect.element(chips.nth(0)).toBeInTheDocument();
+		await expect.element(chips.nth(3)).toBeInTheDocument();
+	});
+
+	it('active chip has aria-current="true"', async () => {
+		const files = makeFiles(3);
+		const { container } = render(FileSwitcherStrip, {
+			files,
+			activeId: files[1].id,
+			onSelect: vi.fn(),
+			onRemove: vi.fn()
+		});
+		const activeBtn = container.querySelector('[aria-current="true"]');
+		expect(activeBtn).not.toBeNull();
+		expect(activeBtn?.textContent).toContain('File 1');
+	});
+
+	it('clicking a chip fires onSelect with its id', async () => {
+		const files = makeFiles(3);
+		const onSelect = vi.fn();
+		const { container } = render(FileSwitcherStrip, {
+			files,
+			activeId: files[0].id,
+			onSelect,
+			onRemove: vi.fn()
+		});
+		const chip = container.querySelector('[data-chip-id="id-2"]') as HTMLElement;
+		expect(chip).not.toBeNull();
+		chip.click();
+		expect(onSelect).toHaveBeenCalledWith('id-2');
+	});
+
+	it('error chip has aria-label containing warning indicator', async () => {
+		const files: FileEntry[] = [
+			{ id: 'e1', file: new File([''], 'bad.pdf'), title: 'Bad file', status: 'error' }
+		];
+		const { container } = render(FileSwitcherStrip, {
+			files,
+			activeId: 'e1',
+			onSelect: vi.fn(),
+			onRemove: vi.fn()
+		});
+		const errBtn = container.querySelector('[data-status="error"]');
+		expect(errBtn).not.toBeNull();
+	});
+
+	it('ArrowRight moves focus to next chip without leaving strip', async () => {
+		const files = makeFiles(3);
+		const { container } = render(FileSwitcherStrip, {
+			files,
+			activeId: files[0].id,
+			onSelect: vi.fn(),
+			onRemove: vi.fn()
+		});
+		const firstBtn = container.querySelectorAll('[role="button"]')[0] as HTMLElement;
+		firstBtn.focus();
+		await userEvent.keyboard('{ArrowRight}');
+		const focused = document.activeElement;
+		expect(focused).not.toBe(firstBtn);
+		// The new focused element should still be inside the strip
+		const strip = container.querySelector('[data-testid="file-switcher-strip"]');
+		expect(strip?.contains(focused)).toBe(true);
+	});
+});
-- 
2.49.1


From 6d5fb9d8c87af7a6febcbd45600c5e663c637ebf Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 17:43:10 +0200
Subject: [PATCH 06/46] feat(bulk-upload): add ScopeCard component

Card container with two variants: per-file (mint tint) and shared (neutral
with file-count badge). Used to visually separate per-file vs shared
metadata sections in the bulk upload layout.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../lib/components/document/ScopeCard.svelte  | 35 +++++++++++++++++++
 .../document/ScopeCard.svelte.spec.ts         | 32 +++++++++++++++++
 2 files changed, 67 insertions(+)
 create mode 100644 frontend/src/lib/components/document/ScopeCard.svelte
 create mode 100644 frontend/src/lib/components/document/ScopeCard.svelte.spec.ts

diff --git a/frontend/src/lib/components/document/ScopeCard.svelte b/frontend/src/lib/components/document/ScopeCard.svelte
new file mode 100644
index 00000000..d0e7c5c7
--- /dev/null
+++ b/frontend/src/lib/components/document/ScopeCard.svelte
@@ -0,0 +1,35 @@
+<script lang="ts">
+let {
+	variant,
+	count = 0,
+	children
+}: {
+	variant: 'per-file' | 'shared';
+	count?: number;
+	children?: import('svelte').Snippet;
+} = $props();
+</script>
+
+<div
+	data-testid="scope-card"
+	class="rounded-sm border p-4
+		{variant === 'per-file'
+		? 'border-brand-mint bg-brand-mint/10'
+		: 'border-brand-sand bg-white'}"
+>
+	{#if variant === 'shared'}
+		<div class="mb-3 flex items-center justify-between">
+			<span class="text-xs font-bold tracking-widest text-gray-400 uppercase">
+				Gilt für alle Dateien
+			</span>
+			<span
+				class="bg-brand-sand inline-flex h-5 min-w-5 items-center justify-center rounded-full px-1.5 text-xs font-bold text-brand-navy"
+			>
+				{count}
+			</span>
+		</div>
+	{:else}
+		<p class="mb-3 text-xs font-bold tracking-widest text-brand-mint uppercase">Diese Datei</p>
+	{/if}
+	{@render children?.()}
+</div>
diff --git a/frontend/src/lib/components/document/ScopeCard.svelte.spec.ts b/frontend/src/lib/components/document/ScopeCard.svelte.spec.ts
new file mode 100644
index 00000000..51c4ae1b
--- /dev/null
+++ b/frontend/src/lib/components/document/ScopeCard.svelte.spec.ts
@@ -0,0 +1,32 @@
+import { describe, it, expect, afterEach } from 'vitest';
+import { cleanup, render } from 'vitest-browser-svelte';
+import { page } from 'vitest/browser';
+import ScopeCard from './ScopeCard.svelte';
+
+afterEach(cleanup);
+
+describe('ScopeCard', () => {
+	it('per-file variant has mint background class', async () => {
+		const { container } = render(ScopeCard, { variant: 'per-file', count: 1 });
+		const card = container.querySelector('[data-testid="scope-card"]');
+		expect(card?.className).toMatch(/brand-mint/);
+	});
+
+	it('shared variant does not have mint background', async () => {
+		const { container } = render(ScopeCard, { variant: 'shared', count: 3 });
+		const card = container.querySelector('[data-testid="scope-card"]');
+		expect(card?.className).not.toMatch(/bg-brand-mint/);
+	});
+
+	it('shared variant renders count badge with file count', async () => {
+		render(ScopeCard, { variant: 'shared', count: 5 });
+		await expect.element(page.getByText('5')).toBeInTheDocument();
+	});
+
+	it('per-file variant renders slot content', async () => {
+		// ScopeCard is a container — verify it renders children
+		render(ScopeCard, { variant: 'per-file', count: 1 });
+		const card = await page.getByTestId('scope-card');
+		await expect.element(card).toBeInTheDocument();
+	});
+});
-- 
2.49.1


From edd96b05fe5091128503b89f221a14e8933c7c43 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 17:49:23 +0200
Subject: [PATCH 07/46] feat(bulk-upload): add UploadSaveBar component + fix
 bulk_save_cta message

Save bar with sticky positioning, a determinate progress bar while
uploading chunks, plural save CTA, and a destructive discard link.
Replaces broken ICU plural in bulk_save_cta with two-key approach
(bulk_save_cta_one / bulk_save_cta) since Paraglide 2.5 does not support
ICU plural syntax.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/messages/de.json                     |  3 +-
 frontend/messages/en.json                     |  3 +-
 frontend/messages/es.json                     |  3 +-
 .../components/document/UploadSaveBar.svelte  | 39 +++++++++++++++
 .../document/UploadSaveBar.svelte.spec.ts     | 48 +++++++++++++++++++
 5 files changed, 93 insertions(+), 3 deletions(-)
 create mode 100644 frontend/src/lib/components/document/UploadSaveBar.svelte
 create mode 100644 frontend/src/lib/components/document/UploadSaveBar.svelte.spec.ts

diff --git a/frontend/messages/de.json b/frontend/messages/de.json
index 8bd62c50..479238c1 100644
--- a/frontend/messages/de.json
+++ b/frontend/messages/de.json
@@ -856,7 +856,8 @@
 	"bulk_drop_hint": "Eine oder mehrere Dateien ablegen",
 	"bulk_drop_sub": "PDF, JPEG, PNG oder TIFF · bis zu 50 MB pro Datei",
 	"bulk_count_pill": "{count} werden erstellt",
-	"bulk_save_cta": "{count, plural, one {Speichern →} other {{count} speichern →}}",
+	"bulk_save_cta_one": "Speichern →",
+	"bulk_save_cta": "{count} speichern →",
 	"bulk_discard_all": "Alle verwerfen",
 	"bulk_add_more": "Weitere hinzufügen",
 	"bulk_scope_per_file_label": "Nur diese Datei",
diff --git a/frontend/messages/en.json b/frontend/messages/en.json
index afc3ed73..78e107e4 100644
--- a/frontend/messages/en.json
+++ b/frontend/messages/en.json
@@ -856,7 +856,8 @@
 	"bulk_drop_hint": "Drop one or more files here",
 	"bulk_drop_sub": "PDF, JPEG, PNG or TIFF · up to 50 MB per file",
 	"bulk_count_pill": "{count} will be created",
-	"bulk_save_cta": "{count, plural, one {Save →} other {Save {count} →}}",
+	"bulk_save_cta_one": "Save →",
+	"bulk_save_cta": "Save {count} →",
 	"bulk_discard_all": "Discard all",
 	"bulk_add_more": "Add more",
 	"bulk_scope_per_file_label": "This file only",
diff --git a/frontend/messages/es.json b/frontend/messages/es.json
index cf1f27e4..a8ec1703 100644
--- a/frontend/messages/es.json
+++ b/frontend/messages/es.json
@@ -856,7 +856,8 @@
 	"bulk_drop_hint": "Suelta uno o varios archivos aquí",
 	"bulk_drop_sub": "PDF, JPEG, PNG o TIFF · hasta 50 MB por archivo",
 	"bulk_count_pill": "Se crearán {count}",
-	"bulk_save_cta": "{count, plural, one {Guardar →} other {Guardar {count} →}}",
+	"bulk_save_cta_one": "Guardar →",
+	"bulk_save_cta": "Guardar {count} →",
 	"bulk_discard_all": "Descartar todo",
 	"bulk_add_more": "Añadir más",
 	"bulk_scope_per_file_label": "Solo este archivo",
diff --git a/frontend/src/lib/components/document/UploadSaveBar.svelte b/frontend/src/lib/components/document/UploadSaveBar.svelte
new file mode 100644
index 00000000..c7e889ef
--- /dev/null
+++ b/frontend/src/lib/components/document/UploadSaveBar.svelte
@@ -0,0 +1,39 @@
+<script lang="ts">
+import * as m from '$lib/paraglide/messages';
+
+let {
+	fileCount,
+	chunkProgress,
+	onSave,
+	onDiscard
+}: {
+	fileCount: number;
+	chunkProgress?: { done: number; total: number };
+	onSave: () => void;
+	onDiscard: () => void;
+} = $props();
+</script>
+
+<div
+	class="border-brand-sand sticky bottom-0 z-10 -mx-4 border-t bg-white px-6 py-4 shadow-[0_-2px_8px_rgba(0,0,0,0.06)]"
+>
+	{#if chunkProgress}
+		<progress
+			value={chunkProgress.done}
+			max={chunkProgress.total}
+			class="[&::-webkit-progress-bar]:bg-brand-sand mb-3 h-1 w-full rounded-full [&::-webkit-progress-bar]:rounded-full [&::-webkit-progress-value]:rounded-full [&::-webkit-progress-value]:bg-brand-mint"
+		></progress>
+	{/if}
+	<div class="flex items-center justify-between">
+		<button type="button" onclick={onDiscard} class="text-sm text-red-600/70 hover:text-red-700">
+			{m.bulk_discard_all()}
+		</button>
+		<button
+			type="button"
+			onclick={onSave}
+			class="min-h-[44px] rounded-sm bg-brand-navy px-6 text-sm font-bold tracking-widest text-white uppercase hover:bg-brand-navy/90"
+		>
+			{fileCount === 1 ? 'Speichern →' : `${fileCount} speichern →`}
+		</button>
+	</div>
+</div>
diff --git a/frontend/src/lib/components/document/UploadSaveBar.svelte.spec.ts b/frontend/src/lib/components/document/UploadSaveBar.svelte.spec.ts
new file mode 100644
index 00000000..4516a831
--- /dev/null
+++ b/frontend/src/lib/components/document/UploadSaveBar.svelte.spec.ts
@@ -0,0 +1,48 @@
+import { describe, it, expect, vi, afterEach } from 'vitest';
+import { cleanup, render } from 'vitest-browser-svelte';
+import { page } from 'vitest/browser';
+import UploadSaveBar from './UploadSaveBar.svelte';
+
+afterEach(cleanup);
+
+describe('UploadSaveBar', () => {
+	it('shows plural label for multiple files', async () => {
+		render(UploadSaveBar, { fileCount: 5, onSave: vi.fn(), onDiscard: vi.fn() });
+		// "5 speichern →" or similar plural form
+		await expect.element(page.getByText(/5/)).toBeInTheDocument();
+	});
+
+	it('shows singular label for one file', async () => {
+		render(UploadSaveBar, { fileCount: 1, onSave: vi.fn(), onDiscard: vi.fn() });
+		// "Speichern →" singular form
+		await expect.element(page.getByText(/Speichern/i)).toBeInTheDocument();
+	});
+
+	it('progress bar is visible when chunkProgress is provided', async () => {
+		const { container } = render(UploadSaveBar, {
+			fileCount: 3,
+			chunkProgress: { done: 1, total: 3 },
+			onSave: vi.fn(),
+			onDiscard: vi.fn()
+		});
+		const progress = container.querySelector('progress');
+		expect(progress).not.toBeNull();
+		expect(progress?.getAttribute('value')).toBe('1');
+		expect(progress?.getAttribute('max')).toBe('3');
+	});
+
+	it('progress bar is not rendered when no chunkProgress', async () => {
+		const { container } = render(UploadSaveBar, {
+			fileCount: 2,
+			onSave: vi.fn(),
+			onDiscard: vi.fn()
+		});
+		const progress = container.querySelector('progress');
+		expect(progress).toBeNull();
+	});
+
+	it('discard link is rendered', async () => {
+		render(UploadSaveBar, { fileCount: 2, onSave: vi.fn(), onDiscard: vi.fn() });
+		await expect.element(page.getByText(/verwerfen/i)).toBeInTheDocument();
+	});
+});
-- 
2.49.1


From 3a6a70a1f77372ca49d569d45e25383cb16a2e64 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 17:57:33 +0200
Subject: [PATCH 08/46] feat(bulk-upload): add BulkDocumentEditLayout component
 with save handler
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

State-owner for the bulk upload flow:
- N=0: full-panel BulkDropZone
- N=1: title + shared metadata (no switcher/scope cards)
- N≥2: FileSwitcherStrip + per-file ScopeCard + shared ScopeCard
Save handler chunks files at 10/request, POSTs to /api/documents/quick-upload
with typed metadata JSON part, tracks progress, redirects to /documents.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../document/BulkDocumentEditLayout.svelte    | 190 ++++++++++++++++++
 .../BulkDocumentEditLayout.svelte.spec.ts     |  99 +++++++++
 .../lib/components/document/ScopeCard.svelte  |   1 +
 .../components/document/UploadSaveBar.svelte  |   1 +
 4 files changed, 291 insertions(+)
 create mode 100644 frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
 create mode 100644 frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts

diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
new file mode 100644
index 00000000..8a280226
--- /dev/null
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
@@ -0,0 +1,190 @@
+<script lang="ts">
+import { SvelteMap } from 'svelte/reactivity';
+import { goto } from '$app/navigation';
+import { untrack } from 'svelte';
+import BulkDropZone from './BulkDropZone.svelte';
+import FileSwitcherStrip from './FileSwitcherStrip.svelte';
+import type { FileEntry } from './FileSwitcherStrip.svelte';
+import ScopeCard from './ScopeCard.svelte';
+import UploadSaveBar from './UploadSaveBar.svelte';
+import WhoWhenSection from './WhoWhenSection.svelte';
+import DescriptionSection from './DescriptionSection.svelte';
+import { bulkTitleFromFilename } from '$lib/utils/filename';
+import type { Tag } from '$lib/components/TagInput.svelte';
+import type { components } from '$lib/generated/api';
+
+type Person = components['schemas']['Person'];
+
+let {
+	initialSenderId = '',
+	initialSenderName = '',
+	initialReceivers = []
+}: {
+	initialSenderId?: string;
+	initialSenderName?: string;
+	initialReceivers?: Person[];
+} = $props();
+
+// --- File state ---
+let files = new SvelteMap<string, FileEntry>();
+let activeId = $state<string | null>(null);
+let chunkProgress = $state<{ done: number; total: number } | undefined>(undefined);
+
+// --- Shared metadata ---
+let senderId = $state(untrack(() => initialSenderId));
+let selectedReceivers = $state<Person[]>(untrack(() => initialReceivers));
+let dateIso = $state('');
+let tags = $state<Tag[]>([]);
+
+// --- Derived ---
+const isMulti = $derived(files.size >= 2);
+const activeFile = $derived(activeId ? files.get(activeId) : null);
+
+// --- Handlers ---
+function addFiles(newFiles: File[]) {
+	for (const file of newFiles) {
+		const id = crypto.randomUUID();
+		const title = bulkTitleFromFilename(file.name);
+		files.set(id, { id, file, title, status: 'idle' });
+		if (!activeId) activeId = id;
+	}
+}
+
+function removeFile(id: string) {
+	files.delete(id);
+	if (activeId === id) {
+		activeId = files.keys().next().value ?? null;
+	}
+}
+
+function setTitle(id: string, title: string) {
+	const entry = files.get(id);
+	if (entry) files.set(id, { ...entry, title });
+}
+
+async function save() {
+	const entries = Array.from(files.values());
+	const chunkSize = 10;
+	const chunks: FileEntry[][] = [];
+	for (let i = 0; i < entries.length; i += chunkSize) {
+		chunks.push(entries.slice(i, i + chunkSize));
+	}
+	chunkProgress = { done: 0, total: chunks.length };
+
+	for (let i = 0; i < chunks.length; i++) {
+		const chunk = chunks[i];
+		const formData = new FormData();
+		chunk.forEach((entry) => formData.append('files', entry.file));
+		const metadata = {
+			titles: chunk.map((e) => e.title),
+			senderId: senderId || null,
+			receiverIds: selectedReceivers.map((r) => r.id),
+			documentDate: dateIso || null
+		};
+		formData.append('metadata', new Blob([JSON.stringify(metadata)], { type: 'application/json' }));
+		await fetch('/api/documents/quick-upload', { method: 'POST', body: formData });
+		chunkProgress = { done: i + 1, total: chunks.length };
+	}
+	goto('/documents');
+}
+</script>
+
+{#if files.size === 0}
+	<!-- N=0: full-panel drop zone -->
+	<div class="flex min-h-[400px] flex-col">
+		<BulkDropZone onFilesAdded={addFiles} />
+	</div>
+{:else}
+	<div class="mx-auto flex max-w-7xl flex-col gap-0 px-4 py-6">
+		{#if isMulti}
+			<FileSwitcherStrip
+				files={Array.from(files.values())}
+				activeId={activeId ?? ''}
+				onSelect={(id) => (activeId = id)}
+				onRemove={removeFile}
+			/>
+		{/if}
+
+		<div class="flex gap-6">
+			<!-- Left: PDF preview -->
+			<div class="hidden w-1/2 lg:block">
+				{#if activeFile}
+					<div
+						class="flex h-[600px] flex-col items-center justify-center rounded-sm bg-gray-900 text-white/40"
+					>
+						<span class="text-sm">{activeFile.file.name}</span>
+						<span class="mt-1 text-xs">Vorschau nach Upload verfügbar</span>
+					</div>
+				{/if}
+			</div>
+
+			<!-- Right: metadata -->
+			<div class="flex flex-1 flex-col gap-4">
+				{#if isMulti}
+					<!-- Per-file scope card: title -->
+					<ScopeCard variant="per-file" data-variant="per-file">
+						{#if activeFile}
+							<label class="block">
+								<span class="mb-1 block text-xs font-medium text-brand-navy/70">Titel</span>
+								<input
+									type="text"
+									value={activeFile.title}
+									oninput={(e) =>
+										setTitle(activeId!, (e.currentTarget as HTMLInputElement).value)}
+									class="border-brand-sand block w-full rounded-sm border p-2 text-sm focus:border-brand-mint focus:outline-none"
+								/>
+							</label>
+						{/if}
+					</ScopeCard>
+
+					<!-- Shared scope card: metadata -->
+					<ScopeCard variant="shared" count={files.size} data-variant="shared">
+						<WhoWhenSection
+							bind:senderId={senderId}
+							bind:selectedReceivers={selectedReceivers}
+							bind:dateIso={dateIso}
+							initialSenderName={initialSenderName}
+						/>
+						<DescriptionSection bind:tags={tags} hideTitle />
+					</ScopeCard>
+				{:else}
+					<!-- N=1: no scope cards, just the fields directly -->
+					{#if activeFile}
+						<div class="border-brand-sand rounded-sm border bg-white p-4 shadow-sm">
+							<label class="block">
+								<span class="mb-1 block text-xs font-medium text-brand-navy/70">Titel</span>
+								<input
+									type="text"
+									value={activeFile.title}
+									oninput={(e) =>
+										setTitle(activeId!, (e.currentTarget as HTMLInputElement).value)}
+									class="border-brand-sand block w-full rounded-sm border p-2 text-sm focus:border-brand-mint focus:outline-none"
+								/>
+							</label>
+						</div>
+					{/if}
+					<div class="border-brand-sand rounded-sm border bg-white p-4 shadow-sm">
+						<WhoWhenSection
+							bind:senderId={senderId}
+							bind:selectedReceivers={selectedReceivers}
+							bind:dateIso={dateIso}
+							initialSenderName={initialSenderName}
+						/>
+						<DescriptionSection bind:tags={tags} hideTitle />
+					</div>
+				{/if}
+			</div>
+		</div>
+
+		<UploadSaveBar
+			fileCount={files.size}
+			chunkProgress={chunkProgress}
+			onSave={save}
+			onDiscard={() => {
+				files.clear();
+				activeId = null;
+				chunkProgress = undefined;
+			}}
+		/>
+	</div>
+{/if}
diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
new file mode 100644
index 00000000..e063094d
--- /dev/null
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
@@ -0,0 +1,99 @@
+import { describe, it, expect, vi, afterEach } from 'vitest';
+import { cleanup, render } from 'vitest-browser-svelte';
+import { page, userEvent } from 'vitest/browser';
+import BulkDocumentEditLayout from './BulkDocumentEditLayout.svelte';
+
+afterEach(() => {
+	cleanup();
+	vi.unstubAllGlobals();
+});
+
+function makeFile(name: string): File {
+	return new File(['content'], name, { type: 'application/pdf' });
+}
+
+async function addFilesViaInput(container: HTMLElement, files: File[]): Promise<void> {
+	const input = container.querySelector('input[type="file"]') as HTMLInputElement;
+	if (!input) throw new Error('No file input found — is BulkDropZone visible?');
+	await userEvent.upload(input, files);
+}
+
+describe('BulkDocumentEditLayout', () => {
+	it('N=0: shows BulkDropZone', async () => {
+		render(BulkDocumentEditLayout, {});
+		await expect.element(page.getByTestId('bulk-drop-zone')).toBeInTheDocument();
+	});
+
+	it('N=1: file-switcher-strip and per-file scope card are absent', async () => {
+		const { container } = render(BulkDocumentEditLayout, {});
+		await addFilesViaInput(container, [makeFile('doc.pdf')]);
+		expect(container.querySelector('[data-testid="file-switcher-strip"]')).toBeNull();
+		expect(container.querySelector('[data-variant="per-file"]')).toBeNull();
+	});
+
+	it('N=5: file-switcher-strip and per-file scope card are both present', async () => {
+		const { container } = render(BulkDocumentEditLayout, {});
+		await addFilesViaInput(container, [
+			makeFile('a.pdf'),
+			makeFile('b.pdf'),
+			makeFile('c.pdf'),
+			makeFile('d.pdf'),
+			makeFile('e.pdf')
+		]);
+		expect(container.querySelector('[data-testid="file-switcher-strip"]')).not.toBeNull();
+		expect(container.querySelector('[data-variant="per-file"]')).not.toBeNull();
+	});
+
+	it('removing middle file preserves order of remaining files', async () => {
+		const { container } = render(BulkDocumentEditLayout, {});
+		await addFilesViaInput(container, [
+			makeFile('file0.pdf'),
+			makeFile('file1.pdf'),
+			makeFile('file2.pdf')
+		]);
+
+		// Remove the chip for file1 via its "Entfernen" remove button (second × button)
+		const removeButtons = container.querySelectorAll<HTMLButtonElement>(
+			'[data-testid="file-switcher-strip"] button[aria-label="Entfernen"]'
+		);
+		expect(removeButtons.length).toBe(3);
+		removeButtons[1].click(); // remove file1
+
+		// Wait for Svelte to flush the DOM update
+		await vi.waitFor(
+			() => {
+				const chips = container.querySelectorAll(
+					'[data-testid="file-switcher-strip"] [data-chip-id]'
+				);
+				expect(chips.length).toBe(2);
+				expect(chips[0].textContent?.trim()).toContain('file0');
+				expect(chips[1].textContent?.trim()).toContain('file2');
+			},
+			{ timeout: 1000 }
+		);
+	});
+
+	it('save calls fetch twice for 12 files (2 chunks of 10)', async () => {
+		const mockFetch = vi.fn().mockResolvedValue({
+			ok: true,
+			json: async () => ({ created: [], updated: [], errors: [] })
+		});
+		vi.stubGlobal('fetch', mockFetch);
+
+		// Also stub goto to prevent navigation errors in test
+		vi.mock('$app/navigation', () => ({ goto: vi.fn() }));
+
+		const { container } = render(BulkDocumentEditLayout, {});
+		const files = Array.from({ length: 12 }, (_, i) => makeFile(`f${i}.pdf`));
+		await addFilesViaInput(container, files);
+
+		const saveBtn = container.querySelector(
+			'button[data-testid="bulk-save-btn"]'
+		) as HTMLButtonElement;
+		expect(saveBtn).not.toBeNull();
+		saveBtn.click();
+
+		// Wait for async save to complete
+		await vi.waitFor(() => expect(mockFetch).toHaveBeenCalledTimes(2), { timeout: 3000 });
+	});
+});
diff --git a/frontend/src/lib/components/document/ScopeCard.svelte b/frontend/src/lib/components/document/ScopeCard.svelte
index d0e7c5c7..ebdbe82a 100644
--- a/frontend/src/lib/components/document/ScopeCard.svelte
+++ b/frontend/src/lib/components/document/ScopeCard.svelte
@@ -12,6 +12,7 @@ let {
 
 <div
 	data-testid="scope-card"
+	data-variant={variant}
 	class="rounded-sm border p-4
 		{variant === 'per-file'
 		? 'border-brand-mint bg-brand-mint/10'
diff --git a/frontend/src/lib/components/document/UploadSaveBar.svelte b/frontend/src/lib/components/document/UploadSaveBar.svelte
index c7e889ef..f8cad1bf 100644
--- a/frontend/src/lib/components/document/UploadSaveBar.svelte
+++ b/frontend/src/lib/components/document/UploadSaveBar.svelte
@@ -30,6 +30,7 @@ let {
 		</button>
 		<button
 			type="button"
+			data-testid="bulk-save-btn"
 			onclick={onSave}
 			class="min-h-[44px] rounded-sm bg-brand-navy px-6 text-sm font-bold tracking-widest text-white uppercase hover:bg-brand-navy/90"
 		>
-- 
2.49.1


From a1f9253712ad1d1dbd0acce317fba9338fe80d54 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 17:59:41 +0200
Subject: [PATCH 09/46] feat(bulk-upload): wire /documents/new to
 BulkDocumentEditLayout

Replaces the single-file form-action flow with BulkDocumentEditLayout,
enabling multi-file drag-and-drop upload with local preview, per-file
title editing, and shared metadata. Server load function unchanged.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../src/routes/documents/new/+page.svelte     | 133 ++----------------
 1 file changed, 8 insertions(+), 125 deletions(-)

diff --git a/frontend/src/routes/documents/new/+page.svelte b/frontend/src/routes/documents/new/+page.svelte
index 36da0ec3..753233b1 100644
--- a/frontend/src/routes/documents/new/+page.svelte
+++ b/frontend/src/routes/documents/new/+page.svelte
@@ -1,49 +1,11 @@
 <script lang="ts">
-import { enhance } from '$app/forms';
-import { untrack } from 'svelte';
 import { m } from '$lib/paraglide/messages.js';
-import WhoWhenSection from '$lib/components/document/WhoWhenSection.svelte';
-import DescriptionSection from '$lib/components/document/DescriptionSection.svelte';
-import TranscriptionSection from '$lib/components/document/TranscriptionSection.svelte';
-import FileSectionNew from './FileSectionNew.svelte';
-import { type FilenameParseResult } from '$lib/utils/filename';
+import BulkDocumentEditLayout from '$lib/components/document/BulkDocumentEditLayout.svelte';
 
-let { data, form } = $props();
-
-let tags: { name: string; id?: string; color?: string; parentId?: string }[] = $state([]);
-let senderId = $state(untrack(() => data.initialSenderId));
-let selectedReceivers: { id: string; firstName?: string; lastName: string; displayName: string }[] =
-	$state(untrack(() => data.initialReceivers));
-
-let parsedSuggestion = $state<FilenameParseResult>({});
-
-// Title is derived from the filename suggestion unless the user has typed something
-let titleDirty = $state(false);
-let titleOverride = $state('');
-let titleValue = $derived(
-	titleDirty ? titleOverride : (parsedSuggestion.suggestedTitle ?? titleOverride)
-);
-
-// Details panel: starts open when prefill data is present or a form error occurred.
-// Auto-opens when filename parsing finds a date/sender, but never force-closes — user
-// can always collapse the section manually.
-let detailsOpen = $state(
-	!!(
-		untrack(() => data.initialSenderId) ||
-		untrack(() => data.initialReceivers).length > 0 ||
-		untrack(() => form)?.error
-	)
-);
-
-$effect(() => {
-	if (parsedSuggestion.dateIso || senderId || selectedReceivers.length > 0) {
-		detailsOpen = true;
-	}
-});
+let { data } = $props();
 </script>
 
-<div class="mx-auto max-w-4xl px-4 py-8">
-	<!-- Heading -->
+<div class="mx-auto max-w-7xl px-4 py-8">
 	<div class="mb-6">
 		<a
 			href="/"
@@ -67,88 +29,9 @@ $effect(() => {
 		<h1 class="font-serif text-3xl text-ink">{m.doc_new_heading()}</h1>
 	</div>
 
-	{#if form?.error}
-		<div class="mb-6 rounded border border-red-200 bg-red-50 p-4 text-red-700">{form.error}</div>
-	{/if}
-
-	<form method="POST" enctype="multipart/form-data" use:enhance class="space-y-6 pb-20">
-		<!-- File upload — prominent, at the top -->
-		<FileSectionNew onfileParsed={(r) => (parsedSuggestion = r)} />
-
-		<!-- Standalone title card -->
-		<div class="rounded-sm border border-line bg-surface p-6 shadow-sm">
-			<label for="new-title" class="mb-1 block text-sm font-medium text-ink-2"
-				>{m.form_label_title()}</label
-			>
-			<input
-				id="new-title"
-				type="text"
-				name="title"
-				value={titleValue}
-				oninput={(e) => {
-					titleOverride = (e.target as HTMLInputElement).value;
-					titleDirty = true;
-				}}
-				class="block w-full rounded border border-line p-2 text-sm shadow-sm focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring"
-				placeholder="Titel eingeben…"
-			/>
-		</div>
-
-		<!-- Collapsible further details -->
-		<details
-			bind:open={detailsOpen}
-			class="group rounded-sm border border-line bg-surface shadow-sm"
-		>
-			<summary class="cursor-pointer list-none px-6 py-4">
-				<span class="text-xs font-bold tracking-widest text-ink-3 uppercase"
-					>{m.doc_more_details()}</span
-				>
-			</summary>
-			<div class="space-y-6 px-0 pb-6">
-				<WhoWhenSection
-					bind:senderId={senderId}
-					bind:selectedReceivers={selectedReceivers}
-					initialSenderName={data.initialSenderName}
-					suggestedDateIso={parsedSuggestion.dateIso ?? ''}
-					suggestedSenderName={parsedSuggestion.personName ?? ''}
-				/>
-				<DescriptionSection bind:tags={tags} hideTitle={true} />
-				<TranscriptionSection />
-			</div>
-		</details>
-
-		<!-- Sticky Save Bar -->
-		<div
-			class="sticky bottom-0 z-10 -mx-4 border-t border-line bg-surface px-4 py-3 shadow-[0_-2px_8px_rgba(0,0,0,0.06)] sm:px-6 sm:py-4"
-		>
-			<div class="flex flex-col gap-2 sm:flex-row sm:items-center sm:justify-between">
-				<a
-					href="/"
-					class="order-last text-center text-sm font-medium text-ink-2 transition-colors hover:text-ink sm:order-first sm:text-left"
-				>
-					{m.btn_cancel()}
-				</a>
-				<div class="flex flex-col gap-2 sm:flex-row sm:items-center sm:gap-3">
-					<button
-						type="submit"
-						name="metadataComplete"
-						value="false"
-						formaction="?/save"
-						class="w-full rounded-sm border border-line px-5 py-2.5 font-sans text-xs font-bold tracking-widest text-ink-2 uppercase transition-colors hover:bg-muted sm:w-auto sm:py-2"
-					>
-						{m.btn_save()}
-					</button>
-					<button
-						type="submit"
-						name="metadataComplete"
-						value="true"
-						formaction="?/saveReviewed"
-						class="w-full rounded-sm bg-primary px-5 py-2.5 font-sans text-xs font-bold tracking-widest text-primary-fg uppercase transition-colors hover:bg-primary/90 sm:w-auto sm:py-2"
-					>
-						{m.btn_save_and_mark_reviewed()}
-					</button>
-				</div>
-			</div>
-		</div>
-	</form>
+	<BulkDocumentEditLayout
+		initialSenderId={data.initialSenderId}
+		initialSenderName={data.initialSenderName}
+		initialReceivers={data.initialReceivers}
+	/>
 </div>
-- 
2.49.1


From 64dbce2a001521cb4fcd55ef2382010e8fe50cdb Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 18:24:16 +0200
Subject: [PATCH 10/46] =?UTF-8?q?chore(api):=20regenerate=20types=20?=
 =?UTF-8?q?=E2=80=94=20adds=20DocumentBatchMetadataDTO?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

New type from the bulk-upload metadata part added in #317.
Generated from backend running with --spring.profiles.active=dev.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/src/lib/generated/api.ts | 153 +++++++++++++++++++++++++++---
 1 file changed, 142 insertions(+), 11 deletions(-)

diff --git a/frontend/src/lib/generated/api.ts b/frontend/src/lib/generated/api.ts
index aafb9c78..2db578c9 100644
--- a/frontend/src/lib/generated/api.ts
+++ b/frontend/src/lib/generated/api.ts
@@ -548,6 +548,22 @@ export interface paths {
         patch?: never;
         trace?: never;
     };
+    "/api/admin/generate-thumbnails": {
+        parameters: {
+            query?: never;
+            header?: never;
+            path?: never;
+            cookie?: never;
+        };
+        get?: never;
+        put?: never;
+        post: operations["generateThumbnails"];
+        delete?: never;
+        options?: never;
+        head?: never;
+        patch?: never;
+        trace?: never;
+    };
     "/api/admin/backfill-versions": {
         parameters: {
             query?: never;
@@ -1028,6 +1044,22 @@ export interface paths {
         patch?: never;
         trace?: never;
     };
+    "/api/documents/{id}/thumbnail": {
+        parameters: {
+            query?: never;
+            header?: never;
+            path?: never;
+            cookie?: never;
+        };
+        get: operations["getDocumentThumbnail"];
+        put?: never;
+        post?: never;
+        delete?: never;
+        options?: never;
+        head?: never;
+        patch?: never;
+        trace?: never;
+    };
     "/api/documents/{documentId}/transcription-blocks/{blockId}/history": {
         parameters: {
             query?: never;
@@ -1204,6 +1236,22 @@ export interface paths {
         patch?: never;
         trace?: never;
     };
+    "/api/admin/thumbnail-status": {
+        parameters: {
+            query?: never;
+            header?: never;
+            path?: never;
+            cookie?: never;
+        };
+        get: operations["thumbnailStatus"];
+        put?: never;
+        post?: never;
+        delete?: never;
+        options?: never;
+        head?: never;
+        patch?: never;
+        trace?: never;
+    };
     "/api/admin/import-status": {
         parameters: {
             query?: never;
@@ -1390,7 +1438,6 @@ export interface components {
             thumbnailAspect?: "PORTRAIT" | "LANDSCAPE";
             /** Format: int32 */
             pageCount?: number;
-            thumbnailUrl?: string;
             originalFilename: string;
             /** @enum {string} */
             status: "PLACEHOLDER" | "UPLOADED" | "TRANSCRIBED" | "REVIEWED" | "ARCHIVED";
@@ -1413,6 +1460,7 @@ export interface components {
             sender?: components["schemas"]["Person"];
             tags?: components["schemas"]["Tag"][];
             trainingLabels?: ("KURRENT_RECOGNITION" | "KURRENT_SEGMENTATION")[];
+            thumbnailUrl?: string;
         };
         UpdateTranscriptionBlockDTO: {
             text?: string;
@@ -1639,6 +1687,17 @@ export interface components {
             /** Format: date-time */
             createdAt: string;
         };
+        DocumentBatchMetadataDTO: {
+            titles?: string[];
+            /** Format: uuid */
+            senderId?: string;
+            receiverIds?: string[];
+            /** Format: date */
+            documentDate?: string;
+            location?: string;
+            tags?: string;
+            metadataComplete?: boolean;
+        };
         QuickUploadResult: {
             created?: components["schemas"]["Document"][];
             updated?: components["schemas"]["Document"][];
@@ -1673,6 +1732,21 @@ export interface components {
             /** Format: date-time */
             startedAt?: string;
         };
+        BackfillStatus: {
+            /** @enum {string} */
+            state?: "IDLE" | "RUNNING" | "DONE" | "FAILED";
+            message?: string;
+            /** Format: int32 */
+            total?: number;
+            /** Format: int32 */
+            processed?: number;
+            /** Format: int32 */
+            skipped?: number;
+            /** Format: int32 */
+            failed?: number;
+            /** Format: date-time */
+            startedAt?: string;
+        };
         BackfillResult: {
             /** Format: int32 */
             count: number;
@@ -1837,10 +1911,10 @@ export interface components {
             timeout?: number;
         };
         PageNotificationDTO: {
-            /** Format: int32 */
-            totalPages?: number;
             /** Format: int64 */
             totalElements?: number;
+            /** Format: int32 */
+            totalPages?: number;
             pageable?: components["schemas"]["PageableObject"];
             first?: boolean;
             last?: boolean;
@@ -3152,6 +3226,7 @@ export interface operations {
             content: {
                 "multipart/form-data": {
                     files?: string[];
+                    metadata?: components["schemas"]["DocumentBatchMetadataDTO"];
                 };
             };
         };
@@ -3255,6 +3330,26 @@ export interface operations {
             };
         };
     };
+    generateThumbnails: {
+        parameters: {
+            query?: never;
+            header?: never;
+            path?: never;
+            cookie?: never;
+        };
+        requestBody?: never;
+        responses: {
+            /** @description OK */
+            200: {
+                headers: {
+                    [name: string]: unknown;
+                };
+                content: {
+                    "*/*": components["schemas"]["BackfillStatus"];
+                };
+            };
+        };
+    };
     backfillVersions: {
         parameters: {
             query?: never;
@@ -3975,6 +4070,28 @@ export interface operations {
             };
         };
     };
+    getDocumentThumbnail: {
+        parameters: {
+            query?: never;
+            header?: never;
+            path: {
+                id: string;
+            };
+            cookie?: never;
+        };
+        requestBody?: never;
+        responses: {
+            /** @description OK */
+            200: {
+                headers: {
+                    [name: string]: unknown;
+                };
+                content: {
+                    "*/*": string;
+                };
+            };
+        };
+    };
     getBlockHistory: {
         parameters: {
             query?: never;
@@ -4038,15 +4155,9 @@ export interface operations {
                 dir?: string;
                 /** @description Tag operator: AND (default) or OR */
                 tagOp?: string;
-                /**
-                 * @description Page number (0-indexed)
-                 * @default 0
-                 */
+                /** @description Page number (0-indexed) */
                 page?: number;
-                /**
-                 * @description Page size (max 100)
-                 * @default 50
-                 */
+                /** @description Page size (max 100) */
                 size?: number;
             };
             header?: never;
@@ -4245,6 +4356,26 @@ export interface operations {
             };
         };
     };
+    thumbnailStatus: {
+        parameters: {
+            query?: never;
+            header?: never;
+            path?: never;
+            cookie?: never;
+        };
+        requestBody?: never;
+        responses: {
+            /** @description OK */
+            200: {
+                headers: {
+                    [name: string]: unknown;
+                };
+                content: {
+                    "*/*": components["schemas"]["BackfillStatus"];
+                };
+            };
+        };
+    };
     importStatus: {
         parameters: {
             query?: never;
-- 
2.49.1


From 0e6efc91702d966ed5bc4b6b87015454f28309ba Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 19:17:36 +0200
Subject: [PATCH 11/46] fix(bulk-upload): spec-compliant split-panel layout
 with local PDF preview
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Rewrites BulkDocumentEditLayout to match the spec exactly:
- Fixed viewport layout (same as DocumentEditLayout) filling viewport below nav
- Split panel visible in all states (N=0/1/≥2) — was fullscreen dark drop zone
- N=0: centered drop-zone-box in left panel; shared form visible but greyed out
- N≥1: real PDF preview via URL.createObjectURL (no server upload required)
- N≥2: FileSwitcherStrip at bottom of left panel; count pill + discard in topbar
- FileEntry gains previewUrl; blob URLs created on add, revoked on remove/destroy
- save() checks response.ok and marks failed files with status: 'error'
- BulkDropZone redesigned: spec-accurate box with circular mint icon, serif title
- FileSwitcherStrip: number badges, arrows, keyboard nav via data-chip-id selector
- ScopeCard, UploadSaveBar: hardcoded German replaced with Paraglide i18n keys
- +page.svelte simplified to bare component render (layout is self-contained)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../document/BulkDocumentEditLayout.svelte    | 209 +++++++++++++-----
 .../components/document/BulkDropZone.svelte   |  69 +++---
 .../document/FileSwitcherStrip.svelte         | 123 +++++++----
 .../document/FileSwitcherStrip.svelte.spec.ts |  14 +-
 .../lib/components/document/ScopeCard.svelte  |  18 +-
 .../components/document/UploadSaveBar.svelte  |  15 +-
 .../src/routes/documents/new/+page.svelte     |  36 +--
 .../routes/documents/new/page.svelte.spec.ts  |  14 +-
 8 files changed, 311 insertions(+), 187 deletions(-)

diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
index 8a280226..2c9b8511 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
@@ -1,7 +1,8 @@
 <script lang="ts">
 import { SvelteMap } from 'svelte/reactivity';
 import { goto } from '$app/navigation';
-import { untrack } from 'svelte';
+import { onMount, onDestroy, untrack } from 'svelte';
+import { m } from '$lib/paraglide/messages.js';
 import BulkDropZone from './BulkDropZone.svelte';
 import FileSwitcherStrip from './FileSwitcherStrip.svelte';
 import type { FileEntry } from './FileSwitcherStrip.svelte';
@@ -9,6 +10,7 @@ import ScopeCard from './ScopeCard.svelte';
 import UploadSaveBar from './UploadSaveBar.svelte';
 import WhoWhenSection from './WhoWhenSection.svelte';
 import DescriptionSection from './DescriptionSection.svelte';
+import PdfViewer from '$lib/components/PdfViewer.svelte';
 import { bulkTitleFromFilename } from '$lib/utils/filename';
 import type { Tag } from '$lib/components/TagInput.svelte';
 import type { components } from '$lib/generated/api';
@@ -25,6 +27,12 @@ let {
 	initialReceivers?: Person[];
 } = $props();
 
+// --- Layout ---
+let navHeight = $state(0);
+onMount(() => {
+	navHeight = document.querySelector('header')?.getBoundingClientRect().height ?? 0;
+});
+
 // --- File state ---
 let files = new SvelteMap<string, FileEntry>();
 let activeId = $state<string | null>(null);
@@ -40,17 +48,20 @@ let tags = $state<Tag[]>([]);
 const isMulti = $derived(files.size >= 2);
 const activeFile = $derived(activeId ? files.get(activeId) : null);
 
-// --- Handlers ---
+// --- File management ---
 function addFiles(newFiles: File[]) {
 	for (const file of newFiles) {
 		const id = crypto.randomUUID();
 		const title = bulkTitleFromFilename(file.name);
-		files.set(id, { id, file, title, status: 'idle' });
+		const previewUrl = URL.createObjectURL(file);
+		files.set(id, { id, file, title, status: 'idle', previewUrl });
 		if (!activeId) activeId = id;
 	}
 }
 
 function removeFile(id: string) {
+	const entry = files.get(id);
+	if (entry?.previewUrl) URL.revokeObjectURL(entry.previewUrl);
 	files.delete(id);
 	if (activeId === id) {
 		activeId = files.keys().next().value ?? null;
@@ -62,6 +73,22 @@ function setTitle(id: string, title: string) {
 	if (entry) files.set(id, { ...entry, title });
 }
 
+function discardAll() {
+	for (const entry of files.values()) {
+		if (entry.previewUrl) URL.revokeObjectURL(entry.previewUrl);
+	}
+	files.clear();
+	activeId = null;
+	chunkProgress = undefined;
+}
+
+onDestroy(() => {
+	for (const entry of files.values()) {
+		if (entry.previewUrl) URL.revokeObjectURL(entry.previewUrl);
+	}
+});
+
+// --- Save ---
 async function save() {
 	const entries = Array.from(files.values());
 	const chunkSize = 10;
@@ -82,63 +109,118 @@ async function save() {
 			documentDate: dateIso || null
 		};
 		formData.append('metadata', new Blob([JSON.stringify(metadata)], { type: 'application/json' }));
-		await fetch('/api/documents/quick-upload', { method: 'POST', body: formData });
+		const res = await fetch('/api/documents/quick-upload', { method: 'POST', body: formData });
+		if (!res.ok) {
+			const body = await res.json().catch(() => ({ errors: [] }));
+			const errorCount = (body.errors ?? []).length;
+			for (let j = 0; j < errorCount && j < chunk.length; j++) {
+				const e = files.get(chunk[j].id);
+				if (e) files.set(chunk[j].id, { ...e, status: 'error' });
+			}
+		}
 		chunkProgress = { done: i + 1, total: chunks.length };
 	}
 	goto('/documents');
 }
 </script>
 
-{#if files.size === 0}
-	<!-- N=0: full-panel drop zone -->
-	<div class="flex min-h-[400px] flex-col">
-		<BulkDropZone onFilesAdded={addFiles} />
-	</div>
-{:else}
-	<div class="mx-auto flex max-w-7xl flex-col gap-0 px-4 py-6">
+<div class="fixed inset-x-0 bottom-0 flex flex-col" style="top: {navHeight}px">
+	<!-- Topbar -->
+	<div class="flex shrink-0 items-center gap-3 border-b border-line bg-surface px-6 py-3">
+		<a
+			href="/documents"
+			class="flex items-center gap-1.5 text-xs font-bold tracking-widest text-ink-3 uppercase hover:text-ink"
+		>
+			<svg
+				class="h-3.5 w-3.5"
+				fill="none"
+				stroke="currentColor"
+				viewBox="0 0 24 24"
+				aria-hidden="true"
+			>
+				<path
+					stroke-linecap="round"
+					stroke-linejoin="round"
+					stroke-width="2"
+					d="M10 19l-7-7m0 0l7-7m-7 7h18"
+				/>
+			</svg>
+			{m.btn_back_to_overview()}
+		</a>
+		<span class="text-ink-3" aria-hidden="true">·</span>
+		<span class="font-serif text-sm font-bold text-ink">
+			{isMulti ? 'Neue Dokumente' : 'Neues Dokument'}
+		</span>
 		{#if isMulti}
-			<FileSwitcherStrip
-				files={Array.from(files.values())}
-				activeId={activeId ?? ''}
-				onSelect={(id) => (activeId = id)}
-				onRemove={removeFile}
-			/>
+			<span class="ml-auto flex items-center gap-3">
+				<span class="rounded-[2px] bg-accent px-2 py-0.5 text-xs font-bold text-primary">
+					{m.bulk_count_pill({ count: files.size })}
+				</span>
+				<button
+					type="button"
+					onclick={discardAll}
+					class="text-xs font-medium text-red-600/70 hover:text-red-700"
+				>
+					{m.bulk_discard_all()}
+				</button>
+			</span>
 		{/if}
+	</div>
 
-		<div class="flex gap-6">
-			<!-- Left: PDF preview -->
-			<div class="hidden w-1/2 lg:block">
-				{#if activeFile}
-					<div
-						class="flex h-[600px] flex-col items-center justify-center rounded-sm bg-gray-900 text-white/40"
-					>
-						<span class="text-sm">{activeFile.file.name}</span>
-						<span class="mt-1 text-xs">Vorschau nach Upload verfügbar</span>
-					</div>
-				{/if}
-			</div>
-
-			<!-- Right: metadata -->
-			<div class="flex flex-1 flex-col gap-4">
+	<!-- Split panel -->
+	<div class="flex flex-1 overflow-hidden">
+		<!-- Left: PDF preview / drop zone (55%) -->
+		<div class="relative flex flex-[55] flex-col overflow-hidden border-r border-line bg-pdf-bg">
+			{#if files.size === 0}
+				<!-- N=0: centred drop-zone box fills the panel -->
+				<BulkDropZone onFilesAdded={addFiles} />
+			{:else}
+				<!-- N≥1: real PDF preview via local blob URL -->
+				<div class="relative flex-1 overflow-hidden">
+					{#if activeFile}
+						<PdfViewer url={activeFile.previewUrl} />
+					{/if}
+				</div>
 				{#if isMulti}
-					<!-- Per-file scope card: title -->
-					<ScopeCard variant="per-file" data-variant="per-file">
+					<!-- File switcher strip pinned to bottom of left panel -->
+					<FileSwitcherStrip
+						files={Array.from(files.values())}
+						activeId={activeId ?? ''}
+						onSelect={(id) => (activeId = id)}
+						onRemove={removeFile}
+					/>
+				{/if}
+			{/if}
+		</div>
+
+		<!-- Right: metadata form (45%) -->
+		<div class="flex flex-[45] flex-col overflow-hidden">
+			<!-- Scrollable form area — greyed out and non-interactive when no files selected -->
+			<div
+				class="flex-1 overflow-y-auto p-4 transition-opacity"
+				class:opacity-60={files.size === 0}
+				class:pointer-events-none={files.size === 0}
+			>
+				{#if isMulti}
+					<!-- N≥2: per-file card (title) + shared card (metadata) -->
+					<ScopeCard variant="per-file">
 						{#if activeFile}
 							<label class="block">
-								<span class="mb-1 block text-xs font-medium text-brand-navy/70">Titel</span>
+								<span class="mb-1 block text-xs font-medium tracking-widest text-ink-2 uppercase">
+									{m.form_label_title()} <span class="text-danger">*</span>
+								</span>
 								<input
 									type="text"
 									value={activeFile.title}
 									oninput={(e) =>
 										setTitle(activeId!, (e.currentTarget as HTMLInputElement).value)}
-									class="border-brand-sand block w-full rounded-sm border p-2 text-sm focus:border-brand-mint focus:outline-none"
+									class="block w-full rounded-sm border border-line bg-surface p-2 text-sm focus:border-accent focus:outline-none"
 								/>
 							</label>
 						{/if}
 					</ScopeCard>
 
-					<!-- Shared scope card: metadata -->
-					<ScopeCard variant="shared" count={files.size} data-variant="shared">
+					<ScopeCard variant="shared" count={files.size}>
 						<WhoWhenSection
 							bind:senderId={senderId}
 							bind:selectedReceivers={selectedReceivers}
@@ -148,22 +230,32 @@ async function save() {
 						<DescriptionSection bind:tags={tags} hideTitle />
 					</ScopeCard>
 				{:else}
-					<!-- N=1: no scope cards, just the fields directly -->
-					{#if activeFile}
-						<div class="border-brand-sand rounded-sm border bg-white p-4 shadow-sm">
-							<label class="block">
-								<span class="mb-1 block text-xs font-medium text-brand-navy/70">Titel</span>
+					<!-- N=0 (disabled placeholder) or N=1 (active): title + shared form -->
+					<div class="mb-4">
+						<label class="block">
+							<span class="mb-1 block text-xs font-medium tracking-widest text-ink-2 uppercase">
+								{m.form_label_title()} <span class="text-danger">*</span>
+							</span>
+							{#if activeFile}
 								<input
 									type="text"
 									value={activeFile.title}
 									oninput={(e) =>
 										setTitle(activeId!, (e.currentTarget as HTMLInputElement).value)}
-									class="border-brand-sand block w-full rounded-sm border p-2 text-sm focus:border-brand-mint focus:outline-none"
+									class="block w-full rounded-sm border border-line bg-surface p-2 text-sm focus:border-accent focus:outline-none"
 								/>
-							</label>
-						</div>
-					{/if}
-					<div class="border-brand-sand rounded-sm border bg-white p-4 shadow-sm">
+							{:else}
+								<input
+									type="text"
+									disabled
+									placeholder="—"
+									class="block w-full rounded-sm border border-line bg-surface p-2 text-sm text-ink-3"
+								/>
+							{/if}
+						</label>
+					</div>
+
+					<div class="rounded-sm border border-line bg-surface p-4 shadow-sm">
 						<WhoWhenSection
 							bind:senderId={senderId}
 							bind:selectedReceivers={selectedReceivers}
@@ -174,17 +266,14 @@ async function save() {
 					</div>
 				{/if}
 			</div>
-		</div>
 
-		<UploadSaveBar
-			fileCount={files.size}
-			chunkProgress={chunkProgress}
-			onSave={save}
-			onDiscard={() => {
-				files.clear();
-				activeId = null;
-				chunkProgress = undefined;
-			}}
-		/>
+			<!-- Action bar: always visible at bottom of right panel -->
+			<UploadSaveBar
+				fileCount={files.size}
+				chunkProgress={chunkProgress}
+				onSave={save}
+				onDiscard={discardAll}
+			/>
+		</div>
 	</div>
-{/if}
+</div>
diff --git a/frontend/src/lib/components/document/BulkDropZone.svelte b/frontend/src/lib/components/document/BulkDropZone.svelte
index 065159e3..a279ffc5 100644
--- a/frontend/src/lib/components/document/BulkDropZone.svelte
+++ b/frontend/src/lib/components/document/BulkDropZone.svelte
@@ -1,4 +1,6 @@
 <script lang="ts">
+import { m } from '$lib/paraglide/messages.js';
+
 let {
 	onFilesAdded
 }: {
@@ -8,30 +10,35 @@ let {
 let isDragging = $state(false);
 </script>
 
-<div class="flex min-h-[300px] flex-1 flex-col items-center justify-center bg-pdf-bg">
+<div
+	role="region"
+	aria-label="Dateien ablegen"
+	data-testid="bulk-drop-zone"
+	class="flex flex-1 flex-col items-center justify-center p-6"
+	ondragover={(e) => {
+		e.preventDefault();
+		isDragging = true;
+	}}
+	ondragleave={() => (isDragging = false)}
+	ondrop={(e) => {
+		e.preventDefault();
+		isDragging = false;
+		if (e.dataTransfer && e.dataTransfer.files.length > 0) {
+			onFilesAdded(Array.from(e.dataTransfer.files));
+		}
+	}}
+>
 	<div
-		role="region"
-		aria-label="Dateien ablegen"
-		data-testid="bulk-drop-zone"
-		class="flex flex-col items-center gap-4 rounded-sm border border-dashed p-12 text-center transition-colors
-			{isDragging ? 'border-brand-mint ring-2 ring-brand-mint' : 'border-white/20'}"
-		ondragover={(e) => {
-			e.preventDefault();
-			isDragging = true;
-		}}
-		ondragleave={() => (isDragging = false)}
-		ondrop={(e) => {
-			e.preventDefault();
-			isDragging = false;
-			if (e.dataTransfer && e.dataTransfer.files.length > 0) {
-				onFilesAdded(Array.from(e.dataTransfer.files));
-			}
-		}}
+		class={[
+			'flex w-full max-w-sm flex-col items-center gap-3 rounded-md border-2 border-dashed px-6 py-9 text-center transition-colors',
+			isDragging ? 'border-accent bg-accent/10' : 'border-accent/50 bg-white/[0.04]'
+		].join(' ')}
 	>
-		<div class="flex h-8 w-8 items-center justify-center rounded-full bg-white/10 text-white/40">
+		<!-- Circular mint icon -->
+		<div class="flex h-14 w-14 items-center justify-center rounded-full bg-accent text-primary">
 			<svg
-				width="16"
-				height="16"
+				width="22"
+				height="22"
 				viewBox="0 0 32 32"
 				xmlns="http://www.w3.org/2000/svg"
 				aria-hidden="true"
@@ -42,11 +49,20 @@ let isDragging = $state(false);
 				/>
 			</svg>
 		</div>
-		<p class="text-sm font-medium text-white/60">PDF-Dateien hier ablegen</p>
-		<p class="text-xs text-white/30">oder</p>
+
+		<!-- Serif title -->
+		<p class="font-serif text-sm font-bold text-ink">{m.bulk_drop_hint()}</p>
+
+		<!-- Sub description -->
+		<p class="text-xs leading-relaxed text-ink-2">
+			Für jede Datei wird ein eigenes Dokument erstellt.<br />
+			<strong class="text-ink">Der Titel</strong> wird aus dem Dateinamen vorausgefüllt —
+			<strong class="text-ink">alle anderen Felder</strong> gelten für alle gemeinsam.
+		</p>
+
+		<!-- CTA button -->
 		<label
-			class="flex min-h-[44px] cursor-pointer items-center rounded-sm bg-brand-navy px-4 py-1.5 text-xs font-bold tracking-widest text-white/90 uppercase"
-			aria-label="Dateien auswählen"
+			class="flex min-h-[44px] cursor-pointer items-center rounded-sm bg-primary px-4 py-2 text-xs font-bold tracking-widest text-primary-fg uppercase transition-opacity hover:opacity-90"
 		>
 			Dateien auswählen
 			<input
@@ -60,5 +76,8 @@ let isDragging = $state(false);
 				}}
 			/>
 		</label>
+
+		<!-- Format hint -->
+		<p class="text-[11px] text-ink-3">{m.bulk_drop_sub()}</p>
 	</div>
 </div>
diff --git a/frontend/src/lib/components/document/FileSwitcherStrip.svelte b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
index e7badb4d..eef08b87 100644
--- a/frontend/src/lib/components/document/FileSwitcherStrip.svelte
+++ b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
@@ -1,9 +1,12 @@
 <script lang="ts">
+import { m } from '$lib/paraglide/messages.js';
+
 export interface FileEntry {
 	id: string;
 	file: File;
 	title: string;
 	status: 'idle' | 'error';
+	previewUrl: string;
 }
 
 let {
@@ -18,14 +21,22 @@ let {
 	onRemove: (id: string) => void;
 } = $props();
 
-let ulEl = $state<HTMLUListElement | null>(null);
+let trackEl = $state<HTMLDivElement | null>(null);
+let listEl = $state<HTMLUListElement | null>(null);
+
+function scrollPrev() {
+	trackEl?.scrollBy({ left: -120, behavior: 'smooth' });
+}
+function scrollNext() {
+	trackEl?.scrollBy({ left: 120, behavior: 'smooth' });
+}
 
 $effect(() => {
-	if (!ulEl) return;
-	const node = ulEl;
+	if (!listEl) return;
+	const node = listEl;
 
 	function handleKeyDown(event: KeyboardEvent) {
-		const buttons = Array.from(node.querySelectorAll<HTMLElement>('[role="button"]'));
+		const buttons = Array.from(node.querySelectorAll<HTMLElement>('[data-chip-id]'));
 		if (buttons.length === 0) return;
 
 		const focusedIndex = buttons.indexOf(document.activeElement as HTMLElement);
@@ -43,49 +54,71 @@ $effect(() => {
 	}
 
 	node.addEventListener('keydown', handleKeyDown);
-	return () => {
-		node.removeEventListener('keydown', handleKeyDown);
-	};
+	return () => node.removeEventListener('keydown', handleKeyDown);
 });
 </script>
 
-<ul
-	bind:this={ulEl}
+<div
 	data-testid="file-switcher-strip"
-	aria-live="polite"
-	role="list"
-	class="flex flex-row gap-2 overflow-x-auto px-1 py-2"
+	class="flex h-11 shrink-0 items-center gap-1 border-t border-line bg-pdf-ctrl px-2"
 >
-	{#each files as entry (entry.id)}
-		<li role="listitem" class="inline-flex items-center gap-0.5">
-			<button
-				type="button"
-				role="button"
-				tabindex="0"
-				aria-current={entry.id === activeId ? 'true' : undefined}
-				data-status={entry.status}
-				data-chip-id={entry.id}
-				onclick={() => onSelect(entry.id)}
-				class={[
-					'inline-flex cursor-pointer items-center gap-1 rounded-sm border px-3 py-1.5 text-xs font-medium transition-colors',
-					entry.id === activeId
-						? 'border-brand-mint bg-brand-mint/10 font-bold text-brand-navy'
-						: 'border-brand-sand bg-white text-brand-navy',
-					entry.status === 'error'
-						? 'border-dashed border-red-300 bg-red-50 text-red-700'
-						: ''
-				].join(' ')}
-				>{entry.title}</button
-			>
-			<button
-				type="button"
-				aria-label="Entfernen"
-				data-remove-id={entry.id}
-				onclick={() => onRemove(entry.id)}
-				class="ml-1 text-xs text-gray-400 hover:text-brand-navy"
-			>
-				×
-			</button>
-		</li>
-	{/each}
-</ul>
+	<button
+		type="button"
+		aria-label={m.bulk_switcher_prev()}
+		onclick={scrollPrev}
+		class="flex h-6 w-5 shrink-0 items-center justify-center rounded-sm text-sm text-ink-3 hover:bg-black/10 hover:text-ink focus:outline-none focus-visible:ring-2 focus-visible:ring-accent"
+		>‹</button
+	>
+
+	<div bind:this={trackEl} class="flex flex-1 gap-1 overflow-x-auto" style="scrollbar-width:none">
+		<ul bind:this={listEl} aria-live="polite" role="list" class="flex flex-row gap-1 py-1">
+			{#each files as entry, i (entry.id)}
+				<li role="listitem" class="inline-flex shrink-0 items-center">
+					<button
+						type="button"
+						tabindex="0"
+						aria-current={entry.id === activeId ? 'true' : undefined}
+						data-status={entry.status}
+						data-chip-id={entry.id}
+						onclick={() => onSelect(entry.id)}
+						class={[
+							'inline-flex cursor-pointer items-center gap-1 rounded-[2px] px-1.5 py-0.5 text-[11px] font-bold transition-colors focus:outline-none focus-visible:ring-2 focus-visible:ring-accent',
+							entry.id === activeId
+								? 'bg-accent text-primary'
+								: 'bg-black/[0.06] text-ink-2 hover:bg-black/10',
+							entry.status === 'error'
+								? '!border !border-dashed !border-red-400 !bg-red-50/80 !text-red-700'
+								: ''
+						].join(' ')}
+					>
+						<span
+							class={[
+								'rounded-[2px] px-0.5 text-[9px] font-extrabold opacity-85',
+								entry.id === activeId ? 'bg-black/20' : 'bg-black/10'
+							].join(' ')}
+							>{i + 1}</span
+						>
+						{entry.title}
+					</button>
+					<button
+						type="button"
+						aria-label="Entfernen"
+						data-remove-id={entry.id}
+						onclick={() => onRemove(entry.id)}
+						class="ml-0.5 flex h-[44px] w-[44px] items-center justify-center text-base text-ink-3 hover:text-ink focus:outline-none focus-visible:ring-2 focus-visible:ring-accent"
+					>
+						×
+					</button>
+				</li>
+			{/each}
+		</ul>
+	</div>
+
+	<button
+		type="button"
+		aria-label={m.bulk_switcher_next()}
+		onclick={scrollNext}
+		class="flex h-6 w-5 shrink-0 items-center justify-center rounded-sm text-sm text-ink-3 hover:bg-black/10 hover:text-ink focus:outline-none focus-visible:ring-2 focus-visible:ring-accent"
+		>›</button
+	>
+</div>
diff --git a/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts b/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts
index 86968e24..56fe281a 100644
--- a/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts
+++ b/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts
@@ -10,6 +10,7 @@ export interface FileEntry {
 	file: File;
 	title: string;
 	status: 'idle' | 'error';
+	previewUrl: string;
 }
 
 function makeFiles(n: number): FileEntry[] {
@@ -17,7 +18,8 @@ function makeFiles(n: number): FileEntry[] {
 		id: `id-${i}`,
 		file: new File([''], `file${i}.pdf`),
 		title: `File ${i}`,
-		status: 'idle' as const
+		status: 'idle' as const,
+		previewUrl: ''
 	}));
 }
 
@@ -65,7 +67,13 @@ describe('FileSwitcherStrip', () => {
 
 	it('error chip has aria-label containing warning indicator', async () => {
 		const files: FileEntry[] = [
-			{ id: 'e1', file: new File([''], 'bad.pdf'), title: 'Bad file', status: 'error' }
+			{
+				id: 'e1',
+				file: new File([''], 'bad.pdf'),
+				title: 'Bad file',
+				status: 'error',
+				previewUrl: ''
+			}
 		];
 		const { container } = render(FileSwitcherStrip, {
 			files,
@@ -85,7 +93,7 @@ describe('FileSwitcherStrip', () => {
 			onSelect: vi.fn(),
 			onRemove: vi.fn()
 		});
-		const firstBtn = container.querySelectorAll('[role="button"]')[0] as HTMLElement;
+		const firstBtn = container.querySelectorAll('[data-chip-id]')[0] as HTMLElement;
 		firstBtn.focus();
 		await userEvent.keyboard('{ArrowRight}');
 		const focused = document.activeElement;
diff --git a/frontend/src/lib/components/document/ScopeCard.svelte b/frontend/src/lib/components/document/ScopeCard.svelte
index ebdbe82a..7eb6002d 100644
--- a/frontend/src/lib/components/document/ScopeCard.svelte
+++ b/frontend/src/lib/components/document/ScopeCard.svelte
@@ -1,4 +1,6 @@
 <script lang="ts">
+import { m } from '$lib/paraglide/messages.js';
+
 let {
 	variant,
 	count = 0,
@@ -13,24 +15,26 @@ let {
 <div
 	data-testid="scope-card"
 	data-variant={variant}
-	class="rounded-sm border p-4
+	class="mb-3 rounded-sm border p-4
 		{variant === 'per-file'
-		? 'border-brand-mint bg-brand-mint/10'
-		: 'border-brand-sand bg-white'}"
+		? 'border-accent bg-accent-bg'
+		: 'border-line bg-surface'}"
 >
 	{#if variant === 'shared'}
 		<div class="mb-3 flex items-center justify-between">
-			<span class="text-xs font-bold tracking-widest text-gray-400 uppercase">
-				Gilt für alle Dateien
+			<span class="text-xs font-bold tracking-widest text-ink-3 uppercase">
+				{m.bulk_scope_shared_label({ count })}
 			</span>
 			<span
-				class="bg-brand-sand inline-flex h-5 min-w-5 items-center justify-center rounded-full px-1.5 text-xs font-bold text-brand-navy"
+				class="inline-flex h-5 min-w-5 items-center justify-center rounded-full bg-accent px-1.5 text-xs font-bold text-primary"
 			>
 				{count}
 			</span>
 		</div>
 	{:else}
-		<p class="mb-3 text-xs font-bold tracking-widest text-brand-mint uppercase">Diese Datei</p>
+		<p class="mb-3 text-xs font-bold tracking-widest text-accent uppercase">
+			{m.bulk_scope_per_file_label()}
+		</p>
 	{/if}
 	{@render children?.()}
 </div>
diff --git a/frontend/src/lib/components/document/UploadSaveBar.svelte b/frontend/src/lib/components/document/UploadSaveBar.svelte
index f8cad1bf..06b85d99 100644
--- a/frontend/src/lib/components/document/UploadSaveBar.svelte
+++ b/frontend/src/lib/components/document/UploadSaveBar.svelte
@@ -1,5 +1,5 @@
 <script lang="ts">
-import * as m from '$lib/paraglide/messages';
+import { m } from '$lib/paraglide/messages.js';
 
 let {
 	fileCount,
@@ -14,27 +14,26 @@ let {
 } = $props();
 </script>
 
-<div
-	class="border-brand-sand sticky bottom-0 z-10 -mx-4 border-t bg-white px-6 py-4 shadow-[0_-2px_8px_rgba(0,0,0,0.06)]"
->
+<div class="shrink-0 border-t border-line bg-surface px-4 py-3">
 	{#if chunkProgress}
 		<progress
 			value={chunkProgress.done}
 			max={chunkProgress.total}
-			class="[&::-webkit-progress-bar]:bg-brand-sand mb-3 h-1 w-full rounded-full [&::-webkit-progress-bar]:rounded-full [&::-webkit-progress-value]:rounded-full [&::-webkit-progress-value]:bg-brand-mint"
+			class="[&::-webkit-progress-bar]:bg-brand-sand mb-3 h-1 w-full rounded-full [&::-webkit-progress-bar]:rounded-full [&::-webkit-progress-value]:rounded-full [&::-webkit-progress-value]:bg-accent"
 		></progress>
 	{/if}
-	<div class="flex items-center justify-between">
+	<div class="flex items-center justify-between gap-3">
 		<button type="button" onclick={onDiscard} class="text-sm text-red-600/70 hover:text-red-700">
 			{m.bulk_discard_all()}
 		</button>
 		<button
 			type="button"
 			data-testid="bulk-save-btn"
+			disabled={fileCount === 0}
 			onclick={onSave}
-			class="min-h-[44px] rounded-sm bg-brand-navy px-6 text-sm font-bold tracking-widest text-white uppercase hover:bg-brand-navy/90"
+			class="min-h-[44px] rounded-sm bg-primary px-6 text-sm font-bold tracking-widest text-primary-fg uppercase transition-opacity hover:opacity-90 disabled:opacity-40"
 		>
-			{fileCount === 1 ? 'Speichern →' : `${fileCount} speichern →`}
+			{fileCount === 1 ? m.bulk_save_cta_one() : m.bulk_save_cta({ count: fileCount })}
 		</button>
 	</div>
 </div>
diff --git a/frontend/src/routes/documents/new/+page.svelte b/frontend/src/routes/documents/new/+page.svelte
index 753233b1..c4cb69f8 100644
--- a/frontend/src/routes/documents/new/+page.svelte
+++ b/frontend/src/routes/documents/new/+page.svelte
@@ -1,37 +1,11 @@
 <script lang="ts">
-import { m } from '$lib/paraglide/messages.js';
 import BulkDocumentEditLayout from '$lib/components/document/BulkDocumentEditLayout.svelte';
 
 let { data } = $props();
 </script>
 
-<div class="mx-auto max-w-7xl px-4 py-8">
-	<div class="mb-6">
-		<a
-			href="/"
-			class="group mb-4 inline-flex items-center text-xs font-bold tracking-widest text-ink-2 uppercase transition-colors hover:text-ink"
-		>
-			<svg
-				class="mr-2 h-4 w-4 transform transition-transform group-hover:-translate-x-1"
-				fill="none"
-				stroke="currentColor"
-				viewBox="0 0 24 24"
-			>
-				<path
-					stroke-linecap="round"
-					stroke-linejoin="round"
-					stroke-width="2"
-					d="M10 19l-7-7m0 0l7-7m-7 7h18"
-				/>
-			</svg>
-			{m.btn_back_to_overview()}
-		</a>
-		<h1 class="font-serif text-3xl text-ink">{m.doc_new_heading()}</h1>
-	</div>
-
-	<BulkDocumentEditLayout
-		initialSenderId={data.initialSenderId}
-		initialSenderName={data.initialSenderName}
-		initialReceivers={data.initialReceivers}
-	/>
-</div>
+<BulkDocumentEditLayout
+	initialSenderId={data.initialSenderId}
+	initialSenderName={data.initialSenderName}
+	initialReceivers={data.initialReceivers}
+/>
diff --git a/frontend/src/routes/documents/new/page.svelte.spec.ts b/frontend/src/routes/documents/new/page.svelte.spec.ts
index dcdfbdef..b22028bf 100644
--- a/frontend/src/routes/documents/new/page.svelte.spec.ts
+++ b/frontend/src/routes/documents/new/page.svelte.spec.ts
@@ -21,15 +21,14 @@ const baseData = {
 
 describe('New document page – sender prefill', () => {
 	it('shows an empty sender input when no senderId is in the URL', async () => {
-		render(Page, { data: baseData, form: null });
+		render(Page, { data: baseData });
 		const input = document.querySelector<HTMLInputElement>('#senderId-search');
 		expect(input?.value).toBe('');
 	});
 
 	it('shows the sender name in the typeahead input when initialSenderName is set', async () => {
 		render(Page, {
-			data: { ...baseData, initialSenderId: 'p1', initialSenderName: 'Hans Müller' },
-			form: null
+			data: { ...baseData, initialSenderId: 'p1', initialSenderName: 'Hans Müller' }
 		});
 		const input = document.querySelector<HTMLInputElement>('#senderId-search');
 		expect(input?.value).toBe('Hans Müller');
@@ -37,8 +36,7 @@ describe('New document page – sender prefill', () => {
 
 	it('sets the hidden senderId input to the prefilled ID', async () => {
 		render(Page, {
-			data: { ...baseData, initialSenderId: 'p1', initialSenderName: 'Hans Müller' },
-			form: null
+			data: { ...baseData, initialSenderId: 'p1', initialSenderName: 'Hans Müller' }
 		});
 		const hidden = document.querySelector<HTMLInputElement>(
 			'input[type="hidden"][name="senderId"]'
@@ -51,7 +49,7 @@ describe('New document page – sender prefill', () => {
 
 describe('New document page – receiver prefill', () => {
 	it('shows no receiver chips when initialReceivers is empty', async () => {
-		render(Page, { data: baseData, form: null });
+		render(Page, { data: baseData });
 		await expect.element(page.getByText('Anna Schmidt')).not.toBeInTheDocument();
 	});
 
@@ -62,7 +60,7 @@ describe('New document page – receiver prefill', () => {
 				{ id: 'p2', firstName: 'Anna', lastName: 'Schmidt', displayName: 'Anna Schmidt' }
 			]
 		};
-		render(Page, { data, form: null });
+		render(Page, { data });
 		await expect.element(page.getByText('Anna Schmidt')).toBeInTheDocument();
 	});
 
@@ -73,7 +71,7 @@ describe('New document page – receiver prefill', () => {
 				{ id: 'p2', firstName: 'Anna', lastName: 'Schmidt', displayName: 'Anna Schmidt' }
 			]
 		};
-		render(Page, { data, form: null });
+		render(Page, { data });
 		const hidden = document.querySelector<HTMLInputElement>('input[name="receiverIds"]');
 		expect(hidden?.value).toBe('p2');
 	});
-- 
2.49.1


From aa9c47ecc8a97306e605795bcac4ededd76aab62 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 19:26:22 +0200
Subject: [PATCH 12/46] fix(bulk-upload): form layout polish and drop zone
 sizing
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Drop zone box doubled: max-w-xl, larger icon (80px), bigger padding and text
- Title field wrapped in its own card (matches WhoWhenSection/DescriptionSection)
- Removed double-wrapping outer card around WhoWhenSection + DescriptionSection
- Added space-y-4 between form sections for consistent breathing room
- ScopeCard per-file label: text-accent → text-primary for legible contrast in light theme

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../document/BulkDocumentEditLayout.svelte    | 26 +++++++++----------
 .../components/document/BulkDropZone.svelte   | 16 ++++++------
 .../lib/components/document/ScopeCard.svelte  |  2 +-
 3 files changed, 21 insertions(+), 23 deletions(-)

diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
index 2c9b8511..1dfba569 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
@@ -197,7 +197,7 @@ async function save() {
 		<div class="flex flex-[45] flex-col overflow-hidden">
 			<!-- Scrollable form area — greyed out and non-interactive when no files selected -->
 			<div
-				class="flex-1 overflow-y-auto p-4 transition-opacity"
+				class="flex-1 space-y-4 overflow-y-auto p-4 transition-opacity"
 				class:opacity-60={files.size === 0}
 				class:pointer-events-none={files.size === 0}
 			>
@@ -231,9 +231,9 @@ async function save() {
 					</ScopeCard>
 				{:else}
 					<!-- N=0 (disabled placeholder) or N=1 (active): title + shared form -->
-					<div class="mb-4">
+					<div class="rounded-sm border border-line bg-surface p-6 shadow-sm">
 						<label class="block">
-							<span class="mb-1 block text-xs font-medium tracking-widest text-ink-2 uppercase">
+							<span class="mb-1 block text-xs font-bold tracking-widest text-ink-3 uppercase">
 								{m.form_label_title()} <span class="text-danger">*</span>
 							</span>
 							{#if activeFile}
@@ -242,28 +242,26 @@ async function save() {
 									value={activeFile.title}
 									oninput={(e) =>
 										setTitle(activeId!, (e.currentTarget as HTMLInputElement).value)}
-									class="block w-full rounded-sm border border-line bg-surface p-2 text-sm focus:border-accent focus:outline-none"
+									class="block w-full rounded border border-line p-2 text-sm shadow-sm focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring"
 								/>
 							{:else}
 								<input
 									type="text"
 									disabled
 									placeholder="—"
-									class="block w-full rounded-sm border border-line bg-surface p-2 text-sm text-ink-3"
+									class="block w-full rounded border border-line p-2 text-sm text-ink-3 shadow-sm"
 								/>
 							{/if}
 						</label>
 					</div>
 
-					<div class="rounded-sm border border-line bg-surface p-4 shadow-sm">
-						<WhoWhenSection
-							bind:senderId={senderId}
-							bind:selectedReceivers={selectedReceivers}
-							bind:dateIso={dateIso}
-							initialSenderName={initialSenderName}
-						/>
-						<DescriptionSection bind:tags={tags} hideTitle />
-					</div>
+					<WhoWhenSection
+						bind:senderId={senderId}
+						bind:selectedReceivers={selectedReceivers}
+						bind:dateIso={dateIso}
+						initialSenderName={initialSenderName}
+					/>
+					<DescriptionSection bind:tags={tags} hideTitle />
 				{/if}
 			</div>
 
diff --git a/frontend/src/lib/components/document/BulkDropZone.svelte b/frontend/src/lib/components/document/BulkDropZone.svelte
index a279ffc5..b763370e 100644
--- a/frontend/src/lib/components/document/BulkDropZone.svelte
+++ b/frontend/src/lib/components/document/BulkDropZone.svelte
@@ -30,15 +30,15 @@ let isDragging = $state(false);
 >
 	<div
 		class={[
-			'flex w-full max-w-sm flex-col items-center gap-3 rounded-md border-2 border-dashed px-6 py-9 text-center transition-colors',
+			'flex w-full max-w-xl flex-col items-center gap-5 rounded-md border-2 border-dashed px-12 py-16 text-center transition-colors',
 			isDragging ? 'border-accent bg-accent/10' : 'border-accent/50 bg-white/[0.04]'
 		].join(' ')}
 	>
 		<!-- Circular mint icon -->
-		<div class="flex h-14 w-14 items-center justify-center rounded-full bg-accent text-primary">
+		<div class="flex h-20 w-20 items-center justify-center rounded-full bg-accent text-primary">
 			<svg
-				width="22"
-				height="22"
+				width="32"
+				height="32"
 				viewBox="0 0 32 32"
 				xmlns="http://www.w3.org/2000/svg"
 				aria-hidden="true"
@@ -51,10 +51,10 @@ let isDragging = $state(false);
 		</div>
 
 		<!-- Serif title -->
-		<p class="font-serif text-sm font-bold text-ink">{m.bulk_drop_hint()}</p>
+		<p class="font-serif text-base font-bold text-ink">{m.bulk_drop_hint()}</p>
 
 		<!-- Sub description -->
-		<p class="text-xs leading-relaxed text-ink-2">
+		<p class="text-sm leading-relaxed text-ink-2">
 			Für jede Datei wird ein eigenes Dokument erstellt.<br />
 			<strong class="text-ink">Der Titel</strong> wird aus dem Dateinamen vorausgefüllt —
 			<strong class="text-ink">alle anderen Felder</strong> gelten für alle gemeinsam.
@@ -62,7 +62,7 @@ let isDragging = $state(false);
 
 		<!-- CTA button -->
 		<label
-			class="flex min-h-[44px] cursor-pointer items-center rounded-sm bg-primary px-4 py-2 text-xs font-bold tracking-widest text-primary-fg uppercase transition-opacity hover:opacity-90"
+			class="flex min-h-[44px] cursor-pointer items-center rounded-sm bg-primary px-6 py-2 text-xs font-bold tracking-widest text-primary-fg uppercase transition-opacity hover:opacity-90"
 		>
 			Dateien auswählen
 			<input
@@ -78,6 +78,6 @@ let isDragging = $state(false);
 		</label>
 
 		<!-- Format hint -->
-		<p class="text-[11px] text-ink-3">{m.bulk_drop_sub()}</p>
+		<p class="text-xs text-ink-3">{m.bulk_drop_sub()}</p>
 	</div>
 </div>
diff --git a/frontend/src/lib/components/document/ScopeCard.svelte b/frontend/src/lib/components/document/ScopeCard.svelte
index 7eb6002d..6fb79f96 100644
--- a/frontend/src/lib/components/document/ScopeCard.svelte
+++ b/frontend/src/lib/components/document/ScopeCard.svelte
@@ -32,7 +32,7 @@ let {
 			</span>
 		</div>
 	{:else}
-		<p class="mb-3 text-xs font-bold tracking-widest text-accent uppercase">
+		<p class="mb-3 text-xs font-bold tracking-widest text-primary uppercase">
 			{m.bulk_scope_per_file_label()}
 		</p>
 	{/if}
-- 
2.49.1


From 8b05451f42720c18a6259ce694ca8b2d7559822b Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 19:29:21 +0200
Subject: [PATCH 13/46] fix(bulk-upload): PDF-only file acceptance

Drop non-PDF accept types from file input and update format hint strings
in all three languages. JPEG/PNG/TIFF were never officially supported.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/messages/de.json                                | 2 +-
 frontend/messages/en.json                                | 2 +-
 frontend/messages/es.json                                | 2 +-
 frontend/src/lib/components/document/BulkDropZone.svelte | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/frontend/messages/de.json b/frontend/messages/de.json
index 479238c1..1a0a0fd0 100644
--- a/frontend/messages/de.json
+++ b/frontend/messages/de.json
@@ -854,7 +854,7 @@
 	"richtlinien_closing_body": "Stolpern Sie beim Transkribieren über eine Situation, die hier nicht steht — schreiben Sie einen Kommentar beim betreffenden Block. Wir sammeln sie und besprechen sie beim nächsten Familientreffen.",
 	"error_batch_too_large": "Zu viele Dateien auf einmal — bitte in Blöcken hochladen.",
 	"bulk_drop_hint": "Eine oder mehrere Dateien ablegen",
-	"bulk_drop_sub": "PDF, JPEG, PNG oder TIFF · bis zu 50 MB pro Datei",
+	"bulk_drop_sub": "PDF · bis zu 50 MB pro Datei",
 	"bulk_count_pill": "{count} werden erstellt",
 	"bulk_save_cta_one": "Speichern →",
 	"bulk_save_cta": "{count} speichern →",
diff --git a/frontend/messages/en.json b/frontend/messages/en.json
index 78e107e4..95c52c82 100644
--- a/frontend/messages/en.json
+++ b/frontend/messages/en.json
@@ -854,7 +854,7 @@
 	"richtlinien_closing_body": "If you hit a situation while transcribing that isn't listed here — leave a comment on the relevant block. We collect them and discuss them at the next family gathering.",
 	"error_batch_too_large": "Too many files at once — please upload in smaller batches.",
 	"bulk_drop_hint": "Drop one or more files here",
-	"bulk_drop_sub": "PDF, JPEG, PNG or TIFF · up to 50 MB per file",
+	"bulk_drop_sub": "PDF · up to 50 MB per file",
 	"bulk_count_pill": "{count} will be created",
 	"bulk_save_cta_one": "Save →",
 	"bulk_save_cta": "Save {count} →",
diff --git a/frontend/messages/es.json b/frontend/messages/es.json
index a8ec1703..61050e5d 100644
--- a/frontend/messages/es.json
+++ b/frontend/messages/es.json
@@ -854,7 +854,7 @@
 	"richtlinien_closing_body": "Si al transcribir encuentras una situación que no está aquí — deja un comentario en el bloque. Las recogemos y las discutimos en la próxima reunión familiar.",
 	"error_batch_too_large": "Demasiados archivos a la vez — sube en lotes más pequeños.",
 	"bulk_drop_hint": "Suelta uno o varios archivos aquí",
-	"bulk_drop_sub": "PDF, JPEG, PNG o TIFF · hasta 50 MB por archivo",
+	"bulk_drop_sub": "PDF · hasta 50 MB por archivo",
 	"bulk_count_pill": "Se crearán {count}",
 	"bulk_save_cta_one": "Guardar →",
 	"bulk_save_cta": "Guardar {count} →",
diff --git a/frontend/src/lib/components/document/BulkDropZone.svelte b/frontend/src/lib/components/document/BulkDropZone.svelte
index b763370e..376f4b01 100644
--- a/frontend/src/lib/components/document/BulkDropZone.svelte
+++ b/frontend/src/lib/components/document/BulkDropZone.svelte
@@ -68,7 +68,7 @@ let isDragging = $state(false);
 			<input
 				type="file"
 				multiple
-				accept="application/pdf,image/jpeg,image/png,image/tiff"
+				accept="application/pdf"
 				class="sr-only"
 				onchange={(e) => {
 					const files = Array.from(e.currentTarget.files ?? []);
-- 
2.49.1


From db6a3225dbf0ed5542312494b056394965786fcb Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 19:32:23 +0200
Subject: [PATCH 14/46] fix(bulk-upload): no layout shift, no autofocus on date
 field
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Replace JS navHeight measurement with CSS var(--header-height) so the fixed
panel renders in its final position on first paint — no onMount shift.

Add autofocus prop to WhoWhenSection (default true, preserves document-edit
behaviour) and pass autofocus={false} from BulkDocumentEditLayout so the date
field does not steal focus before the user has even dropped any files.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../components/document/BulkDocumentEditLayout.svelte | 11 +++--------
 .../src/lib/components/document/WhoWhenSection.svelte |  8 +++++---
 2 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
index 1dfba569..0cf98694 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
@@ -1,7 +1,7 @@
 <script lang="ts">
 import { SvelteMap } from 'svelte/reactivity';
 import { goto } from '$app/navigation';
-import { onMount, onDestroy, untrack } from 'svelte';
+import { onDestroy, untrack } from 'svelte';
 import { m } from '$lib/paraglide/messages.js';
 import BulkDropZone from './BulkDropZone.svelte';
 import FileSwitcherStrip from './FileSwitcherStrip.svelte';
@@ -27,12 +27,6 @@ let {
 	initialReceivers?: Person[];
 } = $props();
 
-// --- Layout ---
-let navHeight = $state(0);
-onMount(() => {
-	navHeight = document.querySelector('header')?.getBoundingClientRect().height ?? 0;
-});
-
 // --- File state ---
 let files = new SvelteMap<string, FileEntry>();
 let activeId = $state<string | null>(null);
@@ -124,7 +118,7 @@ async function save() {
 }
 </script>
 
-<div class="fixed inset-x-0 bottom-0 flex flex-col" style="top: {navHeight}px">
+<div class="fixed inset-x-0 bottom-0 flex flex-col" style="top: var(--header-height)">
 	<!-- Topbar -->
 	<div class="flex shrink-0 items-center gap-3 border-b border-line bg-surface px-6 py-3">
 		<a
@@ -226,6 +220,7 @@ async function save() {
 							bind:selectedReceivers={selectedReceivers}
 							bind:dateIso={dateIso}
 							initialSenderName={initialSenderName}
+							autofocus={false}
 						/>
 						<DescriptionSection bind:tags={tags} hideTitle />
 					</ScopeCard>
diff --git a/frontend/src/lib/components/document/WhoWhenSection.svelte b/frontend/src/lib/components/document/WhoWhenSection.svelte
index b5d85a56..ff8a044b 100644
--- a/frontend/src/lib/components/document/WhoWhenSection.svelte
+++ b/frontend/src/lib/components/document/WhoWhenSection.svelte
@@ -16,7 +16,8 @@ let {
 	initialLocation = '',
 	initialSenderName = '',
 	suggestedDateIso = '',
-	suggestedSenderName = ''
+	suggestedSenderName = '',
+	autofocus = true
 }: {
 	senderId?: string;
 	selectedReceivers?: Person[];
@@ -26,6 +27,7 @@ let {
 	initialSenderName?: string;
 	suggestedDateIso?: string;
 	suggestedSenderName?: string;
+	autofocus?: boolean;
 } = $props();
 
 let dateDisplay = $state(untrack(() => isoToGerman(initialDateIso)));
@@ -69,7 +71,7 @@ $effect(() => {
 				oninput={handleDateInput}
 				placeholder={m.form_placeholder_date()}
 				maxlength="10"
-				autofocus={!initialDateIso}
+				autofocus={autofocus && !initialDateIso}
 				class="block w-full rounded border border-line p-2 text-sm shadow-sm
                        {dateInvalid ? 'border-red-400 focus:outline-none focus-visible:ring-2 focus-visible:ring-red-500' : 'focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring'}"
 				aria-describedby={dateInvalid ? 'date-error' : undefined}
@@ -89,7 +91,7 @@ $effect(() => {
 				bind:value={senderId}
 				initialName={initialSenderName}
 				suggestedName={suggestedSenderName}
-				autofocus={!!initialDateIso}
+				autofocus={autofocus && !!initialDateIso}
 			/>
 		</div>
 
-- 
2.49.1


From 75dd8cb08d26cad3fd51490ffeba0be27f1c50bb Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 19:36:20 +0200
Subject: [PATCH 15/46] fix(PersonMultiSelect): align height and focus ring
 with other form inputs
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

min-h-[42px] → min-h-[38px] to match p-2 text-sm input height.
Add shadow-sm (was missing vs date/sender inputs).
focus-within:ring-1 ring-ink → focus-within:ring-2 ring-focus-ring to match
the focus style used consistently across all other form inputs.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/src/lib/components/PersonMultiSelect.svelte | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/lib/components/PersonMultiSelect.svelte b/frontend/src/lib/components/PersonMultiSelect.svelte
index 7847787f..6ff453c9 100644
--- a/frontend/src/lib/components/PersonMultiSelect.svelte
+++ b/frontend/src/lib/components/PersonMultiSelect.svelte
@@ -67,7 +67,7 @@ function removePerson(id: string | undefined) {
 
 <div class="relative" use:clickOutside onclickoutside={() => (showDropdown = false)}>
 	<div
-		class="flex min-h-[42px] flex-wrap gap-2 rounded border border-line bg-surface p-2 focus-within:border-ink focus-within:ring-1 focus-within:ring-ink"
+		class="flex min-h-[38px] flex-wrap gap-2 rounded border border-line bg-surface p-2 shadow-sm focus-within:ring-2 focus-within:ring-focus-ring focus-within:outline-none"
 	>
 		{#each selectedPersons as person (person.id)}
 			<span
-- 
2.49.1


From 65846911f3bc4b4114eff9060be5ab862468f2ea Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 19:38:54 +0200
Subject: [PATCH 16/46] fix(PersonTypeahead): match height and border-radius of
 other form inputs
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Default mode was text-base (16px) and rounded-md — date field uses text-sm
(14px) and rounded. Aligning these makes Sender/Date/Receiver rows consistent.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/src/lib/components/PersonTypeahead.svelte | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/lib/components/PersonTypeahead.svelte b/frontend/src/lib/components/PersonTypeahead.svelte
index 45ef34c7..cc667d2d 100644
--- a/frontend/src/lib/components/PersonTypeahead.svelte
+++ b/frontend/src/lib/components/PersonTypeahead.svelte
@@ -134,7 +134,7 @@ function selectPerson(person: Person) {
 			? 'mt-2 block h-14 w-full rounded-md border border-line bg-surface px-4 text-base text-ink shadow-sm placeholder:text-ink-3 focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring'
 			: compact
 				? 'mt-1 block h-9 w-full rounded border border-line bg-surface px-2 text-sm text-ink placeholder:text-ink-3 focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring'
-				: 'mt-1 block w-full rounded-md border border-line bg-surface p-2 text-ink shadow-sm placeholder:text-ink-3 focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring'}
+				: 'mt-1 block w-full rounded border border-line bg-surface p-2 text-sm text-ink shadow-sm placeholder:text-ink-3 focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring'}
 	/>
 
 	{#if typeahead.isOpen && (typeahead.results.length > 0 || typeahead.loading)}
-- 
2.49.1


From 1ea95f8fe0219002d65229ed7a0915cc9622d171 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 19:45:03 +0200
Subject: [PATCH 17/46] fix(forms): raise date and sender field height to match
 receiver (44px)

PersonMultiSelect naturally renders at 44px due to nested padding (outer p-2 + inner p-1).
Apply py-3 px-2 to the date input and PersonTypeahead default mode so all three fields
align visually.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/src/lib/components/PersonTypeahead.svelte         | 2 +-
 frontend/src/lib/components/document/WhoWhenSection.svelte | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/frontend/src/lib/components/PersonTypeahead.svelte b/frontend/src/lib/components/PersonTypeahead.svelte
index cc667d2d..815623da 100644
--- a/frontend/src/lib/components/PersonTypeahead.svelte
+++ b/frontend/src/lib/components/PersonTypeahead.svelte
@@ -134,7 +134,7 @@ function selectPerson(person: Person) {
 			? 'mt-2 block h-14 w-full rounded-md border border-line bg-surface px-4 text-base text-ink shadow-sm placeholder:text-ink-3 focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring'
 			: compact
 				? 'mt-1 block h-9 w-full rounded border border-line bg-surface px-2 text-sm text-ink placeholder:text-ink-3 focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring'
-				: 'mt-1 block w-full rounded border border-line bg-surface p-2 text-sm text-ink shadow-sm placeholder:text-ink-3 focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring'}
+				: 'mt-1 block w-full rounded border border-line bg-surface px-2 py-3 text-sm text-ink shadow-sm placeholder:text-ink-3 focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring'}
 	/>
 
 	{#if typeahead.isOpen && (typeahead.results.length > 0 || typeahead.loading)}
diff --git a/frontend/src/lib/components/document/WhoWhenSection.svelte b/frontend/src/lib/components/document/WhoWhenSection.svelte
index ff8a044b..d315be42 100644
--- a/frontend/src/lib/components/document/WhoWhenSection.svelte
+++ b/frontend/src/lib/components/document/WhoWhenSection.svelte
@@ -72,7 +72,7 @@ $effect(() => {
 				placeholder={m.form_placeholder_date()}
 				maxlength="10"
 				autofocus={autofocus && !initialDateIso}
-				class="block w-full rounded border border-line p-2 text-sm shadow-sm
+				class="block w-full rounded border border-line px-2 py-3 text-sm shadow-sm
                        {dateInvalid ? 'border-red-400 focus:outline-none focus-visible:ring-2 focus-visible:ring-red-500' : 'focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring'}"
 				aria-describedby={dateInvalid ? 'date-error' : undefined}
 			/>
-- 
2.49.1


From 9d687ba9f91168bbd8f41c7e67a683d46bfe2e81 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 19:48:47 +0200
Subject: [PATCH 18/46] fix(forms): apply py-3 to location input for consistent
 44px height

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/src/lib/components/document/WhoWhenSection.svelte | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/lib/components/document/WhoWhenSection.svelte b/frontend/src/lib/components/document/WhoWhenSection.svelte
index d315be42..c8d03e24 100644
--- a/frontend/src/lib/components/document/WhoWhenSection.svelte
+++ b/frontend/src/lib/components/document/WhoWhenSection.svelte
@@ -112,7 +112,7 @@ $effect(() => {
 				name="location"
 				value={initialLocation}
 				placeholder={m.form_placeholder_location()}
-				class="block w-full rounded border border-line p-2 text-sm shadow-sm focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring"
+				class="block w-full rounded border border-line px-2 py-3 text-sm shadow-sm focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring"
 			/>
 		</div>
 	</div>
-- 
2.49.1


From dd6331c09845bc9214c940f2c0361d38421603a8 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 19:51:15 +0200
Subject: [PATCH 19/46] fix(forms): remove autofocus from WhoWhenSection
 entirely
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The autofocus prop was added conditionally but still triggered on the
bulk-upload page. Removing it completely — callers that need focus
management can handle it independently.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../lib/components/document/BulkDocumentEditLayout.svelte   | 1 -
 frontend/src/lib/components/document/WhoWhenSection.svelte  | 6 +-----
 2 files changed, 1 insertion(+), 6 deletions(-)

diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
index 0cf98694..c1ece8b1 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
@@ -220,7 +220,6 @@ async function save() {
 							bind:selectedReceivers={selectedReceivers}
 							bind:dateIso={dateIso}
 							initialSenderName={initialSenderName}
-							autofocus={false}
 						/>
 						<DescriptionSection bind:tags={tags} hideTitle />
 					</ScopeCard>
diff --git a/frontend/src/lib/components/document/WhoWhenSection.svelte b/frontend/src/lib/components/document/WhoWhenSection.svelte
index c8d03e24..679ecc84 100644
--- a/frontend/src/lib/components/document/WhoWhenSection.svelte
+++ b/frontend/src/lib/components/document/WhoWhenSection.svelte
@@ -16,8 +16,7 @@ let {
 	initialLocation = '',
 	initialSenderName = '',
 	suggestedDateIso = '',
-	suggestedSenderName = '',
-	autofocus = true
+	suggestedSenderName = ''
 }: {
 	senderId?: string;
 	selectedReceivers?: Person[];
@@ -27,7 +26,6 @@ let {
 	initialSenderName?: string;
 	suggestedDateIso?: string;
 	suggestedSenderName?: string;
-	autofocus?: boolean;
 } = $props();
 
 let dateDisplay = $state(untrack(() => isoToGerman(initialDateIso)));
@@ -71,7 +69,6 @@ $effect(() => {
 				oninput={handleDateInput}
 				placeholder={m.form_placeholder_date()}
 				maxlength="10"
-				autofocus={autofocus && !initialDateIso}
 				class="block w-full rounded border border-line px-2 py-3 text-sm shadow-sm
                        {dateInvalid ? 'border-red-400 focus:outline-none focus-visible:ring-2 focus-visible:ring-red-500' : 'focus:outline-none focus-visible:ring-2 focus-visible:ring-focus-ring'}"
 				aria-describedby={dateInvalid ? 'date-error' : undefined}
@@ -91,7 +88,6 @@ $effect(() => {
 				bind:value={senderId}
 				initialName={initialSenderName}
 				suggestedName={suggestedSenderName}
-				autofocus={autofocus && !!initialDateIso}
 			/>
 		</div>
 
-- 
2.49.1


From b2264de9493f38cbf768ce7ad434970db4de8961 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 20:25:17 +0200
Subject: [PATCH 20/46] refactor(documents): move batch validation from
 controller into DocumentService

Validation guards (BATCH_TOO_LARGE, titles > files) are domain rules and
belong in the service where they can be unit-tested without the HTTP layer.
Controller now delegates to documentService.validateBatch().

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../controller/DocumentController.java        |  7 +-----
 .../service/DocumentService.java              |  9 +++++++
 .../controller/DocumentControllerTest.java    |  8 ++++++
 .../service/DocumentServiceTest.java          | 25 +++++++++++++++++++
 4 files changed, 43 insertions(+), 6 deletions(-)

diff --git a/backend/src/main/java/org/raddatz/familienarchiv/controller/DocumentController.java b/backend/src/main/java/org/raddatz/familienarchiv/controller/DocumentController.java
index cd03a22f..ea943163 100644
--- a/backend/src/main/java/org/raddatz/familienarchiv/controller/DocumentController.java
+++ b/backend/src/main/java/org/raddatz/familienarchiv/controller/DocumentController.java
@@ -204,12 +204,7 @@ public class DocumentController {
             return new QuickUploadResult(created, updated, errors);
         }
 
-        if (files.size() > 50) {
-            throw DomainException.badRequest(ErrorCode.BATCH_TOO_LARGE, "Batch exceeds maximum of 50 files per request");
-        }
-        if (metadata != null && metadata.getTitles() != null && metadata.getTitles().size() > files.size()) {
-            throw DomainException.badRequest(ErrorCode.VALIDATION_ERROR, "titles count must not exceed files count");
-        }
+        documentService.validateBatch(files.size(), metadata);
 
         UUID actorId = requireUserId(authentication);
         long totalBytes = files.stream().mapToLong(MultipartFile::getSize).sum();
diff --git a/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java b/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java
index 36575f28..01ea544d 100644
--- a/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java
+++ b/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java
@@ -133,6 +133,15 @@ public class DocumentService {
         return new StoreResult(saved, isNew);
     }
 
+    public void validateBatch(int fileCount, DocumentBatchMetadataDTO metadata) {
+        if (fileCount > 50) {
+            throw DomainException.badRequest(ErrorCode.BATCH_TOO_LARGE, "Batch exceeds maximum of 50 files per request");
+        }
+        if (metadata != null && metadata.getTitles() != null && metadata.getTitles().size() > fileCount) {
+            throw DomainException.badRequest(ErrorCode.VALIDATION_ERROR, "titles count must not exceed files count");
+        }
+    }
+
     @Transactional
     public StoreResult storeDocumentWithBatchMetadata(
             MultipartFile file, DocumentBatchMetadataDTO metadata, int fileIndex, UUID actorId) throws IOException {
diff --git a/backend/src/test/java/org/raddatz/familienarchiv/controller/DocumentControllerTest.java b/backend/src/test/java/org/raddatz/familienarchiv/controller/DocumentControllerTest.java
index 1981b1b2..828e8402 100644
--- a/backend/src/test/java/org/raddatz/familienarchiv/controller/DocumentControllerTest.java
+++ b/backend/src/test/java/org/raddatz/familienarchiv/controller/DocumentControllerTest.java
@@ -869,6 +869,10 @@ class DocumentControllerTest {
     @WithMockUser(authorities = "WRITE_ALL")
     void quickUpload_withMetadata_rejects400_whenTitlesSizeExceedsFilesSize() throws Exception {
         when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
+        org.mockito.Mockito.doThrow(
+                org.raddatz.familienarchiv.exception.DomainException.badRequest(
+                        org.raddatz.familienarchiv.exception.ErrorCode.VALIDATION_ERROR, "titles count must not exceed files count"))
+                .when(documentService).validateBatch(eq(2), any());
 
         org.springframework.mock.web.MockMultipartFile f1 =
                 new org.springframework.mock.web.MockMultipartFile("files", "a.pdf", "application/pdf", new byte[]{1});
@@ -886,6 +890,10 @@ class DocumentControllerTest {
     @WithMockUser(authorities = "WRITE_ALL")
     void quickUpload_returns400_whenBatchExceedsCap() throws Exception {
         when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
+        org.mockito.Mockito.doThrow(
+                org.raddatz.familienarchiv.exception.DomainException.badRequest(
+                        org.raddatz.familienarchiv.exception.ErrorCode.BATCH_TOO_LARGE, "Batch exceeds maximum of 50 files per request"))
+                .when(documentService).validateBatch(eq(51), any());
 
         var builder = multipart("/api/documents/quick-upload");
         for (int i = 0; i < 51; i++) {
diff --git a/backend/src/test/java/org/raddatz/familienarchiv/service/DocumentServiceTest.java b/backend/src/test/java/org/raddatz/familienarchiv/service/DocumentServiceTest.java
index 0e46196f..052658a8 100644
--- a/backend/src/test/java/org/raddatz/familienarchiv/service/DocumentServiceTest.java
+++ b/backend/src/test/java/org/raddatz/familienarchiv/service/DocumentServiceTest.java
@@ -1813,4 +1813,29 @@ class DocumentServiceTest {
 
         verify(auditService).logAfterCommit(eq(AuditKind.FILE_UPLOADED), isNull(), eq(id), isNull());
     }
+
+    // ─── validateBatch ───────────────────────────────────────────────────────
+
+    @Test
+    void validateBatch_throwsBatchTooLarge_whenFileCountExceedsCap() {
+        assertThatThrownBy(() -> documentService.validateBatch(51, null))
+                .isInstanceOf(DomainException.class)
+                .hasMessageContaining("50");
+    }
+
+    @Test
+    void validateBatch_doesNotThrow_whenFileCountEqualsCapExactly() {
+        documentService.validateBatch(50, null);
+    }
+
+    @Test
+    void validateBatch_throwsValidationError_whenTitlesSizeExceedsFileCount() {
+        org.raddatz.familienarchiv.dto.DocumentBatchMetadataDTO metadata =
+                new org.raddatz.familienarchiv.dto.DocumentBatchMetadataDTO();
+        metadata.setTitles(java.util.List.of("A", "B", "C"));
+
+        assertThatThrownBy(() -> documentService.validateBatch(2, metadata))
+                .isInstanceOf(DomainException.class)
+                .hasMessageContaining("titles");
+    }
 }
-- 
2.49.1


From 1eb833f333cf10eb94c4f78a503df2a32a76f830 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 20:28:12 +0200
Subject: [PATCH 21/46] refactor(documents): change
 DocumentBatchMetadataDTO.tags from String to List<String> tagNames
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Replaces comma-delimited String with a proper JSON array field — callers no
longer need to pre-serialise. Service drops the split/trim/filter step and
passes tagNames directly to updateDocumentTags().

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../dto/DocumentBatchMetadataDTO.java         |  2 +-
 .../service/DocumentService.java              |  6 ++---
 .../controller/DocumentControllerTest.java    | 24 +++++++++++++++++++
 3 files changed, 27 insertions(+), 5 deletions(-)

diff --git a/backend/src/main/java/org/raddatz/familienarchiv/dto/DocumentBatchMetadataDTO.java b/backend/src/main/java/org/raddatz/familienarchiv/dto/DocumentBatchMetadataDTO.java
index 93e17fbf..399105d9 100644
--- a/backend/src/main/java/org/raddatz/familienarchiv/dto/DocumentBatchMetadataDTO.java
+++ b/backend/src/main/java/org/raddatz/familienarchiv/dto/DocumentBatchMetadataDTO.java
@@ -13,6 +13,6 @@ public class DocumentBatchMetadataDTO {
     private List<UUID> receiverIds;
     private LocalDate documentDate;
     private String location;
-    private String tags;
+    private List<String> tagNames;
     private Boolean metadataComplete;
 }
diff --git a/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java b/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java
index 01ea544d..dac82517 100644
--- a/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java
+++ b/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java
@@ -166,11 +166,9 @@ public class DocumentService {
         if (metadata.getMetadataComplete() != null) {
             doc.setMetadataComplete(metadata.getMetadataComplete());
         }
-        if (metadata.getTags() != null && !metadata.getTags().isBlank()) {
-            List<String> tagNames = Arrays.stream(metadata.getTags().split(","))
-                    .map(String::trim).filter(s -> !s.isEmpty()).toList();
+        if (metadata.getTagNames() != null && !metadata.getTagNames().isEmpty()) {
             UUID docId = doc.getId();
-            updateDocumentTags(docId, tagNames);
+            updateDocumentTags(docId, metadata.getTagNames());
             doc = documentRepository.findById(docId)
                     .orElseThrow(() -> DomainException.notFound(ErrorCode.DOCUMENT_NOT_FOUND, "Not found after batch metadata: " + docId));
         }
diff --git a/backend/src/test/java/org/raddatz/familienarchiv/controller/DocumentControllerTest.java b/backend/src/test/java/org/raddatz/familienarchiv/controller/DocumentControllerTest.java
index 828e8402..b80ed173 100644
--- a/backend/src/test/java/org/raddatz/familienarchiv/controller/DocumentControllerTest.java
+++ b/backend/src/test/java/org/raddatz/familienarchiv/controller/DocumentControllerTest.java
@@ -886,6 +886,30 @@ class DocumentControllerTest {
                 .andExpect(status().isBadRequest());
     }
 
+    @Test
+    @WithMockUser(authorities = "WRITE_ALL")
+    void quickUpload_withMetadata_tagNamesJsonArray_parsedCorrectly() throws Exception {
+        Document doc = Document.builder().id(UUID.randomUUID()).title("brief").originalFilename("brief.pdf").build();
+        when(userService.findByEmail(any())).thenReturn(AppUser.builder().id(UUID.randomUUID()).build());
+
+        org.mockito.ArgumentCaptor<DocumentBatchMetadataDTO> captor =
+                org.mockito.ArgumentCaptor.forClass(DocumentBatchMetadataDTO.class);
+        when(documentService.storeDocumentWithBatchMetadata(any(), captor.capture(), anyInt(), any()))
+                .thenReturn(new DocumentService.StoreResult(doc, true));
+
+        org.springframework.mock.web.MockMultipartFile file =
+                new org.springframework.mock.web.MockMultipartFile("files", "brief.pdf", "application/pdf", new byte[]{1});
+        org.springframework.mock.web.MockMultipartFile metadata =
+                new org.springframework.mock.web.MockMultipartFile("metadata", "metadata", "application/json",
+                        "{\"tagNames\":[\"Briefwechsel\",\"Krieg\"]}".getBytes());
+
+        mockMvc.perform(multipart("/api/documents/quick-upload").file(file).file(metadata))
+                .andExpect(status().isOk());
+
+        org.assertj.core.api.Assertions.assertThat(captor.getValue().getTagNames())
+                .containsExactly("Briefwechsel", "Krieg");
+    }
+
     @Test
     @WithMockUser(authorities = "WRITE_ALL")
     void quickUpload_returns400_whenBatchExceedsCap() throws Exception {
-- 
2.49.1


From f90d4b282e9067731abbfacb05f8325c2a0a0e00 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 20:30:15 +0200
Subject: [PATCH 22/46] refactor(documents): extract applyBatchMetadata private
 helper in DocumentService

storeDocumentWithBatchMetadata was a 30-line flat method mixing file storage
with metadata hydration. The private helper makes each concern visible at a
glance.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../raddatz/familienarchiv/service/DocumentService.java  | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java b/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java
index dac82517..06317642 100644
--- a/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java
+++ b/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java
@@ -146,8 +146,11 @@ public class DocumentService {
     public StoreResult storeDocumentWithBatchMetadata(
             MultipartFile file, DocumentBatchMetadataDTO metadata, int fileIndex, UUID actorId) throws IOException {
         StoreResult base = storeDocument(file, actorId);
-        Document doc = base.document();
+        Document doc = applyBatchMetadata(base.document(), metadata, fileIndex);
+        return new StoreResult(documentRepository.save(doc), base.isNew());
+    }
 
+    private Document applyBatchMetadata(Document doc, DocumentBatchMetadataDTO metadata, int fileIndex) {
         if (metadata.getTitles() != null && fileIndex < metadata.getTitles().size()) {
             doc.setTitle(metadata.getTitles().get(fileIndex));
         }
@@ -172,9 +175,7 @@ public class DocumentService {
             doc = documentRepository.findById(docId)
                     .orElseThrow(() -> DomainException.notFound(ErrorCode.DOCUMENT_NOT_FOUND, "Not found after batch metadata: " + docId));
         }
-
-        Document saved = documentRepository.save(doc);
-        return new StoreResult(saved, base.isNew());
+        return doc;
     }
 
     @Transactional
-- 
2.49.1


From 43122c20cbca756fc59be782ba58efa1dc2cbcad Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 20:38:39 +0200
Subject: [PATCH 23/46] fix(bulk-upload): i18n hardcoded strings in
 BulkDropZone and FileSwitcherStrip
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Add bulk_drop_desc, bulk_select_files, bulk_drop_zone_label, bulk_remove_file
  keys to de/en/es message files
- BulkDropZone: use m.bulk_drop_zone_label(), m.bulk_drop_desc(),
  m.bulk_select_files() — removes all hardcoded German
- FileSwitcherStrip: use m.bulk_remove_file() on × button; move aria-live
  from <ul> to a dedicated visually-hidden region above the strip (screen
  readers now announce changes without coupling the live region to the list)
- Spec: import FileEntry from component instead of re-declaring; use
  data-remove-id selector instead of hardcoded German aria-label

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/messages/de.json                              |  6 +++++-
 frontend/messages/en.json                              |  6 +++++-
 frontend/messages/es.json                              |  6 +++++-
 .../document/BulkDocumentEditLayout.svelte.spec.ts     |  4 ++--
 .../src/lib/components/document/BulkDropZone.svelte    | 10 +++-------
 .../lib/components/document/FileSwitcherStrip.svelte   |  5 +++--
 .../document/FileSwitcherStrip.svelte.spec.ts          |  9 +--------
 7 files changed, 24 insertions(+), 22 deletions(-)

diff --git a/frontend/messages/de.json b/frontend/messages/de.json
index 1a0a0fd0..38b52713 100644
--- a/frontend/messages/de.json
+++ b/frontend/messages/de.json
@@ -868,5 +868,9 @@
 	"bulk_file_error_chip_label": "Fehler beim Hochladen",
 	"bulk_upload_progress": "{done} von {total} hochgeladen",
 	"bulk_partial_success": "{created} erstellt, {failed} fehlgeschlagen",
-	"bulk_all_failed": "Alle Uploads fehlgeschlagen"
+	"bulk_all_failed": "Alle Uploads fehlgeschlagen",
+	"bulk_drop_desc": "Für jede Datei wird ein eigenes Dokument erstellt. Der Titel wird aus dem Dateinamen vorausgefüllt — alle anderen Felder gelten für alle gemeinsam.",
+	"bulk_select_files": "Dateien auswählen",
+	"bulk_drop_zone_label": "Dateien ablegen",
+	"bulk_remove_file": "Entfernen"
 }
diff --git a/frontend/messages/en.json b/frontend/messages/en.json
index 95c52c82..7fef157f 100644
--- a/frontend/messages/en.json
+++ b/frontend/messages/en.json
@@ -868,5 +868,9 @@
 	"bulk_file_error_chip_label": "Upload failed",
 	"bulk_upload_progress": "{done} of {total} uploaded",
 	"bulk_partial_success": "{created} created, {failed} failed",
-	"bulk_all_failed": "All uploads failed"
+	"bulk_all_failed": "All uploads failed",
+	"bulk_drop_desc": "A separate document is created for each file. The title is pre-filled from the filename — all other fields apply to all documents.",
+	"bulk_select_files": "Select files",
+	"bulk_drop_zone_label": "Drop files here",
+	"bulk_remove_file": "Remove"
 }
diff --git a/frontend/messages/es.json b/frontend/messages/es.json
index 61050e5d..9be45208 100644
--- a/frontend/messages/es.json
+++ b/frontend/messages/es.json
@@ -868,5 +868,9 @@
 	"bulk_file_error_chip_label": "Error al subir",
 	"bulk_upload_progress": "{done} de {total} subidos",
 	"bulk_partial_success": "{created} creados, {failed} fallidos",
-	"bulk_all_failed": "Todos los uploads fallaron"
+	"bulk_all_failed": "Todos los uploads fallaron",
+	"bulk_drop_desc": "Se crea un documento separado por archivo. El título se rellena desde el nombre del archivo — el resto de campos se aplican a todos.",
+	"bulk_select_files": "Seleccionar archivos",
+	"bulk_drop_zone_label": "Soltar archivos aquí",
+	"bulk_remove_file": "Eliminar"
 }
diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
index e063094d..1feaf4de 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
@@ -52,9 +52,9 @@ describe('BulkDocumentEditLayout', () => {
 			makeFile('file2.pdf')
 		]);
 
-		// Remove the chip for file1 via its "Entfernen" remove button (second × button)
+		// Remove the chip for file1 via its remove button (identified by data-remove-id)
 		const removeButtons = container.querySelectorAll<HTMLButtonElement>(
-			'[data-testid="file-switcher-strip"] button[aria-label="Entfernen"]'
+			'[data-testid="file-switcher-strip"] button[data-remove-id]'
 		);
 		expect(removeButtons.length).toBe(3);
 		removeButtons[1].click(); // remove file1
diff --git a/frontend/src/lib/components/document/BulkDropZone.svelte b/frontend/src/lib/components/document/BulkDropZone.svelte
index 376f4b01..2a8b8e9e 100644
--- a/frontend/src/lib/components/document/BulkDropZone.svelte
+++ b/frontend/src/lib/components/document/BulkDropZone.svelte
@@ -12,7 +12,7 @@ let isDragging = $state(false);
 
 <div
 	role="region"
-	aria-label="Dateien ablegen"
+	aria-label={m.bulk_drop_zone_label()}
 	data-testid="bulk-drop-zone"
 	class="flex flex-1 flex-col items-center justify-center p-6"
 	ondragover={(e) => {
@@ -54,17 +54,13 @@ let isDragging = $state(false);
 		<p class="font-serif text-base font-bold text-ink">{m.bulk_drop_hint()}</p>
 
 		<!-- Sub description -->
-		<p class="text-sm leading-relaxed text-ink-2">
-			Für jede Datei wird ein eigenes Dokument erstellt.<br />
-			<strong class="text-ink">Der Titel</strong> wird aus dem Dateinamen vorausgefüllt —
-			<strong class="text-ink">alle anderen Felder</strong> gelten für alle gemeinsam.
-		</p>
+		<p class="text-sm leading-relaxed text-ink-2">{m.bulk_drop_desc()}</p>
 
 		<!-- CTA button -->
 		<label
 			class="flex min-h-[44px] cursor-pointer items-center rounded-sm bg-primary px-6 py-2 text-xs font-bold tracking-widest text-primary-fg uppercase transition-opacity hover:opacity-90"
 		>
-			Dateien auswählen
+			{m.bulk_select_files()}
 			<input
 				type="file"
 				multiple
diff --git a/frontend/src/lib/components/document/FileSwitcherStrip.svelte b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
index eef08b87..67a17ac2 100644
--- a/frontend/src/lib/components/document/FileSwitcherStrip.svelte
+++ b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
@@ -58,6 +58,7 @@ $effect(() => {
 });
 </script>
 
+<div aria-live="polite" aria-atomic="true" class="sr-only"></div>
 <div
 	data-testid="file-switcher-strip"
 	class="flex h-11 shrink-0 items-center gap-1 border-t border-line bg-pdf-ctrl px-2"
@@ -71,7 +72,7 @@ $effect(() => {
 	>
 
 	<div bind:this={trackEl} class="flex flex-1 gap-1 overflow-x-auto" style="scrollbar-width:none">
-		<ul bind:this={listEl} aria-live="polite" role="list" class="flex flex-row gap-1 py-1">
+		<ul bind:this={listEl} role="list" class="flex flex-row gap-1 py-1">
 			{#each files as entry, i (entry.id)}
 				<li role="listitem" class="inline-flex shrink-0 items-center">
 					<button
@@ -102,7 +103,7 @@ $effect(() => {
 					</button>
 					<button
 						type="button"
-						aria-label="Entfernen"
+						aria-label={m.bulk_remove_file()}
 						data-remove-id={entry.id}
 						onclick={() => onRemove(entry.id)}
 						class="ml-0.5 flex h-[44px] w-[44px] items-center justify-center text-base text-ink-3 hover:text-ink focus:outline-none focus-visible:ring-2 focus-visible:ring-accent"
diff --git a/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts b/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts
index 56fe281a..bb069e86 100644
--- a/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts
+++ b/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts
@@ -2,17 +2,10 @@ import { describe, it, expect, vi, afterEach } from 'vitest';
 import { cleanup, render } from 'vitest-browser-svelte';
 import { page, userEvent } from 'vitest/browser';
 import FileSwitcherStrip from './FileSwitcherStrip.svelte';
+import type { FileEntry } from './FileSwitcherStrip.svelte';
 
 afterEach(cleanup);
 
-export interface FileEntry {
-	id: string;
-	file: File;
-	title: string;
-	status: 'idle' | 'error';
-	previewUrl: string;
-}
-
 function makeFiles(n: number): FileEntry[] {
 	return Array.from({ length: n }, (_, i) => ({
 		id: `id-${i}`,
-- 
2.49.1


From 811baf78da49a606ae8b4f04289098618bab70fa Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 20:41:14 +0200
Subject: [PATCH 24/46] test(bulk-upload): add save-error and discard-all
 coverage to BulkDocumentEditLayout spec
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- save error path: server returns non-ok → fetch is called (error handling wired)
- discard-all: N=2 → click topbar button → N=0 drop-zone restored, switcher gone
- Add data-testid="discard-all-btn" to topbar discard button for reliable selection

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../document/BulkDocumentEditLayout.svelte    |  1 +
 .../BulkDocumentEditLayout.svelte.spec.ts     | 42 +++++++++++++++++++
 2 files changed, 43 insertions(+)

diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
index c1ece8b1..fa0e729e 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
@@ -152,6 +152,7 @@ async function save() {
 				</span>
 				<button
 					type="button"
+					data-testid="discard-all-btn"
 					onclick={discardAll}
 					class="text-xs font-medium text-red-600/70 hover:text-red-700"
 				>
diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
index 1feaf4de..7195c3e3 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
@@ -96,4 +96,46 @@ describe('BulkDocumentEditLayout', () => {
 		// Wait for async save to complete
 		await vi.waitFor(() => expect(mockFetch).toHaveBeenCalledTimes(2), { timeout: 3000 });
 	});
+
+	it('save marks file as error when server returns non-ok response', async () => {
+		const mockFetch = vi.fn().mockResolvedValue({
+			ok: false,
+			json: async () => ({ errors: [{ filename: 'f0.pdf', code: 'FILE_UPLOAD_FAILED' }] })
+		});
+		vi.stubGlobal('fetch', mockFetch);
+		vi.mock('$app/navigation', () => ({ goto: vi.fn() }));
+
+		const { container } = render(BulkDocumentEditLayout, {});
+		await addFilesViaInput(container, [makeFile('f0.pdf')]);
+
+		const saveBtn = container.querySelector(
+			'button[data-testid="bulk-save-btn"]'
+		) as HTMLButtonElement;
+		saveBtn.click();
+
+		await vi.waitFor(() => expect(mockFetch).toHaveBeenCalledTimes(1), { timeout: 3000 });
+	});
+
+	it('discard-all resets to N=0 state and shows drop zone', async () => {
+		const { container } = render(BulkDocumentEditLayout, {});
+		await addFilesViaInput(container, [makeFile('a.pdf'), makeFile('b.pdf')]);
+
+		// Confirm N=2 state — switcher is visible
+		expect(container.querySelector('[data-testid="file-switcher-strip"]')).not.toBeNull();
+
+		// Click the topbar discard-all button (only visible in isMulti state)
+		const discardBtn = container.querySelector(
+			'button[data-testid="discard-all-btn"]'
+		) as HTMLButtonElement;
+		expect(discardBtn).not.toBeNull();
+		discardBtn.click();
+
+		await vi.waitFor(
+			() => {
+				expect(container.querySelector('[data-testid="bulk-drop-zone"]')).not.toBeNull();
+				expect(container.querySelector('[data-testid="file-switcher-strip"]')).toBeNull();
+			},
+			{ timeout: 1000 }
+		);
+	});
 });
-- 
2.49.1


From f75a9601796b07e19c22ba2ef29dab2b68d23acc Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 21:12:02 +0200
Subject: [PATCH 25/46] fix(bulk-upload): include tagNames in quick-upload
 metadata payload

Tags were silently dropped because the metadata object built in save()
never included a tagNames field; they never reached the backend.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../document/BulkDocumentEditLayout.svelte    |  3 ++-
 .../BulkDocumentEditLayout.svelte.spec.ts     | 24 +++++++++++++++++++
 2 files changed, 26 insertions(+), 1 deletion(-)

diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
index fa0e729e..594a9245 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
@@ -100,7 +100,8 @@ async function save() {
 			titles: chunk.map((e) => e.title),
 			senderId: senderId || null,
 			receiverIds: selectedReceivers.map((r) => r.id),
-			documentDate: dateIso || null
+			documentDate: dateIso || null,
+			tagNames: tags.map((t) => t.name)
 		};
 		formData.append('metadata', new Blob([JSON.stringify(metadata)], { type: 'application/json' }));
 		const res = await fetch('/api/documents/quick-upload', { method: 'POST', body: formData });
diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
index 7195c3e3..e61bec16 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
@@ -116,6 +116,30 @@ describe('BulkDocumentEditLayout', () => {
 		await vi.waitFor(() => expect(mockFetch).toHaveBeenCalledTimes(1), { timeout: 3000 });
 	});
 
+	it('save() includes tagNames in metadata payload', async () => {
+		let capturedFormData: FormData | undefined;
+		const mockFetch = vi.fn().mockImplementation(async (_url: string, init: RequestInit) => {
+			capturedFormData = init?.body as FormData;
+			return { ok: true, json: async () => ({ created: [], updated: [], errors: [] }) };
+		});
+		vi.stubGlobal('fetch', mockFetch);
+
+		const { container } = render(BulkDocumentEditLayout, {});
+		await addFilesViaInput(container, [makeFile('doc.pdf')]);
+
+		const saveBtn = container.querySelector(
+			'button[data-testid="bulk-save-btn"]'
+		) as HTMLButtonElement;
+		saveBtn.click();
+
+		await vi.waitFor(() => expect(mockFetch).toHaveBeenCalledTimes(1), { timeout: 3000 });
+
+		expect(capturedFormData).toBeDefined();
+		const metadataBlob = capturedFormData!.get('metadata') as Blob;
+		const metadataJson = JSON.parse(await metadataBlob.text());
+		expect(metadataJson).toHaveProperty('tagNames');
+	});
+
 	it('discard-all resets to N=0 state and shows drop zone', async () => {
 		const { container } = render(BulkDocumentEditLayout, {});
 		await addFilesViaInput(container, [makeFile('a.pdf'), makeFile('b.pdf')]);
-- 
2.49.1


From 3c99030546a5e34e9251c604346c1040b338ad4d Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 21:18:17 +0200
Subject: [PATCH 26/46] fix(bulk-upload): skip navigation when any chunk fails
 to upload

goto('/documents') fired unconditionally, discarding error chips and
leaving the user with no feedback on which files failed. Now only
navigates when hadErrors is false after all chunks complete.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../document/BulkDocumentEditLayout.svelte    |  4 ++-
 .../BulkDocumentEditLayout.svelte.spec.ts     | 27 ++++++++++++++++---
 2 files changed, 26 insertions(+), 5 deletions(-)

diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
index 594a9245..14e9fe51 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
@@ -92,6 +92,7 @@ async function save() {
 	}
 	chunkProgress = { done: 0, total: chunks.length };
 
+	let hadErrors = false;
 	for (let i = 0; i < chunks.length; i++) {
 		const chunk = chunks[i];
 		const formData = new FormData();
@@ -106,6 +107,7 @@ async function save() {
 		formData.append('metadata', new Blob([JSON.stringify(metadata)], { type: 'application/json' }));
 		const res = await fetch('/api/documents/quick-upload', { method: 'POST', body: formData });
 		if (!res.ok) {
+			hadErrors = true;
 			const body = await res.json().catch(() => ({ errors: [] }));
 			const errorCount = (body.errors ?? []).length;
 			for (let j = 0; j < errorCount && j < chunk.length; j++) {
@@ -115,7 +117,7 @@ async function save() {
 		}
 		chunkProgress = { done: i + 1, total: chunks.length };
 	}
-	goto('/documents');
+	if (!hadErrors) goto('/documents');
 }
 </script>
 
diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
index e61bec16..c5dc5e85 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
@@ -1,11 +1,15 @@
 import { describe, it, expect, vi, afterEach } from 'vitest';
+import { goto } from '$app/navigation';
 import { cleanup, render } from 'vitest-browser-svelte';
 import { page, userEvent } from 'vitest/browser';
 import BulkDocumentEditLayout from './BulkDocumentEditLayout.svelte';
 
+vi.mock('$app/navigation', () => ({ goto: vi.fn() }));
+
 afterEach(() => {
 	cleanup();
 	vi.unstubAllGlobals();
+	vi.clearAllMocks();
 });
 
 function makeFile(name: string): File {
@@ -80,9 +84,6 @@ describe('BulkDocumentEditLayout', () => {
 		});
 		vi.stubGlobal('fetch', mockFetch);
 
-		// Also stub goto to prevent navigation errors in test
-		vi.mock('$app/navigation', () => ({ goto: vi.fn() }));
-
 		const { container } = render(BulkDocumentEditLayout, {});
 		const files = Array.from({ length: 12 }, (_, i) => makeFile(`f${i}.pdf`));
 		await addFilesViaInput(container, files);
@@ -103,7 +104,6 @@ describe('BulkDocumentEditLayout', () => {
 			json: async () => ({ errors: [{ filename: 'f0.pdf', code: 'FILE_UPLOAD_FAILED' }] })
 		});
 		vi.stubGlobal('fetch', mockFetch);
-		vi.mock('$app/navigation', () => ({ goto: vi.fn() }));
 
 		const { container } = render(BulkDocumentEditLayout, {});
 		await addFilesViaInput(container, [makeFile('f0.pdf')]);
@@ -140,6 +140,25 @@ describe('BulkDocumentEditLayout', () => {
 		expect(metadataJson).toHaveProperty('tagNames');
 	});
 
+	it('save() does not navigate when chunk returns non-ok response', async () => {
+		const mockFetch = vi.fn().mockResolvedValue({
+			ok: false,
+			json: async () => ({ errors: [{ filename: 'f0.pdf', code: 'FILE_UPLOAD_FAILED' }] })
+		});
+		vi.stubGlobal('fetch', mockFetch);
+
+		const { container } = render(BulkDocumentEditLayout, {});
+		await addFilesViaInput(container, [makeFile('f0.pdf')]);
+
+		const saveBtn = container.querySelector(
+			'button[data-testid="bulk-save-btn"]'
+		) as HTMLButtonElement;
+		saveBtn.click();
+
+		await vi.waitFor(() => expect(mockFetch).toHaveBeenCalledTimes(1), { timeout: 3000 });
+		expect(goto).not.toHaveBeenCalled();
+	});
+
 	it('discard-all resets to N=0 state and shows drop zone', async () => {
 		const { container } = render(BulkDocumentEditLayout, {});
 		await addFilesViaInput(container, [makeFile('a.pdf'), makeFile('b.pdf')]);
-- 
2.49.1


From a2a7d547ee022b027e008a46da39e936b4c44070 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 21:21:07 +0200
Subject: [PATCH 27/46] fix(bulk-upload): i18n topbar title; replace hardcoded
 German strings

'Neues Dokument' / 'Neue Dokumente' in BulkDocumentEditLayout topbar
bypassed Paraglide. Added bulk_title_single and bulk_title_multi keys
to de/en/es message files and switched to m.*() calls.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/messages/de.json                                     | 4 +++-
 frontend/messages/en.json                                     | 4 +++-
 frontend/messages/es.json                                     | 4 +++-
 .../src/lib/components/document/BulkDocumentEditLayout.svelte | 2 +-
 4 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/frontend/messages/de.json b/frontend/messages/de.json
index 38b52713..c20394a1 100644
--- a/frontend/messages/de.json
+++ b/frontend/messages/de.json
@@ -872,5 +872,7 @@
 	"bulk_drop_desc": "Für jede Datei wird ein eigenes Dokument erstellt. Der Titel wird aus dem Dateinamen vorausgefüllt — alle anderen Felder gelten für alle gemeinsam.",
 	"bulk_select_files": "Dateien auswählen",
 	"bulk_drop_zone_label": "Dateien ablegen",
-	"bulk_remove_file": "Entfernen"
+	"bulk_remove_file": "Entfernen",
+	"bulk_title_single": "Neues Dokument",
+	"bulk_title_multi": "Neue Dokumente"
 }
diff --git a/frontend/messages/en.json b/frontend/messages/en.json
index 7fef157f..a57ca05d 100644
--- a/frontend/messages/en.json
+++ b/frontend/messages/en.json
@@ -872,5 +872,7 @@
 	"bulk_drop_desc": "A separate document is created for each file. The title is pre-filled from the filename — all other fields apply to all documents.",
 	"bulk_select_files": "Select files",
 	"bulk_drop_zone_label": "Drop files here",
-	"bulk_remove_file": "Remove"
+	"bulk_remove_file": "Remove",
+	"bulk_title_single": "New Document",
+	"bulk_title_multi": "New Documents"
 }
diff --git a/frontend/messages/es.json b/frontend/messages/es.json
index 9be45208..a64424a0 100644
--- a/frontend/messages/es.json
+++ b/frontend/messages/es.json
@@ -872,5 +872,7 @@
 	"bulk_drop_desc": "Se crea un documento separado por archivo. El título se rellena desde el nombre del archivo — el resto de campos se aplican a todos.",
 	"bulk_select_files": "Seleccionar archivos",
 	"bulk_drop_zone_label": "Soltar archivos aquí",
-	"bulk_remove_file": "Eliminar"
+	"bulk_remove_file": "Eliminar",
+	"bulk_title_single": "Nuevo Documento",
+	"bulk_title_multi": "Nuevos Documentos"
 }
diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
index 14e9fe51..da4e6d3b 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
@@ -146,7 +146,7 @@ async function save() {
 		</a>
 		<span class="text-ink-3" aria-hidden="true">·</span>
 		<span class="font-serif text-sm font-bold text-ink">
-			{isMulti ? 'Neue Dokumente' : 'Neues Dokument'}
+			{isMulti ? m.bulk_title_multi() : m.bulk_title_single()}
 		</span>
 		{#if isMulti}
 			<span class="ml-auto flex items-center gap-3">
-- 
2.49.1


From 9ac01f7cc2c7d607f822a012acd4d1eaaa209ad2 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 21:22:49 +0200
Subject: [PATCH 28/46] fix(bulk-upload): add aria-label to progress bar in
 UploadSaveBar

<progress> had no accessible name, failing WCAG 1.3.1 and 4.1.2.
Labels it with the already-existing bulk_upload_progress i18n key.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/src/lib/components/document/UploadSaveBar.svelte | 1 +
 1 file changed, 1 insertion(+)

diff --git a/frontend/src/lib/components/document/UploadSaveBar.svelte b/frontend/src/lib/components/document/UploadSaveBar.svelte
index 06b85d99..7b53f602 100644
--- a/frontend/src/lib/components/document/UploadSaveBar.svelte
+++ b/frontend/src/lib/components/document/UploadSaveBar.svelte
@@ -19,6 +19,7 @@ let {
 		<progress
 			value={chunkProgress.done}
 			max={chunkProgress.total}
+			aria-label={m.bulk_upload_progress({ done: chunkProgress.done, total: chunkProgress.total })}
 			class="[&::-webkit-progress-bar]:bg-brand-sand mb-3 h-1 w-full rounded-full [&::-webkit-progress-bar]:rounded-full [&::-webkit-progress-value]:rounded-full [&::-webkit-progress-value]:bg-accent"
 		></progress>
 	{/if}
-- 
2.49.1


From e742c36ef6cd8f2aebd50662cf24eb27ed96df23 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 21:25:06 +0200
Subject: [PATCH 29/46] fix(bulk-upload): populate aria-live region with active
 file title

The sr-only aria-live div was always empty, so screen readers never
announced file switches. Derived activeAnnouncement from the active
entry and bound it to the div's text content.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/src/lib/components/document/FileSwitcherStrip.svelte | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/frontend/src/lib/components/document/FileSwitcherStrip.svelte b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
index 67a17ac2..ab1d7b7d 100644
--- a/frontend/src/lib/components/document/FileSwitcherStrip.svelte
+++ b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
@@ -24,6 +24,8 @@ let {
 let trackEl = $state<HTMLDivElement | null>(null);
 let listEl = $state<HTMLUListElement | null>(null);
 
+const activeAnnouncement = $derived(files.find((f) => f.id === activeId)?.title ?? '');
+
 function scrollPrev() {
 	trackEl?.scrollBy({ left: -120, behavior: 'smooth' });
 }
@@ -58,7 +60,7 @@ $effect(() => {
 });
 </script>
 
-<div aria-live="polite" aria-atomic="true" class="sr-only"></div>
+<div aria-live="polite" aria-atomic="true" class="sr-only">{activeAnnouncement}</div>
 <div
 	data-testid="file-switcher-strip"
 	class="flex h-11 shrink-0 items-center gap-1 border-t border-line bg-pdf-ctrl px-2"
-- 
2.49.1


From eb106c9ca74aeab1c2dc4e9067456e4e3c5745e2 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 21:26:49 +0200
Subject: [PATCH 30/46] =?UTF-8?q?fix(bulk-upload):=20enlarge=20scroll=20bu?=
 =?UTF-8?q?tton=20touch=20targets=20to=2044=C3=9744px?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Prev/next scroll buttons were 24×20px, below the WCAG 2.2 SC 2.5.5
minimum of 44×44px. Changed to h-[44px] w-[44px].

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/src/lib/components/document/FileSwitcherStrip.svelte | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/frontend/src/lib/components/document/FileSwitcherStrip.svelte b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
index ab1d7b7d..783c5fff 100644
--- a/frontend/src/lib/components/document/FileSwitcherStrip.svelte
+++ b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
@@ -69,7 +69,7 @@ $effect(() => {
 		type="button"
 		aria-label={m.bulk_switcher_prev()}
 		onclick={scrollPrev}
-		class="flex h-6 w-5 shrink-0 items-center justify-center rounded-sm text-sm text-ink-3 hover:bg-black/10 hover:text-ink focus:outline-none focus-visible:ring-2 focus-visible:ring-accent"
+		class="flex h-[44px] w-[44px] shrink-0 items-center justify-center rounded-sm text-sm text-ink-3 hover:bg-black/10 hover:text-ink focus:outline-none focus-visible:ring-2 focus-visible:ring-accent"
 		>‹</button
 	>
 
@@ -121,7 +121,7 @@ $effect(() => {
 		type="button"
 		aria-label={m.bulk_switcher_next()}
 		onclick={scrollNext}
-		class="flex h-6 w-5 shrink-0 items-center justify-center rounded-sm text-sm text-ink-3 hover:bg-black/10 hover:text-ink focus:outline-none focus-visible:ring-2 focus-visible:ring-accent"
+		class="flex h-[44px] w-[44px] shrink-0 items-center justify-center rounded-sm text-sm text-ink-3 hover:bg-black/10 hover:text-ink focus:outline-none focus-visible:ring-2 focus-visible:ring-accent"
 		>›</button
 	>
 </div>
-- 
2.49.1


From 5131c8da3131c49144d4f625368e9698648ebae5 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 21:30:38 +0200
Subject: [PATCH 31/46] fix(bulk-upload): truncate long chip titles with
 tooltip in FileSwitcherStrip

Long filenames caused chips to overflow the strip. Added max-w-[8rem]
and truncate on the title span, plus a title attribute for full text
on hover.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/src/lib/components/document/FileSwitcherStrip.svelte | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/lib/components/document/FileSwitcherStrip.svelte b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
index 783c5fff..d4ae1089 100644
--- a/frontend/src/lib/components/document/FileSwitcherStrip.svelte
+++ b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
@@ -101,7 +101,7 @@ $effect(() => {
 							].join(' ')}
 							>{i + 1}</span
 						>
-						{entry.title}
+						<span class="max-w-[8rem] truncate" title={entry.title}>{entry.title}</span>
 					</button>
 					<button
 						type="button"
-- 
2.49.1


From c78a1d69dc6abd9c9e5f20864bc0e29d6ec1c202 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Fri, 24 Apr 2026 21:40:17 +0200
Subject: [PATCH 32/46] test(bulk-upload): add unit tests for
 storeDocumentWithBatchMetadata

Covers four behaviours of applyBatchMetadata that had no coverage:
title applied by list index, sender resolved via PersonService,
tags applied via updateDocumentTags, and title left unchanged when
the fileIndex exceeds the titles list length.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../service/DocumentServiceTest.java          | 79 +++++++++++++++++++
 1 file changed, 79 insertions(+)

diff --git a/backend/src/test/java/org/raddatz/familienarchiv/service/DocumentServiceTest.java b/backend/src/test/java/org/raddatz/familienarchiv/service/DocumentServiceTest.java
index 052658a8..4c34862b 100644
--- a/backend/src/test/java/org/raddatz/familienarchiv/service/DocumentServiceTest.java
+++ b/backend/src/test/java/org/raddatz/familienarchiv/service/DocumentServiceTest.java
@@ -1814,6 +1814,85 @@ class DocumentServiceTest {
         verify(auditService).logAfterCommit(eq(AuditKind.FILE_UPLOADED), isNull(), eq(id), isNull());
     }
 
+    // ─── storeDocumentWithBatchMetadata ──────────────────────────────────────
+
+    private MockMultipartFile pdfFile(String name) {
+        return new MockMultipartFile("file", name, "application/pdf", new byte[]{1});
+    }
+
+    private void stubStoreDocument(String filename) throws Exception {
+        when(documentRepository.findFirstByOriginalFilename(filename)).thenReturn(Optional.empty());
+        when(fileService.uploadFile(any(), any())).thenReturn(new FileService.UploadResult("key", "hash"));
+    }
+
+    @Test
+    void storeDocumentWithBatchMetadata_appliesTitleByIndex() throws Exception {
+        stubStoreDocument("scan01.pdf");
+        when(documentRepository.save(any())).thenAnswer(inv -> inv.getArgument(0));
+
+        org.raddatz.familienarchiv.dto.DocumentBatchMetadataDTO meta = new org.raddatz.familienarchiv.dto.DocumentBatchMetadataDTO();
+        meta.setTitles(List.of("Erster Brief", "Zweiter Brief"));
+
+        DocumentService.StoreResult result = documentService.storeDocumentWithBatchMetadata(pdfFile("scan01.pdf"), meta, 0, null);
+
+        assertThat(result.document().getTitle()).isEqualTo("Erster Brief");
+    }
+
+    @Test
+    void storeDocumentWithBatchMetadata_resolvesSenderViaPersonService() throws Exception {
+        UUID senderId = UUID.randomUUID();
+        stubStoreDocument("scan02.pdf");
+        when(documentRepository.save(any())).thenAnswer(inv -> inv.getArgument(0));
+        Person sender = Person.builder().id(senderId).firstName("Anna").build();
+        when(personService.getById(senderId)).thenReturn(sender);
+
+        org.raddatz.familienarchiv.dto.DocumentBatchMetadataDTO meta = new org.raddatz.familienarchiv.dto.DocumentBatchMetadataDTO();
+        meta.setSenderId(senderId);
+
+        DocumentService.StoreResult result = documentService.storeDocumentWithBatchMetadata(pdfFile("scan02.pdf"), meta, 0, null);
+
+        assertThat(result.document().getSender()).isEqualTo(sender);
+    }
+
+    @Test
+    void storeDocumentWithBatchMetadata_appliesTagsViaUpdateDocumentTags() throws Exception {
+        UUID docId = UUID.randomUUID();
+        when(documentRepository.findFirstByOriginalFilename("scan03.pdf")).thenReturn(Optional.empty());
+        when(fileService.uploadFile(any(), any())).thenReturn(new FileService.UploadResult("key", "hash"));
+        when(documentRepository.save(any())).thenAnswer(inv -> {
+            Document d = inv.getArgument(0);
+            if (d.getId() == null) d.setId(docId);
+            return d;
+        });
+        when(documentRepository.findById(docId)).thenAnswer(inv -> {
+            Document d = new Document();
+            d.setId(docId);
+            return Optional.of(d);
+        });
+        Tag tag = Tag.builder().id(UUID.randomUUID()).name("Familie").build();
+        when(tagService.findOrCreate("Familie")).thenReturn(tag);
+
+        org.raddatz.familienarchiv.dto.DocumentBatchMetadataDTO meta = new org.raddatz.familienarchiv.dto.DocumentBatchMetadataDTO();
+        meta.setTagNames(List.of("Familie"));
+
+        documentService.storeDocumentWithBatchMetadata(pdfFile("scan03.pdf"), meta, 0, null);
+
+        verify(tagService).findOrCreate("Familie");
+    }
+
+    @Test
+    void storeDocumentWithBatchMetadata_leavesTitle_whenIndexExceedsTitlesList() throws Exception {
+        stubStoreDocument("scan04.pdf");
+        when(documentRepository.save(any())).thenAnswer(inv -> inv.getArgument(0));
+
+        org.raddatz.familienarchiv.dto.DocumentBatchMetadataDTO meta = new org.raddatz.familienarchiv.dto.DocumentBatchMetadataDTO();
+        meta.setTitles(List.of("Only One Title"));
+
+        DocumentService.StoreResult result = documentService.storeDocumentWithBatchMetadata(pdfFile("scan04.pdf"), meta, 5, null);
+
+        assertThat(result.document().getTitle()).isEqualTo("scan04");
+    }
+
     // ─── validateBatch ───────────────────────────────────────────────────────
 
     @Test
-- 
2.49.1


From f1b3e8c2d8c481e822f051216bc8030e87d72496 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Sat, 25 Apr 2026 01:07:44 +0200
Subject: [PATCH 33/46] fix(i18n): remove orphaned merge conflict markers from
 message files

All three message files had a bare `<<<<<<< HEAD` at line 814 with no
corresponding separator or closing marker, making them invalid JSON and
breaking the Paraglide build.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/messages/de.json | 1 -
 frontend/messages/en.json | 1 -
 frontend/messages/es.json | 1 -
 3 files changed, 3 deletions(-)

diff --git a/frontend/messages/de.json b/frontend/messages/de.json
index c20394a1..25ad5edb 100644
--- a/frontend/messages/de.json
+++ b/frontend/messages/de.json
@@ -811,7 +811,6 @@
 	"pagination_next": "Weiter",
 	"pagination_page_of": "Seite {page} von {total}",
 	"pagination_nav_label": "Seitennavigation",
-<<<<<<< HEAD
 
 	"common_opens_new_tab": "(öffnet in neuem Tab)",
 
diff --git a/frontend/messages/en.json b/frontend/messages/en.json
index a57ca05d..523fa7af 100644
--- a/frontend/messages/en.json
+++ b/frontend/messages/en.json
@@ -811,7 +811,6 @@
 	"pagination_next": "Next",
 	"pagination_page_of": "Page {page} of {total}",
 	"pagination_nav_label": "Pagination",
-<<<<<<< HEAD
 
 	"common_opens_new_tab": "(opens in new tab)",
 
diff --git a/frontend/messages/es.json b/frontend/messages/es.json
index a64424a0..b725d39b 100644
--- a/frontend/messages/es.json
+++ b/frontend/messages/es.json
@@ -811,7 +811,6 @@
 	"pagination_next": "Siguiente",
 	"pagination_page_of": "Página {page} de {total}",
 	"pagination_nav_label": "Paginación",
-<<<<<<< HEAD
 
 	"common_opens_new_tab": "(abre en pestaña nueva)",
 
-- 
2.49.1


From 74b473e3d7a9196037a4ef632888f3f5e887f603 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Sat, 25 Apr 2026 01:15:41 +0200
Subject: [PATCH 34/46] fix(bulk-upload): match error chips by filename, not by
 chunk position

save() was marking the first N files in a chunk as errored (where N = the
error count returned by the backend), but the backend errors are keyed by
filename. A failure for file[2] would incorrectly mark file[0] as the error.

Now builds a Set of error filenames and matches chunk entries by file.name.
Test added: save marks only the file whose filename matches the backend error.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../document/BulkDocumentEditLayout.svelte    | 12 +++++---
 .../BulkDocumentEditLayout.svelte.spec.ts     | 28 +++++++++++++++++++
 2 files changed, 36 insertions(+), 4 deletions(-)

diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
index da4e6d3b..3959a54e 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
@@ -109,10 +109,14 @@ async function save() {
 		if (!res.ok) {
 			hadErrors = true;
 			const body = await res.json().catch(() => ({ errors: [] }));
-			const errorCount = (body.errors ?? []).length;
-			for (let j = 0; j < errorCount && j < chunk.length; j++) {
-				const e = files.get(chunk[j].id);
-				if (e) files.set(chunk[j].id, { ...e, status: 'error' });
+			const errorFilenames = new Set<string>(
+				(body.errors ?? []).map((err: { filename: string }) => err.filename)
+			);
+			for (const entry of chunk) {
+				if (errorFilenames.has(entry.file.name)) {
+					const e = files.get(entry.id);
+					if (e) files.set(entry.id, { ...e, status: 'error' });
+				}
 			}
 		}
 		chunkProgress = { done: i + 1, total: chunks.length };
diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
index c5dc5e85..b800d218 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
@@ -159,6 +159,34 @@ describe('BulkDocumentEditLayout', () => {
 		expect(goto).not.toHaveBeenCalled();
 	});
 
+	it('save marks only the file whose filename matches the backend error, not adjacent files', async () => {
+		// backend returns error keyed to b.pdf — only b.pdf chip should get data-status="error"
+		const mockFetch = vi.fn().mockResolvedValue({
+			ok: false,
+			json: async () => ({ errors: [{ filename: 'b.pdf', code: 'FILE_UPLOAD_FAILED' }] })
+		});
+		vi.stubGlobal('fetch', mockFetch);
+
+		const { container } = render(BulkDocumentEditLayout, {});
+		await addFilesViaInput(container, [makeFile('a.pdf'), makeFile('b.pdf'), makeFile('c.pdf')]);
+
+		const saveBtn = container.querySelector(
+			'button[data-testid="bulk-save-btn"]'
+		) as HTMLButtonElement;
+		saveBtn.click();
+
+		await vi.waitFor(() => expect(mockFetch).toHaveBeenCalledTimes(1), { timeout: 3000 });
+
+		await vi.waitFor(
+			() => {
+				const errorChips = container.querySelectorAll('[data-chip-id][data-status="error"]');
+				expect(errorChips.length).toBe(1);
+				expect(errorChips[0].textContent).toContain('b');
+			},
+			{ timeout: 1000 }
+		);
+	});
+
 	it('discard-all resets to N=0 state and shows drop zone', async () => {
 		const { container } = render(BulkDocumentEditLayout, {});
 		await addFilesViaInput(container, [makeFile('a.pdf'), makeFile('b.pdf')]);
-- 
2.49.1


From 6b10daeeac8e7f3d6f15a33113ebdb8c685d49d1 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Sat, 25 Apr 2026 01:25:03 +0200
Subject: [PATCH 35/46] fix(bulk-upload): accessibility improvements and fetch
 comment

- BulkDropZone: link description <p> to drop zone region via aria-describedby
- UploadSaveBar: add explicit aria-valuenow/aria-valuemin/aria-valuemax to
  <progress> element for consistent screen reader support across browsers
- FileSwitcherStrip: add non-color error indicator (red !) to error chips so
  error state is not communicated by color alone (WCAG 1.4.1)
- BulkDocumentEditLayout: comment explaining why raw fetch is used instead of
  a SvelteKit form action (chunked FormData with per-chunk progress tracking)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../src/lib/components/document/BulkDocumentEditLayout.svelte  | 3 +++
 frontend/src/lib/components/document/BulkDropZone.svelte       | 3 ++-
 frontend/src/lib/components/document/FileSwitcherStrip.svelte  | 3 +++
 frontend/src/lib/components/document/UploadSaveBar.svelte      | 3 +++
 4 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
index 3959a54e..7a171e55 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
@@ -105,6 +105,9 @@ async function save() {
 			tagNames: tags.map((t) => t.name)
 		};
 		formData.append('metadata', new Blob([JSON.stringify(metadata)], { type: 'application/json' }));
+		// Raw fetch is intentional: SvelteKit form actions can't stream chunked
+		// FormData with per-chunk progress. Session cookie is sent automatically
+		// by the browser for same-origin requests.
 		const res = await fetch('/api/documents/quick-upload', { method: 'POST', body: formData });
 		if (!res.ok) {
 			hadErrors = true;
diff --git a/frontend/src/lib/components/document/BulkDropZone.svelte b/frontend/src/lib/components/document/BulkDropZone.svelte
index 2a8b8e9e..204b12fc 100644
--- a/frontend/src/lib/components/document/BulkDropZone.svelte
+++ b/frontend/src/lib/components/document/BulkDropZone.svelte
@@ -13,6 +13,7 @@ let isDragging = $state(false);
 <div
 	role="region"
 	aria-label={m.bulk_drop_zone_label()}
+	aria-describedby="bulk-drop-desc"
 	data-testid="bulk-drop-zone"
 	class="flex flex-1 flex-col items-center justify-center p-6"
 	ondragover={(e) => {
@@ -54,7 +55,7 @@ let isDragging = $state(false);
 		<p class="font-serif text-base font-bold text-ink">{m.bulk_drop_hint()}</p>
 
 		<!-- Sub description -->
-		<p class="text-sm leading-relaxed text-ink-2">{m.bulk_drop_desc()}</p>
+		<p id="bulk-drop-desc" class="text-sm leading-relaxed text-ink-2">{m.bulk_drop_desc()}</p>
 
 		<!-- CTA button -->
 		<label
diff --git a/frontend/src/lib/components/document/FileSwitcherStrip.svelte b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
index d4ae1089..5598fd42 100644
--- a/frontend/src/lib/components/document/FileSwitcherStrip.svelte
+++ b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
@@ -102,6 +102,9 @@ $effect(() => {
 							>{i + 1}</span
 						>
 						<span class="max-w-[8rem] truncate" title={entry.title}>{entry.title}</span>
+						{#if entry.status === 'error'}
+							<span aria-hidden="true" class="ml-0.5 font-extrabold text-red-600">!</span>
+						{/if}
 					</button>
 					<button
 						type="button"
diff --git a/frontend/src/lib/components/document/UploadSaveBar.svelte b/frontend/src/lib/components/document/UploadSaveBar.svelte
index 7b53f602..ac0f0b92 100644
--- a/frontend/src/lib/components/document/UploadSaveBar.svelte
+++ b/frontend/src/lib/components/document/UploadSaveBar.svelte
@@ -19,6 +19,9 @@ let {
 		<progress
 			value={chunkProgress.done}
 			max={chunkProgress.total}
+			aria-valuenow={chunkProgress.done}
+			aria-valuemin={0}
+			aria-valuemax={chunkProgress.total}
 			aria-label={m.bulk_upload_progress({ done: chunkProgress.done, total: chunkProgress.total })}
 			class="[&::-webkit-progress-bar]:bg-brand-sand mb-3 h-1 w-full rounded-full [&::-webkit-progress-bar]:rounded-full [&::-webkit-progress-value]:rounded-full [&::-webkit-progress-value]:bg-accent"
 		></progress>
-- 
2.49.1


From 8cf4f7c2e418579d46c6d079264fba391d1dbeea Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Sat, 25 Apr 2026 09:03:57 +0200
Subject: [PATCH 36/46] test(bulk-upload): fix ScopeCard spec assertions to
 match actual component classes

/brand-mint/ never matched (component uses border-accent bg-accent-bg);
companion test also updated to assert the meaningful negative.
getByText('5') fixed to exact:true to avoid strict-mode ambiguity.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../lib/components/document/ScopeCard.svelte.spec.ts   | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/frontend/src/lib/components/document/ScopeCard.svelte.spec.ts b/frontend/src/lib/components/document/ScopeCard.svelte.spec.ts
index 51c4ae1b..9a04b7d9 100644
--- a/frontend/src/lib/components/document/ScopeCard.svelte.spec.ts
+++ b/frontend/src/lib/components/document/ScopeCard.svelte.spec.ts
@@ -6,21 +6,21 @@ import ScopeCard from './ScopeCard.svelte';
 afterEach(cleanup);
 
 describe('ScopeCard', () => {
-	it('per-file variant has mint background class', async () => {
+	it('per-file variant has accent background class', async () => {
 		const { container } = render(ScopeCard, { variant: 'per-file', count: 1 });
 		const card = container.querySelector('[data-testid="scope-card"]');
-		expect(card?.className).toMatch(/brand-mint/);
+		expect(card?.className).toMatch(/bg-accent-bg/);
 	});
 
-	it('shared variant does not have mint background', async () => {
+	it('shared variant does not have accent background', async () => {
 		const { container } = render(ScopeCard, { variant: 'shared', count: 3 });
 		const card = container.querySelector('[data-testid="scope-card"]');
-		expect(card?.className).not.toMatch(/bg-brand-mint/);
+		expect(card?.className).not.toMatch(/bg-accent-bg/);
 	});
 
 	it('shared variant renders count badge with file count', async () => {
 		render(ScopeCard, { variant: 'shared', count: 5 });
-		await expect.element(page.getByText('5')).toBeInTheDocument();
+		await expect.element(page.getByText('5', { exact: true })).toBeInTheDocument();
 	});
 
 	it('per-file variant renders slot content', async () => {
-- 
2.49.1


From 60f1db1f99ad1dd496db8431a6b59af40d8abb04 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Sat, 25 Apr 2026 09:08:10 +0200
Subject: [PATCH 37/46] fix(bulk-upload): announce error chip status to screen
 readers

The ! indicator was aria-hidden with no sr-only fallback, making failed
uploads invisible to assistive technology. Added sr-only span with
bulk_file_error_chip_label before the visual indicator.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../document/FileSwitcherStrip.svelte         |  1 +
 .../document/FileSwitcherStrip.svelte.spec.ts | 21 +++++++++++++++++++
 2 files changed, 22 insertions(+)

diff --git a/frontend/src/lib/components/document/FileSwitcherStrip.svelte b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
index 5598fd42..d70241bc 100644
--- a/frontend/src/lib/components/document/FileSwitcherStrip.svelte
+++ b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
@@ -103,6 +103,7 @@ $effect(() => {
 						>
 						<span class="max-w-[8rem] truncate" title={entry.title}>{entry.title}</span>
 						{#if entry.status === 'error'}
+							<span class="sr-only">{m.bulk_file_error_chip_label()}</span>
 							<span aria-hidden="true" class="ml-0.5 font-extrabold text-red-600">!</span>
 						{/if}
 					</button>
diff --git a/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts b/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts
index bb069e86..6a6ccc83 100644
--- a/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts
+++ b/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts
@@ -78,6 +78,27 @@ describe('FileSwitcherStrip', () => {
 		expect(errBtn).not.toBeNull();
 	});
 
+	it('error chip contains a screen-reader-only error label', async () => {
+		const files: FileEntry[] = [
+			{
+				id: 'e1',
+				file: new File([''], 'bad.pdf'),
+				title: 'Bad file',
+				status: 'error',
+				previewUrl: ''
+			}
+		];
+		const { container } = render(FileSwitcherStrip, {
+			files,
+			activeId: 'e1',
+			onSelect: vi.fn(),
+			onRemove: vi.fn()
+		});
+		const errBtn = container.querySelector('[data-status="error"]');
+		const srOnly = errBtn?.querySelector('.sr-only');
+		expect(srOnly).not.toBeNull();
+	});
+
 	it('ArrowRight moves focus to next chip without leaving strip', async () => {
 		const files = makeFiles(3);
 		const { container } = render(FileSwitcherStrip, {
-- 
2.49.1


From 262c792654e067a2ed6b679d64e3005fc9ae1df0 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Sat, 25 Apr 2026 09:10:29 +0200
Subject: [PATCH 38/46] fix(bulk-upload): correct stale
 DocumentBatchMetadataDTO type in api.ts

Generated type had tags?: string but Java DTO declares List<String> tagNames.
Corrected to tagNames?: string[] to match the backend contract.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/src/lib/generated/api.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/lib/generated/api.ts b/frontend/src/lib/generated/api.ts
index 2db578c9..3c2e8036 100644
--- a/frontend/src/lib/generated/api.ts
+++ b/frontend/src/lib/generated/api.ts
@@ -1695,7 +1695,7 @@ export interface components {
             /** Format: date */
             documentDate?: string;
             location?: string;
-            tags?: string;
+            tagNames?: string[];
             metadataComplete?: boolean;
         };
         QuickUploadResult: {
-- 
2.49.1


From cb9962f0c27ee05a1fe47303640b4450d4c91434 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Sat, 25 Apr 2026 09:12:21 +0200
Subject: [PATCH 39/46] test(bulk-upload): add positive navigation assertion
 for successful save

The error-path test (goto not called on failure) had no matching positive
assertion. Added: save() navigates to /documents when all chunks succeed.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../BulkDocumentEditLayout.svelte.spec.ts     | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
index b800d218..32e8fdef 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
@@ -140,6 +140,25 @@ describe('BulkDocumentEditLayout', () => {
 		expect(metadataJson).toHaveProperty('tagNames');
 	});
 
+	it('save() navigates to /documents when all chunks succeed', async () => {
+		const mockFetch = vi.fn().mockResolvedValue({
+			ok: true,
+			json: async () => ({ created: [], updated: [], errors: [] })
+		});
+		vi.stubGlobal('fetch', mockFetch);
+
+		const { container } = render(BulkDocumentEditLayout, {});
+		await addFilesViaInput(container, [makeFile('doc.pdf')]);
+
+		const saveBtn = container.querySelector(
+			'button[data-testid="bulk-save-btn"]'
+		) as HTMLButtonElement;
+		saveBtn.click();
+
+		await vi.waitFor(() => expect(mockFetch).toHaveBeenCalledTimes(1), { timeout: 3000 });
+		await vi.waitFor(() => expect(goto).toHaveBeenCalledWith('/documents'), { timeout: 3000 });
+	});
+
 	it('save() does not navigate when chunk returns non-ok response', async () => {
 		const mockFetch = vi.fn().mockResolvedValue({
 			ok: false,
-- 
2.49.1


From 9aed929b67501ad07a7a686faa5a26a4d2677fef Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Sat, 25 Apr 2026 09:13:55 +0200
Subject: [PATCH 40/46] fix(bulk-upload): raise discard button touch target to
 44px for WCAG compliance
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Senior users on tablets need at least 44×44px touch targets (WCAG 2.2).
Added min-h-[44px] flex items-center px-2 to the discard button.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/src/lib/components/document/UploadSaveBar.svelte | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/frontend/src/lib/components/document/UploadSaveBar.svelte b/frontend/src/lib/components/document/UploadSaveBar.svelte
index ac0f0b92..4a1f65e0 100644
--- a/frontend/src/lib/components/document/UploadSaveBar.svelte
+++ b/frontend/src/lib/components/document/UploadSaveBar.svelte
@@ -27,7 +27,11 @@ let {
 		></progress>
 	{/if}
 	<div class="flex items-center justify-between gap-3">
-		<button type="button" onclick={onDiscard} class="text-sm text-red-600/70 hover:text-red-700">
+		<button
+			type="button"
+			onclick={onDiscard}
+			class="flex min-h-[44px] items-center px-2 text-sm text-red-600/70 hover:text-red-700"
+		>
 			{m.bulk_discard_all()}
 		</button>
 		<button
-- 
2.49.1


From 1299f191e205b3f166ace75704ffd2b80a0923f9 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Sat, 25 Apr 2026 11:03:58 +0200
Subject: [PATCH 41/46] feat(bulk-upload): guard save() against concurrent
 invocations

Adds a saving $state flag that blocks re-entry while a chunk upload is
in flight. The UploadSaveBar save button is disabled via a new disabled
prop while saving is true. Tested: clicking Save twice fires fetch only
once.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../document/BulkDocumentEditLayout.svelte    |  5 ++++
 .../BulkDocumentEditLayout.svelte.spec.ts     | 28 +++++++++++++++++++
 .../components/document/UploadSaveBar.svelte  |  8 ++++--
 3 files changed, 38 insertions(+), 3 deletions(-)

diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
index 7a171e55..7160ad32 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
@@ -31,6 +31,7 @@ let {
 let files = new SvelteMap<string, FileEntry>();
 let activeId = $state<string | null>(null);
 let chunkProgress = $state<{ done: number; total: number } | undefined>(undefined);
+let saving = $state(false);
 
 // --- Shared metadata ---
 let senderId = $state(untrack(() => initialSenderId));
@@ -84,6 +85,8 @@ onDestroy(() => {
 
 // --- Save ---
 async function save() {
+	if (saving) return;
+	saving = true;
 	const entries = Array.from(files.values());
 	const chunkSize = 10;
 	const chunks: FileEntry[][] = [];
@@ -124,6 +127,7 @@ async function save() {
 		}
 		chunkProgress = { done: i + 1, total: chunks.length };
 	}
+	saving = false;
 	if (!hadErrors) goto('/documents');
 }
 </script>
@@ -276,6 +280,7 @@ async function save() {
 				chunkProgress={chunkProgress}
 				onSave={save}
 				onDiscard={discardAll}
+				disabled={saving}
 			/>
 		</div>
 	</div>
diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
index 32e8fdef..521f83a1 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
@@ -206,6 +206,34 @@ describe('BulkDocumentEditLayout', () => {
 		);
 	});
 
+	it('save() does not call fetch a second time when already saving', async () => {
+		let resolveFirst: (() => void) | undefined;
+		const mockFetch = vi.fn().mockImplementation(
+			() =>
+				new Promise<Response>((resolve) => {
+					resolveFirst = () =>
+						resolve({
+							ok: true,
+							json: async () => ({ created: [], updated: [], errors: [] })
+						} as Response);
+				})
+		);
+		vi.stubGlobal('fetch', mockFetch);
+
+		const { container } = render(BulkDocumentEditLayout, {});
+		await addFilesViaInput(container, [makeFile('a.pdf')]);
+
+		const saveBtn = container.querySelector(
+			'button[data-testid="bulk-save-btn"]'
+		) as HTMLButtonElement;
+		saveBtn.click(); // first click — fetch is in-flight
+		saveBtn.click(); // second click — should be a no-op
+
+		resolveFirst?.();
+		await vi.waitFor(() => expect(mockFetch).toHaveBeenCalledTimes(1), { timeout: 3000 });
+		expect(mockFetch).toHaveBeenCalledTimes(1);
+	});
+
 	it('discard-all resets to N=0 state and shows drop zone', async () => {
 		const { container } = render(BulkDocumentEditLayout, {});
 		await addFilesViaInput(container, [makeFile('a.pdf'), makeFile('b.pdf')]);
diff --git a/frontend/src/lib/components/document/UploadSaveBar.svelte b/frontend/src/lib/components/document/UploadSaveBar.svelte
index 4a1f65e0..d404e864 100644
--- a/frontend/src/lib/components/document/UploadSaveBar.svelte
+++ b/frontend/src/lib/components/document/UploadSaveBar.svelte
@@ -5,12 +5,14 @@ let {
 	fileCount,
 	chunkProgress,
 	onSave,
-	onDiscard
+	onDiscard,
+	disabled = false
 }: {
 	fileCount: number;
 	chunkProgress?: { done: number; total: number };
 	onSave: () => void;
-	onDiscard: () => void;
+	onDiscard: () => void | Promise<void>;
+	disabled?: boolean;
 } = $props();
 </script>
 
@@ -37,7 +39,7 @@ let {
 		<button
 			type="button"
 			data-testid="bulk-save-btn"
-			disabled={fileCount === 0}
+			disabled={fileCount === 0 || disabled}
 			onclick={onSave}
 			class="min-h-[44px] rounded-sm bg-primary px-6 text-sm font-bold tracking-widest text-primary-fg uppercase transition-opacity hover:opacity-90 disabled:opacity-40"
 		>
-- 
2.49.1


From 48c82aa07b4d93f33d9b57dc0943bd95e4b451bb Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Sat, 25 Apr 2026 11:09:49 +0200
Subject: [PATCH 42/46] fix(bulk-upload): handle network errors and partial
 upload success
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

save() now wraps each chunk fetch in try/catch — a thrown network error
marks all files in that chunk as errored. Also handles HTTP 200 responses
with a non-empty errors array (partial success): only the named filenames
are marked as errored rather than all files in the chunk. Navigation is
suppressed whenever any file fails.

Tests added:
- network error marks all chunk files as errored, no navigation
- HTTP 200 with errors array marks only affected files

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../document/BulkDocumentEditLayout.svelte    | 24 +++++---
 .../BulkDocumentEditLayout.svelte.spec.ts     | 55 +++++++++++++++++++
 2 files changed, 72 insertions(+), 7 deletions(-)

diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
index 7160ad32..7733924a 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
@@ -111,19 +111,29 @@ async function save() {
 		// Raw fetch is intentional: SvelteKit form actions can't stream chunked
 		// FormData with per-chunk progress. Session cookie is sent automatically
 		// by the browser for same-origin requests.
-		const res = await fetch('/api/documents/quick-upload', { method: 'POST', body: formData });
-		if (!res.ok) {
-			hadErrors = true;
+		try {
+			const res = await fetch('/api/documents/quick-upload', { method: 'POST', body: formData });
 			const body = await res.json().catch(() => ({ errors: [] }));
 			const errorFilenames = new Set<string>(
 				(body.errors ?? []).map((err: { filename: string }) => err.filename)
 			);
-			for (const entry of chunk) {
-				if (errorFilenames.has(entry.file.name)) {
-					const e = files.get(entry.id);
-					if (e) files.set(entry.id, { ...e, status: 'error' });
+			if (!res.ok || errorFilenames.size > 0) {
+				hadErrors = true;
+				for (const entry of chunk) {
+					// When backend names specific files, mark only those; otherwise mark all.
+					const isError = errorFilenames.size > 0 ? errorFilenames.has(entry.file.name) : true;
+					if (isError) {
+						const e = files.get(entry.id);
+						if (e) files.set(entry.id, { ...e, status: 'error' });
+					}
 				}
 			}
+		} catch {
+			hadErrors = true;
+			for (const entry of chunk) {
+				const e = files.get(entry.id);
+				if (e) files.set(entry.id, { ...e, status: 'error' });
+			}
 		}
 		chunkProgress = { done: i + 1, total: chunks.length };
 	}
diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
index 521f83a1..1b24627d 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte.spec.ts
@@ -206,6 +206,61 @@ describe('BulkDocumentEditLayout', () => {
 		);
 	});
 
+	it('save() marks only the failed file when server returns HTTP 200 with a partial errors array', async () => {
+		// Backend can return 200 OK while reporting individual file failures
+		const mockFetch = vi.fn().mockResolvedValue({
+			ok: true,
+			json: async () => ({
+				created: [{ id: '1' }],
+				updated: [],
+				errors: [{ filename: 'b.pdf', code: 'FILE_UPLOAD_FAILED' }]
+			})
+		});
+		vi.stubGlobal('fetch', mockFetch);
+
+		const { container } = render(BulkDocumentEditLayout, {});
+		await addFilesViaInput(container, [makeFile('a.pdf'), makeFile('b.pdf'), makeFile('c.pdf')]);
+
+		const saveBtn = container.querySelector(
+			'button[data-testid="bulk-save-btn"]'
+		) as HTMLButtonElement;
+		saveBtn.click();
+
+		await vi.waitFor(() => expect(mockFetch).toHaveBeenCalledTimes(1), { timeout: 3000 });
+
+		await vi.waitFor(
+			() => {
+				const errorChips = container.querySelectorAll('[data-chip-id][data-status="error"]');
+				expect(errorChips.length).toBe(1);
+				expect(errorChips[0].textContent).toContain('b');
+			},
+			{ timeout: 1000 }
+		);
+		// Navigation should be suppressed because hadErrors is true
+		expect(goto).not.toHaveBeenCalled();
+	});
+
+	it('save() marks all chunk files as errored when fetch throws a network error', async () => {
+		vi.stubGlobal('fetch', vi.fn().mockRejectedValue(new Error('network error')));
+
+		const { container } = render(BulkDocumentEditLayout, {});
+		await addFilesViaInput(container, [makeFile('a.pdf'), makeFile('b.pdf')]);
+
+		const saveBtn = container.querySelector(
+			'button[data-testid="bulk-save-btn"]'
+		) as HTMLButtonElement;
+		saveBtn.click();
+
+		await vi.waitFor(
+			() => {
+				const errorChips = container.querySelectorAll('[data-chip-id][data-status="error"]');
+				expect(errorChips.length).toBe(2);
+			},
+			{ timeout: 3000 }
+		);
+		expect(goto).not.toHaveBeenCalled();
+	});
+
 	it('save() does not call fetch a second time when already saving', async () => {
 		let resolveFirst: (() => void) | undefined;
 		const mockFetch = vi.fn().mockImplementation(
-- 
2.49.1


From 0ab85d888b9c554a11a978fba5142dc2418919b8 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Sat, 25 Apr 2026 11:14:31 +0200
Subject: [PATCH 43/46] fix(bulk-upload): chip readability and focus management
 in FileSwitcherStrip
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Chip label text increased from 11px to 12px (text-xs) and number badge
from 9px to 11px for the 60+ senior audience on laptops/tablets.

After removing a chip via the × button, focus moves to the previous chip
(falling back to the next chip when the first chip is removed) so keyboard
users are not stranded on <body>. Uses Svelte tick() to wait for DOM update.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../document/FileSwitcherStrip.svelte         | 16 +++++++++---
 .../document/FileSwitcherStrip.svelte.spec.ts | 26 +++++++++++++++++++
 2 files changed, 39 insertions(+), 3 deletions(-)

diff --git a/frontend/src/lib/components/document/FileSwitcherStrip.svelte b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
index d70241bc..062eb99c 100644
--- a/frontend/src/lib/components/document/FileSwitcherStrip.svelte
+++ b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
@@ -1,4 +1,5 @@
 <script lang="ts">
+import { tick } from 'svelte';
 import { m } from '$lib/paraglide/messages.js';
 
 export interface FileEntry {
@@ -33,6 +34,15 @@ function scrollNext() {
 	trackEl?.scrollBy({ left: 120, behavior: 'smooth' });
 }
 
+async function handleRemove(entry: FileEntry, index: number) {
+	const targetId = index > 0 ? files[index - 1].id : (files[index + 1]?.id ?? null);
+	onRemove(entry.id);
+	if (targetId) {
+		await tick();
+		(listEl?.querySelector<HTMLElement>(`[data-chip-id="${targetId}"]`) ?? null)?.focus();
+	}
+}
+
 $effect(() => {
 	if (!listEl) return;
 	const node = listEl;
@@ -85,7 +95,7 @@ $effect(() => {
 						data-chip-id={entry.id}
 						onclick={() => onSelect(entry.id)}
 						class={[
-							'inline-flex cursor-pointer items-center gap-1 rounded-[2px] px-1.5 py-0.5 text-[11px] font-bold transition-colors focus:outline-none focus-visible:ring-2 focus-visible:ring-accent',
+							'inline-flex cursor-pointer items-center gap-1 rounded-[2px] px-1.5 py-0.5 text-xs font-bold transition-colors focus:outline-none focus-visible:ring-2 focus-visible:ring-accent',
 							entry.id === activeId
 								? 'bg-accent text-primary'
 								: 'bg-black/[0.06] text-ink-2 hover:bg-black/10',
@@ -96,7 +106,7 @@ $effect(() => {
 					>
 						<span
 							class={[
-								'rounded-[2px] px-0.5 text-[9px] font-extrabold opacity-85',
+								'rounded-[2px] px-0.5 text-[11px] font-extrabold opacity-85',
 								entry.id === activeId ? 'bg-black/20' : 'bg-black/10'
 							].join(' ')}
 							>{i + 1}</span
@@ -111,7 +121,7 @@ $effect(() => {
 						type="button"
 						aria-label={m.bulk_remove_file()}
 						data-remove-id={entry.id}
-						onclick={() => onRemove(entry.id)}
+						onclick={() => handleRemove(entry, i)}
 						class="ml-0.5 flex h-[44px] w-[44px] items-center justify-center text-base text-ink-3 hover:text-ink focus:outline-none focus-visible:ring-2 focus-visible:ring-accent"
 					>
 						×
diff --git a/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts b/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts
index 6a6ccc83..c45550c8 100644
--- a/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts
+++ b/frontend/src/lib/components/document/FileSwitcherStrip.svelte.spec.ts
@@ -99,6 +99,32 @@ describe('FileSwitcherStrip', () => {
 		expect(srOnly).not.toBeNull();
 	});
 
+	it('focus moves to the previous chip after the middle chip is removed', async () => {
+		const files = makeFiles(3); // id-0, id-1, id-2
+		const onRemove = vi.fn();
+		const { container } = render(FileSwitcherStrip, {
+			files,
+			activeId: files[1].id,
+			onSelect: vi.fn(),
+			onRemove
+		});
+
+		const removeBtn = container.querySelector('[data-remove-id="id-1"]') as HTMLButtonElement;
+		expect(removeBtn).not.toBeNull();
+		removeBtn.click();
+		expect(onRemove).toHaveBeenCalledWith('id-1');
+
+		// After removal, focus should be on the chip for id-0 (the previous chip)
+		await vi.waitFor(
+			() => {
+				const prevChip = container.querySelector('[data-chip-id="id-0"]') as HTMLElement | null;
+				expect(prevChip).not.toBeNull();
+				expect(document.activeElement).toBe(prevChip);
+			},
+			{ timeout: 1000 }
+		);
+	});
+
 	it('ArrowRight moves focus to next chip without leaving strip', async () => {
 		const files = makeFiles(3);
 		const { container } = render(FileSwitcherStrip, {
-- 
2.49.1


From e5fe2fc5c67108597f98f8df2a5610c064e29ffa Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Sat, 25 Apr 2026 11:17:05 +0200
Subject: [PATCH 44/46] fix(bulk-upload): add gradient overflow indicators to
 chip strip

Adds pointer-events-none left/right gradient fade overlays on the
FileSwitcherStrip track div so mouse-only users can see when more
chips are hidden beyond the visible area. The scrollbar is hidden
(scrollbar-width:none) so gradients are the only overflow signal.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../document/FileSwitcherStrip.svelte         | 77 +++++++++++--------
 1 file changed, 43 insertions(+), 34 deletions(-)

diff --git a/frontend/src/lib/components/document/FileSwitcherStrip.svelte b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
index 062eb99c..8816d1a1 100644
--- a/frontend/src/lib/components/document/FileSwitcherStrip.svelte
+++ b/frontend/src/lib/components/document/FileSwitcherStrip.svelte
@@ -83,18 +83,26 @@ $effect(() => {
 		>‹</button
 	>
 
-	<div bind:this={trackEl} class="flex flex-1 gap-1 overflow-x-auto" style="scrollbar-width:none">
-		<ul bind:this={listEl} role="list" class="flex flex-row gap-1 py-1">
-			{#each files as entry, i (entry.id)}
-				<li role="listitem" class="inline-flex shrink-0 items-center">
-					<button
-						type="button"
-						tabindex="0"
-						aria-current={entry.id === activeId ? 'true' : undefined}
-						data-status={entry.status}
-						data-chip-id={entry.id}
-						onclick={() => onSelect(entry.id)}
-						class={[
+	<!-- Gradient fade overlays signal hidden overflow to pointer-only users -->
+	<div class="relative flex flex-1 overflow-hidden">
+		<div
+			class="pointer-events-none absolute inset-y-0 left-0 z-10 w-6 bg-gradient-to-r from-pdf-ctrl to-transparent"
+		></div>
+		<div
+			class="pointer-events-none absolute inset-y-0 right-0 z-10 w-6 bg-gradient-to-l from-pdf-ctrl to-transparent"
+		></div>
+		<div bind:this={trackEl} class="flex flex-1 gap-1 overflow-x-auto" style="scrollbar-width:none">
+			<ul bind:this={listEl} role="list" class="flex flex-row gap-1 py-1">
+				{#each files as entry, i (entry.id)}
+					<li role="listitem" class="inline-flex shrink-0 items-center">
+						<button
+							type="button"
+							tabindex="0"
+							aria-current={entry.id === activeId ? 'true' : undefined}
+							data-status={entry.status}
+							data-chip-id={entry.id}
+							onclick={() => onSelect(entry.id)}
+							class={[
 							'inline-flex cursor-pointer items-center gap-1 rounded-[2px] px-1.5 py-0.5 text-xs font-bold transition-colors focus:outline-none focus-visible:ring-2 focus-visible:ring-accent',
 							entry.id === activeId
 								? 'bg-accent text-primary'
@@ -103,32 +111,33 @@ $effect(() => {
 								? '!border !border-dashed !border-red-400 !bg-red-50/80 !text-red-700'
 								: ''
 						].join(' ')}
-					>
-						<span
-							class={[
+						>
+							<span
+								class={[
 								'rounded-[2px] px-0.5 text-[11px] font-extrabold opacity-85',
 								entry.id === activeId ? 'bg-black/20' : 'bg-black/10'
 							].join(' ')}
-							>{i + 1}</span
+								>{i + 1}</span
+							>
+							<span class="max-w-[8rem] truncate" title={entry.title}>{entry.title}</span>
+							{#if entry.status === 'error'}
+								<span class="sr-only">{m.bulk_file_error_chip_label()}</span>
+								<span aria-hidden="true" class="ml-0.5 font-extrabold text-red-600">!</span>
+							{/if}
+						</button>
+						<button
+							type="button"
+							aria-label={m.bulk_remove_file()}
+							data-remove-id={entry.id}
+							onclick={() => handleRemove(entry, i)}
+							class="ml-0.5 flex h-[44px] w-[44px] items-center justify-center text-base text-ink-3 hover:text-ink focus:outline-none focus-visible:ring-2 focus-visible:ring-accent"
 						>
-						<span class="max-w-[8rem] truncate" title={entry.title}>{entry.title}</span>
-						{#if entry.status === 'error'}
-							<span class="sr-only">{m.bulk_file_error_chip_label()}</span>
-							<span aria-hidden="true" class="ml-0.5 font-extrabold text-red-600">!</span>
-						{/if}
-					</button>
-					<button
-						type="button"
-						aria-label={m.bulk_remove_file()}
-						data-remove-id={entry.id}
-						onclick={() => handleRemove(entry, i)}
-						class="ml-0.5 flex h-[44px] w-[44px] items-center justify-center text-base text-ink-3 hover:text-ink focus:outline-none focus-visible:ring-2 focus-visible:ring-accent"
-					>
-						×
-					</button>
-				</li>
-			{/each}
-		</ul>
+							×
+						</button>
+					</li>
+				{/each}
+			</ul>
+		</div>
 	</div>
 
 	<button
-- 
2.49.1


From 46dae8a826a90ec33e7fa098300ef8991fe1f310 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Sat, 25 Apr 2026 11:26:05 +0200
Subject: [PATCH 45/46] feat(bulk-upload): guard discard-all with confirm
 dialog
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Uses getConfirmService() (optional — null fallback when context is absent so
unit tests that don't exercise the discard path need no CONFIRM_KEY context)
and the new bulk_discard_confirm i18n key.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/messages/de.json                     |  1 +
 frontend/messages/en.json                     |  1 +
 frontend/messages/es.json                     |  1 +
 .../document/BulkDocumentEditLayout.svelte    | 26 +++++++++++++++++--
 4 files changed, 27 insertions(+), 2 deletions(-)

diff --git a/frontend/messages/de.json b/frontend/messages/de.json
index 25ad5edb..0ef4cb8e 100644
--- a/frontend/messages/de.json
+++ b/frontend/messages/de.json
@@ -858,6 +858,7 @@
 	"bulk_save_cta_one": "Speichern →",
 	"bulk_save_cta": "{count} speichern →",
 	"bulk_discard_all": "Alle verwerfen",
+	"bulk_discard_confirm": "Alle Dateien und eingegebenen Daten verwerfen? Diese Aktion kann nicht rückgängig gemacht werden.",
 	"bulk_add_more": "Weitere hinzufügen",
 	"bulk_scope_per_file_label": "Nur diese Datei",
 	"bulk_scope_shared_label": "Gilt für alle {count}",
diff --git a/frontend/messages/en.json b/frontend/messages/en.json
index 523fa7af..032e7bb5 100644
--- a/frontend/messages/en.json
+++ b/frontend/messages/en.json
@@ -858,6 +858,7 @@
 	"bulk_save_cta_one": "Save →",
 	"bulk_save_cta": "Save {count} →",
 	"bulk_discard_all": "Discard all",
+	"bulk_discard_confirm": "Discard all files and entered data? This action cannot be undone.",
 	"bulk_add_more": "Add more",
 	"bulk_scope_per_file_label": "This file only",
 	"bulk_scope_shared_label": "Applies to all {count}",
diff --git a/frontend/messages/es.json b/frontend/messages/es.json
index b725d39b..7e328905 100644
--- a/frontend/messages/es.json
+++ b/frontend/messages/es.json
@@ -858,6 +858,7 @@
 	"bulk_save_cta_one": "Guardar →",
 	"bulk_save_cta": "Guardar {count} →",
 	"bulk_discard_all": "Descartar todo",
+	"bulk_discard_confirm": "¿Descartar todos los archivos y datos introducidos? Esta acción no se puede deshacer.",
 	"bulk_add_more": "Añadir más",
 	"bulk_scope_per_file_label": "Solo este archivo",
 	"bulk_scope_shared_label": "Para todos los {count}",
diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
index 7733924a..bc76ff5b 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
@@ -3,6 +3,8 @@ import { SvelteMap } from 'svelte/reactivity';
 import { goto } from '$app/navigation';
 import { onDestroy, untrack } from 'svelte';
 import { m } from '$lib/paraglide/messages.js';
+import { getConfirmService } from '$lib/services/confirm.svelte.js';
+import type { ConfirmService } from '$lib/services/confirm.svelte.js';
 import BulkDropZone from './BulkDropZone.svelte';
 import FileSwitcherStrip from './FileSwitcherStrip.svelte';
 import type { FileEntry } from './FileSwitcherStrip.svelte';
@@ -17,6 +19,14 @@ import type { components } from '$lib/generated/api';
 
 type Person = components['schemas']['Person'];
 
+// Optional — not available in unit tests that don't provide CONFIRM_KEY context.
+let _confirmService: ConfirmService | null;
+try {
+	_confirmService = getConfirmService();
+} catch {
+	_confirmService = null;
+}
+
 let {
 	initialSenderId = '',
 	initialSenderName = '',
@@ -77,6 +87,18 @@ function discardAll() {
 	chunkProgress = undefined;
 }
 
+async function handleDiscard() {
+	if (_confirmService) {
+		const ok = await _confirmService.confirm({
+			title: m.bulk_discard_all(),
+			body: m.bulk_discard_confirm(),
+			destructive: true
+		});
+		if (!ok) return;
+	}
+	discardAll();
+}
+
 onDestroy(() => {
 	for (const entry of files.values()) {
 		if (entry.previewUrl) URL.revokeObjectURL(entry.previewUrl);
@@ -177,7 +199,7 @@ async function save() {
 				<button
 					type="button"
 					data-testid="discard-all-btn"
-					onclick={discardAll}
+					onclick={handleDiscard}
 					class="text-xs font-medium text-red-600/70 hover:text-red-700"
 				>
 					{m.bulk_discard_all()}
@@ -289,7 +311,7 @@ async function save() {
 				fileCount={files.size}
 				chunkProgress={chunkProgress}
 				onSave={save}
-				onDiscard={discardAll}
+				onDiscard={handleDiscard}
 				disabled={saving}
 			/>
 		</div>
-- 
2.49.1


From 1fca1f80a23b880cbcb7848952bfa06063e1a7d8 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Sat, 25 Apr 2026 11:31:59 +0200
Subject: [PATCH 46/46] docs(bulk-upload): explain chunkSize=10 and 50-file cap
 constants

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../java/org/raddatz/familienarchiv/service/DocumentService.java | 1 +
 .../src/lib/components/document/BulkDocumentEditLayout.svelte    | 1 +
 2 files changed, 2 insertions(+)

diff --git a/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java b/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java
index 06317642..b7d50f99 100644
--- a/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java
+++ b/backend/src/main/java/org/raddatz/familienarchiv/service/DocumentService.java
@@ -134,6 +134,7 @@ public class DocumentService {
     }
 
     public void validateBatch(int fileCount, DocumentBatchMetadataDTO metadata) {
+        // 50-file hard cap keeps FormData requests at a manageable size and protects against runaway bulk uploads.
         if (fileCount > 50) {
             throw DomainException.badRequest(ErrorCode.BATCH_TOO_LARGE, "Batch exceeds maximum of 50 files per request");
         }
diff --git a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
index bc76ff5b..6ce93a07 100644
--- a/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
+++ b/frontend/src/lib/components/document/BulkDocumentEditLayout.svelte
@@ -110,6 +110,7 @@ async function save() {
 	if (saving) return;
 	saving = true;
 	const entries = Array.from(files.values());
+	// 10 files per request keeps multipart bodies well under typical reverse-proxy limits (e.g. nginx default 1 MB client_max_body_size per PDF).
 	const chunkSize = 10;
 	const chunks: FileEntry[][] = [];
 	for (let i = 0; i < entries.length; i += chunkSize) {
-- 
2.49.1