[wp-trac] [WordPress Trac] #33974: Theme preview breaks HTML code in theme menu

WordPress Trac noreply at wordpress.org
Wed Oct 7 20:01:35 UTC 2015


#33974: Theme preview breaks HTML code in theme menu
--------------------------+------------------------------
 Reporter:  progmastery   |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  General       |     Version:
 Severity:  normal        |  Resolution:
 Keywords:                |     Focuses:  ui, template
--------------------------+------------------------------
Description changed by ocean90:

Old description:

> Theme preview modifies HTML code of navigation menu in some themes.
>
> Here is an example:
> https://wordpress.org/themes/best-magazine/
> https://wp-themes.com/best-magazine/
>
> This is how it should be
> http://themedemo.web-dorado.com/bestmagazine/
>
> nav menu call parameters:
> wp_nav_menu(    array(
> 'show_home' => true,
> 'theme_location'  => 'primary-menu',
> 'container'       => false,
> 'container_class' => '',
> 'container_id'    => '',
> 'menu_class'      => 'top-nav-list',
> 'menu_id'         => '',
> 'echo'            => false,
> 'fallback_cb'     => 'wp_page_menu',
> 'before'          => '',
> 'after'           => '',
> 'link_before'     => '',
> 'link_after'      => '',
> 'items_wrap'      => '<ul id="top-nav-list" class=" %2$s">%3$s</ul>',
> 'depth'           => 0,
> 'walker'          => ''
> ));
>
> It seems there is some problem with wrapping menu, but I don't know if
> this is site issue or something related with WordPress version
> (ver=4.4-alpha-34443). Therefore I submitted this ticket here , not in
> w.org meta.

New description:

 Theme preview modifies HTML code of navigation menu in some themes.

 Here is an example:
 https://wordpress.org/themes/best-magazine/
 https://wp-themes.com/best-magazine/

 This is how it should be
 http://themedemo.web-dorado.com/bestmagazine/

 nav menu call parameters:

 {{{
 wp_nav_menu(    array(
 'show_home' => true,
 'theme_location'  => 'primary-menu',
 'container'       => false,
 'container_class' => '',
 'container_id'    => '',
 'menu_class'      => 'top-nav-list',
 'menu_id'         => '',
 'echo'            => false,
 'fallback_cb'     => 'wp_page_menu',
 'before'          => '',
 'after'           => '',
 'link_before'     => '',
 'link_after'      => '',
 'items_wrap'      => '<ul id="top-nav-list" class=" %2$s">%3$s</ul>',
 'depth'           => 0,
 'walker'          => ''
 ));
 }}}


 It seems there is some problem with wrapping menu, but I don't know if
 this is site issue or something related with WordPress version
 (ver=4.4-alpha-34443). Therefore I submitted this ticket here , not in
 w.org meta.

--

--
Ticket URL: <https://core.trac.wordpress.org/ticket/33974#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list