Closed (fixed)
Project:
Password Policy
Version:
4.0.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
11 Apr 2022 at 10:51 UTC
Updated:
8 Apr 2024 at 05:35 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
tvalimaa commentedHi, I did custom code which hide password policy status thing on user edit and user register form.
hide_pwpolicy.module
Comment #3
paulocsPostponed this issue while #2849271: Decouple display of policy table and policy validation is not committed.
Comment #4
paulocsComment #5
sophiavs commentedI'll work on this
Comment #7
sophiavs commentedComment #8
sophiavs commentedI put a states option on the password policy to show only when the password field is not empty as you can see in the images
Comment #9
gquisini commentedI'll be reviewing
Comment #10
gquisini commentedBefore I switched to MR, which I did:
- Installed password policy and password character length policy.
- I went to /admin/config/security/password-policy/add and created a new policy with the password character length restriction and applied it to Authenticated user
- I went to /user/[user:id]/edit and there are just the recommendations to make the password stronger
- After switching to MR, I went again to edit the user profile and it now shows the recommendations and the password policy restrictions.
I'll attach before/after screenshots to comparison.
Comment #12
rajeshreeputra+1 RTBC, rebased.
Comment #13
kristen polAssigning to myself as I'm reviewing/merging ready RTBC fixes/updates over the next few days.
Comment #14
kristen polI tested and this seems like a simple UX improvement so I think it's okay to add. Note that it doesn't hide it after the password is deleted but that's probably okay as well.
Related, but not part of this issue is that, when editing a user as an admin (or with certain permissions), the status and roles show up between the password fields and the table which pushes the info far below and the user might not see it. A separate issue could be opened for that.
Comment #16
kristen polThanks everyone for their work on this issue.
The fix has been merged and will be part of the next release.
Comment #17
kristen polThis was against 4.0.x as 8.x branches are no longer supported, so changing version.
Comment #19
kristen polThis is part of the new 4.0.1 release.