marcel/familienarchiv
1
0
Fork 0
Code Issues 103 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix(document-page): add .catch() to task deep-link tick promise
Some checks failed
CI / Unit & Component Tests (pull_request) Failing after 3m23s
Details
CI / OCR Service Tests (pull_request) Successful in 29s
Details
CI / Backend Unit Tests (pull_request) Failing after 3m0s
Details
CI / Unit & Component Tests (push) Failing after 3m17s
Details
CI / OCR Service Tests (push) Successful in 34s
Details
CI / Backend Unit Tests (push) Failing after 3m19s
Details

Browse Source 
Addresses @felix — tick().then() had no error handler; console.error
is now logged on failure, matching the existing deep-link scroll pattern.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Marcel
2026-04-29 21:34:41 +02:00
parent 84ba728e08
commit a81a6e7253
1 changed files with 11 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
frontend/src/routes/documents/[id]/+page.svelte
View File

@@ -364,7 +364,8 @@ onMount(() => {
if (page.url.searchParams.get('task') === 'transcribe') { if (page.url.searchParams.get('task') === 'transcribe') {
transcribeMode = true; transcribeMode = true;
tick().then(() => { tick()
.then(() => {
const closeBtn = document.querySelector<HTMLElement>('[data-testid="panel-close"]'); const closeBtn = document.querySelector<HTMLElement>('[data-testid="panel-close"]');
closeBtn?.scrollIntoView({ closeBtn?.scrollIntoView({
behavior: prefersReducedMotion ? 'instant' : 'smooth', behavior: prefersReducedMotion ? 'instant' : 'smooth',
@@ -372,7 +373,8 @@ onMount(() => {
}); });
closeBtn?.focus({ preventScroll: true }); closeBtn?.focus({ preventScroll: true });
replaceState(page.url.pathname, page.state ?? {}); replaceState(page.url.pathname, page.state ?? {});
}); })
.catch((e) => console.error('task deep-link failed', e));
} }
scrollToCommentFromQuery(new URL(page.url), { scrollToCommentFromQuery(new URL(page.url), {