Skip to content

Bugfix: Fix double scrollbar issue in agent selector #3504 - #3505

Merged
Dallas98 merged 2 commits into
developfrom
xyq/bugfix_front
Jul 27, 2026
Merged

Bugfix: Fix double scrollbar issue in agent selector #3504#3505
Dallas98 merged 2 commits into
developfrom
xyq/bugfix_front

Conversation

@xuyaqist

@xuyaqist xuyaqist commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

#3504
修复agent下拉框有两个滚动条的问题
image

@xuyaqist
xuyaqist requested a review from WMC001 as a code owner July 27, 2026 07:11
Copilot AI review requested due to automatic review settings July 27, 2026 07:11
@xuyaqist
xuyaqist requested a review from Dallas98 as a code owner July 27, 2026 07:11
@xuyaqist xuyaqist changed the title Bugfix: Fix double scrollbar issue in agent selector Bugfix: Fix double scrollbar issue in agent selector #3504 Jul 27, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes the “double scrollbar” UX issue in the Agents page’s agent selector dropdown by ensuring scrolling is handled only by the custom list container inside the dropdown.

Changes:

  • Adds a scoped wrapper class to the agent selector dropdown popup to target CSS overrides.
  • Overrides Ant Design dropdown menu overflow/max-height so the menu itself doesn’t create an extra scrollbar.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
frontend/styles/globals.css Adds a scoped CSS override to disable scrolling/max-height on the AntD dropdown menu within the agent selector popup.
frontend/app/[locale]/agents/components/AgentSelectorHeader.tsx Adds a root class hook (agent-selector-dropdown) on the dropdown popup for the new CSS override to apply.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Dallas98
Dallas98 merged commit 2c47784 into develop Jul 27, 2026
12 checks passed
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.

3 participants