Hide dropdown separator when no extra options#8763
Open
Copilot wants to merge 2 commits into
Open
Conversation
Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix separator visibility for merge button with no extra options
Hide dropdown separator when no extra options
Jun 1, 2026
alexr00
approved these changes
Jun 3, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR fixes a small UI polish issue in the merge/action dropdown webview: when there is only a single option available (and the expand chevron is hidden), the vertical split/separator is now also hidden so the button no longer shows a stray divider.
Changes:
- Introduced a
hasMultipleOptionsboolean to centralize the “more than one option” check. - Render the
split/separatorelement only when multiple dropdown options exist. - Reused
hasMultipleOptionsfor both the left-button styling (select-left) and expand-button visibility class to keep behavior consistent.
Show a summary per file
| File | Description |
|---|---|
webviews/components/dropdown.tsx |
Conditionals updated so the split/separator only renders when multiple options exist, avoiding a stray divider in the single-option case. |
Copilot's findings
- Files reviewed: 1/1 changed files
- Comments generated: 0
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The merge/action dropdown rendered its vertical separator unconditionally, leaving a stray divider on the right edge of the button when only a single option was available (the expand chevron is already hidden in this case).
Changes
webviews/components/dropdown.tsx: only render thesplit/separatorelement whenoptionshas more than one entry.hasMultipleOptionsconstant and reused it for the existingselect-leftclass andexpandButtonClasschecks to keep the three branches consistent.