-
-
Notifications
You must be signed in to change notification settings - Fork 59
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Уменьшение размера банда #172
Comments
@webkonstantin подскажешь по |
radio-t-site/hugo/src/js/debug.js Lines 3 to 7 in 0eeca6c
|
Убрал element-closest и hoverintent в моем PR #171 |
Так же используется полифил для customEvent, хотя он появился еще в IE9 https://caniuse.com/customevent |
Так же в идеале вставка всех картинок должна быть через |
насчет CustomEvent, используется конструктор, который в ie не поддерживается |
Ну то есть, если использовать старый конструктор, то можно не юзать полифил :) |
1. Убрать библиотекуilyabirman-likely
Заменить обработку кликов на:
Twitter
https://twitter.com/intent/tweet?url={url}&text={title}
Facebook
https://www.facebook.com/sharer.php?u={url}
VK
'https://vk.com/share.php?url={url}&title={title}
domready
Может быть заменено:
Можно избавиться от
debug
в пользу обычногоconsole.log
если вообще стоит оставлять легирование в продакшн сборкеНепонятно используется ли пакет
hoverintent
,element-closest
Использовать полифилы на основе browserslist через useBuiltIns: 'usage' без ручной вставки
Попробовать добавить purgecss чтобы удалять неиспользуемые стили из CSS.
The text was updated successfully, but these errors were encountered: