chore: cleanup

TalismanSociety · Oct 2, 2024 · 421d517 · 421d517
1 parent 2dfa10f
commit 421d517
Show file tree

Hide file tree

Showing 10 changed files with 7 additions and 448 deletions.
diff --git a/apps/extension/src/ui/apps/popup/components/Navigation/BottomNav.tsx b/apps/extension/src/ui/apps/popup/components/Navigation/BottomNav.tsx
@@ -112,7 +112,7 @@ export const BottomNav = () => {
           <NavButton label={t("Staking")} icon={ZapIcon} onClick={handleStakingClick} />
           <NavButton
             label={t("History")}
-            icon={HistoryIcon} // hasPendingTransactions ? NavIconActivityPending : HistoryIcon
+            icon={HistoryIcon}
             onClick={handleTxHistoryClick}
             route="/tx-history"
           />

diff --git a/apps/extension/src/ui/apps/popup/components/Navigation/QuickSettings.tsx b/apps/extension/src/ui/apps/popup/components/Navigation/QuickSettings.tsx
@@ -28,9 +28,6 @@ export const useQuickSettingsOpenClose = () => useGlobalOpenClose("quick-setting
 export const QuickSettingsOverlay: FC = () => {
   const { isOpen, close } = useQuickSettingsOpenClose()
 
-  //if (!isOpen) return null
-
-  // TODO proper animation in + out
   return (
     <Transition show={isOpen} appear>
       <Transition.Child

diff --git a/apps/extension/src/ui/apps/popup/components/TotalFiatBalance.tsx b/apps/extension/src/ui/apps/popup/components/TotalFiatBalance.tsx
@@ -67,7 +67,6 @@ export const TotalFiatBalance = ({ className, mouseOver, disabled }: Props) => {
           <button
             className={classNames(
               "bg-grey-700/20 border-grey-200 text-grey-200 hover:text-body hover:bg-grey-700/10 hover:border-body pointer-events-auto flex size-16 shrink-0 items-center justify-center rounded-full border text-center transition-colors duration-100 ease-out",
-              "",
               currencyConfig[currency]?.symbol?.length === 2 && "text-xs",
               currencyConfig[currency]?.symbol?.length > 2 && "text-[1rem]"
             )}

diff --git a/apps/extension/src/ui/apps/popup/pages/Portfolio/index.tsx b/apps/extension/src/ui/apps/popup/pages/Portfolio/index.tsx
@@ -60,7 +60,6 @@ export const Portfolio = () => (
       <ErrorBoundary>
         <Content>
           <div className="flex size-full flex-col gap-4 py-8">
-            {/* <AuthorisedSiteToolbar /> */}
             <PortfolioContent />
             <BottomNav />
           </div>

diff --git a/apps/extension/src/ui/atoms/portfolio.ts b/apps/extension/src/ui/atoms/portfolio.ts
@@ -61,7 +61,7 @@ const getNetworkTokenSymbols = ({
 
 const getAccountsType = (accounts?: AccountJsonAny[]) => {
   if (accounts?.every((a) => a.type === "ethereum")) return "ethereum"
-  if (accounts?.every((a) => a.type !== "ethereum")) return "sr25519" // TODO rename substrate
+  if (accounts?.every((a) => a.type !== "ethereum")) return "sr25519" // TODO rename substrate or ss58
   return undefined
 }
 

diff --git a/apps/extension/src/ui/domains/Account/AccountContextMenu.tsx b/apps/extension/src/ui/domains/Account/AccountContextMenu.tsx
@@ -53,7 +53,7 @@ type Props = {
  * If the `address` prop is null, this component will ignore `selectedAccount`
  */
 export const AccountContextMenu = forwardRef<HTMLElement, Props>(function AccountContextMenu(
-  { analyticsFrom, address, placement, trigger, hideManageAccounts, disabled }, // TODO replace trigger like in FolderContextMenu?
+  { analyticsFrom, address, placement, trigger, hideManageAccounts, disabled },
   ref
 ) {
   const { t } = useTranslation()

diff --git a/apps/extension/src/ui/domains/Account/ManageAccounts/TreeItemFolder.tsx b/apps/extension/src/ui/domains/Account/ManageAccounts/TreeItemFolder.tsx
@@ -21,13 +21,7 @@ export const TreeItemFolder: FC<{
   treeName: AccountsCatalogTree
   accounts: AccountJsonAny[]
   disableFolderDrop?: boolean
-}> = ({
-  folder,
-  balanceTotalPerAccount,
-  treeName,
-  accounts,
-  disableFolderDrop, // onHoverMenu
-}) => {
+}> = ({ folder, balanceTotalPerAccount, treeName, accounts, disableFolderDrop }) => {
   const { t } = useTranslation()
   const addresses = useMemo(() => folder.tree.map((item) => item.address), [folder])
   const balanceTotal = useMemo(