Skip to content

improvement(chat): hide the agent-group viewport scrollbar#5664

Merged
waleedlatif1 merged 1 commit into
stagingfrom
improvement/chat-hide-group-scrollbar
Jul 14, 2026
Merged

improvement(chat): hide the agent-group viewport scrollbar#5664
waleedlatif1 merged 1 commit into
stagingfrom
improvement/chat-hide-group-scrollbar

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Hide the scrollbar on the agent-group bounded viewport (the 110px narration window) using the existing scrollbar-hide utility
  • The viewport is an ambient auto-following activity log — overflow is already signaled by the top/bottom fade edges, and wheel/trackpad scrolling (including detach/re-stick) still works

Type of Change

  • Improvement

Testing

Tested manually

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Jul 14, 2026 2:23am

Request Review

@cursor

cursor Bot commented Jul 14, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Single-class styling change on a chat UI scroll region with no logic or data-path changes.

Overview
Adds the existing scrollbar-hide utility to the BoundedViewport scroll container in agent-group.tsx (the 110px capped narration area inside expanded agent groups).

The native scrollbar is hidden so the log reads more like an ambient activity feed; overflow is still indicated by the top/bottom fade gradients, and wheel/trackpad scrolling plus auto-stick-to-bottom behavior are unchanged.

Reviewed by Cursor Bugbot for commit 9f299c0. Configure here.

@greptile-apps

greptile-apps Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR hides the agent-group activity viewport scrollbar. The main changes are:

  • Adds the existing scrollbar-hide utility to the bounded 110px log viewport.
  • Keeps the existing overflow, padding, and fade-edge behavior unchanged.

Confidence Score: 5/5

This looks safe to merge after considering one small usability tradeoff.

  • The added class already exists in the app stylesheet.
  • The scroll APIs used by the viewport stay unchanged.
  • The remaining concern is limited to losing the visible scrollbar thumb for manual navigation.

Important Files Changed

Filename Overview
apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/agent-group/agent-group.tsx Adds scrollbar-hide to the bounded agent-group viewport while leaving the existing scroll logic unchanged.

Reviews (1): Last reviewed commit: "improvement(chat): hide the agent-group ..." | Re-trigger Greptile

@waleedlatif1 waleedlatif1 merged commit 7877713 into staging Jul 14, 2026
18 checks passed
@waleedlatif1 waleedlatif1 deleted the improvement/chat-hide-group-scrollbar branch July 14, 2026 02:40
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cursor review

@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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 9f299c0. Configure here.

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.

1 participant