Skip to content

bugfix: add scroll to dropdown of themes list - #7539

Open
atwof wants to merge 5 commits into
usebruno:mainfrom
atwof:bugix/add-scroll-themes-list
Open

bugfix: add scroll to dropdown of themes list#7539
atwof wants to merge 5 commits into
usebruno:mainfrom
atwof:bugix/add-scroll-themes-list

Conversation

@atwof

@atwof atwof commented Mar 22, 2026

Copy link
Copy Markdown

Description

Added some style to enable the vertically scroll when having more than certain quantity of themes in the statusbar dropdown selector. As i added Kanagawa theme, I saw it was needed.

Summary by CodeRabbit

  • Style
    • Improved the theme dropdown: the theme list now supports internal scrolling to prevent excessive length, and the list header remains visible and visually consistent with the dropdown while scrolling for easier and clearer theme selection.

@atwof atwof changed the title feat: add add scrol to the list of themes feat: add add scroll to the list of themes Mar 22, 2026
@coderabbitai

coderabbitai Bot commented Mar 22, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 62f414fa-c8f0-4555-a408-5e966f174d01

📥 Commits

Reviewing files that changed from the base of the PR and between e2a73c0 and 7a25527.

📒 Files selected for processing (1)
  • packages/bruno-app/src/components/StatusBar/ThemeDropdown/StyledWrapper.js
🚧 Files skipped from review as they are similar to previous changes (1)
  • packages/bruno-app/src/components/StatusBar/ThemeDropdown/StyledWrapper.js

Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review.


Walkthrough

The theme dropdown list now has a 250px maximum height and internal vertical scrolling. Its label uses the dropdown background and remains visible while the list scrolls.

Changes

Theme dropdown styling

Layer / File(s) Summary
Scrollable theme list
packages/bruno-app/src/components/StatusBar/ThemeDropdown/StyledWrapper.js
.theme-list now uses max-height: 250px and overflow-y: auto. .theme-list-label uses the dropdown background and sticky positioning.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Merge Risk: ⚪ Minimal · up to 7a255

The theme selector now bounds its list height and scrolls vertically, keeping the label visible. No current merge-blocking risk is identified.

Poem

The theme list now scrolls with care
Its label stays fixed in the air
A matching background guards the view
The dropdown remains neat and true
Small CSS changes make navigation clear

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: adding scrolling to the theme dropdown list.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
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.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@atwof atwof changed the title feat: add add scroll to the list of themes feat: add add scroll to dronpdown of themes list Mar 22, 2026
@atwof atwof changed the title feat: add add scroll to dronpdown of themes list feat: add add scroll to dropdown of themes list Mar 22, 2026
@atwof atwof changed the title feat: add add scroll to dropdown of themes list bugfix: add add scroll to dropdown of themes list Mar 22, 2026
@atwof atwof changed the title bugfix: add add scroll to dropdown of themes list bugfix: ad scroll to dropdown of themes list Mar 22, 2026
@atwof atwof changed the title bugfix: ad scroll to dropdown of themes list bugfix: add scroll to dropdown of themes list Mar 22, 2026
@coderabbitai

coderabbitai Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Note

GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants