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

Commit

Permalink
Add success dialog after key backup
Browse files Browse the repository at this point in the history
  • Loading branch information
weeman1337 committed Feb 16, 2023
1 parent f0359a5 commit b463226
Show file tree
Hide file tree
Showing 9 changed files with 94 additions and 2 deletions.
4 changes: 4 additions & 0 deletions cypress/e2e/crypto/crypto.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -183,6 +183,10 @@ describe("Cryptography", function () {
cy.contains(".mx_Dialog_primary:not([disabled])", "Continue").click();
cy.contains(".mx_Dialog_title", "Setting up keys").should("exist");
cy.contains(".mx_Dialog_title", "Setting up keys").should("not.exist");

cy.contains("Secure Backup successful").should("exist");
cy.contains("Done").click();
cy.contains("Secure Backup successful").should("not.exist");
});
return;
});
Expand Down
1 change: 1 addition & 0 deletions res/css/_components.pcss
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@
@import "./components/views/settings/shared/_SettingsSubsectionHeading.pcss";
@import "./components/views/spaces/_QuickThemeSwitcher.pcss";
@import "./components/views/typography/_Caption.pcss";
@import "./compound/_SuccessDialog.pcss";
@import "./compound/_Icon.pcss";
@import "./structures/_AutoHideScrollbar.pcss";
@import "./structures/_AutocompleteInput.pcss";
Expand Down
12 changes: 12 additions & 0 deletions res/css/compound/_Icon.pcss
Original file line number Diff line number Diff line change
Expand Up @@ -29,10 +29,22 @@ limitations under the License.
color: $accent;
}

.mx_Icon_bg-accent-light {
background-color: rgba($accent, 0.1);
}

.mx_Icon_alert {
color: $alert;
}

.mx_Icon_circle-40 {
border-radius: 20px;
flex: 0 0 40px;
height: 40px;
padding: 0 12px;
width: 40px;
}

.mx_Icon_8 {
flex: 0 0 8px;
height: 8px;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,14 @@ limitations under the License.
/* never asked. */
width: 560px;

&.mx_SuccessDialog {
padding: 56px; /* 80px from design - 24px wrapper padding */

.mx_Dialog_title {
margin-bottom: $spacing-16;
}
}

.mx_SettingsFlag {
display: flex;
}
Expand Down
20 changes: 20 additions & 0 deletions res/img/element-icons/check.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import { TrustInfo } from "matrix-js-sdk/src/crypto/backup";
import { CrossSigningKeys, UIAFlow } from "matrix-js-sdk/src/matrix";
import { IRecoveryKey } from "matrix-js-sdk/src/crypto/api";
import { CryptoEvent } from "matrix-js-sdk/src/crypto";
import classNames from "classnames";

import { MatrixClientPeg } from "../../../../MatrixClientPeg";
import { _t, _td } from "../../../../languageHandler";
Expand All @@ -48,6 +49,7 @@ import BaseDialog from "../../../../components/views/dialogs/BaseDialog";
import Spinner from "../../../../components/views/elements/Spinner";
import InteractiveAuthDialog from "../../../../components/views/dialogs/InteractiveAuthDialog";
import { IValidationResult } from "../../../../components/views/elements/Validation";
import { Icon as CheckmarkIcon } from "../../../../../res/img/element-icons/check.svg";

// I made a mistake while converting this and it has to be fixed!
enum Phase {
Expand All @@ -59,6 +61,7 @@ enum Phase {
PassphraseConfirm = "passphrase_confirm",
ShowKey = "show_key",
Storing = "storing",
Stored = "stored",
ConfirmSkip = "confirm_skip",
}

Expand Down Expand Up @@ -361,7 +364,10 @@ export default class CreateSecretStorageDialog extends React.PureComponent<IProp
},
});
}
this.props.onFinished(true);

