Skip to content

[codex] Fix playback pause and scroll behavior - #93

Merged
Talljack merged 1 commit into
mainfrom
codex/fix-playback-pause-and-scroll
Jul 29, 2026
Merged

Talljack merged 1 commit into
mainfrom
codex/fix-playback-pause-and-scroll

Conversation

@Talljack

Copy link
Copy Markdown
Owner

What changed

  • Keep Listen controls in the visible viewport with an independently scrollable transcript.
  • Preserve word and sentence highlights while paused; resume the existing browser or audio playback instead of restarting.
  • Align browser speech boundaries with their real character offsets and add regression coverage.
  • Make Write references and typing areas independently scrollable, and keep Speak conversations pinned to the latest message.

Root cause

Listen Pause called the destructive TTS stop path, which canceled the utterance and cleared the playback state. Highlight rendering was also conditioned on active playback.

Validation

  • pnpm test src/components/read-aloud/read-aloud-content.test.tsx src/lib/read-aloud-playback.test.ts
  • PLAYWRIGHT_BASE_URL=http://127.0.0.1:3119 PLAYWRIGHT_REUSE_EXISTING_SERVER=true pnpm exec playwright test e2e/listen.spec.ts --workers=1 --reporter=line
  • pnpm typecheck
  • pnpm lint
  • pnpm build

@vercel

vercel Bot commented Jul 29, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
echo-type Ready Ready Preview, Comment Jul 29, 2026 1:26pm

@Talljack
Talljack marked this pull request as ready for review July 29, 2026 13:30
@Talljack
Talljack merged commit d2901cf into main Jul 29, 2026
5 checks passed
@Talljack
Talljack deleted the codex/fix-playback-pause-and-scroll branch July 29, 2026 13:30

This branch was successfully deployed

1 active deployment
Preview — d7bf9d9a Deployed Jul 29, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant