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.

Sign up for GitHub

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

Jump to bottom

Show Password Should be Available on Reset Password/Change Pages Too #1236

Closed
wonderchook opened this issue Mar 7, 2017 · 3 comments
Closed

Show Password Should be Available on Reset Password/Change Pages Too #1236

wonderchook opened this issue Mar 7, 2017 · 3 comments
Labels
ui/ux

Comments

@wonderchook
Copy link
Contributor

When signing up for an account there is a "show password" option, this is not the case for password reset or the change password pages.
@Divya063
Copy link
Contributor

Divya063 commented Mar 7, 2017

@wonderchook but there is a prompt box of "show password" option by the firefox browser in password reset or the change password pages.

@wonderchook
Copy link
Contributor Author

Can you attach a screenshot? It isn't there on Chrome

@wonderchook wonderchook mentioned this issue Mar 7, 2017
Closed
@clash99 clash99 mentioned this issue Mar 8, 2017
Merged
@clash99
Copy link
Contributor

clash99 commented Mar 8, 2017

Would like same integration as #1240.

@Shruti9520 Shruti9520 mentioned this issue Mar 8, 2017
Closed
14 tasks
niharika1995 added a commit to niharika1995/cadasta-platform that referenced this issue Mar 8, 2017
@niharika1995
Fix #@1216, Cadasta#1217, Cadasta#1236: Replace the Show Password c…
d264c0a 
…heckbox with glyphicons

Changes proposed:
1. Moves JS code to show/hide password to `password_widget.js`.
2. Add a show/hide glyphicon toggle to the right of password widget.
3. Add a border around the glyphicon when form submission returns errors.
niharika1995 added a commit to niharika1995/cadasta-platform that referenced this issue Mar 8, 2017
@niharika1995
Fix Cadasta#1216, Cadasta#1217, Cadasta#1236: Replace the `Show Passw…
a765e5f 
…ord` checkbox with glyphicons

Changes proposed:
1. Moves JS code to show/hide password to `password_widget.js`.
2. Add a show/hide glyphicon toggle to the right of password widget.
3. Add a border around the glyphicon when form submission returns errors.
niharika1995 added a commit to niharika1995/cadasta-platform that referenced this issue Mar 8, 2017
@niharika1995
Fix Cadasta#1216, Cadasta#1217, Cadasta#1236: Replace the `Show Passw…
b648065 
…ord` checkbox with glyphicons

Changes proposed:
1. Moves JS code to show/hide password to `password_widget.js`.
2. Add a show/hide glyphicon toggle to the right of password widget.
3. Add a border around the glyphicon when form submission returns errors.
niharika1995 added a commit to niharika1995/cadasta-platform that referenced this issue Mar 8, 2017
@niharika1995
Fix Cadasta#1216, Cadasta#1217, Cadasta#1236: Replace the `Show Passw…
b5c312c 
…ord` checkbox with glyphicons

Changes proposed:
1. Moves JS code to show/hide password to `password_widget.js`.
2. Add a show/hide glyphicon toggle to the right of password widget.
3. Add a border around the glyphicon when form submission returns errors.
@dpalomino dpalomino added the ui/ux label Mar 9, 2017
niharika1995 added a commit to niharika1995/cadasta-platform that referenced this issue Mar 9, 2017
@niharika1995
Fix Cadasta#1216, Cadasta#1217, Cadasta#1236: Replace the `Show Passw…
5ab3319 
…ord` checkboxes

Changes proposed:
1. Moves JS code to show/hide password to `password_widget.js`.
2. Add a show/hide glyphicon toggle to the right of password widget.
3. Add a border around the glyphicon when form submission returns errors.
oliverroick pushed a commit to niharika1995/cadasta-platform that referenced this issue Mar 10, 2017
@niharika1995 @oliverroick
Fix Cadasta#1216, Cadasta#1217, Cadasta#1236: Replace the `Show Passw…
4e976ed 
…ord` checkboxes

Changes proposed:
1. Moves JS code to show/hide password to `password_widget.js`.
2. Add a show/hide glyphicon toggle to the right of password widget.
3. Add a border around the glyphicon when form submission returns errors.
amplifi pushed a commit that referenced this issue Mar 10, 2017
@niharika1995 @amplifi
Fix #1216, #1217, #1236: Replace the Show Password checkboxes (#1240)
cd3865a 
Changes proposed:
1. Moves JS code to show/hide password to `password_widget.js`.
2. Add a show/hide glyphicon toggle to the right of password widget.
3. Add a border around the glyphicon when form submission returns errors.
laura-barluzzi pushed a commit to laura-barluzzi/cadasta-platform that referenced this issue Mar 14, 2017
@niharika1995 @laura-barluzzi
Fix Cadasta#1216, Cadasta#1217, Cadasta#1236: Replace the `Show Passw…
d38ac1a 
…ord` checkboxes (Cadasta#1240)

Changes proposed:
1. Moves JS code to show/hide password to `password_widget.js`.
2. Add a show/hide glyphicon toggle to the right of password widget.
3. Add a border around the glyphicon when form submission returns errors.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
ui/ux
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixed-issue:1236 Shruti9520/cadasta-platform
4 participants
@wonderchook @clash99 @dpalomino @Divya063