Merge pull request #41 from whale4113/feat/lujunji/auto-scroll

fix: missing diagrams when downloading long documents
fix: pictures and files still download when downloading is canceled

Author: whale4113

.changeset/curvy-carrots-drop.md

@@ -0,0 +1,5 @@
+---
+'@dolphin/common': minor
+---
+
+feat: add `waitFor()` utility and `imageDataToBlob()` support `onDispose()` option

.changeset/neat-shoes-sip.md

@@ -0,0 +1,5 @@
+---
+'@dolphin/chrome-extension': patch
+---
+
+fix: pictures and files still download when downloading is canceled

.changeset/purple-bulldogs-accept.md

@@ -0,0 +1,5 @@
+---
+'@dolphin/lark': minor
+---
+
+feat: support scroll document

.changeset/ten-clocks-attend.md

@@ -0,0 +1,5 @@
+---
+'@dolphin/common': minor
+---
+
+feat: add `waitForSelector()` utility

.changeset/warm-forks-turn.md

@@ -0,0 +1,5 @@
+---
+'@dolphin/chrome-extension': patch
+---
+
+fix: the scroll position of a whiteboard's view element may not be recorded

.changeset/witty-moons-matter.md

@@ -0,0 +1,5 @@
+---
+'@dolphin/chrome-extension': patch
+---
+
+fix: missing diagrams when downloading long documents

apps/chrome-extension/src/common/legacy.ts

@@ -1,3 +1,4 @@
+import { Second } from '@dolphin/common'
 import { FirstFileSaveOptions } from 'browser-fs-access'
 
 export const legacyFileSave = (
@@ -19,7 +20,7 @@ export const legacyFileSave = (
 
       // `setTimeout()` due to
       // https://github.com/LLK/scratch-gui/issues/1783#issuecomment-426286393
-      setTimeout(() => URL.revokeObjectURL(a.href), 30 * 1000)
+      setTimeout(() => URL.revokeObjectURL(a.href), 30 * Second)
     },
     true,
   )

apps/chrome-extension/src/common/utils.ts

@@ -0,0 +1,41 @@
+interface Ref<T> {
+  current: T
+}
+
+interface WithSignalOptions {
+  signal?: AbortSignal
+  onAbort?: () => void
+}
+
+export const withSignal = async <T>(
+  inner: (abortedRef: Ref<boolean>) => Promise<T> = Promise.resolve,
+  options: WithSignalOptions = {},
+): Promise<T | null> => {
+  const { signal, onAbort } = options
+
+  let ref: Ref<boolean> = { current: false }
+  const handler = () => {
+    ref.current = true
+
+    signal?.removeEventListener('abort', handler)
+
+    onAbort?.()
+  }
+
+  signal?.addEventListener('abort', handler)
+
+  let result = null
+
+  try {
+    result = await inner(ref)
+  } catch (error) {
+    console.error(error)
+  }
+
+  signal?.removeEventListener('abort', handler)
+
+  // @ts-expect-error remove reference
+  ref = null
+
+  return result
+}