Нужна помощь с многоуровневм раскрывающимся меню

[karlitos32]

Подскажите, чего не хватает в коде для того чтобы при щелчке по пункту меню вместе с открытием этого пункта закрывался другой уже открытый пунтк. Либо поделитесь ссылкой где про это можно прочесть.

Код следующий:

$(document).ready(function() {
  $('ul#my-menu ul').each(function(index) {
    $(this).prev().addClass('collapsible').click(function() {
      if ($(this).next().css('display') == 'none') {
        $(this).next().slideDown(200, function () {
          $(this).prev().removeClass('collapsed').addClass('expanded');
        });
      }else {
        $(this).next().slideUp(200, function () {
          $(this).prev().removeClass('expanded').addClass('collapsed');
          $(this).find('ul').each(function() {
            $(this).hide().prev().removeClass('expanded').addClass('collapsed');
          });
        });
      }
      return false;
    });
  });
});

[s0rr0w]

Подскажите, чего не хватает в коде для того чтобы при щелчке по пункту меню вместе с открытием этого пункта закрывался другой уже открытый пунтк. Либо поделитесь ссылкой где про это можно прочесть.

Код следующий:

Не хватает кода убирания класса с активного элемента

[Veseloff]

Я бы немного иначе сделал.

$(document).ready(function() {

заменил бы на

$(function(){

вместо

$(this).next().css('display') == 'none'

использовал бы

$(this).next().is(':hidden')

И вообще использовал бы slideToggle.

Непонятно зачем вот так сделано

$(this).find('ul').each(function() {
            $(this).hide().prev().removeClass('expanded').addClass('collapsed');
          });

Когда можно не использовать each

$(this).find('ul').hide().prev().removeClass('expanded').addClass('collapsed');

[karlitos32]

Спасибо за подсказки. Походу рановато я взялся за менюшку, надо бы еще мат часть подучить...