Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#25941 closed defect (bug) (fixed)

JSHint configuration: Set "smarttabs" to false

Reported by: kadamwhite's profile kadamwhite Owned by:
Milestone: 3.8 Priority: normal
Severity: normal Version:
Component: Build/Test Tools Keywords: has-patch
Focuses: Cc:

Description

jQuery has the JSHint "smarttabs" option set to "true," meaning they use a combination of tabs and spaces to align code within blocks (tabs to get to the level of the block, spaces to align code within that block). We don't follow this practice, so we should turn it off in our jshintrc.

Attachments (1)

25941.diff (318 bytes) - added by kadamwhite 11 years ago.
jshint change to set "smarttabs" to false

Download all attachments as: .zip

Change History (3)

@kadamwhite
11 years ago

jshint change to set "smarttabs" to false

#2 @nacin
11 years ago

  • Milestone changed from Awaiting Review to 3.8
  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.