WordPress.org
Get WordPress

Make WordPress Core

Opened 16 years ago

Closed 16 years ago

Last modified 16 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 16 years ago.
adds a + 1 on li 183

Download all attachments as: .zip

Change History (3)

@junsuijin
16 years ago

adds a + 1 on li 183

#1 @markjaquith
16 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.