Make WordPress Core

Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#24643 closed defect (bug) (fixed)

Twenty Thirteen: Minor js cleanup

Reported by: obenland Owned by: lancewillett
Milestone: 3.6 Priority: low
Severity: normal Version: 3.6
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:


I noticed some unused and overused variables in our main js file. With the removal of fixed navbar code from functions.js in r24169, it gives us the chance to optimize it further.

Attachments (1)

24643.patch (1.9 KB) - added by obenland 2 years ago.

Download all attachments as: .zip

Change History (5)

2 years ago

#1 @obenland
2 years ago

  • Priority changed from normal to low

#2 @lancewillett
2 years ago

JS changes look good.

However, you missed bumping the JS version in wp_enqueue_script() call. I'll bump it in the commit.

#3 @lancewillett
2 years ago

  • Owner set to lancewillett
  • Resolution set to fixed
  • Status changed from new to closed

In 24511:

Twenty Thirteen: functions.js optimizations, props obenland. Fixes #24643.

#4 @lancewillett
2 years ago

In 24514:

Twenty Thirteen: JS fix, after r24511 changes -- add missing semicolon to end variable definition. See #24643.

Note: See TracTickets for help on using tickets.