update avatar setting

plebbit · Feb 26, 2024 · 548b1eb · 548b1eb
1 parent bc57479
commit 548b1eb
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 16 deletions.
diff --git a/src/views/settings/avatar-settings/avatar-settings.module.css b/src/views/settings/avatar-settings/avatar-settings.module.css
@@ -2,6 +2,7 @@
   width: 70px;
   height: 70px;
   border: 1px solid var(--border-text);
+  cursor: pointer;
 }
 
 .avatar img {

diff --git a/src/views/settings/avatar-settings/avatar-settings.tsx b/src/views/settings/avatar-settings/avatar-settings.tsx
@@ -1,16 +1,13 @@
 import { useEffect, useMemo, useState } from 'react';
 import { setAccount, useAccount, useAuthorAvatar } from '@plebbit/plebbit-react-hooks';
-import { useTranslation } from 'react-i18next';
 import styles from './avatar-settings.module.css';
 
 interface AvatarSettingsProps {
-  areSettingsShown?: boolean;
   avatar?: any;
   showSettings?: () => void;
 }
 
-const AvatarPreview = ({ avatar, showSettings, areSettingsShown }: AvatarSettingsProps) => {
-  const { t } = useTranslation();
+const AvatarPreview = ({ avatar, showSettings }: AvatarSettingsProps) => {
   const account = useAccount();
   let author = useMemo(() => ({ ...account?.author, avatar }), [account, avatar]);
 
@@ -30,18 +27,14 @@ const AvatarPreview = ({ avatar, showSettings, areSettingsShown }: AvatarSetting
 
   return (
     <>
-      <div className={styles.avatar}>
-        {imageUrl && state !== 'initializing' ? (
-          <img src={imageUrl} alt='avatar' />
-        ) : (
-          <span className={styles.emptyAvatar} onClick={showSettings}>
-            {areSettingsShown ? '–' + t('hide') : '+' + t('add')}
-          </span>
-        )}
-      </div>
-      <div className={styles.state}>
-        {stateText} {error?.message}
+      <div className={styles.avatar} onClick={showSettings}>
+        {imageUrl && state !== 'initializing' && <img src={imageUrl} alt='avatar' />}
       </div>
+      {state !== 'succeeded' && account?.author?.avatar && (
+        <div className={styles.state}>
+          {stateText} {error?.message}
+        </div>
+      )}
     </>
   );
 };
@@ -128,7 +121,7 @@ const AvatarSettings = () => {
 
   return (
     <div className={styles.avatarSettings}>
-      <AvatarPreview avatar={avatar} showSettings={() => setShowSettings(!showSettings)} areSettingsShown={showSettings} />
+      <AvatarPreview avatar={avatar} showSettings={() => setShowSettings(!showSettings)} />
       {showSettings && (
         <div className={styles.avatarSettingsForm}>
           <div className={styles.avatarSettingInput}>