-
Notifications
You must be signed in to change notification settings - Fork 2
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
'User'-related UX improvements #829
Comments
Great plan, fully agree. We might want to add "register" to the "log in" link text so it's obvious that the link serves both functions |
I agree with most points. Regarding hiding the |
That sounds fine for me. The UX issue there ATM is the redirect to the login page. We can instead display a "you need to log in or create an account" message (with links). |
@bh-ethz, here are some suggestions regarding the UX. |
This is almost entirely implemented. The only thing missing here is to scrub I implemented some of it in #1109 |
That's included in #1224 |
The below is my opinion of how we could improve UX around User page and login
When you are not logged in:
andSubmit
Revise
buttons should be hiddenUser
button should be replaced by aLog in
buttonLog in
page (which needs general restyling with keycloak at some point - Keycloak style improvements #743 ) already provides the option toRegister
if you don't have an account.When you are logged in:
Submit
button visible (Revise
feature should be linked from submit page)My account
button visible, providingLog out
feature and group selection feature (Sequence overview is mostly moved under theSubmit
page as part of the wizard).The text was updated successfully, but these errors were encountered: