WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#25941 closed defect (bug) (fixed)

JSHint configuration: Set "smarttabs" to false

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

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 6 years ago.
jshint change to set "smarttabs" to false

Download all attachments as: .zip

Change History (3)

@kadamwhite
6 years ago

jshint change to set "smarttabs" to false

#2 @nacin
6 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.