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

Add Password Strength and Toggle Password #44

Merged
merged 2 commits into from
May 31, 2024

Conversation

kushalkumar1362
Copy link
Contributor

This PR adds a password strength feature to the password input component.

It includes:

  • Real-time password strength evaluation using the zxcvbn library.
  • Visual indicators for password strength.
  • Password suggestions with FontAwesome icons to indicate whether specific password criteria are met.
  • Add Toggle password eye using react-icons library.

Changes

  • Updated the PasswordInput component to include the password strength meter.
  • Added checks for lowercase & uppercase letters, numbers, special characters, and minimum length.
  • Add Toggle password eye using react-icons library in both signup and login page.

Screenshots

image

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings

Linked Issues: #37

Copy link

vercel bot commented May 30, 2024

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

Name Status Preview Comments Updated (UTC)
style-share ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 31, 2024 4:09pm

Copy link

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@kushalkumar1362
Copy link
Contributor Author

please also add labels in PR as per GSSOC guidelines

Copy link
Owner

@VaibhavArora314 VaibhavArora314 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

resolve merge conflicts, rest is good

@kushalkumar1362
Copy link
Contributor Author

resolve merge conflicts, rest is good

when is make request there is no conflicts let me check

@kushalkumar1362
Copy link
Contributor Author

please merge this before merge another request this conflicts that arire because you merge some another request
also add label in the PR

@kushalkumar1362
Copy link
Contributor Author

@VaibhavArora314

@VaibhavArora314 VaibhavArora314 merged commit b8507ed into VaibhavArora314:main May 31, 2024
2 checks passed
Copy link

🎉 Your pull request has been successfully merged! 🎉 Thank you for your valuable contribution to our project. Your efforts are greatly appreciated. Feel free to reach out if you have any more contributions or if there's anything else we can assist you with. Keep up the fantastic work! 🚀

@kushalkumar1362
Copy link
Contributor Author

@VaibhavArora314 as per guidelines please add level label , gssoc label here

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

Successfully merging this pull request may close these issues.

3 participants