Teplitsa / Leyka

Плагин для WordPress, упрощающий сбор, управление и отчетность по онлайн-пожертвованиям, Leyka - Crowdfunding Fundraising and Donations Management System
https://leyka.org
GNU General Public License v2.0
82 stars 51 forks source link

Вынести SourceMaps в отдельные файлы #374

Closed versusbassz closed 1 week ago

versusbassz commented 2 years ago

Файл leyka/assets/css/public.css?ver=3.25 содержит внутри себя sourcemaps (итоговый размер: ~900 КБ). sourcemaps занимают ~65% размера файла. В итоге после gzip файл весит 152KB это довольно плохо для загрузки страницы (core web vitals и пр.), особенно учитывая что это только один из плагинов сайта.
Такая же проблема с assets/css/admins.css (он ещё больше весит).

Настройте, пожалуйста, сборку assets так, чтобы sourcemaps генерировались в отдельные файлы, ссылки на эти отдельные файлы оставались в оригинальных и эти sourcemaps загружались бы только при включенных dev-tools. Как пример, webpack так может, см. его документацию.

foralien commented 2 years ago

Здравствуйте!

Спасибо за ваше обращение! Мы добавим этот пункт в наши планы по разработке.

--

С уважением, Вячеслав Олейник

Full-stack developer

Служба поддержки Теплица социальных технологий https://te-st.ru

Teplitsa/Leyka, on May 6 @ 13:46 (Etc/GMT-3):

Файл leyka/assets/css/public.css?ver=3.25 содержит внутри себя sourcemaps. Они занимают ~65% размера файла. В итоге после gzip файл весит 152KB это довольно плохо для загрузки страницы (core web vitals и пр.), особенно учитывая что это только один из плагинов сайта.

Настройте, пожалуйста, сборку assets так, чтобы sourcemaps генерировались в отдельные файлы, ссылки на эти отдельные файлы оставались в оригинальных и эти sourcemaps загружались бы только при включенных dev-tools. Как пример, webpack так может, см. его документацию.

— Reply to this email directly, view it on GitHub [https://github.com/Teplitsa/Leyka/issues/374], or unsubscribe [https://github.com/notifications/unsubscribe-auth/AA4G7UZXBWVSKV5C25QFHZTVITZ4BANCNFSM5VHYSEYA]. You are receiving this because you are subscribed to this thread. https://github.com/notifications/beacon/AA4G7U5TFOLCX3DH7RWQMYLVITZ4BA5CNFSM5VHYSEYKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4SJNIRUA.gifMessage ID: @.***>

versusbassz commented 1 week ago

Fixed in Dec 2023. See the PR above.