Merge pull request #1414 from matrix-org/dbkr/member_list_profiles
Show displayname & avatar url in group member list
This commit is contained in:
commit
77febdedfc
3 changed files with 11 additions and 5 deletions
|
@ -90,11 +90,10 @@ export default withMatrixClient(React.createClass({
|
|||
let memberList = this.state.members;
|
||||
if (query) {
|
||||
memberList = memberList.filter((m) => {
|
||||
// TODO: add this when we have this info from the API
|
||||
//const matchesName = m.name.toLowerCase().indexOf(query) !== -1;
|
||||
const matchesName = m.displayname.toLowerCase().indexOf(query) !== -1;
|
||||
const matchesId = m.userId.toLowerCase().includes(query);
|
||||
|
||||
if (/*!matchesName &&*/ !matchesId) {
|
||||
if (!matchesName && !matchesId) {
|
||||
return false;
|
||||
}
|
||||
|
||||
|
|
|
@ -47,10 +47,13 @@ export default withMatrixClient(React.createClass({
|
|||
const BaseAvatar = sdk.getComponent('avatars.BaseAvatar');
|
||||
const EntityTile = sdk.getComponent('rooms.EntityTile');
|
||||
|
||||
const name = this.props.member.userId;
|
||||
const name = this.props.member.displayname || this.props.member.userId;
|
||||
|
||||
const av = (
|
||||
<BaseAvatar name={this.props.member.userId} width={36} height={36} />
|
||||
<BaseAvatar name={this.props.member.userId}
|
||||
width={36} height={36}
|
||||
url={this.props.matrixClient.mxcUrlToHttp(this.props.member.avatarUrl)}
|
||||
/>
|
||||
);
|
||||
|
||||
return (
|
||||
|
|
|
@ -18,10 +18,14 @@ import PropTypes from 'prop-types';
|
|||
|
||||
export const GroupMemberType = PropTypes.shape({
|
||||
userId: PropTypes.string.isRequired,
|
||||
displayname: PropTypes.string,
|
||||
avatarUrl: PropTypes.string,
|
||||
});
|
||||
|
||||
export function groupMemberFromApiObject(apiObject) {
|
||||
return {
|
||||
userId: apiObject.user_id,
|
||||
displayname: apiObject.displayname,
|
||||
avatarUrl: apiObject.avatar_url,
|
||||
};
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue