WordPress.org

Make WordPress Core

Opened 8 months ago

Closed 8 months ago

Last modified 8 months 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: 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 8 months ago.
52597.diff (1.8 KB) - added by mukesh27 8 months ago.
Initial Patch.
52597-after-applying-patch.png (177.2 KB) - added by hellofromTonya 8 months ago.
User profile UI after applying patch 52597.diff

Download all attachments as: .zip

Change History (11)

#1 @SergeyBiryukov
8 months ago

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

#2 @hellofromTonya
8 months ago

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

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


8 months ago

@mukesh27
8 months ago

Initial Patch.

#4 @mukesh27
8 months ago

  • Keywords has-patch added; needs-patch removed

@hellofromTonya
8 months ago

User profile UI after applying patch 52597.diff

#5 @hellofromTonya
8 months 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.


8 months ago

#7 @SergeyBiryukov
8 months 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
8 months 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.