[wp-trac] [WordPress Trac] #43633: Duplicate links to edit profile dropdown in toolbar
WordPress Trac
noreply at wordpress.org
Tue Mar 27 07:54:47 UTC 2018
#43633: Duplicate links to edit profile dropdown in toolbar
-------------------------------------------------+-------------------------
Reporter: danieltj | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting
Component: Toolbar | Review
Severity: normal | Version:
Keywords: has-patch needs-testing 2nd-opinion | Resolution:
| Focuses: ui
-------------------------------------------------+-------------------------
Changes (by danieltj):
* keywords: has-patch needs-testing => has-patch needs-testing 2nd-opinion
Comment:
Replying to [comment:7 netweb]:
> I stand by what I wrote in comment:5, the primary purpose of the
`wp_admin_bar_my_account_menu()` function is to add "My Account" items to
the admin bar which are the ''user profile'' links, first the user profile
[https://core.trac.wordpress.org/browser/trunk/src/wp-includes/admin-
bar.php#L291 #L291], and then, if it is not multisite
[https://core.trac.wordpress.org/browser/trunk/src/wp-includes/admin-
bar.php#L266 #L266] the "Edit My Profile" link is added.
>
> [attachment:43633.2.diff] changes multisite behaviour by changing the
first link to link to a users post archive rather than the current link to
the dashboard [https://core.trac.wordpress.org/browser/trunk/src/wp-
includes/admin-bar.php#L267 #L267], this is currently used I expect due to
all of the multisite blog/site/network juggling complexities that are
required to retrieve any user related content in multisite environments.
>
> Also adding a filter as proposed in [attachment:43633.2.diff] puts the
onus on developers to opt-out of a what is a breaking change being
introduced into WordPress and these types of change should be avoided.
How do we fix the issue of having two links to the same page though? As it
stands, single-site installs have two links to the same place which isn't
useful and is confusing. In [attachment:43633.2.diff], the ids are kept
the same to ensure additional links are still added correctly to the
toolbar, the link location has simply changed.
There is another way to the dashboard on multisites anyway on the left
hand side, so access to these account pages isn't lost and still very
accessible. The aim here is to change the fact we have two links going to
the same page right next to each other. A much better use of toolbar UI
would be to provide a link to somewhere that is hard to find.
Adding 2nd-opinion to get more thoughts on this.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/43633#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list