Как добавить removeClass в селектор в jQuery

Question

Как добавить removeClass в селектор в jQuery

У меня есть два меню на странице (см. Jsfiddle ниже, если необходимо ... это отзывчиво, поэтому вам нужно минимизировать окно). В настоящее время, когда кто-то открывается, и вы щелкаете другим, они оба остаются открытыми. Как мне отредактировать javascript, поэтому, когда кто-то откроется, другой закроется? Я знаю, что мне нужно добавить removeClass, но я не уверен, как его реализовать. Большое спасибо за любую помощь.

http://jsfiddle.net/Jq6sZ/

$(document).ready(function() {
    $('body').addClass('js');
    var $menu = $('#menu'),
        $menulink = $('.menu-link'),
        $menuTrigger = $('.has-subnav > a');

    $menulink.click(function(e) {
        e.preventDefault();
        $menulink.toggleClass('active');
        $menu.toggleClass('active');
    });

    $menuTrigger.click(function(e) {
        e.preventDefault();
        var $this = $(this);
        $this.toggleClass('active').next('ul').toggleClass('active');
    });

});

$(document).ready(function() {
    $('body').addClass('js');
    var $menu = $('#menu2'),
        $menulink = $('.menu-link2'),
        $menuTrigger = $('.has-subnav > a');

    $menulink.click(function(e) {
        e.preventDefault();
        $menulink.toggleClass('active');
        $menu.toggleClass('active');
    });

    $menuTrigger.click(function(e) {
        e.preventDefault();
        var $this = $(this);
        $this.toggleClass('active').next('ul').toggleClass('active');
    });
});

2

добавлено 09 Январь 2013 в 03:41 автор Adam Robertson отредактировано 09 Январь 2013 в 03:48

Просмотры: 3

источник

nl ja de

@wirey Он не разбирается в id #menu и # menu2

добавлено 09 Январь 2013 в 03:58, автор andlrc, источник

@wirey Он не разбирается в id #menu и # menu2

добавлено 09 Январь 2013 в 03:58, автор andlrc, источник

Вы знаете, что вы можете использовать $ ('# menu, # menu2'). Each (function() {var $ menu = $ (this); ... вместо дублирования кода.

добавлено 09 Январь 2013 в 03:59, автор andlrc, источник

Вы знаете, что вы можете использовать $ ('# menu, # menu2'). Each (function() {var $ menu = $ (this); ... вместо дублирования кода.

добавлено 09 Январь 2013 в 03:59, автор andlrc, источник

есть ли причина, по которой вы дважды используете один и тот же код js? Это была ошибка копирования/вставки?

добавлено 09 Январь 2013 в 03:54, автор ᾠῗᵲᄐᶌ, источник

4 ответы

Видеокурс по Java для начинающих

Получи видеокурс по Java для начинающих с упражнениями

Видеокурс "Создание приложений для Android"

Получи множество видеоуроков по Android сейчас

Система управления ссылками ReDirector 3.0

Автор: Евгений Попов

Мини-курс по Angular и NgRx

16 видео-уроков

автор c_kick · Answer 1 · 2013-01-09T04:09Z

Ваш код может быть намного проще и короче, у вас есть 2 оператора docready, и вы дважды устанавливаете класс body в «js». Это можно сократить. В целом, вы немного ограничены тем, как HTML выложен, но есть способ:

$(document).ready(function() {
    $('body').addClass('js');

    var $menulink = $('.menu-link, .menu-link2'), 
        $menuTrigger = $('.has-subnav > a');

    $menulink.click(function(e) {
        var menu_to_toggle = $(this).next(); //which menu will this button toggle?
        e.preventDefault();
        $menulink.not($(this)).removeClass('active'); //remove active class from all menu links, but not this one
        $(this).toggleClass('active'); //toggle active class for this link
        $('.menu.active').not(menu_to_toggle).removeClass('active'); //remove active class from all menus, but not this one
        menu_to_toggle.toggleClass('active'); //toggle active class for the menu
    });

    $menuTrigger.click(function(e) {
        e.preventDefault();
        var $this = $(this);
        $this.toggleClass('active').next('ul').toggleClass('active');
    });
});

Смотрите этот скрипт для демонстрации

Примечание: сценарий не редуцирует подменю.

автор c_kick · Answer 2 · 2013-01-09T04:09Z

Ваш код может быть намного проще и короче, у вас есть 2 оператора docready, и вы дважды устанавливаете класс body в «js». Это можно сократить. В целом, вы немного ограничены тем, как HTML выложен, но есть способ:

$(document).ready(function() {
    $('body').addClass('js');

    var $menulink = $('.menu-link, .menu-link2'), 
        $menuTrigger = $('.has-subnav > a');

    $menulink.click(function(e) {
        var menu_to_toggle = $(this).next(); //which menu will this button toggle?
        e.preventDefault();
        $menulink.not($(this)).removeClass('active'); //remove active class from all menu links, but not this one
        $(this).toggleClass('active'); //toggle active class for this link
        $('.menu.active').not(menu_to_toggle).removeClass('active'); //remove active class from all menus, but not this one
        menu_to_toggle.toggleClass('active'); //toggle active class for the menu
    });

    $menuTrigger.click(function(e) {
        e.preventDefault();
        var $this = $(this);
        $this.toggleClass('active').next('ul').toggleClass('active');
    });
});

Смотрите этот скрипт для демонстрации

Примечание: сценарий не редуцирует подменю.

автор Chase · Answer 3 · 2013-01-09T03:47Z

Вы можете проверить, находится ли другое меню hasClass active . Например, в вашем событии click для menu-link у вас будет что-то вроде:

if($('.menu-link2').hasClass("active")){
    $('.menu-link2').trigger("click");
}

Затем используйте trigger , чтобы вызвать событие click в другом меню.

http://jsfiddle.net/Jq6sZ/1/

автор Chase · Answer 4 · 2013-01-09T03:47Z

Вы можете проверить, находится ли другое меню hasClass active . Например, в вашем событии click для menu-link у вас будет что-то вроде:

if($('.menu-link2').hasClass("active")){
    $('.menu-link2').trigger("click");
}

Затем используйте trigger , чтобы вызвать событие click в другом меню.

http://jsfiddle.net/Jq6sZ/1/