Opened 5 years ago
Closed 5 years ago
#48417 closed enhancement (fixed)
Change text wordpress to WordPress
Reported by: | 1naveengiri | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 5.4 | Priority: | normal |
Severity: | normal | Version: | 5.3 |
Component: | General | Keywords: | has-patch |
Focuses: | docs, coding-standards | Cc: |
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
@
5 years ago
- Keywords has-patch added; 2nd-opinion removed
- Milestone changed from Awaiting Review to 5.4
- Owner set to SergeyBiryukov
- Status changed from new to 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
@
5 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.
made the change from wordpress to WordPress