xdevelx
Forum Replies Created
-
В версии 2.2.3 пофиксили
Fixed error in Lighthouse Report: Does not use passive listeners to improve scrolling performance
Спасибо за мгновенную реакцию на issue.
В плагине заменил
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}В этом я полностью согласен. На одном из тестируемых сайтов после внедрения true lazy analytics балы с 80/99 изменились на 100/100. Но появилось это предупреждение. События scroll и touch естественно не повлияют на саму загрузку страницы. Но pagespeed тем не менее контролирует обработчики таких событий. Да и добавить доп параметры в addEventListener очень просто. Зато исчезнет это предупреждение.
- This reply was modified 4 years, 11 months ago by xdevelx.