Не срабатывает Javascript?

Question

[De-12]

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

Но почему-то не срабатывает ничего, происходит переход по ссылке сразу:

$('.has-submenu > a').dblclick(function() {
  window.location.assign($(this).attr('href'))
});
$('.has-submenu > a').click(function() {
  event.preventDefault();
  $('.nav-sidebar__submenu').css('display', 'block');
});

<li class="nav-sidebar__section has-submenu">
    <a href="/divany"><img src="/image/cache/catalog/icon-category/single-couch-25x25.png" alt="Диваны" title="Диваны">Диваны</a>
    <div class="nav-sidebar__submenu">
      <div class="submenu-title">
        <a href="/divany">Диваны</a>
      </div>
                  <a href="/divany/divany-krovati">Диваны-кровати</a>
            <a href="/divany/kuhonnye-divany">Кухонные диваны</a>
            <a href="/divany/pryamye-divany">Прямые диваны</a>
            <a href="/divany/uglovye-divany">Угловые диваны</a>
                </div>
  </li>

Answer 1

[Владислав Дворянинов]

Так перепишите:
$('.has-submenu > a').click(function(event) {
event.preventDefault();
$('.nav-sidebar__submenu').css('display', 'block');
return false;
});

Comments

[De-12]

Не понял, как тогда будет срабатывать? Должна быть лоигка такая:

1 клик - добавляет сабменю style
Двойной клик - открывает линк.

[Владислав Дворянинов]

De-12, я переписал неправильную часть кода остальное оставь как есть.

[Владислав Дворянинов]

У Вас нет event в аргументах функции и нет return false для предотвращения перехода по ссылке по умолчанию.

[De-12]

Владислав Дворянинов, если не сложно, дайте полный код JS, спасибо.

[Владислав Дворянинов]

$('.has-submenu > a').dblclick(function() {
window.location.assign($(this).attr('href'))
});
$('.has-submenu > a').click(function(event) {
event.preventDefault();
$('.nav-sidebar__submenu').css('display', 'block');
return false;
});

[De-12]

Владислав Дворянинов, почему-то не срабатывает все равно ничего, переходит по ссылке все также..

[Владислав Дворянинов]

Тогда так:
$(function(){
$('.has-submenu > a').dblclick(function() {
window.location.assign($(this).attr('href'))
});
$('.has-submenu > a').click(function(event) {
event.preventDefault();
$('.nav-sidebar__submenu').css('display', 'block');
return false;
});
});

[De-12]

Владислав Дворянинов, благодарю, работает, а как сделать, чтобы не display block добавляло, а класс?

[Владислав Дворянинов]

$('.nav-sidebar__submenu').addClass("someclassname"); https://api.jquery.com/addClass/

[De-12]

Владислав Дворянинов, огромное спасибо и последний вопрос, как-то можно сделать, что если кликаем на другой элемент, у предыдущего пропадал класс? а то сейчас если кликать - то у всех остается

[De-12]

Владислав Дворянинов, и еще как-то можно сделать, чтобы скрипт срабатывал при определенном разрешении (на мобильных хотим это сделать)

[De-12]

Владислав Дворянинов, оно почему-то добавляет абсолютно всем с .nav-sidebar__submenu теперь класс, а не только тому, к которому относится ссылка..

[Владислав Дворянинов]

$('.has-submenu > a').click(function(event) {
event.preventDefault();
$('.nav-sidebar__submenu').removeClass('classname');
$(this).addClass('classname');
return false;
});
Если я правильно Вас понял то так. А по поводу разрешений вот - https://developer.mozilla.org/ru/docs/Web/API/Wind... скорее всего поможет.

[De-12]

Владислав Дворянинов, теперь он класс ссылке добавляет.

В общем предыдущий вариант нормально работает, но он добавляет класс абсолютно всем подменю.

А нужно, чтобы только той, по которому кликнули пункту, если кликаем другой - у предыдущего пропадает.

[De-12]

Владислав Дворянинов,

$(function(){
$('.has-submenu > a').dblclick(function() {
window.location.assign($(this).attr('href'))
});
$('.has-submenu > a').click(function(event) {
event.preventDefault();
$(this).siblings('.nav-sidebar__submenu').addClass("mobhid");
return false;
});
});

Вот так добавляет только на нажатый, но при нажатии на другой, у предыдущего оставляет и не убирает. Можно как-то сделать, чтобы убирало?

[De-12]

Разобрался вроде

$(function(){
$('.has-submenu > a').dblclick(function() {
window.location.assign($(this).attr('href'))
});
$('.has-submenu > a').click(function(event) {
event.preventDefault();
$('.nav-sidebar__submenu').removeClass('mobhid');
$(this).siblings('.nav-sidebar__submenu').addClass("mobhid");
return false;
});
});

[De-12]

Владислав Дворянинов,

$(function(){
$('.has-submenu > a').dblclick(function() {
window.location.assign($(this).attr('href'))
});
$('.has-submenu > a').click(function(event) {
event.preventDefault();
$('.nav-sidebar__submenu').removeClass('mobhid');
$(this).siblings('.nav-sidebar__submenu').addClass("mobhid");
return false;
});
});

Вот так работает как надо все, один момент только. Как можно сделать, чтобы если еще раз кликаю на меню, после того, как выпало, чтобы оно сворачивало подменю?

[Владислав Дворянинов]

https://api.jquery.com/toggleclass/ - попробуйте с помощью этого

[De-12]

Владислав Дворянинов, $(this).siblings('.nav-sidebar__submenu').toggleClass('mobhid');
не помогает почему-то..

Answer 2

[Борис Черепанов]

Надо просто добавить передачу event

$('.has-submenu > a').click(function(event) {
  event.preventDefault();
  $('.nav-sidebar__submenu').css('display', 'block');
});