Opened 7 years ago
Closed 7 years ago
#48417 closed enhancement (fixed)
Change text wordpress to WordPress
| Reported by: | 1naveengiri | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.4 |
| Component: | General | Version: | 5.3 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | docs, coding-standards |
Description
So far everything is done to convert "Wordpress" or "wordpress" to "WordPress" after capital_P_dangit().
I have noticed a few lines in a code comment. where we are still using small case "wordpress".
so I am wondering. it will be good to convert it to "WordPress".
Thanks
Attachments (4)
Change History (7)
#1
@
7 years ago
- Keywords has-patch added; 2nd-opinion removed
- Milestone Awaiting Review → 5.4
- Owner set to
- Status new → reviewing
Hi there, thanks for the ticket and the patch!
The change in wp-admin/css/customize-widgets.css looks good. Note that there's no need to patch RTL CSS files, as they are automatically generated during the build process.
The change in wp-includes/css/dist/edit-post/style.css should be submitted upstream, as edit-post is an external package developed on GitHub.
#2
@
7 years ago
Upon a closer look, wordpress here is a Dashicon slug reference and should not be capitalized.
That said, there are a couple of inconsistencies between these comments and actual Dashicons, let's fix them for clarity.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
made the change from wordpress to WordPress