[nwingt]

No description provided.

[notion-workspace]

閱讀器支援以 App 音量鍵翻頁（網頁設定與翻頁）

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing. Review the Copilot code review billing settings.

[Copilot]

🟡 Changes recommended

Volume-key paging lets reading-time tracking stop after two minutes of continued use, and the EPUB and PDF volume-key switches lack accessible names.

2 open issues

• Reset the idle timer on page navigation · New
• Add an accessible name to the volume-key switch · New

_{Review effort: Balanced}

[Copilot]

🔵 Needs a closer look

A same-tick guard deactivation and reader unmount can skip releasing native volume keys, leaving the shell in paging mode; cleanup must release supported keys and include a regression test.

2 open issues

• Add an accessible name to the volume-key switch
• Reset the idle timer on page navigation

1 previously missed issue

In code that hasn't changed since last review

Release native volume keys when unmounting during capture deactivation

app/​composables/​use-volume-key-paging.ts:41

If a capture guard becomes false and the reader unmounts in the same tick, this condition skips releasing the native volume keys. The queued watcher is stopped during unmount before it sends enabled: false, so the shell can remain in paging mode after the reader is gone. Release whenever the capability is supported, and add a regression test that deactivates and unmounts without awaiting nextTick().

_{Review effort: Balanced}

[Copilot]

🔵 Needs a closer look

The inspected 3ook-com-app lacks volumeKeyPaging support, so the paging contract and hardware-key release behavior on iOS and Android require validation with the companion native build.

1 open issue

• Reset the idle timer on page navigation

1 resolved since last review

• Add an accessible name to the volume-key switch

_{Review effort: Balanced}

[Copilot]

🔵 Needs a closer look

The inspected native shell lacks the required bridge support, so protocol compatibility, hardware-key capture/release, and volume restoration require validation on supporting iOS/Android builds and real devices.

No open issues 🎉

1 resolved since last review

• Reset the idle timer on page navigation

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

No active review issues require resolution for the volume-key page-turning change.

No open issues 🎉

_{Review effort: Balanced}

[Copilot]

🔵 Needs a closer look

Native-device validation is needed for hardware-volume interception and restoration across reader, TTS, and navigation transitions because unit tests simulate bridge messages rather than exercise Android key handling.

No open issues 🎉

_{Review effort: Balanced}