some comments to explain the mess

pull/21833/head
Bruno Windels 2020-04-02 16:44:42 +02:00
parent 0307361fa2
commit 1baaafe7b9
1 changed files with 5 additions and 0 deletions

View File

@ -162,6 +162,8 @@ export default class VerificationPanel extends React.PureComponent {
renderQRReciprocatePhase() { renderQRReciprocatePhase() {
const {member} = this.props; const {member} = this.props;
let Button; let Button;
// a bit of a hack, but the FormButton should only be used in the right panel
// they should probably just be the same component with a css class applied to it?
if (this.props.inDialog) { if (this.props.inDialog) {
Button = sdk.getComponent("elements.AccessibleButton"); Button = sdk.getComponent("elements.AccessibleButton");
} else { } else {
@ -175,6 +177,9 @@ export default class VerificationPanel extends React.PureComponent {
let body; let body;
if (this.state.reciprocateQREvent) { if (this.state.reciprocateQREvent) {
// riot web doesn't support scanning yet, so assume here we're the client being scanned. // riot web doesn't support scanning yet, so assume here we're the client being scanned.
//
// we're passing both a label and a child string to Button as
// FormButton and AccessibleButton expect this differently
body = <React.Fragment> body = <React.Fragment>
<p>{description}</p> <p>{description}</p>
<E2EIcon isUser={true} status="verified" size={128} hideTooltip={true} /> <E2EIcon isUser={true} status="verified" size={128} hideTooltip={true} />