search

kblincoe / VisualGit_SE701_2019_1

1 stars 0 forks source link

Redesign sign out button for reaching main menu screen #159

Closed cyrus-raitava closed 5 years ago

cyrus-raitava commented 5 years ago

This issue relates to #63 and #108

As of currently, there is functionality for going back from having logged in, to the main screen the user is first shown.

Access to this screen however, does not currently exist, as far as I'm aware, if the user clicks 'Continue without Sign In'.

My proposal is that instead of being greeted with the 'Sign In' button in the top left-hand corner, they be given a 'back' button, that takes them back to the main screen. This makes greater logical sense to me, and provides a more consistent experience for the user.

CURRENT FLOW:

PROPOSED FLOW:

See the attached screenshots for reference:

Screen Shot 2019-03-27 at 11 59 24 PM Screen Shot 2019-03-28 at 12 00 26 AM
DionBalmforth commented 5 years ago

I agree, replacing the "sign in" button with a "back" button would be better for the user - Approved.

hhagenson28 commented 5 years ago

I think this is a good idea to remove duplicate functionality. I think you could just leave the grey "Sign In" button in the top right hand corner and instead of the sign in header panel appearing, clicking the sign in button would just redirect the user back to the login page, instead of changing it to "Back". This is because when a user is logged in, clicking sign out sends them back to the login page so this would keep things more consistent with Sign In/ Sign Out, which is the only reason for a user to go 'back' anyway.

liamtbrand commented 5 years ago

I agree with Holly, I don't think the button should be changed to a back button, it should be a sign in button that takes you to the sign in page instead of opening an alternative sign in. Approved.

cyrus-raitava commented 5 years ago

Great suggestions, I agree with both of you @hhagenson28 @liamtbrand !! I will implement the changes as you both explained.

cyrus-raitava commented 5 years ago

Note @liamtbrand @hhagenson28 that #171 redesigned the way the user logs out, if they are logged in. To keep the user experience consistent, I modified the use path for if the user isn't logged in, to be near the same experience. See the screenshots in #183 for more information!

(Justification for not deviating from the advice both of you gave)