Skip to content

fix(desktop): highlight active notes in sidebar timeline#5433

Merged
ComputelessComputer merged 1 commit into
mainfrom
fix/sidebar-active-note-indicator
Jun 1, 2026
Merged

fix(desktop): highlight active notes in sidebar timeline#5433
ComputelessComputer merged 1 commit into
mainfrom
fix/sidebar-active-note-indicator

Conversation

@ComputelessComputer
Copy link
Copy Markdown
Collaborator

@ComputelessComputer ComputelessComputer commented Jun 1, 2026

Show active recording notes as red rows in sidebar timeline mode with a live activity glyph and stop control.


Note

Low Risk
Desktop sidebar UI and stop wiring only; no auth, data, or backend changes.

Overview
Active recording sessions in the sidebar timeline now stand out with a red row and white text instead of the usual neutral selection styling.

Each live session row shows a live activity control (DancingSticks driven by mic/speaker amplitude) that becomes a Stop listening action on hover; stopping calls the listener stop handler without opening the note tab. Finalizing/enhancing spinners are suppressed while a session is live.

A Vitest case in item.test.tsx locks in the red styling, amplitude wiring, and stop behavior.

Reviewed by Cursor Bugbot for commit 50f9c55. Bugbot is set up for automated code reviews on this repo. Configure here.

Show active recording notes as red rows in sidebar timeline mode with a live activity glyph and stop control.
@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 1, 2026

Deploy Preview for old-char canceled.

Name Link
🔨 Latest commit 50f9c55
🔍 Latest deploy log https://app.netlify.com/projects/old-char/deploys/6a1d696f1f08490007fb91ca

@ComputelessComputer ComputelessComputer merged commit d34c80e into main Jun 1, 2026
10 checks passed
@ComputelessComputer ComputelessComputer deleted the fix/sidebar-active-note-indicator branch June 1, 2026 11:31
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