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

Add WebAuthn UI stub #87

Closed
6 tasks done
iandunn opened this issue Mar 22, 2023 · 0 comments · Fixed by #95
Closed
6 tasks done

Add WebAuthn UI stub #87

iandunn opened this issue Mar 22, 2023 · 0 comments · Fixed by #95
Assignees
Milestone

Comments

@iandunn
Copy link
Member

iandunn commented Mar 22, 2023

We should stub out the rough code and UI for the WebAuthn front-end, so we have something concrete to build off of. This is just the rough work, refinement will be done in #193 and #194.

  • this should be behind a flag so that production users don't see it until the backend is in place

WPCOM design as a reference, but it won't translate directly, and #194 will have a more refined version.

  • entry screen

this might not really fit into our design. we have the account status screen instead

Screenshot 2023-03-24 at 1 11 03 PM

  • naming key

Screenshot 2023-03-24 at 1 11 13 PM

  • spinner while waiting on browser to receive token
    note: i don't think we should require a verification pin. it hurts the UX more than it helps security IMO

Screenshot 2023-03-24 at 1 11 46 PM

  • success notice

Screenshot 2023-03-24 at 1 13 51 PM

  • confirm removing key

Screenshot 2023-03-24 at 1 15 01 PM

@iandunn iandunn added this to the Iteration 1 milestone Mar 22, 2023
@iandunn iandunn self-assigned this Mar 22, 2023
This was referenced Mar 24, 2023
@iandunn iandunn changed the title Add WebAuthn UI stub Add WebAuthn UI May 26, 2023
@iandunn iandunn changed the title Add WebAuthn UI Add WebAuthn UI stub May 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant