fix: make track visibility toggle button clickable (#658)

The eye icon in the track labels was not responding to clicks because
the onClick handler was placed directly on HugeiconsIcon which doesn't
forward click events to the underlying SVG. Fixed by wrapping the icon
in a button element.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Abelino Chavez 2026-01-30 21:13:23 -06:00
parent 905238d424
commit 71bdafac75
1 changed files with 10 additions and 15 deletions

View File

@ -539,20 +539,15 @@ function TrackToggleIcon({
onClick: () => void;
}) {
return (
<>
{isOff ? (
<HugeiconsIcon
icon={icons.off}
className="text-destructive size-4 cursor-pointer"
onClick={onClick}
/>
) : (
<HugeiconsIcon
icon={icons.on}
className="text-muted-foreground size-4 cursor-pointer"
onClick={onClick}
/>
)}
</>
<button
type="button"
onClick={onClick}
className="flex items-center justify-center"
>
<HugeiconsIcon
icon={isOff ? icons.off : icons.on}
className={`size-4 ${isOff ? "text-destructive" : "text-muted-foreground"}`}
/>
</button>
);
}