fix: remove duplicate color dot from connection selector trigger
SelectValue already renders the color dot from the selected item, so the explicit one in the trigger was redundant. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -22,23 +22,13 @@ export function ConnectionSelector() {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
const activeConn = connectedList.find((c) => c.id === activeConnectionId);
|
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<Select
|
<Select
|
||||||
value={activeConnectionId ?? undefined}
|
value={activeConnectionId ?? undefined}
|
||||||
onValueChange={setActiveConnectionId}
|
onValueChange={setActiveConnectionId}
|
||||||
>
|
>
|
||||||
<SelectTrigger className="h-7 w-[200px] text-xs">
|
<SelectTrigger className="h-7 w-[200px] text-xs">
|
||||||
<div className="flex items-center gap-1.5">
|
|
||||||
{activeConn?.color && (
|
|
||||||
<span
|
|
||||||
className="h-2.5 w-2.5 shrink-0 rounded-full"
|
|
||||||
style={{ backgroundColor: activeConn.color }}
|
|
||||||
/>
|
|
||||||
)}
|
|
||||||
<SelectValue placeholder="Select connection" />
|
<SelectValue placeholder="Select connection" />
|
||||||
</div>
|
|
||||||
</SelectTrigger>
|
</SelectTrigger>
|
||||||
<SelectContent>
|
<SelectContent>
|
||||||
{connectedList.map((conn) => (
|
{connectedList.map((conn) => (
|
||||||
|
|||||||
Reference in New Issue
Block a user