Skip to content

Commit e8baef3

Browse files
committed
Fix unnecessary checks in UserDropdown
1 parent ff68093 commit e8baef3

File tree

1 file changed

+13
-17
lines changed

1 file changed

+13
-17
lines changed

site/components/Navbar/UserDropdown.tsx

Lines changed: 13 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -35,11 +35,9 @@ export const UserDropdown: React.FC<UserDropdownProps> = ({ user, onSignOut }: U
3535
<div>
3636
<MenuItem onClick={handleDropdownClick}>
3737
<div className={styles.inner}>
38-
{user && (
39-
<Badge overlap="circle">
40-
<UserAvatar user={user} />
41-
</Badge>
42-
)}
38+
<Badge overlap="circle">
39+
<UserAvatar user={user} />
40+
</Badge>
4341
{anchorEl ? (
4442
<KeyboardArrowUp className={`${styles.arrowIcon} ${styles.arrowIconUp}`} />
4543
) : (
@@ -65,20 +63,18 @@ export const UserDropdown: React.FC<UserDropdownProps> = ({ user, onSignOut }: U
6563
variant="user-dropdown"
6664
onClose={onPopoverClose}
6765
>
68-
{user && (
69-
<div className={styles.userInfo}>
70-
<UserProfileCard user={user} />
66+
<div className={styles.userInfo}>
67+
<UserProfileCard user={user} />
7168

72-
<Divider className={styles.divider} />
69+
<Divider className={styles.divider} />
7370

74-
<MenuItem className={styles.menuItem} onClick={onSignOut}>
75-
<ListItemIcon className={styles.icon}>
76-
<LogoutIcon />
77-
</ListItemIcon>
78-
<ListItemText primary="Sign Out" />
79-
</MenuItem>
80-
</div>
81-
)}
71+
<MenuItem className={styles.menuItem} onClick={onSignOut}>
72+
<ListItemIcon className={styles.icon}>
73+
<LogoutIcon />
74+
</ListItemIcon>
75+
<ListItemText primary="Sign Out" />
76+
</MenuItem>
77+
</div>
8278
</BorderedMenu>
8379
</>
8480
)

0 commit comments

Comments
 (0)