Make WordPress Core

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#13920 closed defect (bug) (fixed)

Incorrect HTML with custom menu widget?

Reported by: jowo's profile jowo Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.0
Component: Menus Keywords:
Focuses: Cc:

Description

I created a custom menu widget with a title and added some items to the menu.

This it he HTML output I get:

<ul>
<h3>Information</h3><div class="menu-information-container"><ul id="menu-information-1" class="menu"><li id="menu-item-6" class="menu-item menu-item-type-post_type"><a href="http://www.mydomain.com/about-us/">About us</a></li>

<li id="menu-item-5" class="menu-item menu-item-type-post_type"><a href="http://www.mydomain.com/contact-us/">Contact us</a></li>
</ul></div>
</ul>

Which seems like a weird format. Isn't this better:

<h3>Information</h3>
<div class="menu-information-container">
<ul id="menu-information-1" class="menu">
<li id="menu-item-6" class="menu-item menu-item-type-post_type"><a href="http://www.mydomain.com/about-us/">About us</a></li>

<li id="menu-item-5" class="menu-item menu-item-type-post_type"><a href="http://www.mydomain.com/contact-us/">Contact us</a></li>
</ul>
</div>

Putting headings and DIV above UL and not using multiple UL inside the menu.

Change History (2)

#1 @jowo
15 years ago

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

Nevermind it was the template designer who hardcoded a UL above dynamic_sidebar for some reason.

#2 @ocean90
15 years ago

  • Milestone Unassigned deleted
Note: See TracTickets for help on using tickets.