- 🇩🇪Germany Anybody Porta Westfalica
I just wanted to create a UX issue like this. The "current password" field should definitely only be shown, if it's needed, otherwise it confuses regular users a lot.
Similar request: ✨ Move the request for current password on email/password change to a modal dialog. Active
- First commit to issue fork.
- Status changed to Needs review
11 months ago 8:19am 20 December 2023 All the tests are fixed.Marking it needs review for an initial round of reviews.
- 🇮🇳India narendraR Jaipur, India
Needs IS update for proposed solution along with SS.
I have attached the screen shots for the updated approach that we are adopting and also updated the proposed resolution in the issue summary.
- Status changed to Needs work
11 months ago 1:45pm 21 December 2023 - 🇮🇳India narendraR Jaipur, India
Currently administrator can change email/password without filling `current password` for other users. This case is missing.
Also I am not sure but splitting form in multiple forms may impact api's and may require framework manager review. - Status changed to Needs review
11 months ago 6:50am 26 December 2023 - Status changed to Needs work
11 months ago 10:28pm 26 December 2023 - 🇺🇸United States smustgrave
Removed the tags from almost 8 years ago that I don't think bring much value today.
NW for the failing tests,
Been almost 2 years since the usability review, may need another look when near complete as this will be a big change.
- Status changed to Needs review
11 months ago 8:11am 28 December 2023 I have addressed all the feedbacks and all the tests are now passing.Marking it NR for another round of review.
- Status changed to Needs work
11 months ago 9:39am 28 December 2023 - 🇮🇳India narendraR Jaipur, India
With latest changes, email field is now started appearing on 'Edit profile' page.
- Status changed to Needs review
11 months ago 10:49am 28 December 2023 I reverted the change, now it appears conditionally as it was in the older version.
- 🇺🇸United States worldlinemine
At the request/suggestion of @smustgrave in the Drupal Slack #UX channel the UX group discussed this issue while gathered today and we considered the latest patch. In general we agree and confirm the patch is an improvement in useability though we need to work on some specific recommendations for the text of labels and the URLs for better consistency. We will be addressing this issue further at our meeting next Friday so please feel free to join us with any suggestions or concerns.
- 🇮🇳India narendraR Jaipur, India
All threads are resolved. Changes looks good to me. Waiting for accessibility & Usability review
- 🇺🇸United States smustgrave
Fyi, just showed a client this and they loved the cleanup. We don't do a lot of user customization as they usually don't ask for it but they said this made it clearer for them.
- Status changed to Needs work
9 months ago 10:11am 22 February 2024 The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.