cfpb-chart-builder - npm Package Compare versions

Comparing version 5.1.2 to 6.0.0

package.json

 {
   "name": "cfpb-chart-builder",
-  "version": "5.1.2",
+  "version": "6.0.0",
   "description": "Charts for the Consumer Financial Protection Bureau",
@@ -40,9 +40,9 @@ "main": "src/js/index.js",
     "cf-typography": "5.0.1",
-    "highcharts": "6.2.0",
+    "highcharts": "7.0.0",
     "xdr": "0.5.3"
   },
   "devDependencies": {
-    "@babel/core": "7.1.6",
-    "@babel/preset-env": "7.1.6",
-    "autoprefixer": "9.3.1",
+    "@babel/core": "7.2.0",
+    "@babel/preset-env": "7.2.0",
+    "autoprefixer": "9.4.2",
     "babel-core": "^7.0.0-bridge.0",
@@ -68,3 +68,3 @@ "babel-jest": "23.6.0",
     "request": "2.88.0",
-    "require-dir": "1.1.0",
+    "require-dir": "1.2.0",
     "sauce-connect-launcher": "1.2.4",
@@ -71,0 +71,0 @@ "serve-static": "1.13.2",

src/js/charts/BarChart.js

@@ -1,3 +0,3 @@
-import Highcharts from 'highcharts/js/highstock';
-import accessibility from 'highcharts/js/modules/accessibility';
+import Highcharts from 'highcharts/highstock';
+import accessibility from 'highcharts/modules/accessibility';
 import { processYoyData } from '../utils/process-json';
@@ -25,2 +25,3 @@
         marginRight: 20,
+        styledMode: true,
         zoomType: 'none'
@@ -27,0 +28,0 @@ },

src/js/charts/GeoMap.js

@@ -1,3 +0,3 @@
-import Highcharts from 'highcharts/js/highmaps';
-import accessibility from 'highcharts/js/modules/accessibility';
+import Highcharts from 'highcharts/highmaps';
+import accessibility from 'highcharts/modules/accessibility';
 import colorRange from '../utils/color-range';
@@ -4,0 +4,0 @@ import outlines from '../utils/state-outlines';

src/js/charts/LineChart.js

@@ -1,3 +0,3 @@
-import Highcharts from 'highcharts/js/highstock';
-import accessibility from 'highcharts/js/modules/accessibility';
+import Highcharts from 'highcharts/highstock';
+import accessibility from 'highcharts/modules/accessibility';
 import { getFirstNumber } from '../utils/calculation';
@@ -75,2 +75,3 @@ import { processNumOriginationsData } from '../utils/process-json';
         marginRight: 20,
+        styledMode: true,
         zoomType: 'none'
@@ -241,2 +242,5 @@ },
               xAxis: {
+                labels: {
+                  y: 26
+                },
                 plotLines: [ {
@@ -249,3 +253,3 @@ value: data.projectedDate.timestamp,
                     x: -300,
-                    y: -24
+                    y: -20
                   }
@@ -252,0 +256,0 @@ } ]

src/js/charts/LineChartComparison.js

@@ -1,3 +0,3 @@
-import Highcharts from 'highcharts/js/highstock';
-import accessibility from 'highcharts/js/modules/accessibility';
+import Highcharts from 'highcharts/highstock';
+import accessibility from 'highcharts/modules/accessibility';
 import { processDelinquencies } from '../utils/process-json';
@@ -34,6 +34,7 @@
       chart: {
+        animation: false,
         marginRight: 0,
         marginTop: 100,
-        zoomType: 'none',
-        animation: false
+        styledMode: true,
+        zoomType: 'none'
       },
@@ -83,3 +84,4 @@ className: 'cfpb-chart_line-comparison',
         labels: {
-          useHTML: true
+          useHTML: true,
+          y: 26
         }
@@ -86,0 +88,0 @@ },

src/js/charts/LineChartIndex.js

@@ -1,3 +0,3 @@
-import Highcharts from 'highcharts/js/highstock';
-import accessibility from 'highcharts/js/modules/accessibility';
+import accessibility from 'highcharts/modules/accessibility';
+import Highcharts from 'highcharts/highstock';
 import { processNumOriginationsData } from '../utils/process-json';
@@ -25,2 +25,3 @@
         marginRight: 20,
+        styledMode: true,
         zoomType: 'none'
@@ -182,2 +183,5 @@ },
               xAxis: {
+                labels: {
+                  y: 26
+                },
                 plotLines: [ {
@@ -190,3 +194,3 @@ value: data.projectedDate.timestamp,
                     x: -300,
-                    y: -24
+                    y: -20
                   }
@@ -193,0 +197,0 @@ } ]

src/js/charts/TileMap.js

@@ -1,3 +0,3 @@
-import Highcharts from 'highcharts/js/highmaps';
-import accessibility from 'highcharts/js/modules/accessibility';
+import Highcharts from 'highcharts/highmaps';
+import accessibility from 'highcharts/modules/accessibility';
 import getTileMapColor from '../utils/get-tile-map-color';
@@ -75,3 +75,4 @@ import { processMapData } from '../utils/process-json';
       chart: {
-        marginTop: 150
+        marginTop: 150,
+        styledMode: true
       },
@@ -78,0 +79,0 @@ title: false,

test/unit_tests/charts/GeoMap-spec.js

 const shapes = require( '../../sample_data/mortgage-performance/map-data/30-89/states/us-states.geo.json' );
 
-jest.mock( 'highcharts/js/highmaps', () => ( {
+jest.mock( 'highcharts/highmaps', () => ( {
   setOptions: jest.fn(),
@@ -17,3 +17,3 @@ geojson: () => [],
 
-jest.mock( 'highcharts/js/modules/accessibility', () => jest.fn() );
+jest.mock( 'highcharts/modules/accessibility', () => jest.fn() );
 
@@ -20,0 +20,0 @@ import GeoMap from '../../../src/js/charts/GeoMap';

test/unit_tests/charts/LineChartComparison-spec.js

@@ -1,2 +0,2 @@
-jest.mock( 'highcharts/js/highstock', () => ( {
+jest.mock( 'highcharts/highstock', () => ( {
   setOptions: jest.fn(),
@@ -14,3 +14,3 @@ stockChart: ( props, opts ) => {
 
-jest.mock( 'highcharts/js/modules/accessibility', () => jest.fn() );
+jest.mock( 'highcharts/modules/accessibility', () => jest.fn() );
 
@@ -17,0 +17,0 @@ import LineChart from '../../../src/js/charts/LineChartComparison';

No alert changes

Improved metrics

Lines of code

41073

increased by0.03%

Worsened metrics

Total package byte prevSize

4381353

Dependency changes

• + Addedhighcharts@7.0.0(transitive)

• - Removedhighcharts@6.2.0(transitive)

• Updatedhighcharts@7.0.0