mirrors/element-web
1
0
Fork 0
mirror of https://github.com/element-hq/element-web.git synced 2024-12-12 16:37:51 +03:00
Code Issues Projects Releases Packages Wiki Activity

Hopefully ake cancel dialog a bit less weird

Browse source 
There's no design on how to fix this so I've switched the buttons
and made the primary not a danger button. We could also try some
different wording, eg. 'abort' rather than 'cancel' because with
'ancel' it's not clear if you're cancelling whatever you were trying
to do or the dialog asking you if you want to cancel...

Ideal might be to make the cancel button red but that means making
it a separate button or adding support for doing so to DialogButtons,
so not going to do that unless we're sure that's what we want.

Fixes https://github.com/vector-im/riot-web/issues/14140
This commit is contained in:
David Baker 2020-06-25 14:52:59 +01:00
parent 086177d808
commit 497ab0f212
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/CrossSigningManager.js
View file

@ -54,11 +54,11 @@ async function confirmToDismiss(name) {
const [sure] = await Modal.createDialog(QuestionDialog, {
title: _t("Cancel entering passphrase?"),
description,
danger: true,
cancelButton: _t("Enter passphrase"),
button: _t("Cancel"),
danger: false,
button: _t("Enter passphrase"),
cancelButton: _t("Cancel"),
}).finished;
return sure;
return !sure;
}
async function getSecretStorageKey({ keys: keyInfos }, ssssItemName) {