1

Всем привет

Есть такая функция:

<script>
    $(window).on('hashchange', function(){
        var id = location.hash, targ=$(id+'+section.module'); //запоминаем в targ нужный блок
        if (!targ[0]) return // выходим, если "левый" хэш, например, ввели вручную
        var top = targ.offset().top;
        $(".header__list a").removeClass(('active'));
        $('.header__list a[href="'+id+'"]').addClass('active');
    }
    //
    // if (windowRes <= 1024) {
    //  $('body,html').animate({scrollTop: top})
    // }

    $('body,html').animate({
        scrollTop: top + 3
    }, 1500);
    });
    }).on('load', function(){$(window).hashchange()}) // переходим к нужному разделу, если, напрмер, ссылку в новой вкладке открыли
</script>

Веб-инспектор кричит об ошибке: joxi.ru/4AkzJvnHBlen2q

SyntaxError: missing ) after argument list
8
  • не пользуйся PhpStorm :) Тут кстати и кроме этого еще и }); лишний
    – Grundy
    11 авг 2016 в 10:38
  • 1
    А че не в поддержку phpstorm, там же очень отзывчивые ребята 11 авг 2016 в 10:41
  • @Grundy Можете, пожалуйста, переписать правильно функцию? Я ее копипастнул и не вник в смысл. Пробовал переписать, но ломается лоигка 11 авг 2016 в 10:42
  • @VladimirGamalian Я думаю проблема в том, что JS в index.html, поэтому и не видит ошибку 11 авг 2016 в 10:43
  • может вы проверки отключили просто? у меня все отлично показывает goo.gl/vxeuYK 11 авг 2016 в 11:04

2 ответа 2

1

Учитывая ваш сриншот, я пришел к выводу, что все нужные code inspections у вас активны. У себя мне не удалось воспроизвести проблему отключением каких бы то ни было проверок.

Я нашел только один способ добиться такого, как у вас внешнего вида кода - убрав цветное подчеркивание в code style. Проверьте эти настройки у себя: https://goo.gl/zL4mt1 (Editor -> Colors & Fonts -> General -> Errors and Warnings -> Error). Они должны быть такими же. Если галочек справа нет или цвет не контрастный - получается ситуация как у вас.

Если это не поможет - мое мнение, что проблема в глюке шторма или плагинов.

Верный код, в вашем случае, будет примерно таким, если я правильно его понял:

<script>
    $(window).on('hashchange', function(){
        var id = location.hash,
            targ = $(id+'+section.module'); //запоминаем в targ нужный блок

        if (!targ[0]) return; // выходим, если "левый" хэш, например, ввели вручную

        var top = targ.offset().top;
        $(".header__list a").removeClass(('active'));
        $('.header__list a[href="'+id+'"]').addClass('active');

        //
        // if (windowRes <= 1024) {
        //  $('body,html').animate({scrollTop: top})
        // }

        $('body,html').animate({
            scrollTop: top + 3
        }, 1500);
    }).on('load', function(){$(window).trigger('hashchange')}); // переходим к нужному разделу, если, напрмер, ссылку в новой вкладке открыли
</script>

Для меня осталось загадкой, откуда там скобки в середине блока кода и странные }) в конце.

11
  • Мне бы код переписать на правильный лад) У меня не вышло 11 авг 2016 в 11:46
  • По ошибке - вроде все круто joxi.ru/eAOeJl9UqDB5mo 11 авг 2016 в 11:48
  • @НикитаЩипилов тогда у меня нет идей, почему ошибки не показывает. Попробуйте переустановить) Версия, кстати, актуальная? А код в чем сложность поправить - по строкам читаешь код и смотришь где неожиданные скобки и все делов. 11 авг 2016 в 11:54
  • Да не всего делов, не выходит переписать. Последняя строка все ломает. Можешь глянуть? 11 авг 2016 в 11:56
  • Предпоследняя версия, вроде как 11 авг 2016 в 11:56
1

PhpStorm 2017.3.4/manjaro

Галочка на пункте "Power Save Mode" может привести к отключению функций подсказки и проверки синтаксиса. Уберите её.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.