JavaScript Поможете найти ошибкув коде?

Question

Olga17527 @Olga17527

JavaScript

JavaScript Поможете найти ошибкув коде?

Что неправильно написала в этом коде? Не добавляет класса ._active.

if (isMobile.any()) {
  document.body.classList.add('_touch');

  let menuArrows = document.querySelectorAll('.menu_arrow');
  if (menuArrows.length > 0) {
    for (let index = 0; index < menuArrows.length; index++) {
      const menuArrow = menuArrows[index];
      menuArrow.addEventListener("click", function (e) {
        menuArrow.parentElement.classList.toggle('_active');
      });
    }
  }

} else {
  document.body.classList.add('_pc');
}

Вопрос задан более трёх лет назад
171 просмотр

8 комментариев

Подписаться 1 Простой 8 комментариев

Сергей Сергей @hahenty

на мобилке проверено?

Написано более трёх лет назад
twobomb @twobomb

А можно на песочницу залить? Не понятно что за метод any и выполняется ли он вообще, есть ли там элементы с классом menu_arrow у которых есть парент. Ну и класс должен меняться так-то только по клику

Написано более трёх лет назад
WapSter @wapster92 Куратор тега JavaScript
С кодом всё нормально, ну по крайней мере с малым его куском. Я бы его оптимизировал, но и так сойдет snippet

const btnsList = document.querySelector('ul'); btnsList.addEventListener('click', ({target}) => { if (target.classList.contains('menu_arrow')) { target.parentElement.classList.toggle('_active'); } })
избавился от ненужного цикла
Написано более трёх лет назад

Olga17527 @Olga17527 Автор вопроса

twobomb, Вот полный код (Я делаю меню)

<!DOCTYPE html>
<html lang="en" dir="ltr">
  <head>
    <meta charset="utf-8">
    <link rel="stylesheet" href="style.css">
    <meta name="viewport" content="width=device-width, initial-scale=1">
    <title></title>
  </head>
  <body>
        <script src="scrips.js"></script>
    <div class="wrapper">
      <header class="header">
        <head class="header_container">
          <a href="#"class="logo"></a>
          <div class="header_menu menu">
            <div class="menu_icon">
              <span></span>
            </div>
            <nav class="menu_body">
              <ul class="menu_list">
                <li><a href="#"class="menu_link">Раздел 1</a></li>
                <li><a href="#"class="menu_link">Раздел 2</a></li>
                <li><a href="#"class="menu_link">Раздел 2</a></li>
                <li><a href="#"class="menu_link">Страница 1</a>
                  <span class="menu_arrow"></span>
                  <ul class="menu_sub-list">
                    <li><a href="#"class="menu_sub-link">hi</a></li>
                    <li><a href="#"class="menu_sub-link">hello</a></li>
                    <li><a href="#"class="menu_sub-link">hey</a></li>
                  </ul>
                </li>
              </ul>
            </nav>
          </div>
        </head>

      </header>
      <main class="page">
        <section class="page_saction">
          <h1>qqqqqqqqqqqqqqqqqqqqqqq</h1>
        </section>

      </main>

    </div>




  </body>
</html>

body {
  padding: 0;
  margin: 0;
  font-family: 'Open Sans', sans-serif;
  font-size: 16px;
  color: #000;
  -webkit-font-smoothing: antialiased;
  background-color: #F8F8F8;
}

*,
*:before,
*:after {
  box-sizing: border-box;
}

h1,
h2,
h3,
h4,
h5,
h6 {
  margin: 0;
  padding: 0;
}

p {
  margin: 0;
  padding: 0;
}

hr {
  margin: 0;
  padding: 0;
}

ul,
li {
  display: block;
  padding: 0;
  margin: 0;
}

a {
  text-decoration: none;
}

.header {
  position: fixed;
  top: 0;
  left: 0;
  width: 100%;
  z-index: 10;
  background-color: #246;
}
.header_container {
  max-width: 1200px;
  margin: 0px auto;
  display: flex;
  padding: 0px 30px;
  align-items: center;
  justify-content: space-between;
  min-height: 70px;
}
.logo {
  border-radius: 50%;
  background-color: #fff;
  position: relative;
  z-index: 5;
  width: 40px;
  height: 40px;
}
.header_menu {

}
.menu {

}
.menu_icon {

}
.menu_body {

}

