-
Notifications
You must be signed in to change notification settings - Fork 48
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
Feat: Toggle View for Password field during signup and login #18
Comments
Hi! I would love to work on this issue. As per my understanding, the login page has to be modified to include a toggle feature to show and hide the password as necessary by the user. I have worked with HTML, CSS and JS for about a year now and I hope I can make a valid contribution. Thanks! |
@aswinikalyan30 can I please work on this issue? |
Hey @aindree-2005 , gotta assign it to @Arya-Hari as she had commented before |
@aswinikalyan30 I have been facing an issue importing the database schema and so i haven't been able to view the pages properly on my local system. I have made some changes to the login page only and created a PR. Kindly let me know if the changes are proper, only then will I proceed for the signup page. Sorry for the hinderance :( |
No worries @Arya-Hari , let me know if you need help setting up , we can connect over discord. The readme is still getting prepared so it is quite confusing. |
@aswinikalyan30 i have done changes accept it going to do more |
No @sk-upadhyay that's not how it works. You first get assigned an issue and then raise a PR. @Arya-Hari is already working on this. Do whatever is assigned to you |
@aswinikalyan30 can you add me as reviewer in this |
@aswinikalyan30 I'm so sorry but I've been having issues all week with my laptop and am not able to run any medium or heavy duty applications...I haven't been able to make any changes and I don't think I'll be able to for a few more days. Since I don't want to keep this issue open and since there are other interested contributors for these issues, I'm okay with this issue and the other being reassigned to some other contributor. Once again, sorry for the delay |
@aswinikalyan30 assign me and merge my code |
Sure @Arya-Hari . Thanks for letting me know |
@aswinikalyan30 As my previous code has solved the issue so you can close this issue |
Describe the feature
The field currently shows as password field which can be modified to have a show/hide toggle
Add ScreenShots
Record
The text was updated successfully, but these errors were encountered: