WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#31817 closed defect (bug) (fixed)

Invalid markup in "nav-menu.php"

Reported by: sagarjadhav Owned by: wonderboymusic
Milestone: 4.3 Priority: normal
Severity: normal Version: 3.0
Component: Menus Keywords: has-patch
Focuses: ui, administration Cc:

Description

Following error found when validate markup on W3C Validator.

Element dl is missing a required instance of child element dd

Attached screenshot for more information.

http://i.imgur.com/74ChYk6.png

Attachments (2)

31817.patch (1.3 KB) - added by sagarjadhav 4 years ago.
Fixed issue
31817.2.patch (1.8 KB) - added by sagarjadhav 4 years ago.
Patch updated with CSS changes

Download all attachments as: .zip

Change History (7)

@sagarjadhav
4 years ago

Fixed issue

#1 @sagarjadhav
4 years ago

  • Keywords has-patch added

#2 @SergeyBiryukov
4 years ago

  • Version changed from 4.1.1 to 3.0

Introduced in [13257]. Seems like this would need a CSS change as well.

@sagarjadhav
4 years ago

Patch updated with CSS changes

#3 @SergeyBiryukov
4 years ago

  • Milestone changed from Awaiting Review to 4.3

#4 @wonderboymusic
4 years ago

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

In 32879:

Use valid markup in Walker_Nav_Menu_Edit::start_el() - replace <d(l|t)> with <div> for .menu-item-bar and .menu-item-handle.

Props sagarjadhav.
Fixes #31817.

#5 @ocean90
4 years ago

In 32889:

Customizer: Use valid markup for menu items.

Replace <d(l|t)> with <div> for .menu-item-bar and .menu-item-handle.

see #32724, #32721, #31817.

Note: See TracTickets for help on using tickets.