feat(webapp): fix agent overview page scroll bug + layout fixes on task and agent pages - #4454
Conversation
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.
…age-layout-improvements
|
WalkthroughAdded title-style tab and header primitives, including the public 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 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 |
…age-layout-improvements
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.
stickyHeaderswaps the table'soverflow-x-autoforoverflow-visible, so the overflow escaped up to the page scroll box, and setting onlyoverflow-y-autoon that box leaves the computedoverflow-xatvisible, which CSS then promotes toauto. 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
PageContainerwrapping the agent routes.PageContainerisgrid-rows-[auto_1fr], so a lone child lands in theautorow and itsh-fullresolves 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