Как добавить к function click переменную?

@westyes · Регистрация: 15.08.2014

Студворк — интернет-сервис помощи студентам

Всем здравствуйте.
Простое меню, по клику на "#touch-menu" разворачивает и сворачивает список меню ".menu" с ссылками.
Как сделать так чтобы меню сворачивалось обратно не только по клику на "#touch-menu",
но и по клику по ссылке?
p.s. Эти ссылки перелистывают картинки, а не переходят на другую страницу.

HTML5
1
2
3
4
5
6
7
8
9
10
<div id="menu-content"> <a id="touch-menu" class="mobile-menu" href="#">Меню</a>
  <nav id="nav-menu">
    <ul class="menu">
      <li class="nav-menu-current"><a href="#">Первая</a></li>
      <li><a  href="#">Вторая</a></li>
      <li><a  href="#">Третья</a></li>
      <li><a  href="#">Четвертая</a></li>
    </ul>
  </nav>
</div>

Такой код:

JavaScript
1
2
3
4
5
6
7
8
9
$(document).ready(function(){ 
    var touch   = $('#touch-menu');
    var menu    = $('.menu');
 
        $(touch).on('click', function(e) {
        e.preventDefault();
        menu.slideToggle();
 
});

@Sombrero · 22.08.2014, 07:31

westyes, добавить для них общий класс, либо создать еще один обработчик onclick для ссылки

mixenik · 22.08.2014, 09:50

есть такая штука

JavaScript
1
$('#touch-menu li').click();

Называется сложный селектор.

@westyes · 22.08.2014, 15:40 **[ТС]**

Получилось сделать сворачивание меню по клику на li, но появилась проблема.
У меня одно меню, но имеет разный вид при разной ширине экрана.

Сворачиваться должно только при экране меньше чем 767px, а с такой функцией у меня при любой ширине сворачивается. Как это исправить?
Я дуб в js

Весь код выглядит так теперь:

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
$(document).ready(function(){ 
    var touch   = $('#touch-menu');
    var menu    = $('.menu');
 
        $(touch).on('click', function(e) {
        e.preventDefault();
        menu.slideToggle();
 
            $('li').on('click', function() {
            menu.slideUp();
            });
});
 
    $(window).resize(function(){
        var w = $(window).width();
        if(w > 767 && menu.is(':hidden')) {
            menu.removeAttr('style');
        }
    });
    
});

Добавлено через 1 час 14 минут
Получилось то, что я хотел вот так:

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
$(document).ready(function(){ 
    var touch   = $('#touch-menu');
    var menu    = $('.menu');
 
        $(touch).on('click', function(e) {
        e.preventDefault();
        menu.slideToggle();
 
            $('li').on('click', function() {
            var wi  = $(window).width();                    
            if(wi < 767) {
            menu.slideUp(); }
            });
});
 
    $(window).resize(function(){
        var w = $(window).width();
        if(w > 767 && menu.is(':hidden')) {
            menu.removeAttr('style');
        }
    });
    
});

Этот мой измененный код не кривой? просто я методом тыка сделал.

mixenik · 22.08.2014, 17:33

вместо

JavaScript
1
$('li').on('click', function()

сделайте

JavaScript
1
$('li', touch).on('click', function()

Иначе, когда вы еще добавите теги li на странице, у вас будет срабатывать меню)

@westyes · 22.08.2014, 17:51 **[ТС]**

Да, вы правы, другие ли тоже захватывает.
Но с этим кодом перестал сворачивать почему то:

JavaScript
1
$('li', touch).on('click', function()

Тогда я добавил в html в меню к li id="fff"

и вот такой код сделал:

JavaScript
1
$('li#fff').on('click', function()

Стало работать и другие ли не захватывает.
Большое спс за наводку. Но почему ваш метод не сработал? Может еще вариант есть?

mixenik · 22.08.2014, 17:56

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

Добавлено через 1 минуту
Стоп, это же и есть объект О_о... странно, должно работать

Добавлено через 1 минуту
ясно, menu нужно было, а не touch. Моя невнимательность

@westyes · 22.08.2014, 18:34 **[ТС]**

Сообщение от mixenik

ясно, menu нужно было, а не touch

Да теперь работает без дополнительного id!
Благодарю.

Новые блоги и статьи Все статьи Все блоги /
Валидация и контроль данных табличной части документа перед записью Maks 22.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в КА2. Задача: контроль и валидация данных табличной части документа перед записью с учетом регламента компании. . .	Отчёт о затраченных материалах за определенный период с макетом печатной формы Maks 21.04.2026 Отчёт из решения ниже размещён в конфигурации КА2. Задача: разработка отчёта по затраченным материалам за определённый период, с возможностью вывода печатной формы отчёта с шапкой и подвалом. В. . .	Отчёт о спецтехнике находящейся в ремонте Maks 20.04.2026 Отчёт из решения ниже размещен в конфигурации КА2. Задача: отобразить спецтехнику, которая на данный момент находится в ремонте. Есть нетиповой документ "Заявка на ремонт спецтехники" который. . .	Памятка для бота и "визитка" для читателей "Semantic Universe Layer (Слой семантической вселенной)" Hrethgir 19.04.2026 Сгенерировано для краткого описания по случаю сборки и компиляции скелета серверного приложения. И пусть после этого скажут, что статьи сгенерированные AI - туфта и не интересно. И это не реклама -. . .
Запрет удаления строк ТЧ документа при определённом условии Maks 19.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "Аккумуляторы", разработанного в конфигурации КА2. У данного документа есть ТЧ, в которой в зависимости от прав доступа. . .	Модель заражения группы наркоманов alhaos 17.04.2026 Условия задачи сформулированы тут Суть: - Группа наркоманов из 10 человек. - Только один инфицирован ВИЧ. - Колются одной иглой. - Колются раз в день. - Колются последовательно через. . .	Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .

@Sombrero 74 / 70 / 40 Регистрация: 31.03.2014 Сообщений: 298
	22.08.2014, 07:31
	westyes, добавить для них общий класс, либо создать еще один обработчик onclick для ссылки 1

mixenik Все элементарное - сложно 133 / 118 / 50 Регистрация: 14.04.2014 Сообщений: 509
	22.08.2014, 17:56
	Прошу прощения, сила привычки. Я обычно в переменную вытаскиваю весь объект. Перепутал в общем. Добавлено через 1 минуту Стоп, это же и есть объект О_о... странно, должно работать Добавлено через 1 минуту ясно, menu нужно было, а не touch. Моя невнимательность 1