fix(web): work log Show more — cursor, transition & scroll by Marve10s · Pull Request #1569 · pingdotgg/t3code

Marve10s · 2026-03-30T09:10:26Z

Summary

Cursor pointer: The "Show more" button in work log / tool call groups was missing cursor-pointer, showing a default arrow on hover instead of a hand. Fixed by switching to the existing CollapsibleTrigger component which includes it.
Smooth transition: Replaced the instant show/hide with the existing Base UI Collapsible + CollapsiblePanel component, giving a smooth 200ms height animation when expanding/collapsing overflow entries.
No auto-scroll on expand: Expanding a work log group during a running session would yank the viewport to the bottom via 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 in chat timeline work log expansion behavior; primary risk is subtle regressions in ordering/scroll anchoring when expanding during active sessions.

Overview
Updates WorkGroupSection in MessagesTimeline.tsx to use the shared Collapsible UI components for work-log/tool-call overflow, replacing the manual toggle.

Overflow entries are split into always visible vs collapsible entries, with the hidden portion rendered inside a CollapsiblePanel keepMounted for animated expand/collapse and a CollapsibleTrigger (adds pointer cursor). The toggle now includes data-scroll-anchor-ignore to avoid unwanted auto-scroll/anchor interference when expanding while the timeline is live.

