Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 9 years ago

#19470 closed defect (bug) (fixed)

Poor UX after redirecting to user ID anchor on Users screen

Reported by: johnbillion's profile johnbillion Owned by: nacin's profile nacin
Milestone: 3.4 Priority: normal
Severity: minor Version: 3.0
Component: Administration Keywords: has-patch
Focuses: Cc:

Description

After adding a new user from the Users screen you're redirected to an anchor on the Users page (wp-admin/users.php?usersearch=newuser&update=add#user-2). This doesn't work well with the admin bar toolbar in place because it mostly covers up the point where the anchor is. This is more prominent now the toolbar is persistent in the admin area in 3.3.

Off the top of my head I can't think of any other screen which redirects to an anchor. Maybe we should just remove the anchor in the redirect? The redirect includes the usersearch parameter which should be enough in most cases anyway.

Attachments (4)

usersearch.png (15.9 KB) - added by johnbillion 13 years ago.
19470.patch (658 bytes) - added by SergeyBiryukov 13 years ago.
19470.2.patch (1.4 KB) - added by merty 13 years ago.
Here is the patch for my suggestion.
19470.3.patch (9.5 KB) - added by SergeyBiryukov 13 years ago.
An attempt to add the link to all the "Add User" screens (unfinished)

Download all attachments as: .zip

Change History (17)

#1 @SergeyBiryukov
13 years ago

  • Keywords has-patch added

#2 @SergeyBiryukov
13 years ago

  • Milestone changed from Awaiting Review to 3.4

#3 follow-up: @merty
13 years ago

  • Cc merty92@… added

What about adding an "Edit Profile" link to the end of the success message displayed after adding the new user succeeds, in order to make the life easier for those who want to edit the newly-added-user's profile immediately after?

@merty
13 years ago

Here is the patch for my suggestion.

#4 follow-up: @merty
13 years ago

  • Keywords ux-feedback added

#5 in reply to: ↑ 4 ; follow-up: @jane
13 years ago

  • Keywords ux-feedback removed

Replying to merty:
Would probably say "Edit User" rather than Edit Profile.

#6 in reply to: ↑ 5 @merty
13 years ago

Replying to jane: Me too, I used "Edit User" instead of "Edit Profile" in the patch. I wrote my initial post during a lecture so... (Yes, I was that bored) :)

Last edited 13 years ago by merty (previous) (diff)

@SergeyBiryukov
13 years ago

An attempt to add the link to all the "Add User" screens (unfinished)

#7 in reply to: ↑ 3 ; follow-ups: @SergeyBiryukov
13 years ago

Replying to merty:

What about adding an "Edit Profile" link to the end of the success message

I guess we'd want to do that consistently for all the "Add User" screens, including several screens in Multisite. The patch turned out bigger than I initially thought. 19470.3.patch (unfinished) illustrates the idea.

Perhaps we should just fix the issue outlined in the description for now, and move the link addition to a new ticket.

#8 in reply to: ↑ 7 @merty
13 years ago

Replying to SergeyBiryukov:

Replying to merty:

What about adding an "Edit Profile" link to the end of the success message

I guess we'd want to do that consistently for all the "Add User" screens, including several screens in Multisite. The patch turned out bigger than I initially thought. 19470.3.patch (unfinished) illustrates the idea.

Perhaps we should just fix the issue outlined in the description for now, and move the link addition to a new ticket.

Yes, all of them should have the "Edit User" link. My patch was to give a general idea about my proposal but the real patch is way bigger than it as you mentioned so I agree that it deserves its own ticket for further discussions.

#9 @nacin
13 years ago

In [20630]:

Clean up messages on edit.php. see #19470.

#10 @nacin
13 years ago

A lot of the code proposed in this ticket is designed to clean up how we handle multisite redirects, which when dealing with users, is a bit of a mess. (I think I just re-defined the phrase "a bit" to mean "a lot".)

Let's just remove usersearch plus the anchor as proposed, and then provide a simple "Edit user" link.

#11 @nacin
13 years ago

And obviously the rest can happen in a future release.

#12 @nacin
13 years ago

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

In [20631]:

After adding a user, take them back to users.php with an 'Edit user' link in the message.

This removes the (possibly ineffective) search from the redirect, as well as the URL hash that will get covered up by the toolbar.

fixes #19470. props merty and SergeyBiryukov for initial patches.

#13 in reply to: ↑ 7 @SergeyBiryukov
9 years ago

Replying to SergeyBiryukov:

What about adding an "Edit Profile" link to the end of the success message

I guess we'd want to do that consistently for all the "Add User" screens, including several screens in Multisite.

Follow-up: #35705

Note: See TracTickets for help on using tickets.