Skip to content

Commit

Permalink
accountDetails: Use UserAvatar instead of UserAvatarWithPresence.
Browse files Browse the repository at this point in the history
Because we are already showing presence indicator below avatar image, no
need to show it over avatar image. Which also doesn't look good.

Fixes: zulip#3361
  • Loading branch information
jainkuniya authored and gnprice committed Feb 28, 2019
1 parent 78f27d6 commit d1a3976
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions src/account-info/AccountDetails.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import { View, Dimensions, StyleSheet } from 'react-native';
import { connect } from 'react-redux';

import type { GlobalState, User } from '../types';
import { UserAvatarWithPresence, ComponentList, RawLabel } from '../common';
import { UserAvatar, ComponentList, RawLabel } from '../common';
import { getCurrentRealm } from '../selectors';
import PresenceStatusIndicator from '../common/PresenceStatusIndicator';
import ActivityText from '../title/ActivityText';
Expand Down Expand Up @@ -34,9 +34,8 @@ class AccountDetails extends PureComponent<Props, void> {

return (
<View>
<UserAvatarWithPresence
<UserAvatar
avatarUrl={getAvatarFromUser(user, realm, screenWidth)}
email={user.email}
size={screenWidth}
shape="square"
/>
Expand Down

0 comments on commit d1a3976

Please sign in to comment.