Skip to content

fix: Remove double scrollbar from repo selector#1537

Merged
charlesvien merged 1 commit intomainfrom
04-07-remove_double_scrollbar_from_repo_selector
Apr 8, 2026
Merged

fix: Remove double scrollbar from repo selector#1537
charlesvien merged 1 commit intomainfrom
04-07-remove_double_scrollbar_from_repo_selector

Conversation

@charlesvien
Copy link
Copy Markdown
Member

@charlesvien charlesvien commented Apr 7, 2026

Problem

Repo selector dropdown shows two scrollbars when the list overflows.

Closes #1490

Changes

  1. Add overflow: hidden to .rt-PopoverContent combobox reset so the outer wrapper never scrolls
  2. Remove redundant inline maxHeight: "300px" from GitHubRepoPicker, deferring to the existing [cmdk-root] CSS constraint

How did you test this?

Manually

@charlesvien charlesvien force-pushed the 04-07-remove_double_scrollbar_from_repo_selector branch from 074bba6 to 7e29b50 Compare April 8, 2026 22:55
Copy link
Copy Markdown
Member Author

charlesvien commented Apr 8, 2026

Merge activity

  • Apr 8, 11:25 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Apr 8, 11:25 PM UTC: @charlesvien merged this pull request with Graphite.

@charlesvien charlesvien merged commit a2b92ee into main Apr 8, 2026
16 checks passed
@charlesvien charlesvien deleted the 04-07-remove_double_scrollbar_from_repo_selector branch April 8, 2026 23:25
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.

repo selector has two scrollbars

2 participants