From 6378ec653851fe03a3c275a22b4da52e0b809798 Mon Sep 17 00:00:00 2001
From: JellyWang <38491708+ezailWang@users.noreply.github.com>
Date: Thu, 5 Sep 2024 10:38:28 +0800
Subject: [PATCH] Fix/swap some bugs fix (#5721)

---
 packages/kit-bg/src/services/ServiceSwap.ts   |  1 +
 .../src/components/TokenListItem/index.tsx    | 17 ++++++---
 .../Swap/components/SwapPopularTokenGroup.tsx |  6 ++--
 .../kit/src/views/Swap/hooks/useSwapQuote.ts  |  2 --
 .../Swap/pages/components/SwapQuoteResult.tsx | 11 ++++--
 .../Swap/pages/modal/SwapTokenSelectModal.tsx | 13 +++++--
 .../types/swap/SwapProvider.constants.ts      | 36 +++++++++++++++++++
 7 files changed, 72 insertions(+), 14 deletions(-)

diff --git a/packages/kit-bg/src/services/ServiceSwap.ts b/packages/kit-bg/src/services/ServiceSwap.ts
index 539753af7b4..a62b8e2d22c 100644
--- a/packages/kit-bg/src/services/ServiceSwap.ts
+++ b/packages/kit-bg/src/services/ServiceSwap.ts
@@ -392,6 +392,7 @@ export default class ServiceSwap extends ServiceBase {
   }
 
   @backgroundMethod()
+  @toastIfError()
   async fetchQuotesEvents({
     fromToken,
     toToken,
diff --git a/packages/kit/src/components/TokenListItem/index.tsx b/packages/kit/src/components/TokenListItem/index.tsx
index 95abdd671b4..2b86719ca6e 100644
--- a/packages/kit/src/components/TokenListItem/index.tsx
+++ b/packages/kit/src/components/TokenListItem/index.tsx
@@ -37,9 +37,6 @@ export function TokenListItem({
   return (
     <ListItem
       userSelect="none"
-      {...(disabled && {
-        opacity: 0.5,
-      })}
       {...(isSearch && {
         $md: {
           alignItems: 'flex-start',
@@ -47,8 +44,17 @@ export function TokenListItem({
       })}
       {...rest}
     >
-      <Token tokenImageUri={tokenImageSrc} networkImageUri={networkImageSrc} />
+      <Token
+        {...(disabled && {
+          opacity: 0.5,
+        })}
+        tokenImageUri={tokenImageSrc}
+        networkImageUri={networkImageSrc}
+      />
       <ListItem.Text
+        {...(disabled && {
+          opacity: 0.5,
+        })}
         flex={1}
         primary={tokenSymbol}
         primaryMatch={titleMatchStr}
@@ -79,6 +85,9 @@ export function TokenListItem({
         }
       />
       <ListItem.Text
+        {...(disabled && {
+          opacity: 0.5,
+        })}
         align="right"
         primary={
           <NumberSizeableText
diff --git a/packages/kit/src/views/Swap/components/SwapPopularTokenGroup.tsx b/packages/kit/src/views/Swap/components/SwapPopularTokenGroup.tsx
index 3627c91d7e6..1ca1abd6e9e 100644
--- a/packages/kit/src/views/Swap/components/SwapPopularTokenGroup.tsx
+++ b/packages/kit/src/views/Swap/components/SwapPopularTokenGroup.tsx
@@ -15,17 +15,19 @@ const SwapPopularTokenGroup = ({
   selectedToken,
   tokens,
 }: ISwapPopularTokenGroupProps) => (
-  <XStack pt="$1" pb="$3" gap="$2" flexWrap="wrap">
+  <XStack pt="$1" pb="$3" gap="$1.5" flexWrap="wrap">
     {tokens.map((token) => (
       <Button
         key={token.contractAddress}
         variant="secondary"
         borderRadius="$4"
+        px="$1.5"
         size="small"
         borderColor="$borderSubdued"
         flexDirection="row"
+        justifyContent="center"
+        alignItems="center"
         backgroundColor="$background"
-        gap="$0.5"
         disabled={
           !!equalTokenNoCaseSensitive({
             token1: selectedToken,
diff --git a/packages/kit/src/views/Swap/hooks/useSwapQuote.ts b/packages/kit/src/views/Swap/hooks/useSwapQuote.ts
index 88da1d8e17f..559722ae433 100644
--- a/packages/kit/src/views/Swap/hooks/useSwapQuote.ts
+++ b/packages/kit/src/views/Swap/hooks/useSwapQuote.ts
@@ -238,8 +238,6 @@ export function useSwapQuote() {
         if (isFocus) {
           appEventBus.off(EAppEventBusNames.SwapQuoteEvent, quoteEventHandler);
           appEventBus.on(EAppEventBusNames.SwapQuoteEvent, quoteEventHandler);
-        } else {
-          appEventBus.off(EAppEventBusNames.SwapQuoteEvent, quoteEventHandler);
         }
       }
       setTimeout(() => {
diff --git a/packages/kit/src/views/Swap/pages/components/SwapQuoteResult.tsx b/packages/kit/src/views/Swap/pages/components/SwapQuoteResult.tsx
index f7835753aac..c986c178d25 100644
--- a/packages/kit/src/views/Swap/pages/components/SwapQuoteResult.tsx
+++ b/packages/kit/src/views/Swap/pages/components/SwapQuoteResult.tsx
@@ -1,6 +1,6 @@
 import { memo, useCallback, useRef, useState } from 'react';
 
-import { isNil } from 'lodash';
+import BigNumber from 'bignumber.js';
 import { useIntl } from 'react-intl';
 
 import type { IDialogInstance } from '@onekeyhq/components';
@@ -94,8 +94,13 @@ const SwapQuoteResult = ({
     slippageOnSave,
     setSwapSlippageDialogOpening,
   ]);
-
-  if (!fromToken || !toToken || !fromTokenAmount) {
+  const fromTokenAmountBN = new BigNumber(fromTokenAmount ?? 0);
+  if (
+    !fromToken ||
+    !toToken ||
+    fromTokenAmountBN.isZero() ||
+    fromTokenAmountBN.isNaN()
+  ) {
     return null;
   }
   return (
diff --git a/packages/kit/src/views/Swap/pages/modal/SwapTokenSelectModal.tsx b/packages/kit/src/views/Swap/pages/modal/SwapTokenSelectModal.tsx
index 7cd323b1400..4de2bb770ad 100644
--- a/packages/kit/src/views/Swap/pages/modal/SwapTokenSelectModal.tsx
+++ b/packages/kit/src/views/Swap/pages/modal/SwapTokenSelectModal.tsx
@@ -1,4 +1,4 @@
-import { useCallback, useEffect, useMemo, useState } from 'react';
+import { useCallback, useEffect, useMemo, useRef, useState } from 'react';
 
 import { useRoute } from '@react-navigation/core';
 import BigNumber from 'bignumber.js';
@@ -68,6 +68,7 @@ import { useSwapTokenList } from '../../hooks/useSwapTokens';
 import { SwapProviderMirror } from '../SwapProviderMirror';
 
 import type { RouteProp } from '@react-navigation/core';
+import type { FlatList } from 'react-native';
 
 const SwapTokenSelectPage = () => {
   const navigation =
@@ -140,6 +141,7 @@ const SwapTokenSelectPage = () => {
   const [currentSelectNetwork, setCurrentSelectNetwork] = useState<
     ISwapNetwork | undefined
   >(syncDefaultNetworkSelect);
+  const listViewRef = useRef<FlatList>(null);
 
   useEffect(() => {
     const accountNet =
@@ -233,6 +235,10 @@ const SwapTokenSelectPage = () => {
           networkId: network.networkId,
         });
       }
+      listViewRef.current?.scrollToOffset({
+        offset: 0,
+        animated: true,
+      });
     },
     [type, updateSelectedAccountNetwork],
   );
@@ -434,7 +440,7 @@ const SwapTokenSelectPage = () => {
       />
       <Page.Body>
         <XStack px="$5" pb="$2">
-          <SizableText size="$headingSm" color="$textSubdued" pr="$2">
+          <SizableText size="$bodyMd" color="$textSubdued" pr="$2">
             {`${intl.formatMessage({
               id: ETranslations.token_selector_network,
             })}`}
@@ -472,6 +478,7 @@ const SwapTokenSelectPage = () => {
         <Divider mt="$2" />
         <YStack flex={1}>
           <ListView
+            ref={listViewRef}
             data={currentTokens}
             renderItem={renderItem}
             estimatedItemSize={60}
@@ -479,7 +486,7 @@ const SwapTokenSelectPage = () => {
               currentNetworkPopularTokens.length > 0 &&
               !searchKeywordDebounce ? (
                 <YStack px="$5" pt="$3" gap="$2">
-                  <SizableText size="$headingSm" color="$textSubdued" pr="$2">
+                  <SizableText size="$bodyMd" color="$textSubdued" pr="$2">
                     {`${intl.formatMessage({
                       id: ETranslations.swap_token_selector_popular_token,
                     })}`}
diff --git a/packages/shared/types/swap/SwapProvider.constants.ts b/packages/shared/types/swap/SwapProvider.constants.ts
index 1c683ba9736..f2fa4ddc29e 100644
--- a/packages/shared/types/swap/SwapProvider.constants.ts
+++ b/packages/shared/types/swap/SwapProvider.constants.ts
@@ -432,6 +432,42 @@ export const swapDefaultSetTokens: Record<
       'isNative': false,
     },
   },
+  'sui--mainnet': {
+    fromToken: {
+      'networkId': 'sui--mainnet',
+      'contractAddress': '0x2::sui::SUI',
+      'name': 'Sui',
+      'symbol': 'SUI',
+      'decimals': 9,
+      'logoURI':
+        'https://uni-test.onekey-asset.com/server-service-onchain/sui--mainnet/tokens/0x2::sui::SUI.png',
+      'isNative': true,
+    },
+  },
+  'ton--mainnet': {
+    fromToken: {
+      'networkId': 'ton--mainnet',
+      'contractAddress': '',
+      'name': 'Toncoin',
+      'symbol': 'TON',
+      'decimals': 9,
+      'logoURI':
+        'https://uni-test.onekey-asset.com/server-service-onchain/ton--mainnet/tokens/native.png',
+      'isNative': true,
+    },
+  },
+  'aptos--1': {
+    fromToken: {
+      'networkId': 'aptos--1',
+      'contractAddress': '0x1::aptos_coin::AptosCoin',
+      'name': 'Aptos Coin',
+      'symbol': 'APT',
+      'decimals': 8,
+      'logoURI':
+        'https://uni-test.onekey-asset.com/server-service-onchain/aptos--1/tokens/0x1::aptos_coin::AptosCoin.png',
+      'isNative': true,
+    },
+  },
 };
 
 export const swapPopularTokens: Record<string, ISwapToken[]> = {