@feedzai/brushable-histogram - npm Package Compare versions

Comparing version 1.1.7 to 1.1.8

CHANGELOG.md

 # Brushable Histogram
 
+## 1.1.8 (2019/03/15)
+
+- Fix a bug related with screen resizing events
+
 ## 1.1.7 (2019/02/19)
@@ -4,0 +8,0 @@ - Fix a bug related with the zooming interactions between the histogram

lib/DensityChart/DensityChart.js

@@ -88,3 +88,6 @@ "use strict";
     let max = this.props.brushDomainMax;
-    const densityChartXScale = this.props.densityChartXScale;
+    const _this$props2 = this.props,
+          densityChartXScale = _this$props2.densityChartXScale,
+          width = _this$props2.width,
+          height = _this$props2.height;
 
@@ -95,2 +98,7 @@ if (max >= this.props.overallTimeDomainMax) {
       max = this.props.overallTimeDomainMax;
+    } // We need to resize the max value of the brush when the screen has resized
+
+
+    if (prevProps.width !== width || prevProps.height !== height) {
+      this.brush.extent([[0, 0], [width, height]]);
     }
@@ -170,12 +178,12 @@
   _drawDensityChart() {
-    const _this$props2 = this.props,
-          width = _this$props2.width,
-          height = _this$props2.height,
-          densityChartXScale = _this$props2.densityChartXScale,
-          brushDomainMax = _this$props2.brushDomainMax,
-          brushDomainMin = _this$props2.brushDomainMin,
-          xAccessor = _this$props2.xAccessor,
-          data = _this$props2.data,
-          brushDensityChartColor = _this$props2.brushDensityChartColor,
-          brushDensityChartFadedColor = _this$props2.brushDensityChartFadedColor;
+    const _this$props3 = this.props,
+          width = _this$props3.width,
+          height = _this$props3.height,
+          densityChartXScale = _this$props3.densityChartXScale,
+          brushDomainMax = _this$props3.brushDomainMax,
+          brushDomainMin = _this$props3.brushDomainMin,
+          xAccessor = _this$props3.xAccessor,
+          data = _this$props3.data,
+          brushDensityChartColor = _this$props3.brushDensityChartColor,
+          brushDensityChartFadedColor = _this$props3.brushDensityChartFadedColor;
     (0, _canvasRenderUtils.clearCanvas)(this.densityChartCanvasContext, width, height);
@@ -203,10 +211,10 @@
   _renderPlayButton() {
-    const _this$props3 = this.props,
-          width = _this$props3.width,
-          densityChartXScale = _this$props3.densityChartXScale,
-          brushDomainMax = _this$props3.brushDomainMax,
-          brushDomainMin = _this$props3.brushDomainMin,
-          frameStep = _this$props3.frameStep,
-          frameDelay = _this$props3.frameDelay,
-          renderPlayButton = _this$props3.renderPlayButton;
+    const _this$props4 = this.props,
+          width = _this$props4.width,
+          densityChartXScale = _this$props4.densityChartXScale,
+          brushDomainMax = _this$props4.brushDomainMax,
+          brushDomainMin = _this$props4.brushDomainMin,
+          frameStep = _this$props4.frameStep,
+          frameDelay = _this$props4.frameDelay,
+          renderPlayButton = _this$props4.renderPlayButton;
 
@@ -230,6 +238,6 @@ if (!renderPlayButton) {
     let leftPadding = 0;
-    const _this$props4 = this.props,
-          width = _this$props4.width,
-          height = _this$props4.height,
-          padding = _this$props4.padding;
+    const _this$props5 = this.props,
+          width = _this$props5.width,
+          height = _this$props5.height,
+          padding = _this$props5.padding;
 
@@ -236,0 +244,0 @@ if (!this.props.renderPlayButton) {

lib/Histogram/Histogram.js

@@ -84,3 +84,3 @@ "use strict";
       // This is an early return in order to avoid processing brush event
-      if (_d3Selection.event.sourceEvent && _d3Selection.event.sourceEvent.type === "brush") {
+      if (_d3Selection.event.sourceEvent && _d3Selection.event.sourceEvent.target.name === "brush") {
         return;
@@ -87,0 +87,0 @@ }

package.json

 {
   "name": "@feedzai/brushable-histogram",
-  "version": "1.1.7",
+  "version": "1.1.8",
   "description": "A time histogram with a time brush that renders a summary of the events",
@@ -34,3 +34,4 @@ "main": "lib/index.js",
     "[e-mail]",
-    "[e-mail]"
+    "[e-mail]",
+    "[e-mail]"
   ],
@@ -37,0 +38,0 @@ "devDependencies": {

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

504063

increased by0.11%

Lines of code

2171

increased by0.28%

Number of medium supply chain risk alerts

0

decreased by-100%

No dependency changes