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

Me: Fix the current flow for Sign Out when using 2FA #3666

Closed
Tug opened this issue Feb 29, 2016 · 18 comments
Closed

Me: Fix the current flow for Sign Out when using 2FA #3666

Tug opened this issue Feb 29, 2016 · 18 comments
Assignees
Labels
[Feature] User & Account Settings (/me) Settings and tools for managing your WordPress.com user account. [Type] Bug When a feature is broken and / or not performing as intended

Comments

@Tug
Copy link
Contributor

Tug commented Feb 29, 2016

Steps to reproduce

  1. Have 2FA enabled on your account
  2. Starting at URL: http://calypso.localhost:3000/
  3. You want to log out
  4. You click on your avatar on the top right and are redirected to http://calypso.localhost:3000/me
  5. You are asked for a verification code before you can do anything else, including signing out

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

just wanna log out

@Tug Tug added [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it [Feature] User & Account Settings (/me) Settings and tools for managing your WordPress.com user account. labels Feb 29, 2016
@Tug Tug self-assigned this Feb 29, 2016
@Tug Tug changed the title Fix the current flow for Sign Out when using 2FA Me: Fix the current flow for Sign Out when using 2FA Feb 29, 2016
@lancewillett
Copy link
Contributor

Might be a duplicate of #202

@aduth
Copy link
Contributor

aduth commented Apr 7, 2016

Might be a duplicate of #202

#202 was technically first, but yes, I agree with assessment this is duplicate.

@lancewillett
Copy link
Contributor

Duplicate of means that one is first, yes. :)

@lancewillett
Copy link
Contributor

Closing in favor of older issue.

@aduth
Copy link
Contributor

aduth commented Apr 7, 2016

Err, sorry, I was referring to #4272, which is newer than this issue and should have been closed, keeping this one open. #202 I feel is a separate issue.

@lancewillett
Copy link
Contributor

See also #960 — similar UX problem for changing a password.

@lancewillett lancewillett added [Type] Bug When a feature is broken and / or not performing as intended and removed [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it labels Apr 7, 2016
@lancewillett
Copy link
Contributor

One quick improvement would be to make the Not you? Sign out link stand out more:

just wanna log out

@Tug
Copy link
Contributor Author

Tug commented Apr 7, 2016

Agreed, I didn't notice it was there actually!

@aduth
Copy link
Contributor

aduth commented Apr 11, 2016

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 /me/settings/profile-links endpoint to be requested infinitely.

@Tug
Copy link
Contributor Author

Tug commented Apr 12, 2016

So this is design issue IMO.
I think we could show a small card about the current user profile with the gravatar image and the name of the account. Then you would notice if it's not yours and find quickly the "Not you?" attached to this card.

@lancewillett
Copy link
Contributor

@Tug Love that idea, much more visually rich and communicates which user is logged in.

@ryanboren
Copy link
Contributor

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.

When I’m on the desktop computer at the office and finish ordering books on Amazon.com for an article I’m working on, for instance, I can never find a simple log-out button. Instead, I scroll down a column marked “Hello, Natasha — Your Account” and then click on an option that reads: “Not Natasha? Sign Out.” But the notion that in order to sign off, I have to deny that I am me always gives me pause.

“It’s really clever — they want you to do something that is counterintuitive and maybe a little freaky to you,” Mr. Nodder said. “There’s an element of ‘Maybe I just won’t sign out this time.’”

http://www.nytimes.com/2016/05/15/technology/personaltech/when-websites-wont-take-no-for-an-answer.html?_r=0

I overlook sign out. Once I do finally notice it, I have to deny my identity.

@ryanboren
Copy link
Contributor

Also, on iOS, tapping Notifications doesn't dismiss the 2FA dialog, contributing to a trapped feeling.

@designsimply designsimply added the [Status] Needs Design Review Add this when you'd like to get a review / feedback from the Design team on your PR label May 16, 2016
@ryanboren
Copy link
Contributor

I suggest "Sign out" standing alone. This dialog is an intrusion in flow that usually features a prominent Sign out button.

screen shot 2016-05-16 at 11 14 36 am

img_9663

@folletto
Copy link
Contributor

The quick fix would be just to move "Sign Out" in the dialog below, since all the Dialog actions are meant to stay below:

screen shot 2016-05-16 at 17 36 10

(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):

2fa-inpage-i1

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.

@davewhitley
Copy link
Contributor

davewhitley commented Jun 2, 2016

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.

@kriskarkoski
Copy link
Contributor

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.

@mcsf
Copy link
Member

mcsf commented Jul 6, 2017

Closed per #15095

@mcsf mcsf closed this as completed Jul 6, 2017
@matticbot matticbot removed the [Status] Needs Design Review Add this when you'd like to get a review / feedback from the Design team on your PR label Jul 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] User & Account Settings (/me) Settings and tools for managing your WordPress.com user account. [Type] Bug When a feature is broken and / or not performing as intended
Projects
None yet
Development

No branches or pull requests

10 participants