Jump to content
DenKP

fa-minus в мобильной версии не скрывает подкатегорию

Recommended Posts

Всем Алоха! Надеюсь корона Вас еще не замучала.

 

Может кто подсказать в каком направлении копать, в мобильной версии работает открытие категории на + но на минус нет события.

http://webmark.com.ua/mobile/?url=dev.oruzheinik.kz вот ссылка на отображение в мобильной версии

Буду признателен любой наводки.

Share this post


Link to post
Share on other sites

Возможно это из-за ошибок верстки. Или же проблема в подключаемых скриптах.

 

Snap2.jpg

Snap1.jpg

Share this post


Link to post
Share on other sites

Событие на закрытие , скорее всего родное бутстраповское, может не срабатывать по нескольким причинам

Посмотрите ошибки в консоли, скрин

 

 

 

sshot-10.png

sshot-11.png

Share this post


Link to post
Share on other sites

Понял, спасибо пошел копать

Share this post


Link to post
Share on other sites

а CartLayer разве не к части яндекса относится?

 ym(68722264, "init", {
        clickmap:true,
        trackLinks:true,
        accurateTrackBounce:true,
        webvisor:true,
        ecommerce:"CartLayer"
   });
</script>
<noscript><div><img src="https://mc.yandex.ru/watch/68722264" style="position:absolute; left:-9999px;" alt="" /></div></noscript>
<!-- /Yandex.Metrika counter -->

<script type="text/javascript">
window.CartLayer = window.CartLayer || [];
</script>

<script type="text/javascript">
window.CartLayer.push ({...});
</script>

Share this post


Link to post
Share on other sites
5 минут назад, DenKP сказал:

window.CartLayer.push ({...});

должны быть данные, а не многоточие
https://yandex.ru/support/metrica/data/e-commerce.html

Share this post


Link to post
Share on other sites

В целом убрал, ошибок ноль, но они не связаны с менюшкой были

Share this post


Link to post
Share on other sites
4 минуты назад, thentru сказал:

должны быть данные, а не многоточие
https://yandex.ru/support/metrica/data/e-commerce.html

 

Спасибо, насколько помню там для Казахстана не сделали еще, поэтому и остался нерабочий кусок

Share this post


Link to post
Share on other sites
5 часов назад, DenKP сказал:

но на минус нет события

событие есть, при нажатии на кнопку всегда срабатывает одно и то же условие
в этом проблема
 

Цитата
$menu
                  .find('.dropdown')
                    .removeClass('open')
                      .find('.ochow-open')
                        .removeClass('ochow-open')
                        .addClass('ochow-close');

 

Share this post


Link to post
Share on other sites

В самом начале я как раз события перебирал тут. Но не пришел к логичной реализации
 

Снимок_оп.JPG

Share this post


Link to post
Share on other sites
6 минут назад, DenKP сказал:

В самом начале я как раз события перебирал тут. Но не пришел к логичной реализации
 

Снимок_оп.JPG

Можно ли посадить на кнопку минус событие по изменению класса с dropdown open на dropdown?

Share this post


Link to post
Share on other sites
1 час назад, thentru сказал:

событие есть, при нажатии на кнопку всегда срабатывает одно и то же условие
в этом проблема
 

 

Спасибо в js нашел это место, 
пока что выглядит логично код, но раз событие реально я вижу что одно и тоже значит тут переделать нужно

//Меню
            $(document).ready(function () {
              var $menu = $('#menu');

              $('.ochow-menu-item-toggle').on('click', function () {
                var $btn = $(this);

                $menu
                  .find('.dropdown')
                    .removeClass('open')
                      .find('.ochow-open')
                        .removeClass('ochow-open')
                        .addClass('ochow-close');
                

                if ($btn.hasClass('ochow-close')) {
                  $btn
                    .removeClass('ochow-close')
                    .addClass('ochow-open')
                      .parent()
                        .addClass('open')
                } else {
                  $btn
                    .removeClass('ochow-open')
                    .addClass('ochow-close')
                      .parent()
                        .removeClass('open')
                }

Share this post


Link to post
Share on other sites
42 минуты назад, DenKP сказал:

if ($btn.hasClass('ochow-close')) {

у элемента всегда будет класс ochow-close

Share this post


Link to post
Share on other sites
4 часа назад, thentru сказал:

у элемента всегда будет класс ochow-close

 

Сколько раз прогонял, вижу что класс меняется. А как Вы определили что всегда класс этот?

Скрин вложил

Что самое для меня непонятное, я понимаю что именно в if трабла, он не переходит ни как на условие else

 

Снимок_оп.JPG

Share this post


Link to post
Share on other sites
8 минут назад, DenKP сказал:

 

Сколько раз прогонял, вижу что класс меняется. А как Вы определили что всегда класс этот?

Скрин вложил

Что самое для меня непонятное, я понимаю что именно в if трабла, он не переходит ни как на условие else

 

Снимок_оп.JPG

 

 

Вот всегда как сюда пишу, неравнодушные помогают, вообщем решил вот таким способом
image.png.6182a842fa73214f1b91ac9d65d25ffd.png

 

теперь оно закрывается, но блин повторно не открывается)))

 

@thentru отдельный респект!!!

Edited by DenKP

Share this post


Link to post
Share on other sites

Все тему можно точно закрывать, проблема возникала в find и removeClass addClass, они выставлялись в конце классов а изначально располагались в середине.

 

После полной перезамены классов, все работает как нужно. Был вариант в коде конечно разместить классы замены в конце, но пока классы менять там не буду и если потребуется буду уже делать по уму, наверное это по уму!

Еще раз спасибо @thentru

Снимок_оп.JPG

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
You are posting as a guest. If you have an account, please sign in.
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.


  • Recently Browsing   0 members

    No registered users viewing this page.

×

Important Information

On our site, cookies are used and personal data is processed to improve the user interface. To find out what and what personal data we are processing, please go to the link. If you click "I agree," it means that you understand and accept all the conditions specified in this Privacy Notice.