^{Reviewed by Cursor Bugbot for commit dce8655. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Fix work log "Show more" cursor, transition, and scroll behavior in `WorkGroupSection`

Replaces manual expand/collapse logic in MessagesTimeline.tsx with Collapsible, CollapsiblePanel, and CollapsibleTrigger components. Overflow entries (beyond MAX_VISIBLE_WORK_LOG_ENTRIES) are rendered inside a <CollapsiblePanel keepMounted>, while the always-visible entries render outside it. The trigger button gains data-scroll-anchor-ignore to fix scroll behavior, and the toggle count is derived from overflowEntries.length.

^{Macroscope summarized 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.

coderabbitai · 2026-03-30T09:12:30Z

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 61566563-63bd-4404-8f49-1dc9849c60f5

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

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.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

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.

Marve10s · 2026-03-30T09:43:40Z

2026-03-30.12.07.23.mp4

Sorry, took a second for Codex and I also needed to approve access to folders for Claude :)

juliusmarminge · 2026-04-06T17:45:03Z

2026-03-30.12.07.23.mp4
Sorry, 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

Marve10s · 2026-04-06T17:46:35Z

2026-03-30.12.07.23.mp4
Sorry, 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

juliusmarminge · 2026-04-06T17:48:03Z

I didn't test it out but overall seems like a solid change except for that yes

Marve10s · 2026-04-06T20:20:20Z

@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
Also, is there anything I can help with in your draft PRs or any tasks? I'd love to set up T3 on my WSL, saw a draft for this some time ago, right now I'm only running it on Mac

macroscopeapp · 2026-04-06T20:29:25Z

Approvability

Verdict: Approved

A straightforward UI fix replacing a plain button with existing Collapsible UI components to improve cursor, transition, and scroll behavior. The functional logic for showing/hiding overflow entries remains unchanged, and the author has prior contributions to this file.

^{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

…r-and-transition # Conflicts: # apps/web/src/components/ChatView.tsx # apps/web/src/components/chat/MessagesTimeline.test.tsx # apps/web/src/components/chat/MessagesTimeline.tsx

Dismissing prior approval to re-evaluate 79c52c7

cursor

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 f98f234. Configure here.}

Dismissing prior approval to re-evaluate dce8655

Marve10s added 2 commits March 30, 2026 11:52

fix(web): add cursor-pointer to work log "Show more" button

c6e6b1c

The raw <button> element was missing cursor-pointer, causing the default arrow cursor on hover instead of the expected hand pointer.

feat(web): add smooth expand/collapse transition to work log groups

644b808

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.

github-actions bot added size:M 30-99 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list. labels Mar 30, 2026

Marve10s force-pushed the fix/show-more-cursor-and-transition branch from 59c3df1 to 8e290fc Compare March 30, 2026 09:13

cursor bot reviewed Mar 30, 2026

View reviewed changes

Comment thread apps/web/src/components/ChatView.tsx Outdated

Marve10s force-pushed the fix/show-more-cursor-and-transition branch from 954a90d to 8caccfb Compare March 30, 2026 09:29

Merge branch 'main' into fix/show-more-cursor-and-transition

66cb7cf

Marve10s added 4 commits April 2, 2026 11:53

Merge branch 'main' into fix/show-more-cursor-and-transition

1b90c1a

Merge branch 'main' into fix/show-more-cursor-and-transition

420b9c8

Merge branch 'main' into fix/show-more-cursor-and-transition

f4e55ee

Merge branch 'main' into fix/show-more-cursor-and-transition

69d2ff8

Marve10s and others added 2 commits April 6, 2026 23:08

fix(web): stabilize work log expansion scroll

3db43d0

Merge branch 'main' into fix/show-more-cursor-and-transition

6e135ca

macroscopeapp bot previously approved these changes Apr 6, 2026

View reviewed changes

Merge branch 'main' into fix/show-more-cursor-and-transition

703721b

cursor bot reviewed Apr 7, 2026

View reviewed changes

Comment thread apps/web/src/components/ChatView.tsx Outdated

Marve10s and others added 2 commits April 9, 2026 18:20

Merge branch 'main' into fix/show-more-cursor-and-transition

61d72c7

fix(web): add missing timeline test prop

9b05426

macroscopeapp bot previously approved these changes Apr 9, 2026

View reviewed changes

Resolve merge conflicts for PR pingdotgg#1569

e87ae40

Fix typecheck regression from merge conflict resolution

b7ba60b

macroscopeapp bot previously approved these changes Apr 10, 2026

View reviewed changes

Marve10s and others added 2 commits April 12, 2026 19:06

Merge branch 'main' into fix/show-more-cursor-and-transition

ae7fbb2

fix(web): add missing messages timeline test props

3ce44bf

macroscopeapp bot previously approved these changes Apr 12, 2026

View reviewed changes

Merge remote-tracking branch 'upstream/main' into fix/show-more-curso…

79c52c7

…r-and-transition # Conflicts: # apps/web/src/components/ChatView.tsx # apps/web/src/components/chat/MessagesTimeline.test.tsx # apps/web/src/components/chat/MessagesTimeline.tsx

macroscopeapp bot previously approved these changes Apr 13, 2026

View reviewed changes

Merge branch 'main' into fix/show-more-cursor-and-transition

f98f234

cursor bot reviewed Apr 14, 2026

View reviewed changes

Comment thread apps/web/src/components/chat/MessagesTimeline.tsx

Marve10s and others added 2 commits April 15, 2026 14:45

Merge branch 'main' into fix/show-more-cursor-and-transition

4e4f57d

Merge upstream/main into show more branch

dce8655

macroscopeapp bot approved these changes Apr 16, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(web): work log Show more — cursor, transition & scroll#1569

fix(web): work log Show more — cursor, transition & scroll#1569
Marve10s wants to merge 21 commits intopingdotgg:mainfrom
Marve10s:fix/show-more-cursor-and-transition

Marve10s commented Mar 30, 2026 •

edited by macroscopeapp bot

Loading

Uh oh!

coderabbitai bot commented Mar 30, 2026 •

edited

Loading

Review skipped

Uh oh!

Uh oh!

Marve10s commented Mar 30, 2026

Uh oh!

juliusmarminge commented Apr 6, 2026

Uh oh!

Marve10s commented Apr 6, 2026

Uh oh!

juliusmarminge commented Apr 6, 2026

Uh oh!

Marve10s commented Apr 6, 2026

Uh oh!

macroscopeapp bot commented Apr 6, 2026 •

edited

Loading

Uh oh!

Uh oh!

cursor bot left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

Marve10s commented Mar 30, 2026 • edited by macroscopeapp bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Fix work log "Show more" cursor, transition, and scroll behavior in WorkGroupSection

Uh oh!

coderabbitai bot commented Mar 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Uh oh!

Uh oh!

Marve10s commented Mar 30, 2026

Uh oh!

juliusmarminge commented Apr 6, 2026

Uh oh!

Marve10s commented Apr 6, 2026

Uh oh!

juliusmarminge commented Apr 6, 2026

Uh oh!

Marve10s commented Apr 6, 2026

Uh oh!

macroscopeapp bot commented Apr 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Approvability

Uh oh!

Uh oh!

cursor bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Marve10s commented Mar 30, 2026 •

edited by macroscopeapp bot

Loading

Fix work log "Show more" cursor, transition, and scroll behavior in `WorkGroupSection`

coderabbitai bot commented Mar 30, 2026 •

edited

Loading

macroscopeapp bot commented Apr 6, 2026 •

edited

Loading