search

jquery / jquery-wp-content

WordPress themes and plugins for the jQuery sites
GNU General Public License v2.0
253 stars 169 forks source link

learn.jquery.com: change highlighting links #344

Closed sigorilla closed 9 years ago

sigorilla commented 9 years ago

It is the incorrect selection active chapters in the sidebar in learn.jquery.com. For example, only jQuery UI should be highlighted on the page jQuery UI, without Widget Factory and Using jQuery UI. There is same situation in other links with subchapters (Using jQuery Core and Code Organization).

This PR fixes this problem.

arthurvr commented 9 years ago

@Sigorilla Thanks!

sigorilla commented 9 years ago

@arthurvr I was wrong when I deleted some code in sidebar.php. It's necessary to highlight top categories (without subcategories) in the sidebar. I made the commit (https://github.com/Sigorilla/jquery-wp-content/commit/f90fd13cef75483e1342fc5987ce0b63060fb305), but cannot add to this PR, cause it was already closed.

Can you reopen PR?

jzaefferer commented 9 years ago

The PR was merged, reopening it doesn't really make sense. You can create a new one with the necessary commit.

sigorilla commented 9 years ago

@jzaefferer Thank you!