Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UX : "New Recovery Method" dialog does not cover the most common case (it was not an attacker, it was me on mobile) #825

Closed
2 tasks
estellecomment opened this issue Nov 14, 2023 · 2 comments
Labels

Comments

@estellecomment
Copy link
Contributor

estellecomment commented Nov 14, 2023

When I create a new recovery code on mobile, I see this dialog on web :

image

I don't know what to do next : it was not an attacker, it was me. This is the most common use case (users are rarely attacked for real)

  • I need info : Is there anything I should do, or is everything ok ?
  • Where do I click to make this go away ?

For dev : also need to understand :

  • does this dialog show up every time I create a new recovery code? (@NicolasBuquet saw it while debugging a messed up session, what happens in a normal case?)
  • what does the primary button actually do?
@estellecomment
Copy link
Contributor Author

does this dialog show up every time I create a new recovery code?

No.

When does it show up then ?
In https://github.com/matrix-org/matrix-react-sdk/blob/81c62e51c474581c3d3f0f1e72aa0d1eb26ac45a/src/components/structures/MatrixChat.tsx#L1688..L1711
NewRecoveryMethodDialog is created when cli.on(CryptoEvent.KeyBackupFailed), in the case where there is another keybackup detected.

When I'm in the process of recreating a code, but another client recreates a code ?

@estellecomment
Copy link
Contributor Author

Could not reproduce in 3 tries.
This is a rare enough case that it's fine to have a bad UX on it. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant