diff options
author | Jacob Schatz <jschatz@gitlab.com> | 2016-06-15 21:11:31 +0000 |
---|---|---|
committer | Jacob Schatz <jschatz@gitlab.com> | 2016-06-15 21:11:31 +0000 |
commit | 1a832c8f00f55a0a5edb49d53e757e2bb6398060 (patch) | |
tree | 28820728def0da8c3b954bc6a13e780e94b314c8 /Gemfile | |
parent | b4e8d59b0e343d8c34f83417175d631b101bc045 (diff) | |
parent | 7163229738c4fa534d7909ea168d0612ef89fbef (diff) | |
download | gitlab-ce-1a832c8f00f55a0a5edb49d53e757e2bb6398060.tar.gz |
Merge branch 'network-assets' into 'master'
Replace raphael-rails with raphael.js and split it from the rest of the JS
## What does this MR do?
Removes raphael.js and its related libraries from the default JS bundle.
It also removes raphael-rails because the gem is unmaintained and – for whatever reason – didn't want to cooperate with the way I was trying to require it.
This brings the default (compressed and minified) application.js from 354KB after the d3 MR (!4394) down to 324KB.
## Are there points in the code the reviewer needs to double check?
That tests pass and that there aren't errors on any pages (especially the Network page).
## Why was this MR needed?
It's only used on the Network page and was using way too much bandwidth for just one page.
## What are the relevant issue numbers?
#14372 #13165
Follow-up to !4394 and !4516.
cc: @jschatz1
See merge request !4519
Diffstat (limited to 'Gemfile')
-rw-r--r-- | Gemfile | 1 |
1 files changed, 0 insertions, 1 deletions
@@ -227,7 +227,6 @@ gem 'gon', '~> 6.0.1' gem 'jquery-atwho-rails', '~> 1.3.2' gem 'jquery-rails', '~> 4.1.0' gem 'jquery-ui-rails', '~> 5.0.0' -gem 'raphael-rails', '~> 2.1.2' gem 'request_store', '~> 1.3.0' gem 'select2-rails', '~> 3.5.9' gem 'virtus', '~> 1.0.1' |