Merge branch 'downgrade-d3' into 'master'

Downgrade D3 to 4.13

See merge request gitlab-org/gitlab!23313

app/assets/javascripts/vue_shared/components/bar_chart.vue

@@ -124,7 +124,9 @@ export default {
     },
   },
   mounted() {
-    this.draw();
+    if (!this.allValuesEmpty) {
+      this.draw();
+    }
   },
   methods: {
     draw() {
@@ -151,14 +153,7 @@ export default {
       this.yScale = d3.scaleLinear().rangeRound([this.vbHeight, 0]);
 
       this.xScale.domain(this.graphData.map(d => d.name));
-      /*
-      If we have all-zero graph we want graph to center 0 on axis and not to draw
-      any kind of ticks on Y axis. Infinity allows us to do that.
-
-      See https://gitlab.com/gitlab-org/gitlab/merge_requests/20627#note_251484582
-      for detailed explanation
-      */
-      this.yScale.domain([0, d3.max(this.graphData.map(d => d.value)) || Infinity]);
+      this.yScale.domain([0, d3.max(this.graphData.map(d => d.value))]);
 
       // Zoom/Panning Function
       this.zoom = d3

package.json

@@ -66,7 +66,7 @@
     "core-js": "^3.2.1",
     "cropper": "^2.3.0",
     "css-loader": "^1.0.0",
-    "d3": "^5.12.0",
+    "d3": "^4.13.0",
     "d3-scale": "^1.0.7",
     "d3-selection": "^1.2.0",
     "dateformat": "^3.0.3",