Opened 17 years ago
Closed 17 years ago
#10669 closed enhancement (fixed)
Dashboard includes cleanup
| Reported by: | niallkennedy | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 2.9 |
| Component: | Administration | Version: | 2.8.4 |
| Severity: | normal | Keywords: | has-patch performance |
| Cc: | Focuses: |
Description
Style cleanup of /wp-admin/includes/dashboard.php for speed and consistency.
Single-quote static strings for speed. Ex: '</div>'
Double-quote HTML element attributes for consistency. Ex: "<span class=\"bar\">$foo</span>"
Attachments (1)
Change History (6)
#3
in reply to: ↑ description
@
17 years ago
- Cc added
- Keywords needs-patch added; has-patch removed
- Milestone 2.8.5 → 2.9
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Replying to niallkennedy:
This is against the WordPress Coding Standards.
Also not 2.8 branch material as this is an enhancement.
The switch to single rather than double quotes for strings that it is appropriate sounds good - could you redo the patch with just those changes please.