Skip to content

fix(timeline): move meetings to top timeline#5340

Merged
ComputelessComputer merged 1 commit into
mainfrom
fix/top-meeting-timeline
May 28, 2026
Merged

fix(timeline): move meetings to top timeline#5340
ComputelessComputer merged 1 commit into
mainfrom
fix/top-meeting-timeline

Conversation

@ComputelessComputer
Copy link
Copy Markdown
Collaborator

@ComputelessComputer ComputelessComputer commented May 27, 2026

Replace the default sidebar timeline with a top meeting timeline, add 30-minute duration bars with pan and zoom support, and keep custom sidebars intact.


Note

Medium Risk
Large UI shell change plus session delete/open and calendar-ignore paths on the new timeline; sidebar lock/unmount behavior was adjusted for custom tabs.

Overview
Moves the default meetings experience from the left sidebar into a horizontal top carousel (TopMeetingTimeline) under tab chrome when the sidebar is expanded and the tab is not onboarding, devtool, or a custom-sidebar type (calendar, settings, etc.).

Layout & chrome: ClassicMainBody conditionally renders the top timeline and relocates ToastArea to a floating corner; ClassicMainSidebar no longer shows LeftSidebar for the default case—only devtool or custom-sidebar tabs. The sidebar toggle uses top-panel icons and “Toggle timeline” when that mode applies.

Timeline behavior: New helpers default missing/invalid ends to 30 minutes (normalizeEndMs) and derive session spans from transcript rows (buildSessionRecordingRanges). Cards support open/delete/ignore flows, previews, a Today jump chip, and horizontal scroll (including vertical wheel → horizontal). Calendar events skip all-day rows; future events only; session/event deduping by tracking id.

Supporting fixes: showNativeContextMenu extracted for reuse; useCustomSidebarEffect cleans up on unmount; session preview omits blank participant names; tests updated for the new layout.

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

@netlify
Copy link
Copy Markdown

netlify Bot commented May 27, 2026

Deploy Preview for old-char canceled.

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

Comment thread apps/desktop/src/main/top-meeting-timeline.tsx
@ComputelessComputer ComputelessComputer force-pushed the fix/top-meeting-timeline branch from 6eecece to 30844a6 Compare May 27, 2026 16:33
Comment thread apps/desktop/src/main/top-meeting-timeline.tsx
Comment thread apps/desktop/src/main/top-meeting-timeline.tsx Outdated
@ComputelessComputer ComputelessComputer force-pushed the fix/top-meeting-timeline branch from 30844a6 to ec9d7de Compare May 27, 2026 18:17
Comment thread apps/desktop/src/main/top-meeting-timeline.tsx
Comment thread apps/desktop/src/main/meeting-timeline-layout.ts Outdated
@ComputelessComputer ComputelessComputer force-pushed the fix/top-meeting-timeline branch 2 times, most recently from 6dd93d1 to 76c471e Compare May 27, 2026 19:48
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 76c471e. Configure here.

Comment thread apps/desktop/src/main/meeting-timeline-layout.ts Outdated
Replace the time-axis rail with compact date-ordered carousel cards, group dense meetings, and keep note and event context menus on cards.
@ComputelessComputer ComputelessComputer force-pushed the fix/top-meeting-timeline branch from 76c471e to c6e9d90 Compare May 28, 2026 02:46
@ComputelessComputer ComputelessComputer merged commit b94bf8d into main May 28, 2026
15 checks passed
@ComputelessComputer ComputelessComputer deleted the fix/top-meeting-timeline branch May 28, 2026 04:21
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