.menu_list {

}
.menu_list > li {
  position: relative;
  margin: 0px 0px 0px 20px;
}

.menu_link {
  color: #fff;
  font-size: 18px;
}
.menu_link:hover{
  text-decoration: underline;
}

.menu_sub-list {
  position: absolute;
  top: 100%;
  right: 0;
  background-color: #000;
  padding: 15px;
  min-width: 200px;

}
.menu_sub-list li {
  margin: 0px 0px 10px 0px;
}
.menu_sub-list li:last-child {
  margin: 0;
}

.menu_sub-link {
  color: #fff;
}
.menu_sub-link:hover{
  text-decoration: underline;
}
.menu_arrow {
  display: none;;
}
/* --------------------------- */
body._pc .menu_list > li:hover .menu_sub-list {
  opacity: 1;
  visibility: visible;
  transform: translate(0px, 0px);
  pointer-events: all;
}

body._touch .menu_list > li {
  display: flex;
  align-items: center;
}
body._touch .menu_link {
  flex: 1 1 auto;
}
body._touch .menu_arrow {
  display: block;
  width: 0;
  height: 0;
  margin: 0px 0px 0px 5px;
  transition: transform 0.3s ease 0s;
  border-left: 5px solid transparent;
  border-right: 5px solid transparent;
  border-top: 10px solid #fff;
}

body._touch .menu_list > li._active .menu_sub-list {
  opacity: 1;
  visibility: visible;
  transform: translate(0px, 0px);
  pointer-events: all;
}
/* --------------------------- */
@media (min-width: 767px) {
  .menu_list {
    display: flex;
    align-items: center;
  }
  .menu_list > li {
    padding: 10px 0;
  }
  .menu_sub-list {
    transform: translate (0px, 10%);
    opacity: 0;
    visibility: hidden;
    pointer-events: none;
  }

}

"use strict"

var isMobile = {
    Android: function () {
        return navigator.userAgent.match(/Android/i);
    },
    BlackBerry: function () {
        return navigator.userAgent.match(/BlackBerry/i);
    },
    iOS: function () {
        return navigator.userAgent.match(/iPhone|iPad|iPod/i);
    },
    Opera: function () {
        return navigator.userAgent.match(/Opera Mini/i);
    },
    Windows: function () {
        return navigator.userAgent.match(/IEMobile/i);
    },
    any: function () {
        return (
                isMobile.Android()
                || isMobile.BlackBerry()
                || isMobile.iOS()
                || isMobile.Opera()
                || isMobile.Windows()
                );
    }
};

if (isMobile.any()) {
  document.body.classList.add('_touch');

  let menuArrows = document.querySelectorAll('.menu_arrow');
  if (menuArrows.length > 0) {
    for (let index = 0; index < menuArrows.length; index++) {
      const menuArrow = menuArrows[index];
      menuArrow.addEventListener("click", function (e) {
        menuArrow.parentElement.classList.toggle('_active');
      });
    }
  }

} else {
  document.body.classList.add('_pc');
}

Написано более трёх лет назад

twobomb @twobomb

