Make WordPress Core

Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#12339 closed defect (bug) (fixed)

custom-navigation-dynamic-functions.js bug

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

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

Download all attachments as: .zip

Change History (3)

@junsuijin
17 years ago

adds a + 1 on li 183

#1 @markjaquith
17 years ago

  • Resolution → fixed
  • Status new → closed

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

Note: See TracTickets for help on using tickets.