iProgYou commented 3 years ago

User Auth

Functionality

[ ] The /login page should not display a link to Log In. Same for /signup
[ ] Going to a random route /#/oweiniouewbrviuwebv should redirect or display a 404 page
[ ] Errors should display for both /signup and /login.
[ ] Errors should clear when moving between /signup and /login.
[ ] Can sign up a user
[ ] Can sign in as a user
[ ] Can log out a user
[ ] Can't sign up with the same username/email
[ ] Pressing enter after filling out the session form should use the form data, not the demo user

[ ] The site should look exactly like the actual site
[ ] All relevant elements should have cursor: pointer on hover
[ ] When errors are displayed, most elements should not move around the page (particularly input boxes)
[ ] Remove Redux logger and all console.log()'s from production

iProgYou commented 3 years ago

The functionality seems mainly there, but I need you to make it look as close to Youtube sign in as possible. This means the text and spacing and everything.

iProgYou commented 3 years ago

Make all of the font the same. You can probably put the font on the body tag in your application.css
Currently, I can proceed through the sign in form without having typed anything. I think you want to probably restrict that ability.
Your user icon is too small and needs some right padding
The dropdown menu was displayed when i navigated back to the splash page. Are you tracking the dropdown display in redux state or local component state? (it should be local)
Left padding on placeholder text
Sign in and create forms should look more similar. Make the input fields in the create form the same size as the fields in sign in