fix(document-page): add .catch() to task deep-link tick promise
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 was merged in pull request #377.
This commit is contained in:
@@ -364,15 +364,17 @@ onMount(() => {
|
|||||||
|
|
||||||
if (page.url.searchParams.get('task') === 'transcribe') {
|
if (page.url.searchParams.get('task') === 'transcribe') {
|
||||||
transcribeMode = true;
|
transcribeMode = true;
|
||||||
tick().then(() => {
|
tick()
|
||||||
const closeBtn = document.querySelector<HTMLElement>('[data-testid="panel-close"]');
|
.then(() => {
|
||||||
closeBtn?.scrollIntoView({
|
const closeBtn = document.querySelector<HTMLElement>('[data-testid="panel-close"]');
|
||||||
behavior: prefersReducedMotion ? 'instant' : 'smooth',
|
closeBtn?.scrollIntoView({
|
||||||
block: 'nearest'
|
behavior: prefersReducedMotion ? 'instant' : 'smooth',
|
||||||
});
|
block: 'nearest'
|
||||||
closeBtn?.focus({ preventScroll: true });
|
});
|
||||||
replaceState(page.url.pathname, page.state ?? {});
|
closeBtn?.focus({ preventScroll: true });
|
||||||
});
|
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), {
|
||||||
|
|||||||
Reference in New Issue
Block a user