Skip to content

Flatten transcript scroll chips#5862

Merged
ComputelessComputer merged 2 commits into
mainfrom
fix/transcript-scroll-chip-spacing
Jul 1, 2026
Merged

Flatten transcript scroll chips#5862
ComputelessComputer merged 2 commits into
mainfrom
fix/transcript-scroll-chip-spacing

Conversation

@ComputelessComputer

@ComputelessComputer ComputelessComputer commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Summary:

  • Tighten transcript scroll chip spacing and add directional icons
  • Align the Swift floating panel bottom chip with the flat transcript style
  • Preserve user scroll position by preventing bottom snapping when content changes after manual scroll

Testing:

  • Not run in this PR creation pass

Note

Low Risk
UI and transcript scroll UX only; behavior is covered by new/updated unit tests with no auth, data, or security impact.

Overview
Desktop transcript scroll chips get a flatter look: bordered muted pills with up/down icons, tighter insets (1.5rem), and top chips anchored at the top instead of only bottom-offset styling. Gradient, shadow, and scale hover effects are removed; floating-chat CSS nudges the bottom chip offset to 3.75rem.

Scroll behavior changes so manual “scroll away” isn’t lost when a session toggles inactive/active: useScrollDetection takes currentActive and only resets auto-scroll on re-activation if the user hadn’t scrolled away. useAutoScroll re-pins when auto-scroll turns back on and skips resize-driven snaps when auto-scroll is off, so new transcript content doesn’t yank the viewport after the user scrolled up.

Windows Swift floating bar aligns its bottom chip with the same flat capsule styling (“Go to bottom”), drops extra shadow/padding, and simplifies scroll pinning observers (bounds-only).

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

@netlify

netlify Bot commented Jul 1, 2026

Copy link
Copy Markdown

Deploy Preview for old-char canceled.

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

--transcript-scroll-chip-bottom: calc(5rem + env(safe-area-inset-bottom));
--transcript-scroll-chip-bottom: calc(
3.75rem + env(safe-area-inset-bottom)
);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Chat hover chip clearance removed

Medium Severity

When the bottom data-chat-cta-trigger is hovered or focused, --transcript-scroll-chip-bottom now matches the default 3.75rem instead of the previous 5rem. The expanded chat bar and the transcript scroll chip can overlap, making either control harder to tap while both are visible.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 9b951b4. Configure here.

@ComputelessComputer ComputelessComputer force-pushed the fix/transcript-scroll-chip-spacing branch 3 times, most recently from 4f73192 to d63eb3b Compare July 1, 2026 18:00
Tighten the transcript scroll chip spacing, add directional icons, and align the Swift floating panel bottom chip with the same flat style.
@ComputelessComputer ComputelessComputer force-pushed the fix/transcript-scroll-chip-spacing branch from d63eb3b to f04d9e6 Compare July 1, 2026 18:16
Keep the Swift live transcript panel from re-enabling bottom snapping when transcript content changes while the user has scrolled away.
@ComputelessComputer ComputelessComputer force-pushed the fix/transcript-scroll-chip-spacing branch from f04d9e6 to 9f5c71a Compare July 1, 2026 18:26

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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 using default effort and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

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 9f5c71a. Configure here.

}
lastHeightRef.current = nextHeight;
schedule();
schedule(enabled);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

schedule treats enabled as force

High Severity

In useAutoScroll, schedule’s first parameter is force, but the new calls pass enabled. When auto-scroll is on, that becomes schedule(true), which skips the enabled and isPinned() guards and always jumps to the bottom on dependency updates and resizes, even if the user is not pinned.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 9f5c71a. Configure here.

@ComputelessComputer ComputelessComputer merged commit a2bcb91 into main Jul 1, 2026
13 checks passed
@ComputelessComputer ComputelessComputer deleted the fix/transcript-scroll-chip-spacing branch July 1, 2026 18:46
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