this.setState({
phase: Phase.Stored,
});
} catch (e) {
if (this.state.canUploadKeysWithPasswordOnly && e.httpStatus === 401 && e.data.flows) {
this.setState({
Expand Down Expand Up @@ -785,6 +791,19 @@ export default class CreateSecretStorageDialog extends React.PureComponent<IProp
);
}

private renderStoredPhase(): JSX.Element {
return (
<>
<p className="mx_Dialog_content">{_t("Your keys are now being backed up from this device.")}</p>
<DialogButtons
primaryButton={_t("Done")}
onPrimaryButtonClick={() => this.props.onFinished(true)}
hasCancel={false}
/>
</>
);
}

private renderPhaseLoadError(): JSX.Element {
return (
<div>
Expand Down Expand Up @@ -837,11 +856,27 @@ export default class CreateSecretStorageDialog extends React.PureComponent<IProp
return _t("Save your Security Key");
case Phase.Storing:
return _t("Setting up keys");
case Phase.Stored:
return _t("Secure Backup successful");
default:
return "";
}
}

private get topComponent(): React.ReactNode | null {
if (this.state.phase === Phase.Stored) {
return <CheckmarkIcon className="mx_Icon mx_Icon_circle-40 mx_Icon_accent mx_Icon_bg-accent-light" />;
}

return null;
}

private get classNames(): string {
return classNames("mx_CreateSecretStorageDialog", {
mx_SuccessDialog: this.state.phase === Phase.Stored,
});
}

public render(): React.ReactNode {
let content;
if (this.state.error) {
Expand Down Expand Up @@ -884,6 +919,9 @@ export default class CreateSecretStorageDialog extends React.PureComponent<IProp
case Phase.Storing:
content = this.renderBusyPhase();
break;
case Phase.Stored:
content = this.renderStoredPhase();
break;
case Phase.ConfirmSkip:
content = this.renderPhaseSkipConfirm();
break;
Expand Down Expand Up @@ -912,8 +950,9 @@ export default class CreateSecretStorageDialog extends React.PureComponent<IProp

return (
<BaseDialog
className="mx_CreateSecretStorageDialog"
className={this.classNames}
onFinished={this.props.onFinished}
top={this.topComponent}
title={this.titleForPhase(this.state.phase)}
titleClass={titleClass}
hasCancel={this.props.hasCancel && [Phase.Passphrase].includes(this.state.phase)}
Expand Down
4 changes: 4 additions & 0 deletions src/components/views/dialogs/BaseDialog.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,9 @@ interface IProps extends IDialogProps {
// determine its size. Default: true.
"fixedWidth"?: boolean;

// To be displayed at the top of the dialog. Even above the title.
"top"?: React.ReactNode;

// Title for the dialog.
"title"?: JSX.Element | string;
// Specific aria label to use, if not provided will set aria-labelledBy to mx_Dialog_title
Expand Down Expand Up @@ -161,6 +164,7 @@ export default class BaseDialog extends React.Component<IProps> {
mx_Dialog_fixedWidth: this.props.fixedWidth,
})}
>
{this.props.top}
<div
className={classNames("mx_Dialog_header", {
mx_Dialog_headerWithButton: !!this.props.headerButton,
Expand Down
2 changes: 2 additions & 0 deletions src/components/views/dialogs/InfoDialog.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ import BaseDialog from "./BaseDialog";
import DialogButtons from "../elements/DialogButtons";

interface IProps extends IDialogProps {
top?: ReactNode;
title?: string;
description?: ReactNode;
className?: string;
Expand All @@ -49,6 +50,7 @@ export default class InfoDialog extends React.Component<IProps> {
<BaseDialog
className="mx_InfoDialog"
onFinished={this.props.onFinished}
top={this.props.top}
title={this.props.title}
contentId="mx_Dialog_content"
hasCancel={this.props.hasCloseButton}
Expand Down
2 changes: 2 additions & 0 deletions src/i18n/strings/en_EN.json
Original file line number Diff line number Diff line change
Expand Up @@ -3621,6 +3621,8 @@
"Success!": "Success!",
"Create key backup": "Create key backup",
"Unable to create key backup": "Unable to create key backup",
"Secure Backup successful": "Secure Backup successful",
"Your keys are now being backed up from this device.": "Your keys are now being backed up from this device.",
"Generate a Security Key": "Generate a Security Key",
"We'll generate a Security Key for you to store somewhere safe, like a password manager or a safe.": "We'll generate a Security Key for you to store somewhere safe, like a password manager or a safe.",
"Use a secret phrase only you know, and optionally save a Security Key to use for backup.": "Use a secret phrase only you know, and optionally save a Security Key to use for backup.",
Expand Down

0 comments on commit b463226

Please sign in to comment.