Skip to content

Hw lesson 6#2

Open
VSidhArt wants to merge 2 commits intospajic:masterfrom
VSidhArt:hw_lesson_6
Open

Hw lesson 6#2
VSidhArt wants to merge 2 commits intospajic:masterfrom
VSidhArt:hw_lesson_6

Conversation

@VSidhArt
Copy link

@VSidhArt VSidhArt commented Apr 25, 2019 •
edited
Loading

Проблема

больший размер загружаемого js файла

Диагностика

Проверка бандла при помощи webpack-bundle-analyzer

Защита от регрессии

sitispeed.io с лимитом бюджета:

{
  "budget": {
    "transferSize": {
      "javascript": 460000
    }
  }
}

Анализ

После анализа было выявлено, что основной размер бандла занимают библиотеки chart.js и moment.js
Бандл vendor.js включающий эти библиотеки
vendor.js with chart
Без
vendor.js wo chart
moment.js непосредственно в коде не используется, а является зависимостью chart.js, которая используется только на странице dashboard pro пользователей

Решение

Исключил chart.js из банда vendor.js путем настройки плагина CommonsChunkVendor.

Результат

Размер бандла загружаемого на главной странице уменьшился с 1037 kb до 454 kb

Ссылка на сравнение исходного варианта и результата sitespeed.io

Compare sitespeedio

spajic
spajic approved these changes Apr 27, 2019
View reviewed changes
Copy link
Owner

@spajic spajic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

config/webpack/environment.js
@@ -1,4 +1,4 @@
// const { environment } = require('@rails/webpacker');
// const { environment } = require('@rails/webpacker')j
Copy link
Owner

@spajic spajic Apr 27, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ой

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@spajic spajic spajic approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@VSidhArt @spajic