Skip to content

feat(webapp): fix agent overview page scroll bug + layout fixes on task and agent pages - #4454

Merged
samejr merged 3 commits into
mainfrom
samejr/task-landing-page-layout-improvements
Aug 1, 2026
Merged

feat(webapp): fix agent overview page scroll bug + layout fixes on task and agent pages#4454
samejr merged 3 commits into
mainfrom
samejr/task-landing-page-layout-improvements

Conversation

@samejr

@samejr samejr commented Aug 1, 2026

Copy link
Copy Markdown
Member

Summary

The task, scheduled task and agent pages now name their runs table with its own title bar, and the controls that page the table sit beside it rather than in the bar at the top of the page. The top bar keeps just the date filter.

Two agent page layout bugs are fixed along the way: scrolling a wide runs table sideways dragged the charts off screen with it, and the details panel stopped short of the bottom of the window.

Fix

The charts moved because the runs table had no horizontal scroller of its own. stickyHeader swaps the table's overflow-x-auto for overflow-visible, so the overflow escaped up to the page scroll box, and setting only overflow-y-auto on that box leaves the computed overflow-x at visible, which CSS then promotes to auto. The chart grid is a sibling inside that box, so it scrolled too. The table now keeps its own scroller (the same rule the queues list already documents) and the page box clips x so this cannot recur.

The short panel was a second PageContainer wrapping the agent routes. PageContainer is grid-rows-[auto_1fr], so a lone child lands in the auto row and its h-full resolves against content height instead of the viewport.

This also reverts the global tooltip max-w-[230px] introduced in #4131, so longer tooltips are no longer squeezed into a narrow column.

Agent overview page showing table now scrolling

CleanShot 2026-08-01 at 12 04 38@2x

samejr added 2 commits August 1, 2026 13:31
The task, scheduled task and agent pages now name their runs table with
its own title bar, with the page controls beside the table they act on
rather than in the top bar.

Also fixes two agent page layout bugs: scrolling a wide runs table
sideways dragged the charts off screen with it, and the details panel
stopped short of the bottom of the window. Reverts the global tooltip
max-width so longer tooltips are no longer squeezed into a column.
@changeset-bot

changeset-bot Bot commented Aug 1, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: dcb5592

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@coderabbitai

coderabbitai Bot commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

Added title-style tab and header primitives, including the public TitleBar component. Updated agent and task run pages to move controls and pagination into table headers. Removed the agent route container wrapper, disabled sticky table headers, and constrained table scrolling. Updated metrics overflow handling, exported headerVariants, and removed the tooltip width limit.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 36.36% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ⚠️ Warning The description explains the changes and includes a screenshot, but it omits the issue reference, checklist, testing section, and changelog required by the template. Add the required issue reference, checklist, testing steps, and changelog section; confirm which checklist items were completed.
✅ Passed checks (3 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly summarizes the agent scroll fix and related layout changes on task and agent pages.
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch samejr/task-landing-page-layout-improvements

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.

devin-ai-integration[bot]

This comment was marked as resolved.

@samejr samejr changed the title feat(webapp): runs title bar and layout fixes on task and agent pages feat(webapp): fix agent overview page scroll bug + layout fixes on task and agent pages Aug 1, 2026
@samejr
samejr merged commit 14824b0 into main Aug 1, 2026
33 checks passed
@samejr
samejr deleted the samejr/task-landing-page-layout-improvements branch August 1, 2026 15:26
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