marcel
0 Followers · 0 Following
  • Joined on 2026-03-17
Repositories
3
 Projects Packages Public Activity Starred Repositories
54b4b96411 feat(person-mention): update transcription placeholder with @mention discoverability hint
marcel created branch feat/issue-370-at-mention-placeholder-hint in marcel/familienarchiv 2026-04-30 10:36:23 +02:00
marcel closed issue marcel/familienarchiv#362 2026-04-30 10:21:26 +02:00
feat: Person @mentions in transcription blocks with hover card
marcel created pull request marcel/familienarchiv#378 2026-04-30 10:18:45 +02:00
fix(training): use KURRENT_RECOGNITION label for sender-based block queries
marcel pushed to fix/sender-training-use-kurrent-label at marcel/familienarchiv 2026-04-30 10:16:57 +02:00
c905b81fd3 fix(training): use KURRENT_RECOGNITION label for sender-based block queries
marcel created branch fix/sender-training-use-kurrent-label in marcel/familienarchiv 2026-04-30 10:16:57 +02:00
marcel deleted branch feat/issue-376-auto-open-transcription-panel from marcel/familienarchiv 2026-04-29 21:38:17 +02:00
marcel closed issue marcel/familienarchiv#376 2026-04-29 21:38:16 +02:00
feat: auto-open transcription panel when navigating from mission-control cards
marcel pushed to main at marcel/familienarchiv 2026-04-29 21:38:16 +02:00
db66d0cc61 fix(document-page): add .catch() to task deep-link tick promise
7dc5dc6f71 feat(document-page): auto-open transcription panel when ?task=transcribe is present
d974d39d17 feat(TranscriptionColumn): deep-link to transcription panel via ?task=transcribe
5e4e487d5f feat(SegmentationColumn): deep-link to transcription panel via ?task=transcribe
Compare 4 commits »
marcel merged pull request marcel/familienarchiv#377 2026-04-29 21:38:15 +02:00
feat: auto-open transcription panel when navigating from mission-control cards
marcel commented on pull request marcel/familienarchiv#377 2026-04-29 21:36:46 +02:00
feat: auto-open transcription panel when navigating from mission-control cards

🛠️ Tobias Wendt — DevOps & Platform Engineer

Verdict: ✅ Approved

No infrastructure, CI, or deployment impact.

What I checked

  • New dependencies: none — no new npm packages,…
marcel commented on pull request marcel/familienarchiv#377 2026-04-29 21:36:42 +02:00
feat: auto-open transcription panel when navigating from mission-control cards

🎨 Leonie Voss — UX Designer & Accessibility Strategist

Verdict: ✅ Approved

Focus management and motion handling are done correctly. No accessibility concerns.

What I checked

-…

marcel commented on pull request marcel/familienarchiv#377 2026-04-29 21:36:35 +02:00
feat: auto-open transcription panel when navigating from mission-control cards

🧪 Sara Holt — QA Engineer & Test Strategist

Verdict: ✅ Approved

Test coverage is appropriate for the agreed scope. No blockers.

What I checked

  • Column component tests: both…
marcel commented on pull request marcel/familienarchiv#377 2026-04-29 21:36:29 +02:00
feat: auto-open transcription panel when navigating from mission-control cards

🔒 Nora "NullX" Steiner — Application Security Engineer

Verdict: ✅ Approved

No security concerns in this diff.

What I checked

  • User-controlled param reflection: `page.url.searc…
marcel commented on pull request marcel/familienarchiv#377 2026-04-29 21:36:23 +02:00
feat: auto-open transcription panel when navigating from mission-control cards

🏛️ Markus Keller — Application Architect

Verdict: ✅ Approved

Clean, minimal solution. No architectural concerns.

What I checked

  • Layer placement: the ?task=transcribe logic…
marcel commented on pull request marcel/familienarchiv#377 2026-04-29 21:36:19 +02:00
feat: auto-open transcription panel when navigating from mission-control cards

👨‍💻 Felix Brandt — Senior Fullstack Developer

Verdict: ✅ Approved

Cycle 1 concern addressed — tick().then().catch() now matches the pattern of the scrollToCommentFromQuery chain…

marcel commented on pull request marcel/familienarchiv#377 2026-04-29 21:35:16 +02:00
feat: auto-open transcription panel when navigating from mission-control cards

Review Cycle 1 — Changes

Addressed

  • [@felix, @sara] tick().then() missing .catch() — added .catch((e) => console.error('task deep-link failed', e)) to match the existing deep-link…
a81a6e7253 fix(document-page): add .catch() to task deep-link tick promise
marcel deleted branch fix/hover-card-placeholder-maiden-name from marcel/familienarchiv 2026-04-29 21:33:19 +02:00
marcel pushed to main at marcel/familienarchiv 2026-04-29 21:33:19 +02:00
b3fe9b1171 refactor(PersonMentionEditor): use data-editor-inner attribute for stable querySelector
3c7c7a9aa4 refactor(TranscriptionReadView): rename handleMentionLeave, closeTimer to \$state, add 150ms comment
9908f7afdc test(TranscriptionReadView): cover hover card timer and keyboard focus behavior
96d9ff5db1 fix(PersonHoverCard): move chip colon into DOM for consistent screen reader announcement
0113367d05 refactor(TranscriptionReadView): remove dead else branch in handleMentionLeave
Compare 12 commits »