#60977 closed enhancement (fixed)
"Edit site" means two different things on a Multisite network
Reported by: | johnbillion | Owned by: | audrasjb |
---|---|---|---|
Milestone: | 6.6 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Toolbar | Keywords: | has-patch commit |
Focuses: | administration, multisite | Cc: |
Description
I've just been working with a client who runs a Multisite installation of sites that use full site editing. Several times they have clicked on the wrong "Edit site" link in the admin toolbar and been taken to the network admin site editing screen instead of the site editor.
This only affects Super Admin users.
- When viewing the front end of the site there is a top level "Edit site" link in the admin toolbar. This takes you to the site editor for full site editing (
site-editor.php
). - When viewing the admin area and hovering over the site name in the admin toolbar, there is an "Edit Site" link in its menu. This takes you to the site editing screen in the network admin area (
network/site-info.php
).
It would be good to more clearly differentiate between editing a site and editing a site.
Attachments (2)
Change History (37)
#4
@
5 months ago
- Keywords needs-patch added; needs-design 2nd-opinion removed
- Milestone changed from Awaiting Review to 6.6
- Owner set to audrasjb
- Status changed from new to accepted
#5
@
5 months ago
- Keywords good-first-bug added
Marking this as a good-first-bug
as it is an ideal ticket to onboard people who never contributed to a Core ticket before.
#6
@
5 months ago
hi, @audrasjb . I would like to take this ticket as my first one.
Please correct me if I am wrong, so we have to change the name for the edit site menu to "Site Editor" here โhttps://github.com/WordPress/WordPress/blob/a6b9c3d5423cbe97036aaad25b01e0e6b21ba11e/wp-includes/admin-bar.php#L479 right ?
#7
@
5 months ago
- Keywords has-patch added; needs-patch removed
@pratiklondhe Thanks for the patch! Yes, changing line 479 is correct, within the wp_admin_bar_edit_site_menu()
function.
#8
follow-up:
โย 9
@
5 months ago
I like that the string 'Site Editor' is โalready translated, though its current use is in the theme administration's โfeatures list (since 6.1.1 with [54786]). I did not notice a problem with reusing it in this context, but could that be inappropriate?
For more history, the 'Edit site' link was added to the plugin in GB27135 and merged into Core with [52158].
#9
in reply to:
โย 8
@
5 months ago
- Keywords commit added
The patch looks good to me, thanks!
Replying to sabernhardt:
I like that the string 'Site Editor' is โalready translated, though its current use is in the theme administration's โfeatures list (since 6.1.1 with [54786]). I did not notice a problem with reusing it in this context, but could that be inappropriate?
In this cas, there is no problem reusing it without adding a translating context since it's about the very same thing, so I think the patch can ship as it is. Thanks for checking! :)
#11
@
5 months ago
- Resolution fixed deleted
- Status changed from closed to reopened
Catching up to this late but wanted to see if we could consider going in a different direction to address this. Instead of renaming the "Edit site" to "Site Editor" (verb/action to noun), could we rename the "Edit Site" link? Perhaps make it "Edit site info" or something else more action oriented or "Site Info". I'm spitballing on the names but, right now, changing something from more action oriented (Edit site) for most users to something more passive, especially when folks often see an "Edit post" or "Edit page" near "Edit site" feels like a step backwards.
#13
@
5 months ago
Any chance we can change the verbiage only when you're on a multi-site install? Anne has good points about "Edit site" being more active language, whereas "Site editor" is likely to be meaningless to most people. Alternatively, "Edit design" might even be better.
#14
@
5 months ago
@Joen I don't think we should change the wording of the site editor link (which was changed to "Site Editor" in [58035]) depending on whether the installation is a network or not. Because it would be inconsistent for people using both network installations and single-site installations.
I would rather change the wording of the site information link. Perhaps we could use "Edit Site Info" as proposed by @annezazu.
#16
@
5 months ago
- Keywords has-patch added
We are running out of synonyms in American English. ๐
What I originally liked about "Site Editor" is that it sounds like a tangible place and tool; like a Photo Editor or a Code Editor, a Site Editor is the place I go to edit site stuffs.
Juxtaposed to the actions/verbs of "Edit Site" or "Edit Design", which are unintentionally ambiguous (to me); what aspects of the Site or Design will I be editing (especially as a Super Admin with all kinds of access to things, which this ticket aims specifically at).
If we are considering revising this (which I am not opposed to), here are my alternative suggestions:
Inspired by the admin area menu name hierarchy: Appearance > Editor (thinking of Anne ๐ซถ):
- Appearance Editor
- Edit Appearance
Inspired by site-editor.php
's h1
tag contents: Design (thinking of Joen ๐ซถ):
- Design Editor
- Edit Design
Because so much of WordPress is now inconsistent in look & feel, it becomes even more important that terminology stay consistent when sending users between them. If a click takes a user to a whole-new interface, the key words there should match-up with what they clicked to get there, as a way to cross-check & confirm that they are where they wanted to be โ ignoring for now that "Design" & "Appearance" are also used interchangeably ๐คช
#17
@
5 months ago
No strong opinions, generally, I mainly echo Anne's instincts on using active language (Edit N, etc.) For me it's conceptually about wp-admin itself being the site editor, with the design of your site being just one aspect. That's a bit of a semantic rabbit hole and I'm not suggesting any changes there, mainly using that concept as a vessel for finding other terms. Thanks JJJ!
#18
@
5 months ago
- Keywords has-patch removed
Of the options listed above, I like "Edit Design" for the top-level link.
For another idea, "Open site editor" (already โused on the admin dashboard) starts with a verb and suggests "site editor" is a place or tool. However, that can be rather long for the toolbar's top level.
The multisite link goes to the page for editing site info, though Info is the first of four tab-styled links (the other three are Users, Themes and Settings).
#20
@
5 months ago
I still think it would be better to change the multisite option over the option most users are going to see for the site editor. Some other ideas for verbiage that takes you to the site editing screen in the network admin area:
- "Manage Site"
- "Edit Site Admin"
#21
@
4 months ago
- Keywords needs-patch added
@nilovelez, @tobifjellner, @fierevere and @zodiac1978, can you place provide your ideas about phrases that need to be used to avoid confusion? Sorry for pinging a lot of people, ticket looks simple, but there should be informed decision and we have 2 weeks until Beta 1. Thank you ๐
#22
@
4 months ago
For me if would be easier to keep the toolbar link as "Edit site" and, for the multisite option, change it to "Manage site" as @annezazu suggested.
"Site settings" is also a good candidate for the multisite option.
@
4 months ago
restores "Edit site" on block editor link, replaces multisite link with "Manage Site"
#24
@
4 months ago
- Keywords has-patch added; needs-patch removed
"Manage Site" meets the preference for a verb, so I used that for the multisite link in 60977.2.diffโ.
This ticket was mentioned in โSlack in #core by oglekler. โView the logs.
4 months ago
#26
@
4 months ago
- Keywords needs-testing added
We have just a few days before the Beta 1, so, we need to see how it looks like and if everything is logically correct, merge it.
This ticket was mentioned in โSlack in #core-test by oglekler. โView the logs.
4 months ago
#28
@
4 months ago
Test Report
Patch tested - https://core.trac.wordpress.org/attachment/ticket/60977/60977.2.diff
Environment:
OS: Windows 11
Browser: Chrome v125
WordPress: 6.5.3
Actual Result: When viewing the admin area and hovering over the site name in the admin toolbar, "Manage site" is now visible instead of "Edit site" and clicking on it redirects correctly โ
Screenshots:
โhttps://prnt.sc/YIl8-yWaKGFo
#29
@
4 months ago
Test report
I have tested the below patch.
https://core.trac.wordpress.org/attachment/ticket/60977/60977.2.diff
Environment
OS: macOS Sonoma 14.4
Web Server: Nginx
Platform: Local By Flywheel
PHP: 8.3
WordPress: 6.5.3
Browser: Safari 7.4 (macOS), Chrome Version 123.0.6312.106
Theme: Twenty Twenty-Four Version: 1.1
Actual Result:
When viewing the admin area and hovering over the site name in the admin toolbar, "Manage site" is now visible instead of "Edit site" and redirects to the correct URL. โ
Screenshot
โhttps://share.cleanshot.com/PysG8cv4RYLWdWl1BSgP
GIF
โhttps://share.cleanshot.com/0MXLgCd2KyZ8JZqTkS6q
Thank you.
#30
@
4 months ago
- Keywords needs-testing removed
@sabernhardt can we mark this ticket to commit? :)
#34
@
3 months ago
Bug Report
Description
Several times,@johnbillion has clicked on the wrong "Edit site" link in the admin toolbar and been taken to the network admin site editing screen instead of the site editor.
This only affects Super Admin users.
When viewing the front end of the site there is a top-level "Edit site" link in the admin toolbar. This takes you to the site editor for full site editing (site-editor.php).
When viewing the admin area and hovering over the site name in the admin toolbar, there is an "Edit Site" link in its menu. This takes you to the site editing screen in the network admin area (network/site-info.php).
It would be good to more clearly differentiate between editing a site and editing a site.
Environment
- WordPress: 6.5.4
- PHP: 8.3
- Server: Apache/2.4.10 (Debian)
- Database: mysqli (Server: 5.5.59-MariaDB-1~wheezy / Client: 5.5.62)
- Browser: Chrome 125.0.0.0 (macOS)
- Theme: Could not determine
- MU-Plugins: None activated
- Plugins:
- File Manager Advanced 5.2.4
- WordPress Beta Tester 3.5.5
Steps to Reproduce
I have tested the below patch.
https://core.trac.wordpress.org/attachment/ticket/60977/60977.2.diff
Expected Results
- โ When viewing the admin area and hovering over the site name in the admin toolbar, "Manage site" is now visible instead of "Edit site" and redirects to the correct URL.
I'd say the easier way to fix this issue is to change the link to the Site Editor (aka FSE). It should not be "Edit Site" but rather "Site Editor".