TheCyberHUB
TheCyberHUB copied to clipboard
thecyberworld
[FEATURE] Add Show Password Option
Describe the bug
When a user enters an incorrect password in the password field, the field does not automatically clear itself. Instead, the incorrect password remains in the field until manually deleted by the user.
To Reproduce
Navigate to the login page. Enter an incorrect password in the password field. Observe that the incorrect password remains in the field.
Expected Behavior
After entering an incorrect password, the password field should automatically clear itself, ready for the user to re-enter their password without having to manually delete the incorrect input.
Screenshot/ Video
No response
Code of Conduct
- [X] I agree to follow this project's Code of Conduct
It's great having you contribute to this project
🌟 Welcome to the community 🌟
- If you would like to continue contributing to open source and would like to do it with an awesome inclusive community.
- You should join our Discord chat and our GitHub Organisation.
- We help and encourage each other to contribute to open source little and often 😄.
- Any questions let us know.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
@AhmedKarray005 If we would have "show password" it is not relevant.
@kabir0x23 what do you think?
so you want to add the show password functionality + fixing the bug I told u about?
@AhmedKarray005 You don't need to clear input if we have show password so the user could change typo and so on. Let's wait @kabir0x23 to hear his opinion
hey @AhmedKarray005 @ArkadiK94 sorry for the late response
You don't need to clear input if we have show password so the user could change typo and so on.
this one is also a good option
about clearing password
if user keep entering wrong password, it will lock their account for 30mins example user is using caps lock without knowingly.
i thhink this is better option
You don't need to clear input if we have show password so the user could change typo and so on.
@AhmedKarray005 I changed the title, to add show password button, would you like to handle it? Don't remove the incorrect password on wrong submission.
@ArkadiK94
https://github.com/thecyberworld/TheCyberHUB/assets/104280658/d5344314-895f-469b-9328-76503f0606d3
@AhmedKarray005 I think it is better to add some margin bottom to emphasize that this related to the password input box. Beside it, looks great thanks. Could you open a pr?
https://react-icons.github.io/react-icons/ u can get icons from here
and u can add in same like
password input show hide icon
import { BiHide,BiShowAlt } from "react-icons/bi";
<BiHide />
<BiShowAlt />
@kabir0x23 @ArkadiK94
https://github.com/thecyberworld/TheCyberHUB/assets/104280658/6bbc4dc5-8986-4491-82db-e47509cf470b
@AhmedKarray005 yes, even though, I am not sure of the icons. I think when you see the password the eye should be without cross and the opposite.
@kabir0x23 I need to look for similar examples. Do you know which way is correct? Of the icons?
I found example :) I don't know haha very hard question, who is right
https://github.com/thecyberworld/TheCyberHUB/assets/76536506/7ce31203-698e-48a2-be4b-8a86c281b095
yes in facebook, it is showing what is there
in my example and his pr, it is about what to do
on buttons we do that task
here
arrow is above, when we click, it will bring the tab and icon will be down after words
without getting confused, it's fine for now, will it decide later
@AhmedKarray005 is it ok the someone else will work on it? I saw that someone opened new pr