Skip to content

fix(tts): skip note tooltip text during narration#453

Open
codedogQBY wants to merge 1 commit into
mainfrom
codex/fix-tts-skip-note-tooltips
Open

fix(tts): skip note tooltip text during narration#453
codedogQBY wants to merge 1 commit into
mainfrom
codex/fix-tts-skip-note-tooltips

Conversation

@codedogQBY

@codedogQBY codedogQBY commented Jun 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • mark desktop note tooltip DOM as TTS-skippable
  • add note tooltip selectors to the shared TTS skip list
  • mirror the skip list in the Expo reader template and rebuild the generated mobile reader asset
  • add a focused core test for injected note tooltip filtering

Refs #149

Verification

  • pnpm --filter @readany/app-expo run build:reader
  • pnpm --filter @readany/core test -- text-utils
  • pnpm --filter @readany/core exec tsc --noEmit
  • pnpm --filter @readany/app-expo exec tsc --noEmit
  • pnpm --filter app exec tsc --noEmit
  • pnpm exec biome check packages/core/src/tts/text-utils.ts packages/core/src/tts/text-utils.test.ts
  • git diff --check

Note

This covers the #149 observation where TTS can read user notes/comments injected into the reader document. The repeat/jump symptoms still need validation with the existing TTS navigation PRs, so #149 should stay open.

@codedogQBY codedogQBY added bug Something isn't working priority:p1 High: important feature broken or major platform/workflow regression area:reader Reader, pagination, scrolling, layout, TOC area:tts Text-to-speech and read-aloud playback labels Jun 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:reader Reader, pagination, scrolling, layout, TOC area:tts Text-to-speech and read-aloud playback bug Something isn't working priority:p1 High: important feature broken or major platform/workflow regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant