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

Clear input box after user tries to reset password #120

Open
umih874 opened this issue Mar 18, 2018 · 4 comments
Open

Clear input box after user tries to reset password #120

umih874 opened this issue Mar 18, 2018 · 4 comments
Labels

Comments

@umih874
Copy link
Contributor

umih874 commented Mar 18, 2018

Issue

Currently, when the user tries to reset the password, after selecting save, a feedback message comes up indicating whether the password was reset properly (2nd image) or not (3rd image). However, the text inside the input box stays there and the user has to clear the box before it can try again. I think it would be nice if the box clears after the user clicks the save button.

Recreate

Log in as 'User' and go to password section. Enter current, new and confirmation passwords and click save.

Issue Difficulty

Initially, I thought it would be a quick fix by just clearing input box. So I tried doing it. However, I found that after I clear input box the new warning shows up underneath the box (4th image). It doesn't seem like a big deal but it could be removed.
If we want the issue to be very-small than maybe we can leave the warning message after the input box is cleared. Otherwise, maybe we should make it small? Or maybe all together can be very-small?

Image evidence

priortochange
postchange
postfail
fixattempt

@umih874
Copy link
Contributor Author

umih874 commented Mar 18, 2018

@softeng-701

@softeng-701 softeng-701 added small and removed small labels Mar 18, 2018
@umih874
Copy link
Contributor Author

umih874 commented Mar 20, 2018

@softeng-701 I have looked at this in more detailed and I believe it should be classed as a small issue.

@softeng-701
Copy link
Collaborator

@umih874 APPROVED!

@umih874
Copy link
Contributor Author

umih874 commented Mar 21, 2018

Claiming this one

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

No branches or pull requests

2 participants