fix(explore/ui): chain switcher popover opens (remove double-toggle) (v1.1.6)

ChainSwitcher put onClick={handleToggle} on the PopoverTrigger's button, which
double-toggled against PopoverTrigger's own open/close handling (net: stays
closed) — the switcher never opened, so no chains were selectable. Removed the
redundant onClick + unused handleToggle, matching the working popover pattern
(UserProfileDesktop). PopoverTrigger now drives open state via onOpenChange.

Co-authored-by: Hanzo Dev <dev@hanzo.ai>
This commit is contained in:
zeekay
2026-07-14 08:47:39 -07:00
co-authored by Hanzo Dev
parent 2180e95b0b
commit fa15c73096
2 changed files with 2 additions and 7 deletions
+1 -1
View File
@@ -1,6 +1,6 @@
{
"name": "@luxfi/explore",
"version": "1.1.5",
"version": "1.1.6",
"private": false,
"homepage": "https://github.com/luxfi/explore#readme",
"engines": {
+1 -6
View File
@@ -1,9 +1,9 @@
import { PopoverBody, PopoverContent, PopoverRoot, PopoverTrigger } from '@luxfi/ui/popover';
import React from 'react';
import { getCurrentChain, getCurrentNetwork, getChainsForNetwork, NETWORKS } from 'configs/app/chainRegistry';
import { cn } from 'lib/utils/cn';
import { Link } from 'toolkit/next/link';
import { PopoverBody, PopoverContent, PopoverRoot, PopoverTrigger } from '@luxfi/ui/popover';
// DropdownNavigator carries per-item state for the anchor's onClick so the
// handler reference stays stable across renders (satisfies react/jsx-no-bind)
@@ -30,10 +30,6 @@ const ChainSwitcher = () => {
setOpen(isOpen);
}, []);
const handleToggle = React.useCallback(() => {
setOpen((prev) => !prev);
}, []);
const closePopover = React.useCallback(() => {
setOpen(false);
}, []);
@@ -80,7 +76,6 @@ const ChainSwitcher = () => {
'dark:hover:bg-[var(--color-whiteAlpha-50)]',
'transition-all duration-150',
) }
onClick={ handleToggle }
>
<svg
xmlns="http://www.w3.org/2000/svg"