Olga17527, Ну всё прекрасно работает, если конечно я меняю if (isMobile.any()) { на if(true){
Не знаю что это за уникальный метод и для чего он нужен

Написано более трёх лет назад
Владислав Лысков @Vlatqa

Olga17527, попросили залить в песочгицу, а ты всё портянкой выложил, но тебе ведь это в первую очередь нужно

Написано более трёх лет назад
Spaceoddity @Spaceoddity

Пропишите вот это:
alert(navigator.userAgent);
И посмотрите что выведет на девайсе, на котором вы тестите.

Вообще детектить мобилы по тому, что отдаёт клиент в заголовке user agent - не самое разумное решение. Если вам нужны узкие экраны - используйте медиа-запросы в css, если поддержка touch - проверяйте именно на поддержку touch-cобытий. А в поле user-agent может быть что угодно.

Написано более трёх лет назад
Olga17527 @Olga17527 Автор вопроса

Владислав Лысков, Добрый день, залила) https://jsfiddle.net/vcq5urh3/15/

Написано более трёх лет назад

Пригласить эксперта

Ваш ответ на вопрос

Войдите, чтобы написать ответ

Похожие вопросы

JavaScript

Простой
Как изменить html при помощи jquery?
- 1 подписчик
- 2 часа назад
- 42 просмотра
0

ответов
JavaScript

+1 ещё

Средний
Сканирование QR в поле input. Как реализовать блокировку нажатия клавиш клавиатуры?
- 1 подписчик
- 18 часов назад
- 82 просмотра
1

ответ
JavaScript

Простой
Надо ли делать отложенную загрузку для изображений формата webp/avif?
- 1 подписчик
- вчера
- 75 просмотров
0

ответов
JavaScript

Простой
Поясните мне почему код работает JS?
- 1 подписчик
- вчера
- 155 просмотров
1

ответ
JavaScript

Простой
Язык сайта в адресной строке, как доработать код?
- 1 подписчик
- вчера
- 168 просмотров
0

ответов
JavaScript

+2 ещё

Средний
Почему FullCalendar не правильно отображает диапазоны событий (в клиентской части)?
- 1 подписчик
- 04 мар.
- 78 просмотров
1

ответ
JavaScript

+3 ещё

Средний
Как уменьшить масштаб блока в зависимости от размеров родителя?
- 1 подписчик
- 04 мар.
- 122 просмотра
2

ответа
JavaScript

+1 ещё

Средний
Как в Fancybox 4 отключить воможность перетаскивания окон?
- 1 подписчик
- 04 мар.
- 65 просмотров
1

ответ
JavaScript

Простой
Почему closest() и parentElement не видят родителя?
- 2 подписчика
- 03 мар.
- 265 просмотров
1

ответ
JavaScript

+1 ещё

Простой
React. При отправке axios-запросов компонент ререндерится слишком много раз (или нет)?
- 2 подписчика
- 01 мар.
- 166 просмотров
2

ответа
Показать ещё Загружается…

JavaScript FullStack разработчик

Rocket • Смоленск

от 120 000 ₽

Бэкэнд-разработчик JavaScript

Wanted. • Москва

от 250 000 до 400 000 ₽

Senior Frontend (JavaScript) разработчик

Vital Partners

от 350 000 до 420 000 ₽

А можно на песочницу залить? Не понятно что за метод any и выполняется ли он вообще, есть ли там элементы с классом menu_arrow у которых есть парент. Ну и класс должен меняться так-то только по клику
С кодом всё нормально, ну по крайней мере с малым его куском. Я бы его оптимизировал, но и так сойдет snippet

const btnsList = document.querySelector('ul'); btnsList.addEventListener('click', ({target}) => { if (target.classList.contains('menu_arrow')) { target.parentElement.classList.toggle('_active'); } })
избавился от ненужного цикла
Olga17527, Ну всё прекрасно работает, если конечно я меняю if (isMobile.any()) { на if(true){
Не знаю что это за уникальный метод и для чего он нужен
Olga17527, попросили залить в песочгицу, а ты всё портянкой выложил, но тебе ведь это в первую очередь нужно
Пропишите вот это:
alert(navigator.userAgent);
И посмотрите что выведет на девайсе, на котором вы тестите.

Вообще детектить мобилы по тому, что отдаёт клиент в заголовке user agent - не самое разумное решение. Если вам нужны узкие экраны - используйте медиа-запросы в css, если поддержка touch - проверяйте именно на поддержку touch-cобытий. А в поле user-agent может быть что угодно.
Владислав Лысков, Добрый день, залила) https://jsfiddle.net/vcq5urh3/15/

JavaScript Поможете найти ошибкув коде?

Войдите, чтобы написать ответ

Минуточку внимания

Войдите на сайт