WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#12339 closed defect (bug) (fixed)

custom-navigation-dynamic-functions.js bug

Reported by: junsuijin's profile junsuijin Owned by: junsuijin's profile junsuijin
Milestone: 3.0 Priority: normal
Severity: normal Version: 3.0
Component: Menus Keywords: has-patch, tested
Focuses: Cc:

Description

The first li should be id="menu-1" rather than id="menu-0", since this is the id# (randomnumber in the script) that its data refers to. As it is now the first item will not display the correct information when the user attempts to edit it. Additionally, the change in this patch will reduce the number of "tests" for new items by 1.

Attachments (1)

12339.patch​ (543 bytes) - added by junsuijin 15 years ago.
adds a + 1 on li 183

Download all attachments as: .zip

Change History (3)

@junsuijin
15 years ago

adds a + 1 on li 183

#1 @markjaquith
15 years ago

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

(In [13365]) Increment the menu ID so the first menu works. props junsuijin. fixes #12339

Note: See TracTickets for help on using tickets.