WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#12339 closed defect (bug) (fixed)

custom-navigation-dynamic-functions.js bug

Reported by: junsuijin Owned by: 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 6 years ago.
adds a + 1 on li 183

Download all attachments as: .zip

Change History (3)

@junsuijin6 years ago

adds a + 1 on li 183

comment:1 @markjaquith6 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.