Make WordPress Core

Opened 13 years ago

Last modified 13 years ago

#18256 closed defect (bug)

Close the current-cat li before the children — at Initial Version

Reported by: betsyk's profile betsyk Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.2.1
Component: Template Keywords:
Focuses: Cc:

Description

In a hierarchical category list, children adopt the current-cat class from parents. This is due to the Walker_Category structure which nests <ul class='children'> inside the <li> of the parent list. The parent list should be closed before the child <ul> begins.

Let's say I have a list of categories:

<ul><?php wp_list_categories('style=list&hierarchical=1&depth=0'); ?></ul>

And I want to style the current category:

.current-cat {
background-color: #cccccc;
}

The HTML output is this:

<ul><li class="cat-item cat-item-21 current-cat"><a href="http://mysite/category/food/cookies">cookies</a>
<ul class='children'><li class="cat-item cat-item-107"><a href="http://mysite/category/food/cookies/oatmeal">oatmeal</a></li></ul></li></ul>

It should be this:

<ul><li class="cat-item cat-item-21 current-cat"><a href="http://mysite/category/food/cookies">cookies</a></li><ul class='children'><li class="cat-item cat-item-107"><a href="http://mysite/category/food/cookies/oatmeal">oatmeal</a></li></ul></ul>

Didn't see anything related in the open tickets - hopefully this isn't a duplicate bug report.

Change History (1)

@betsyk
13 years ago

changes are commented and prefixed with FIX

Note: See TracTickets for help on using tickets.