Skip to content

feat(overlay): add recording overlay with waveform visualization#502

Open
NathanSkene wants to merge 1 commit intoOpenWhispr:mainfrom
NathanSkene:pr/recording-overlay
Open

feat(overlay): add recording overlay with waveform visualization#502
NathanSkene wants to merge 1 commit intoOpenWhispr:mainfrom
NathanSkene:pr/recording-overlay

Conversation

@NathanSkene
Copy link
Copy Markdown
Contributor

Summary

  • Adds a small always-on-top overlay window during recording with live audio waveform
  • Overlay follows cursor across monitors for multi-display support
  • Toggle in Settings to enable/disable the overlay
  • Includes audio level forwarding from main process to overlay renderer

Test plan

  • Enable overlay in Settings
  • Start recording - overlay appears with waveform
  • Move between monitors - overlay follows
  • Stop recording - overlay disappears
  • Disable overlay in Settings - no overlay during recording

Adds a small always-on-top overlay window that appears during recording,
showing a live audio waveform. The overlay follows the cursor across
monitors and can be toggled in settings.

- New RecordingOverlay component with real-time waveform canvas
- RecordingOverlayWindow managed by windowManager
- Audio level forwarding from main process to overlay renderer
- Settings toggle for enabling/disabling overlay
@NathanSkene
Copy link
Copy Markdown
Contributor Author

Manual validation update on macOS with the packaged installed app:

This feature passed validation.

What I verified:

  • the recording overlay appears during recording
  • the waveform is visible and responsive
  • overlay positioning follows the configured setting
  • the overlay behavior works in the real installed app, not just in development

Follow-up note: PR #506 contains the refinement layer I actually want to ship day-to-day (smaller pill, bottom positioning, duplicate indicator suppression), but the base overlay behavior in this PR itself is working.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant