From e384aaf6ddb02cd180896aba7e6e4f2da5a34dce Mon Sep 17 00:00:00 2001
From: John Factotum <50942278+johnfactotum@users.noreply.github.com>
Date: Mon, 30 Sep 2024 19:28:04 +0800
Subject: [PATCH] Media Overlay: add `stop()` method

And don't remove highlight when pausing
---
 epub.js | 19 ++++++++++++-------
 1 file changed, 12 insertions(+), 7 deletions(-)

diff --git a/epub.js b/epub.js
index 86e8f2e..d8629d0 100644
--- a/epub.js
+++ b/epub.js
@@ -443,11 +443,8 @@ class MediaOverlay extends EventTarget {
         this.dispatchEvent(new CustomEvent('unhighlight', { detail: this.#activeItem }))
     }
     async #play(audioIndex, itemIndex) {
-        if (this.#audio) {
-            this.#audio.pause()
-            URL.revokeObjectURL(this.#audio.src)
-            this.#audio = null
-        }
+        const paused = this.#audio?.paused
+        if (this.#audio) this.stop()
         this.#audioIndex = audioIndex
         this.#itemIndex = itemIndex
         const src = this.#activeAudio?.src
@@ -477,14 +474,14 @@ class MediaOverlay extends EventTarget {
         audio.addEventListener('error', () =>
             this.#error(new Error(`Failed to load ${src}`)))
         audio.addEventListener('playing', () => this.#highlight())
-        audio.addEventListener('pause', () => this.#unhighlight())
         audio.addEventListener('ended', () => {
             this.#unhighlight()
             URL.revokeObjectURL(url)
             this.#audio = null
             this.#play(audioIndex + 1, 0).catch(e => this.#error(e))
         })
-        audio.addEventListener('canplaythrough', () =>
+        if (paused) this.#highlight()
+        else audio.addEventListener('canplaythrough', () =>
             audio.play().catch(e => this.#error(e)), { once: true })
     }
     async start(sectionIndex, filter = () => true) {
@@ -512,6 +509,14 @@ class MediaOverlay extends EventTarget {
     resume() {
         this.#audio?.play().catch(e => this.#error(e))
     }
+    stop() {
+        if (this.#audio) {
+            this.#audio.pause()
+            URL.revokeObjectURL(this.#audio.src)
+            this.#audio = null
+            this.#unhighlight()
+        }
+    }
     prev() {
         if (this.#itemIndex > 0) this.#play(this.#audioIndex, this.#itemIndex - 1)
         else if (this.#audioIndex > 0) this.#play(this.#audioIndex - 1,