temp4ccount
commented
9 months ago Looks good to merge. Could you add the correct font for the login page (#80). I will merge this PR after that.
Closed BarryAlanPan closed 9 months ago
temp4ccount
commented
9 months ago Looks good to merge. Could you add the correct font for the login page (#80). I will merge this PR after that.
BarryAlanPan
commented
9 months ago Looks good to merge. Could you add the correct font for the login page (#80). I will merge this PR after that.
Font problem solved
BarryAlanPan
commented
9 months ago DO NOT MERGE, Testing modification in progress.
BarryAlanPan
commented
9 months ago Testing issue fixed
Please check if the PR fulfills these requirements
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) New password visibility feature let user decide password field's showing or hiding state. Tests for login page added.
What is the current behavior? (You can also link to an open issue here) User can't see password they input.
What is the new behavior (if this is a feature change)? User can see password they input if they choose to.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?) None
Add screenshot if possible. (Add screenshot of the change)
Other information: Close #58 Implement #55