Support » Plugin: True Lazy Analytics » Does not use passive listeners to improve scrolling performance

  • Resolved xdevelx

    (@xdevelx)


    PageSpeed Insights выдает предупреждение Does not use passive listeners to improve scrolling performance и в справочной информации https://web.dev/uses-passive-event-listeners/ рекомендация для обработчиков событий передавать параметр {passive: true}

    document.addEventListener('touchstart', onTouchStart, {passive: true});

    • This topic was modified 5 months, 3 weeks ago by xdevelx.
Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Author seojacky

    (@seojacky)

    Спасибо. Протестирую. Но, честно говоря, замечание Does not use passive listeners не влияет ни на баллы, ни на LCP, CLS. Это замечание для перфекционистов.

    Thread Starter xdevelx

    (@xdevelx)

    В этом я полностью согласен. На одном из тестируемых сайтов после внедрения true lazy analytics балы с 80/99 изменились на 100/100. Но появилось это предупреждение. События scroll и touch естественно не повлияют на саму загрузку страницы. Но pagespeed тем не менее контролирует обработчики таких событий. Да и добавить доп параметры в addEventListener очень просто. Зато исчезнет это предупреждение.

    • This reply was modified 5 months, 3 weeks ago by xdevelx.
    Plugin Author seojacky

    (@seojacky)

    Кстати у меня на моих сайтах нет такого предупреждения. Я даже не могу оттестировать изменения. Вы разработчик? Можете внедрить этот параметр у себя, путём редактирования файла плагина? Если замечание уйдёт, то внедрю в следующей версии

    Thread Starter xdevelx

    (@xdevelx)

    В плагине заменил

    window.addEventListener( 'scroll', loadTLAnalytics );
    window.addEventListener( 'touchstart', loadTLAnalytics );

    на

    window.addEventListener( 'scroll', loadTLAnalytics, {passive: true} );
    window.addEventListener( 'touchstart', loadTLAnalytics, {passive: true} );

    И пункт Does not use passive listeners to improve scrolling performance исчез.

    Единственный нюанс: у меня в js был handler на touchstart. Но он не вызывал этого предупреждения. Когда появился еще один из скрипта плагина, он уже выдает предупреждение. Тестировал в разных вариантах и получилось, когда есть один только обработчик на touchstart, гугл это устраивает. Когда появляется два и более, то сразу же предупреждение, которое убирается через {passive: true}

    • This reply was modified 5 months, 3 weeks ago by xdevelx.
    • This reply was modified 5 months, 3 weeks ago by xdevelx.
    Thread Starter xdevelx

    (@xdevelx)

    В версии 2.2.3 пофиксили

    Fixed error in Lighthouse Report: Does not use passive listeners to improve scrolling performance

    Спасибо за мгновенную реакцию на issue.

Viewing 5 replies - 1 through 5 (of 5 total)
  • You must be logged in to reply to this topic.