feat: rewrite signup and signin page with tailwind UI

[olamide203]

Fixes

• a rewrite of the signup and signing page with tailwind UI

Closes #100

📄 Note to reviewers

• used react-icons for social media icons on signup and sign in page

📷 Screenshots

[vercel]

Someone is attempting to deploy a commit to a Personal Account owned by @ykdojo on Vercel.

@ykdojo first needs to authorize it.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
defaang	✅ Ready (Inspect)	Visit Preview	Aug 22, 2022 at 9:07AM (UTC)

[ykdojo]

Thank you! It looks much better than what we had before, but I think the registration/signin box should be in a white background.

[olamide203]

I've made changes to the areas that were pointed out
also used the image from the code template in place of the former one

📷 Screenshots

[ykdojo]

Thank you. The new UI has a lot of whitespace on the bottom, though:

Also, the "sign in with" options should be Google and GitHub - or we can comment them out for now.

[olamide203]

Fixes

• Removed extra space at the bottom of UI
• Used react-icons for social icons
• Replaced social signup options with google and github

[ykdojo]

Thank you so much! It looks great.

A few minor things:

1. I feel like the heading ("Sign in to your account") is slightly too thick?

That might be a styling thing - I might make a separate issue for it instead:

2. Should we comment out the social logins/signups until we implement them?

[olamide203]

• The heading does look a bit wordy. I think just 'Sign in' should be okay.
• we could also comment out the social logins/signups till they are implemented. I think there is an open issue addressing it.
  should I make a new PR with these changes?

[ykdojo]

Yes! Thank you. I think you can just update this PR

[ykdojo]

One more thing - let's remove the username field from the signup form.

Context: #95

[olamide203]

what about the confirm password field? it not currently in use. should it be commented out till the feature gets implemented?

[ykdojo]

Let's actually remove that one. I don't see a lot of websites using that field nowadays, and I don't think it's necessary.

[iShibi]

Looks good to me. We can use some SignIn and SignUp components here to keep the pages tidy, what do you think?

[olamide203]

yea, both forms could be moved to separate components with a handleSubmit prop which would get called on submit. what are your thoughts on this?

[iShibi]

I think this is ready to merge. In the next iteration, we can add options for usernames and socials. Just run npm run format:fix to resolve the prettier error and we are good to go.

[ykdojo]

Since I have to interact with this part of the code as I work on #108, I'm going to merge it as is.

I might raise issues/propose fixes as I go.