[wp-trac] Re: [WordPress Trac] #5844: remove extra <ul> in walker class

WordPress Trac wp-trac at lists.automattic.com
Wed Feb 13 19:59:01 GMT 2008


#5844: remove extra <ul> in walker class
---------------------+------------------------------------------------------
 Reporter:  hailin   |        Owner:  anonymous
     Type:  defect   |       Status:  new      
 Priority:  normal   |    Milestone:  2.5      
Component:  General  |      Version:  2.5      
 Severity:  normal   |   Resolution:           
 Keywords:           |  
---------------------+------------------------------------------------------
Changes (by lloydbudd):

  * version:  => 2.5
  * milestone:  2.6 => 2.5

Old description:

> Hi,
>
> I'm wondering why the html generated by wp_list_pages and
> wp_list_categories has changed in the current 2.4-bleeding. I know
> things in this are not set in stone so I'm hoping it's not a permanent
> change as it makes coding CSS menus that work with 2.3 and 2.4 very
> difficult especially as child item are no longer in a single container.
> The new structure also makes less sense semantically as each child is
> now a separate UL.
>
> Changed from:
>
> <ul>
>   <li>
>    Text
>    <ul>
>     <li>text</li>
>     <li>text</li>
>     <li>text</li>
>    <ul>
>   </li>
> </ul>
>
> To:
>
> <ul>
>   <li>
>    Text
>    <ul>
>     <li>text</li>
>    <ul>
>    <ul>
>     <li>text</li>
>    <ul>
>    <ul>
>     <li>text</li>
>    <ul>
>   </li>
> </ul>
>
> Thanks,
>
> --
> James R Whitehead
>
> email:          James at interconnectit.com
> URL:            http://www.interconnectit.com
> Phone:          +44 (0)151 709 7977

New description:

 As reported by James R Whitehead of interconnectit.com on wp-hackers:

 I'm wondering why the html generated by wp_list_pages and
 wp_list_categories has changed in the current 2.4-bleeding. I know
 things in this are not set in stone so I'm hoping it's not a permanent
 change as it makes coding CSS menus that work with 2.3 and 2.4 very
 difficult especially as child item are no longer in a single container.
 The new structure also makes less sense semantically as each child is now
 a separate UL.

 Changed from:

 <ul>
   <li>
    Text
    <ul>
     <li>text</li>
     <li>text</li>
     <li>text</li>
    <ul>
   </li>
 </ul>

 To:

 <ul>
   <li>
    Text
    <ul>
     <li>text</li>
    <ul>
    <ul>
     <li>text</li>
    <ul>
    <ul>
     <li>text</li>
    <ul>
   </li>
 </ul>

-- 
Ticket URL: <http://trac.wordpress.org/ticket/5844#comment:1>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list