-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Me: Fix the current flow for Sign Out when using 2FA #3666
Comments
Might be a duplicate of #202 |
Duplicate of means that one is first, yes. :) |
Closing in favor of older issue. |
See also #960 — similar UX problem for changing a password. |
Agreed, I didn't notice it was there actually! |
Possibly better tracked as a separate issue and certainly worrying: following the flow described in the original comment, another consequence of the stale token is that navigating to the Me section causes the |
So this is design issue IMO. |
@Tug Love that idea, much more visually rich and communicates which user is logged in. |
I always overlook sign out links that are prefixed with "Not you?" type text. I've stared at this screen looking for and missing the sign out link on several occasions. I am certain to overlook sign out links embedded in text. Amazon is the classic case of this.
I overlook sign out. Once I do finally notice it, I have to deny my identity. |
Also, on iOS, tapping Notifications doesn't dismiss the 2FA dialog, contributing to a trapped feeling. |
The quick fix would be just to move "Sign Out" in the dialog below, since all the Dialog actions are meant to stay below: (btw apparently there's a bug there on mobile where the button falls below) However... that screen shouldn't be a Dialog. Dialogs should be avoided as much as possible, and in this scenario it doesn't make sense design-wise (even I'm sure there was a technical reason to make it that way): In this design concept the "dialog" isn't really a dialog (we could even design it to look entirely different) but a "lock" on the specific area of the screen that is impacted in practice by the lock. The sidebar is still functional (navigating isn't locked) and this allows the access to any section that doesn't have the lock... or to see again the log if even other sections are actually locked. |
I remember the discussion of the design of this modal, but I can't find it. I quicker solution would be to make "Send SMS Code" a link instead of a button (and moved up), and make "Sign Out" a button at the bottom. |
I don't have a stale 2fa to grab a screenshot, but this still exists as in the original screenshot with the Sign Out link inline and easy to overlook. +1 to @drw158 quick fix suggestion above since I don't think a revamp of this design is on anyone's roadmap near-term. |
Closed per #15095 |
Steps to reproduce
What I expected
Being able to log out from an account without knowing the verification code for it.
What happened instead
I was asked for a verification code I may not have (if it is not my account).
Screenshot
The text was updated successfully, but these errors were encountered: