Skip to content

feat: enable scrolling for long tooltips and timeline charts - #1156

Merged
govindavashishtha merged 3 commits into
mainfrom
bugfix/hv/better-design-for-sessions-with-many-turns
Oct 23, 2026
Merged

govindavashishtha merged 3 commits into
mainfrom
bugfix/hv/better-design-for-sessions-with-many-turns

Conversation

@harshil-2096

@harshil-2096 harshil-2096 commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Closes AGE-2536
image
image

Changes

How was this tested?

Checklist

  • I have read the contributing guidelines
  • pnpm build, pnpm test, pnpm typecheck, pnpm lint:ci, and pnpm format:check pass locally
  • Tests added/updated where it makes sense
  • No hand-edits to generated code (packages/trueforge-sdk, python/trueforge_sdk, .github/fern/openapi/openapi.json, docs/openapi.json) — fork PRs omit SDK regen; maintainers regenerate after merge
  • Docs / .env.example updated if configuration or behavior changed

Note

Low Risk
Agent-session visualization and tooltip layout only; no API or auth changes, with unit tests covering scroll and axis bounds.

Overview
Session timeline can scroll horizontally when many turns or minimum turn-bar width forces the chart wider than the pane. Layout now stretches short turns to a 32px minimum via getTimelineTurnStretch / timelineChartMs, maps segments and axis ticks through that stretched coordinate system, and skips turn labels that would not fit. Axis labels still use rounded duration text while tick density grows on wide charts (~one per 120px).

Metric tooltips cap list height and scroll: cost-per-turn and tool-call breakdowns scroll vertically (truncated labels + title where needed); the context breakdown uses scrollX so long turn labels stay on one line and scroll horizontally inside the popup.

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

@changeset-bot

changeset-bot Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: ede5bde

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@truefoundry/trueforge-ui Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@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 3 potential issues.

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 42b1196. Configure here.

Comment thread packages/trueforge-ui/src/utils/sessionEventTimelineChart.ts Outdated
Comment thread packages/trueforge-ui/src/utils/sessionEventTimelineChart.ts
@govindavashishtha
govindavashishtha merged commit 7fbb394 into main Oct 23, 2026
14 checks passed
@govindavashishtha
govindavashishtha deleted the bugfix/hv/better-design-for-sessions-with-many-turns branch October 23, 2026 13:13
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.

2 participants