fix(web): work log Show more — cursor, transition & scroll#1569
fix(web): work log Show more — cursor, transition & scroll#1569Marve10s wants to merge 7 commits intopingdotgg:mainfrom
Conversation
The raw <button> element was missing cursor-pointer, causing the default arrow cursor on hover instead of the expected hand pointer.
Use the existing Base UI Collapsible component to animate the overflow entries in tool call / work log groups with a 200ms height transition instead of an instant show/hide.
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
59c3df1 to
8e290fc
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Expanding a work log group during a running session would trigger scheduleStickToBottom via the timelineEntries effect, yanking the viewport to the bottom. Disable auto-scroll on toggle so the user stays where they clicked; the scroll-to-bottom button is shown if they want to jump back.
954a90d to
8caccfb
Compare
2026-03-30.12.07.23.mp4Sorry, took a second for Codex and I also needed to approve access to folders for Claude :) |

Summary
cursor-pointer, showing a default arrow on hover instead of a hand. Fixed by switching to the existingCollapsibleTriggercomponent which includes it.Collapsible+CollapsiblePanelcomponent, giving a smooth 200ms height animation when expanding/collapsing overflow entries.scheduleStickToBottom. Now disables auto-scroll on toggle so the user stays where they clicked; the scroll-to-bottom button appears if they want to jump back.Note
Medium Risk
Changes chat timeline scrolling behavior and work-log expansion rendering; regressions could impact viewport stability during active sessions, but scope is limited to UI interaction logic.
Overview
Fixes work log/tool-call group expansion to use the shared
Collapsiblecomponents, adding proper pointer cursor and a smooth height transition while keeping overflow rows mounted for animation.Prevents unwanted jump-to-bottom when expanding/collapsing a work group by cancelling pending stick-to-bottom, disabling auto-scroll, and forcing the scroll-to-bottom affordance to appear.
Reviewed by Cursor Bugbot for commit f4e55ee. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Fix work log 'Show more' cursor, transition, and scroll behavior
Collapsible/CollapsibleTriggercomponents, showing 'Show N more' or 'Show less' based on overflow count.keepMounted) instead of being unmounted, fixing transition behavior.Macroscope summarized f4e55ee.