From fa12bf5d877773a4b0719b35d70475380be24042 Mon Sep 17 00:00:00 2001
From: Hailey <me@haileyok.com>
Date: Fri, 23 Aug 2024 13:28:56 -0700
Subject: [PATCH] Revert "Make settings account buttons a little nicer" (#4987)

---
 src/view/com/util/AccountDropdownBtn.tsx | 78 +++++++++++-------------
 src/view/screens/Settings/index.tsx      | 71 +++++++++------------
 2 files changed, 66 insertions(+), 83 deletions(-)

diff --git a/src/view/com/util/AccountDropdownBtn.tsx b/src/view/com/util/AccountDropdownBtn.tsx
index 034fbfc6..fa2553d3 100644
--- a/src/view/com/util/AccountDropdownBtn.tsx
+++ b/src/view/com/util/AccountDropdownBtn.tsx
@@ -1,59 +1,53 @@
 import React from 'react'
-import {msg, Trans} from '@lingui/macro'
+import {Pressable} from 'react-native'
+import {
+  FontAwesomeIcon,
+  FontAwesomeIconStyle,
+} from '@fortawesome/react-native-fontawesome'
+import {msg} from '@lingui/macro'
 import {useLingui} from '@lingui/react'
 
 import {SessionAccount, useSessionApi} from '#/state/session'
-import {HITSLOP_10} from 'lib/constants'
-import {Button, ButtonIcon} from '#/components/Button'
+import {usePalette} from 'lib/hooks/usePalette'
+import {s} from 'lib/styles'
 import {useDialogControl} from '#/components/Dialog'
-import {DotGrid_Stroke2_Corner0_Rounded as Ellipsis} from '#/components/icons/DotGrid'
-import {Trash_Stroke2_Corner0_Rounded as Trash} from '#/components/icons/Trash'
-import * as Menu from '#/components/Menu'
 import * as Prompt from '#/components/Prompt'
 import * as Toast from '../../com/util/Toast'
+import {DropdownItem, NativeDropdown} from './forms/NativeDropdown'
 
 export function AccountDropdownBtn({account}: {account: SessionAccount}) {
-  const {_} = useLingui()
+  const pal = usePalette('default')
   const {removeAccount} = useSessionApi()
   const removePromptControl = useDialogControl()
+  const {_} = useLingui()
 
+  const items: DropdownItem[] = [
+    {
+      label: _(msg`Remove account`),
+      onPress: removePromptControl.open,
+      icon: {
+        ios: {
+          name: 'trash',
+        },
+        android: 'ic_delete',
+        web: ['far', 'trash-can'],
+      },
+    },
+  ]
   return (
     <>
-      <Menu.Root>
-        <Menu.Trigger label={_(`Account options`)}>
-          {({props}) => {
-            return (
-              <Button
-                {...props}
-                testID="accountSettingsDropdownBtn"
-                label={_(`Account options`)}
-                hitSlop={HITSLOP_10}
-                size="xsmall"
-                shape="round"
-                color="secondary"
-                variant="ghost">
-                <ButtonIcon icon={Ellipsis} size="sm" />
-              </Button>
-            )
-          }}
-        </Menu.Trigger>
-
-        <Menu.Outer style={{minWidth: 170}}>
-          <Menu.Group>
-            <Menu.Item
-              label={_(msg`Remove account`)}
-              onPress={() => {
-                removePromptControl.open()
-              }}>
-              <Menu.ItemText>
-                <Trans>Remove account</Trans>
-              </Menu.ItemText>
-              <Menu.ItemIcon icon={Trash} />
-            </Menu.Item>
-          </Menu.Group>
-        </Menu.Outer>
-      </Menu.Root>
-
+      <Pressable accessibilityRole="button" style={s.pl10}>
+        <NativeDropdown
+          testID="accountSettingsDropdownBtn"
+          items={items}
+          accessibilityLabel={_(msg`Account options`)}
+          accessibilityHint="">
+          <FontAwesomeIcon
+            icon="ellipsis-h"
+            style={pal.textLight as FontAwesomeIconStyle}
+          />
+        </NativeDropdown>
+      </Pressable>
       <Prompt.Basic
         control={removePromptControl}
         title={_(msg`Remove from quick access?`)}
diff --git a/src/view/screens/Settings/index.tsx b/src/view/screens/Settings/index.tsx
index fc93f9c9..fe449fcd 100644
--- a/src/view/screens/Settings/index.tsx
+++ b/src/view/screens/Settings/index.tsx
@@ -18,7 +18,6 @@ import {useLingui} from '@lingui/react'
 import {useFocusEffect, useNavigation} from '@react-navigation/native'
 import {useQueryClient} from '@tanstack/react-query'
 
-import {sanitizeHandle} from '#/lib/strings/handles'
 import {isNative} from '#/platform/detection'
 import {useModalControls} from '#/state/modals'
 import {clearStorage} from '#/state/persisted'
@@ -56,11 +55,8 @@ import {UserAvatar} from 'view/com/util/UserAvatar'
 import {ScrollView} from 'view/com/util/Views'
 import {DeactivateAccountDialog} from '#/screens/Settings/components/DeactivateAccountDialog'
 import {atoms as a, useTheme} from '#/alf'
-import {Button, ButtonContext} from '#/components/Button'
 import {useDialogControl} from '#/components/Dialog'
 import {BirthDateSettingsDialog} from '#/components/dialogs/BirthDateSettings'
-import {Link as NewLink} from '#/components/Link'
-import {Text as NewText} from '#/components/Typography'
 import {Email2FAToggle} from './Email2FAToggle'
 import {ExportCarDialog} from './ExportCarDialog'
 
@@ -76,69 +72,62 @@ function SettingsAccountCard({
     logContext: 'Settings',
   ) => void
 }) {
+  const pal = usePalette('default')
   const {_} = useLingui()
   const t = useTheme()
   const {currentAccount} = useSession()
   const {data: profile} = useProfileQuery({did: account.did})
   const isCurrentAccount = account.did === currentAccount?.did
 
-  const contents = (ctx: ButtonContext) => (
+  const contents = (
     <View
       style={[
-        a.w_full,
-        a.flex_row,
-        a.align_center,
-        a.gap_md,
-        a.py_md,
-        t.atoms.bg,
-        {
-          paddingHorizontal: 18,
-          marginBottom: 1,
-        },
-        account.did === pendingDid && [t.atoms.bg_contrast_25],
-        ctx.pressed && [t.atoms.bg_contrast_25],
+        pal.view,
+        styles.linkCard,
+        account.did === pendingDid && t.atoms.bg_contrast_25,
       ]}>
-      <UserAvatar
-        size={40}
-        avatar={profile?.avatar}
-        type={profile?.associated?.labeler ? 'labeler' : 'user'}
-      />
-      <View style={[a.flex_1]}>
-        <NewText
-          style={[a.text_md, a.font_bold, a.leading_tight]}
-          numberOfLines={1}>
+      <View style={styles.avi}>
+        <UserAvatar
+          size={40}
+          avatar={profile?.avatar}
+          type={profile?.associated?.labeler ? 'labeler' : 'user'}
+        />
+      </View>
+      <View style={[s.flex1]}>
+        <Text type="md-bold" style={[pal.text, a.self_start]} numberOfLines={1}>
           {profile?.displayName || account.handle}
-        </NewText>
-        <NewText
-          style={[t.atoms.text_contrast_medium, a.leading_tight]}
-          numberOfLines={1}>
-          {sanitizeHandle(account.handle, '@')}
-        </NewText>
+        </Text>
+        <Text type="sm" style={pal.textLight} numberOfLines={1}>
+          {account.handle}
+        </Text>
       </View>
       <AccountDropdownBtn account={account} />
     </View>
   )
 
   return isCurrentAccount ? (
-    <NewLink
-      to={makeProfileLink({
+    <Link
+      href={makeProfileLink({
         did: currentAccount?.did,
         handle: currentAccount?.handle,
       })}
-      label={_(msg`Your profile`)}
-      style={[a.w_full]}>
+      title={_(msg`Your profile`)}
+      noFeedback>
       {contents}
-    </NewLink>
+    </Link>
   ) : (
-    <Button
+    <TouchableOpacity
       testID={`switchToAccountBtn-${account.handle}`}
+      key={account.did}
       onPress={
         pendingDid ? undefined : () => onPressSwitchAccount(account, 'Settings')
       }
-      label={_(msg`Switch to ${account.handle}`)}
-      style={[a.w_full]}>
+      accessibilityRole="button"
+      accessibilityLabel={_(msg`Switch to ${account.handle}`)}
+      accessibilityHint={_(msg`Switches the account you are logged in to`)}
+      activeOpacity={0.8}>
       {contents}
-    </Button>
+    </TouchableOpacity>
   )
 }