Как добавить к 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!
Благодарю.

Новые блоги и статьи Все статьи Все блоги /
сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .
Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .	Расчёт токов в цепи постоянного тока igorrr37 05.01.2026 / * Дана цепь постоянного тока с сопротивлениями и источниками (напряжения, ЭДС и тока). Найти токи и напряжения во всех элементах. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и. . .

@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