Skip to content

Commit ce3bbd9

Browse files
committed
chore: apply changes after review
1 parent b23f833 commit ce3bbd9

File tree

2 files changed

+9
-15
lines changed

2 files changed

+9
-15
lines changed

packages/ui/src/react/components/ui/accordion.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import type { ComponentPropsWithoutRef, ComponentRef } from "react";
66
import * as React from "react";
77
import { cn } from "../../../lib/utils";
88

9-
const Accordion: React.FC<ComponentPropsWithoutRef<typeof AccordionPrimitive.Root>> = AccordionPrimitive.Root;
9+
const Accordion = AccordionPrimitive.Root;
1010

1111
const AccordionItem = React.forwardRef<
1212
ComponentRef<typeof AccordionPrimitive.Item>,

packages/ui/src/react/hooks/use-filtered-sorted-assets.ts

Lines changed: 8 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -6,25 +6,19 @@ import { useDebouncedEffect } from "./use-debounced-effect";
66

77
export function useFilteredSortedAssets() {
88
const { balancesByChain } = useSwapKit();
9-
10-
// internal and public state required to support debouncing
119
const [filters, setFilters] = useState<UseFilteredSortedAssetsFilters>({ searchQuery: "", selectedNetworks: [] });
12-
const [internalFiltersState, setInternalFilterState] = useState(() => filters);
1310

14-
useDebouncedEffect(
15-
() => {
16-
setInternalFilterState(filters);
17-
},
18-
[filters],
19-
500,
20-
);
11+
// internal state for handling debouncing
12+
const [_internalFiltersState, _setInternalFilterState] = useState(() => filters);
13+
14+
useDebouncedEffect(() => _setInternalFilterState(filters), [filters], 500);
2115

2216
const filteredAssets = useMemo(() => {
23-
const filteredAssetsMap = filterAssetsMap({ assetsMap, filters: internalFiltersState });
17+
const filteredAssetsMap = filterAssetsMap({ assetsMap, filters: _internalFiltersState });
2418

2519
Array.from(balancesByChain.values())
2620
?.flat()
27-
.forEach(({ identifier, balance }) => {
21+
?.forEach(({ identifier, balance }) => {
2822
const matchingAsset = filteredAssetsMap.get(identifier);
2923

3024
if (!matchingAsset) return;
@@ -34,8 +28,8 @@ export function useFilteredSortedAssets() {
3428

3529
const assets = Array.from(filteredAssetsMap.values());
3630

37-
return sortAssets({ assets, filters: internalFiltersState });
38-
}, [internalFiltersState, balancesByChain]);
31+
return sortAssets({ assets, filters: _internalFiltersState });
32+
}, [_internalFiltersState, balancesByChain]);
3933

4034
return useMemo(() => ({ assets: filteredAssets, filters, setFilters }), [filters, filteredAssets]);
4135
}

0 commit comments

Comments
 (0)