Opened 18 months ago
Closed 18 months ago
#62971 closed defect (bug) (fixed)
Change casing in admin bar for "Edit site"
| Reported by: | bronsonquick | Owned by: | audrasjb |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.8 |
| Component: | Toolbar | Version: | 5.9 |
| Severity: | minor | Keywords: | has-patch |
| Cc: | Focuses: | administration |
Description
All other words in the admin bar use uppercase so we should change "Edit site" to be "Edit Site" for consistency. This was originally introduced in this ticket: https://core.trac.wordpress.org/changeset/52158
Attachments (1)
Change History (10)
#2
@
18 months ago
- Owner set to
- Status new → accepted
Hello and thank you for the ticket,
This change is different from the things pointed out by the related tickets @sabernhardt pointed out, so let's milestone it for 6.8.
#4
follow-up:
↓ 5
@
18 months ago
site-editor.php also includes the sentence case Edit site as a heading when the error message displays due to a lack of JavaScript (code reference).
As all the other page headings in the dashboard use title case, let's change that too.
@audrasjb to aid translation teams, do you recommend a context be added to differentiate the page heading from the admin bar link?
#5
in reply to: ↑ 4
@
18 months ago
Replying to peterwilsoncc:
@audrasjb to aid translation teams, do you recommend a context be added to differentiate the page heading from the admin bar link?
Yes this is 100% recommended in this case :)
#6
@
18 months ago
@bronsonquick Do you have time to fix the site heading and upload a new patch?
I recommend the following for each of the strings respectively
_x( 'Edit Site', 'site editor link from admin bar' ) _x( 'Edit Site', 'site editor heading' )
#7
@
18 months ago
Related fix for "Collapse menu" -> "Collapse Menu" - https://core.trac.wordpress.org/ticket/59133
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)

Related discussions are on #60977 and #62368.