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

Code Review - UI/UX #60

Open
VatsKan opened this issue Aug 21, 2020 · 0 comments
Open

Code Review - UI/UX #60

VatsKan opened this issue Aug 21, 2020 · 0 comments

Comments

@VatsKan
Copy link

VatsKan commented Aug 21, 2020

  • looks beaut! love the creativity.
  • perhaps hide the form on the home page until someone is signed in. As a user may think its possible to make a submission, and they put all the effort in writing a message to find true dog love and they submit and find out they cannot :'( cause theyre not logged in. Alternatively a message which says 'you need to log in before submitting an entry'.
  • nice warning people on the sign up form about the requirements. really fun personalised error messages and form validation...love it! one imporvement here is that if i do not enter something in a required field, it gives the error 'enter a valid email' which isn't the correct error message.
  • after user has signed in, would be nice to see something which tells me im signed in (e.g. perhaps hide log in/sign up option in nav bar)
  • after being logged out, i am not able to go anywhere, as the nav bar is not displayed. would copy over your html to the log-in page too. similarly if the password is incorrect on login, do not get to retry.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant