Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Format mxids when disambiguation needed #5880

Merged
merged 16 commits into from
Jun 14, 2021
Merged
Show file tree
Hide file tree
Changes from 4 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions res/css/views/messages/_SenderProfile.scss
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,18 @@ limitations under the License.
*/

.mx_SenderProfile_name {
display: flex;
align-items: center;
}

.mx_SenderProfile_displayName {
font-weight: 600;
}

.mx_SenderProfile_mxid {
font-weight: 600;
font-family: monospace;
font-size: 1rem;
color: gray;
margin-left: 5px;
}
Original file line number Diff line number Diff line change
Expand Up @@ -15,26 +15,37 @@
*/

import React from 'react';
import PropTypes from 'prop-types';
import Flair from '../elements/Flair.js';
import FlairStore from '../../../stores/FlairStore';
import {getUserNameColorClass} from '../../../utils/FormattingUtils';
import MatrixClientContext from "../../../contexts/MatrixClientContext";
import {replaceableComponent} from "../../../utils/replaceableComponent";
import MatrixEvent from "matrix-js-sdk/src/models/event";

@replaceableComponent("views.messages.SenderProfile")
export default class SenderProfile extends React.Component {
static propTypes = {
mxEvent: PropTypes.object.isRequired, // event whose sender we're showing
onClick: PropTypes.func,
};
interface IProps {
mxEvent: MatrixEvent;
onClick(): void;
enableFlair: boolean;
}

interface IState {
userGroups;
relatedGroups;
}

@replaceableComponent("views.messages.SenderProfile")
export default class SenderProfile extends React.Component<IProps, IState> {
static contextType = MatrixClientContext;
private unmounted: boolean;

state = {
userGroups: null,
relatedGroups: [],
};
constructor(props: IProps) {
super(props)

this.state = {
userGroups: null,
relatedGroups: [],
};
}

componentDidMount() {
this.unmounted = false;
Expand Down Expand Up @@ -89,14 +100,17 @@ export default class SenderProfile extends React.Component {
render() {
const {mxEvent} = this.props;
const colorClass = getUserNameColorClass(mxEvent.getSender());
const name = mxEvent.sender ? mxEvent.sender.name : mxEvent.getSender();
const {msgtype} = mxEvent.getContent();

const disambiguate = mxEvent.sender?.disambiguate;
const displayName = mxEvent.sender?.rawDisplayName || mxEvent.getSender() || "";
const mxid = mxEvent.sender?.userId || mxEvent.getSender() || "";

if (msgtype === 'm.emote') {
return <span />; // emote message must include the name so don't duplicate it
}

let flair = <div />;
let flair;
if (this.props.enableFlair) {
const displayedGroups = this._getDisplayedGroups(
this.state.userGroups, this.state.relatedGroups,
Expand All @@ -108,20 +122,29 @@ export default class SenderProfile extends React.Component {
/>;
}

const nameElem = name || '';

// Name + flair
const nameFlair = <span>
<span className={`mx_SenderProfile_name ${colorClass}`}>
{ nameElem }
const displayNameElement = (
<span className={`mx_SenderProfile_displayName ${colorClass}`}>
{ displayName }
</span>
{ flair }
</span>;
);

let mxidElement;
if (disambiguate) {
mxidElement = (
<span className="mx_SenderProfile_mxid">
{ `[${mxid}]` }
</span>
);
}

return (
<div className="mx_SenderProfile" dir="auto" onClick={this.props.onClick}>
<div className="mx_SenderProfile_hover">
{ nameFlair }
<div className="mx_SenderProfile_name">
{ displayNameElement }
{ mxidElement }
{ flair }
</div>
</div>
</div>
);
Expand Down