Skip to content

Commit c1e4fc9

Browse files
Copilotalexr00
andauthored
Hide dropdown separator when no extra options (#8763)
* Initial plan * Hide merge button separator when no extra options Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
1 parent faff7a4 commit c1e4fc9

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

webviews/components/dropdown.tsx

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,20 +83,21 @@ export const Dropdown = ({ options, defaultOption, disabled, submitAction, chang
8383
}
8484
};
8585

86-
const expandButtonClass = Object.entries(options).length === 1 ? 'hidden' : areOptionsVisible ? 'open' : '';
86+
const hasMultipleOptions = Object.entries(options).length > 1;
87+
const expandButtonClass = !hasMultipleOptions ? 'hidden' : areOptionsVisible ? 'open' : '';
8788

8889
return (
8990
<div className="select-container" onKeyDown={onKeyDown}>
9091
<div className="select-control">
9192
<Confirm
9293
dropdownId={dropdownId}
93-
className={Object.keys(options).length > 1 ? 'select-left' : ''}
94+
className={hasMultipleOptions ? 'select-left' : ''}
9495
options={options}
9596
selected={selectedMethod}
9697
submitAction={submitAction}
9798
disabled={!!disabled}
9899
/>
99-
<div className={`split${disabled ? ' disabled' : ''}`}><div className={`separator${disabled ? ' disabled' : ''}`}></div></div>
100+
{hasMultipleOptions ? <div className={`split${disabled ? ' disabled' : ''}`}><div className={`separator${disabled ? ' disabled' : ''}`}></div></div> : null}
100101
<button id={EXPAND_OPTIONS_BUTTON} className={'select-right ' + expandButtonClass} aria-label='Expand button options' onClick={onClick}>
101102
{chevronDownIcon}
102103
</button>

0 commit comments

Comments
 (0)