if menu classes are empty the empty attribute will be added to the <li> (solution inside)

[janw.oostendorp]

I often remove the classes form the <li> in menu's with the filter 'nav_menu_css_class'.
That results in empty 'class="" ' attributes in my menu's.
Fixing this is easy to do in the core:

/wp-includes/nav-menu-template.php
line 77

$class_names = ' class="' . esc_attr( $class_names ) . '"';

should be replaced with:

if (!empty($class_names )) {
    $class_names = ' class="' . esc_attr( $class_names ) . '"';
}

[scribu]

Having an empty class attribute doesn't really cause any problems; it's valid HTML.

[janw.oostendorp]

true, but it's ugly and the fix is really simple

[nacin]

[20513]