WordPress.org

Make WordPress Core

Opened 5 days ago

Closed 4 days ago

Last modified 4 days ago

#52597 closed defect (bug) (fixed)

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

Reported by: audrasjb Owned by: SergeyBiryukov
Milestone: 5.7 Priority: normal
Severity: normal Version: trunk
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 5 days ago.
52597.diff (1.8 KB) - added by mukesh27 4 days ago.
Initial Patch.
52597-after-applying-patch.png (177.2 KB) - added by hellofromTonya 4 days ago.
User profile UI after applying patch 52597.diff

Download all attachments as: .zip

Change History (11)

#1 @SergeyBiryukov
5 days ago

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

#2 @hellofromTonya
5 days ago

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

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


5 days ago

@mukesh27
4 days ago

Initial Patch.

#4 @mukesh27
4 days ago

  • Keywords has-patch added; needs-patch removed

@hellofromTonya
4 days ago

User profile UI after applying patch 52597.diff

#5 @hellofromTonya
4 days 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.


4 days ago

#7 @SergeyBiryukov
4 days 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
4 days 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.