Make WordPress Core

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#52597 closed defect (bug) (fixed)

Consider moving the Password reset feature under the Account management section of User profile screen

Reported by: audrasjb's profile audrasjb Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.7 Priority: normal
Severity: normal Version: 5.7
Component: Users Keywords: has-patch commit
Focuses: ui Cc:

Description

The newly introduced "Password Reset" feature is located in the "About the user" section, right above the "Account management" section.

What about moving it the the "Account management" section? Indeed it's about managing users password, and not so much about user personal information.

Attachments (3)

Capture d’écran 2021-02-22 à 11.56.30.png (218.3 KB) - added by audrasjb 3 years ago.
52597.diff (1.8 KB) - added by mukesh27 3 years ago.
Initial Patch.
52597-after-applying-patch.png (177.2 KB) - added by hellofromTonya 3 years ago.
User profile UI after applying patch 52597.diff

Download all attachments as: .zip

Change History (11)

#1 @SergeyBiryukov
3 years ago

Yes, I think it would make more sense in the Account Management section :)

#2 @hellofromTonya
3 years ago

  • Keywords needs-patch added; 2nd-opinion removed

This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.


3 years ago

@mukesh27
3 years ago

Initial Patch.

#4 @mukesh27
3 years ago

  • Keywords has-patch added; needs-patch removed

@hellofromTonya
3 years ago

User profile UI after applying patch 52597.diff

#5 @hellofromTonya
3 years ago

  • Keywords commit added

Patch works as expected to relocate the "Password Reset" button to the "Account Management" section. Patch is a straight up copy-paste to relocate. Marking for commit.

This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.


3 years ago

#7 @SergeyBiryukov
3 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 50411:

Users: Relocate the Password Reset feature to the Account Management section.

Follow-up to [50129].

Props mukesh27, audrasjb, hellofromTonya.
Fixes #52597.

#8 @SergeyBiryukov
3 years ago

In 50412:

Users: Give the Password Reset table row its own wrapper class.

Previously, the class of the Sessions table row was duplicated.

Follow-up to [50129], [50411].

See #52597.

Note: See TracTickets for help on using tickets.