Skip to content

Conversation

ruchagosavi
Copy link

πŸ› οΈ Fixes Issue

#1058

πŸ‘¨β€πŸ’» Changes proposed

Removed Some buttons and textfields

Added message box after successful signup

βœ”οΈ Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

πŸ“„ Note to reviewers

πŸ“· Screenshots

ruchagosavi and others added 2 commits May 20, 2023 20:11
This is modified UI of Signup page
Some changes made. Removed some buttons and text fields.
@netlify
Copy link

netlify bot commented Jun 4, 2023

βœ… Deploy Preview for aec-library-wbsite ready!

Name Link
πŸ”¨ Latest commit 6c65964
πŸ” Latest deploy log https://app.netlify.com/sites/aec-library-wbsite/deploys/647cac067d44de0008ae7e4c
😎 Deploy Preview https://deploy-preview-1306--aec-library-wbsite.netlify.app/sign-up-page
πŸ“± Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@rohanbabbar04
Copy link
Collaborator

image

I really like the vertical layout but I feel you have missed some important fields which we require

  • Username
  • Mobile Number
  • Password also does not have an option to show/hide.
  • Also add a Confirm Password field.
  • Login button does not look nice(the text is very light and does not go with the color theme of the page, maybe think on it a little bit)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants