Display show/hide button on selected form, not on all forms.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | selected_form-2851993-4.patch | 2.17 KB | davps |
| #2 | selected_form-2851993-2.patch | 2.16 KB | davps |
Display show/hide button on selected form, not on all forms.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | selected_form-2851993-4.patch | 2.17 KB | davps |
| #2 | selected_form-2851993-2.patch | 2.16 KB | davps |
Comments
Comment #2
davps commentedCreated a settings page where user can select necessary form.
Comment #3
davps commentedComment #4
davps commentedAdded user_register_form form.
Comment #5
riddhi.addweb commentedAs per the password toggle concept, it should toggle input field type to text from password when the user checked show password checkbox(instead of showing password below the text-field) and vice versa.
So by this way user don't have to re-enter their password to edit it. If we show password in text-field, the user can easily change a single letter instead of the whole password.
Comment #6
davps commented@Jigar.addweb I think it would be correct to make it as a separate issue.
Comment #7
IRuslan commented@david.pashaev I guess you need to change path for the settings form from /admin/settings/password_toggle
to /admin/config/people/password_toggle
Comment #8
jacob.embree commentedThe patch does not apply.
IRuslan is correct about /admin/config/people, but I would use a hyphen, thus, /admin/config/people/password-toggle.