WordPress.org
Get WordPress

Make WordPress Core

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)

dashboard.diff​ (2.1 KB ) - added by niallkennedy 17 years ago.
single quotes for static strings

Download all attachments as: .zip

Change History (6)

#1 @niallkennedy
17 years ago

  • Keywords has-patch added

#2 @niallkennedy
17 years ago

  • Milestone Unassigned → 2.8.5
  • Version → 2.8.4

#3 in reply to: ↑ description @westi
17 years ago

  • Cc westi added
  • Keywords needs-patch added; has-patch removed
  • Milestone 2.8.5 → 2.9

Replying to niallkennedy:

Double-quote HTML element attributes for consistency. Ex: "<span class=\"bar\">$foo</span>"

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.

@niallkennedy
17 years ago

single quotes for static strings

#4 @niallkennedy
17 years ago

  • Keywords has-patch performance added; markup needs-patch removed

#5 @westi
17 years ago

  • Resolution → fixed
  • Status new → closed

(In [11953]) Switch round some quotes. Fixes #10669 props niallkennedy

Note: See TracTickets for help on using tickets.