• Joined on 2026-03-17
ef11cbee4e feat(transcription): clicking annotation focuses corresponding block
676d3cb6a7 fix(pdf): prevent scroll-sync effect from hijacking page navigation
d389dc2023 feat(annotations): dim non-active annotations when a block is focused
Compare 2 commits »
b4212f5e86 feat(transcription): mobile stacked layout + cross-page scroll-sync
c22f2e41b1 fix(transcription): replace broken HTML5 drag with pointer-based drag
7d2d615e0c feat(transcription): add drag-and-drop + arrow button reordering
4a88b3ba82 feat(transcription): add dashed next-block CTA below block list
marcel commented on pull request marcel/familienarchiv#178 2026-04-05 22:50:23 +02:00
feat: Expandable metadata drawer + transcription system (#175, #176)

🔧 Tobias Wendt — DevOps & Platform Engineer (Round 3)

Verdict: Approved

What changed since Round 2

  • 9 dead component files deleted — reduces frontend build surface
  • No new…
marcel commented on pull request marcel/familienarchiv#178 2026-04-05 22:50:08 +02:00
feat: Expandable metadata drawer + transcription system (#175, #176)

🎨 Leonie Voss — UI/UX Design Lead (Round 3)

Verdict: Approved

Significant UI improvements since round 2. All my previous concerns have been addressed:

Resolved concerns

marcel commented on pull request marcel/familienarchiv#178 2026-04-05 22:49:44 +02:00
feat: Expandable metadata drawer + transcription system (#175, #176)

🔒 Nora "NullX" Steiner — Application Security Engineer (Round 3)

Verdict: Approved

All blockers from rounds 1-2 resolved. New changes reviewed:

What improved

  • **Regex HTML…
marcel commented on pull request marcel/familienarchiv#178 2026-04-05 22:49:25 +02:00
feat: Expandable metadata drawer + transcription system (#175, #176)

🧪 Sara Holt — QA Engineer & Test Strategist (Round 3)

Verdict: ⚠️ Approved with concerns

Test coverage — current state

Backend: solid

  • TranscriptionServiceTest — 13 unit…
marcel commented on pull request marcel/familienarchiv#178 2026-04-05 22:49:04 +02:00
feat: Expandable metadata drawer + transcription system (#175, #176)

🏗️ Markus Keller — Application Architect (Round 3)

Verdict: Approved

The architectural decisions from rounds 1-2 are cleanly implemented:

  • Block as aggregate root — delete…
marcel commented on pull request marcel/familienarchiv#178 2026-04-05 22:48:47 +02:00
feat: Expandable metadata drawer + transcription system (#175, #176)

👨‍💻 Felix Brandt — Senior Fullstack Developer (Round 3)

Verdict: ⚠️ Approved with concerns

What improved since Round 2

Massive progress. The PR now has 30 commits addressing…

6dc81ef2e3 fix(ui): match delete icon size + add cursor-pointer to interactive elements
cef1810700 fix(comments): stop Escape propagation in edit mode
351f31b183 feat(comments): inline edit on click + trash icon for own comments
e6432846a1 fix(topbar): use brand navy for transcribe button, not turquoise
a66bec1971 fix(comments): increase text size for readability
82d5a34f76 fix(comments): use semantic tokens for comment box dark mode
3d086bd1fb fix(transcription): auto-capture quote on text selection, smart comment button
e384c87eef refactor(comments): streamline input — Enter to send, no buttons