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 into
Conversation
|
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
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 :) |
scroll behavior is kinda jank when it sometimes expands up and sometimes down. your scroll position shouldn't change when expanding: CleanShot.2026-04-06.at.10.44.49.mp4 |
I can address that in next commit if you're okay with this change overall |
|
I didn't test it out but overall seems like a solid change except for that yes |
|
@juliusmarminge should be good to go , not sure if I should tag you or if you'd come back to this PR eventually. Thank you for reviewing and helping make the software better |
ApprovabilityVerdict: Approved This is a self-contained UI polish fix that replaces manual expand/collapse logic with existing Collapsible UI components, adding proper cursor styling, smooth transitions, and scroll anchor behavior. No runtime behavior changes beyond UX improvements. You can customize Macroscope's approvability policy. Learn more. |
Dismissing prior approval to re-evaluate 9b05426
Dismissing prior approval to re-evaluate e87ae40
Dismissing prior approval to re-evaluate 3ce44bf
Dismissing prior approval to re-evaluate 79c52c7
Dismissing prior approval to re-evaluate dce8655
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.
dce8655 to
b154fb4
Compare
Dismissing prior approval to re-evaluate b154fb4
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.
Reviewed by Cursor Bugbot for commit b154fb4. Configure here.
Dismissing prior approval to re-evaluate 62553cd

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
Low Risk
UI-only change to work log expand/collapse behavior plus a small test; minimal risk beyond potential minor rendering/interaction regressions.
Overview
Improves work log/tool-call group expansion in
MessagesTimelineby switching the overflow UI to the sharedCollapsiblecomponents, enabling the built-in cursor styling and smooth height transition.Work entries are now split into always-visible vs overflow sections (with overflow kept mounted inside
CollapsiblePanel), and the expand/collapse control is marked withdata-scroll-anchor-ignoreto avoid interfering with scroll anchoring.Adds a regression test ensuring the "Show N more" control renders and includes the scroll-anchor ignore attribute.
Reviewed by Cursor Bugbot for commit 557faca. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Fix work log 'Show more' cursor, transition, and scroll anchor in
WorkGroupSectionWorkGroupSectionin MessagesTimeline.tsx to useCollapsible/CollapsibleTrigger/CollapsiblePanelcomponents instead of a manual button toggle.overflowEntries(hidden when collapsed) andalwaysVisibleEntries(always visible), with overflow entries kept mounted viakeepMountedbut hidden until expanded.data-scroll-anchor-ignoreso it is excluded from scroll anchoring logic.data-scroll-anchor-ignoreattribute.Macroscope summarized 557faca.