Skip to content

Commit bc86184

Browse files
committed
chore: move onClick handlers to functions
1 parent 27f3c1e commit bc86184

File tree

4 files changed

+46
-40
lines changed

4 files changed

+46
-40
lines changed

src/components/FindWallet/LanguageSupportFilter.tsx

Lines changed: 11 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,16 @@ export const LanguageSupportFilter = () => {
5454
})
5555
}
5656

57+
const handleTopLanguageClick = (language) => {
58+
setSupportedLanguage(language.code)
59+
60+
trackCustomEvent({
61+
eventCategory: "WalletFilterSidebar",
62+
eventAction: `Language search`,
63+
eventName: getLanguageCodeName(language.code, locale!),
64+
})
65+
}
66+
5767
return (
5868
<AccordionItem
5969
background="background.highlight"
@@ -123,18 +133,7 @@ export const LanguageSupportFilter = () => {
123133
color="primary.base"
124134
size="sm"
125135
cursor="pointer"
126-
onClick={() => {
127-
setSupportedLanguage(language.code)
128-
129-
trackCustomEvent({
130-
eventCategory: "WalletFilterSidebar",
131-
eventAction: `Language search`,
132-
eventName: getLanguageCodeName(
133-
language.code,
134-
locale!
135-
),
136-
})
137-
}}
136+
onClick={handleTopLanguageClick}
138137
>
139138
{language.langName}
140139
</Text>

src/components/FindWallet/MobileFiltersButton.tsx

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,16 @@ export const MobileFiltersButton = ({
3333
// Context API
3434
const { supportedLanguage } = useContext(WalletSupportedLanguageContext)
3535

36+
const handleClick = () => {
37+
showMobileSidebar ? onClose() : onOpen()
38+
39+
trackCustomEvent({
40+
eventCategory: "MobileFilterToggle",
41+
eventAction: `Tap MobileFilterToggle`,
42+
eventName: `show mobile filters ${!showMobileSidebar}`,
43+
})
44+
}
45+
3646
return (
3747
<Box
3848
position="sticky"
@@ -56,14 +66,7 @@ export const MobileFiltersButton = ({
5666
circle: { stroke: "primary.base" },
5767
},
5868
}}
59-
onClick={() => {
60-
showMobileSidebar ? onClose() : onOpen()
61-
trackCustomEvent({
62-
eventCategory: "MobileFilterToggle",
63-
eventAction: `Tap MobileFilterToggle`,
64-
eventName: `show mobile filters ${!showMobileSidebar}`,
65-
})
66-
}}
69+
onClick={handleClick}
6770
>
6871
<Box>
6972
<Text align="start">{t("page-find-wallet-filters")}</Text>

src/components/FindWallet/ResetFiltersButton.tsx

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,17 @@ export const ResetFiltersButton = ({
1515
}: ResetFiltersProps) => {
1616
const { t } = useTranslation("page-wallets-find-wallet")
1717

18+
const handleResetClick = () => {
19+
resetFilters()
20+
resetWalletFilter.current()
21+
22+
trackCustomEvent({
23+
eventCategory: "WalletFilterSidebar",
24+
eventAction: `Reset button`,
25+
eventName: `reset_click`,
26+
})
27+
}
28+
1829
return (
1930
<Center
2031
as="button"
@@ -24,16 +35,7 @@ export const ResetFiltersButton = ({
2435
_hover={{
2536
color: "primary.hover",
2637
}}
27-
onClick={() => {
28-
resetFilters()
29-
resetWalletFilter.current()
30-
31-
trackCustomEvent({
32-
eventCategory: "WalletFilterSidebar",
33-
eventAction: `Reset button`,
34-
eventName: `reset_click`,
35-
})
36-
}}
38+
onClick={handleResetClick}
3739
>
3840
<Icon as={BsArrowCounterclockwise} aria-hidden="true" fontSize="sm" />
3941
<Text as="span" textTransform="uppercase">

src/components/FindWallet/WalletTable/WalletEmptyState.tsx

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,17 @@ export const WalletEmptyState = ({
2121
eventName: `triggered`,
2222
})
2323

24+
const handleClick = () => {
25+
resetFilters()
26+
resetWalletFilter.current()
27+
28+
trackCustomEvent({
29+
eventCategory: "Wallet_empty_state",
30+
eventAction: `reset`,
31+
eventName: `reset_button_clicked`,
32+
})
33+
}
34+
2435
return (
2536
<Flex
2637
justifyContent="center"
@@ -40,16 +51,7 @@ export const WalletEmptyState = ({
4051
color="primary.base"
4152
textDecoration="underline"
4253
cursor="pointer"
43-
onClick={() => {
44-
resetFilters()
45-
resetWalletFilter.current()
46-
47-
trackCustomEvent({
48-
eventCategory: "Wallet_empty_state",
49-
eventAction: `reset`,
50-
eventName: `reset_button_clicked`,
51-
})
52-
}}
54+
onClick={handleClick}
5355
>
5456
{t("page-find-wallet-reset-filters")}
5557
</Text>

0 commit comments

Comments
 (0)