apexcharts - npm Package Compare versions

Comparing version 1.4.0 to 1.4.1

package.json

 {
   "name": "apexcharts",
   "description": "A JavaScript Chart Library",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "main": "dist/apexcharts.min.js",
@@ -6,0 +6,0 @@ "directories": {

src/apexcharts.js

@@ -394,4 +394,11 @@ import Annotations from './modules/Annotations'
     let series = new Series(this.ctx)
+
     w.globals.dataChanged = true
 
+    // while updateing pie/donut series, user completely changed the series length too
+    if (!w.globals.axisCharts && newSeries.length !== w.globals.series.length) {
+      w.globals.dataChanged = false
+      animate = false
+    }
+
     if (animate) {
@@ -398,0 +405,0 @@ series.getPreviousPaths()

src/charts/Pie.js

@@ -263,11 +263,2 @@ import Fill from '../modules/Fill'
 
-      this.animatePaths(elPath, {
-        endAngle,
-        startAngle,
-        i,
-        totalItems: sectorAngleArr.length - 1,
-        animBeginArr: this.animBeginArr,
-        dur
-      })
-
       if (this.dynamicAnim && w.globals.dataChanged) {
@@ -279,7 +270,16 @@ this.animatePaths(elPath, {
           prevEndAngle,
+          animateStartingPos: true,
           i,
-          animateStartingPos: true,
           animBeginArr: this.animBeginArr,
           dur: w.config.chart.animations.dynamicAnimation.speed
         })
+      } else {
+        this.animatePaths(elPath, {
+          endAngle,
+          startAngle,
+          i,
+          totalItems: sectorAngleArr.length - 1,
+          animBeginArr: this.animBeginArr,
+          dur
+        })
       }
@@ -382,13 +382,2 @@
 
-    if (w.globals.dataChanged) {
-      // to avoid flickering, set prev path first and then we will animate from there
-      path = me.getPiePath({
-        me,
-        startAngle,
-        angle: prevAngle,
-        size
-      })
-      el.attr({ d: path })
-    }
-
     if (opts.dur !== 0) {
@@ -404,5 +393,5 @@ el.animate(opts.dur, w.globals.easing, opts.animBeginArr[opts.i]).afterAll(function () {
         if (params.animateStartingPos) {
-          startAngle = fromAngle + (toStartAngle - fromAngle) * pos
-          currAngle = fromStartAngle + (angle - fromStartAngle) * pos
+          startAngle = (fromStartAngle - currAngle) + (toStartAngle - (fromStartAngle - currAngle)) * pos
         }
+
         path = me.getPiePath({
@@ -409,0 +398,0 @@ me,

src/modules/Legend.js

@@ -342,4 +342,7 @@ import Dimensions from './Dimensions'
 
+    const legendRect = elLegendWrap.getBBox()
+    const legendTopPlusHeight = legendRect.y + legendRect.height
+
     let x = w.config.legend.containerMargin.left - w.config.legend.containerMargin.right
-    let y = w.config.legend.containerMargin.top - w.config.legend.markers.size - 2
+    let y = w.config.legend.containerMargin.top - w.config.legend.markers.size - 3
 
@@ -350,2 +353,6 @@ if (w.config.legend.position === 'bottom') {
       }
+
+      if (legendTopPlusHeight - 10 > w.globals.svgHeight) {
+        y = y - (w.globals.svgHeight - legendRect.y + legendRect.height) / 8
+      }
     }
@@ -352,0 +359,0 @@

src/modules/settings/Defaults.js

@@ -294,2 +294,3 @@ /**
       fill: {
+        opacity: 1,
         gradient: {
@@ -334,2 +335,3 @@ shade: 'dark',
       fill: {
+        opacity: 1,
         gradient: {
@@ -336,0 +338,0 @@ shade: 'dark',

src/modules/settings/Options.js

@@ -433,3 +433,3 @@ /**
         colors: undefined, // array of colors,
-        opacity: 0.9,
+        opacity: 0.85,
         gradient: {
@@ -436,0 +436,0 @@ shade: 'dark',

src/modules/Theme.js

@@ -30,4 +30,2 @@ import Utils from '../utils/Utils'
 
-    const defaultColors = w.globals.colors.slice()
-
     if (w.config.theme.monochrome.enabled) {
@@ -46,3 +44,3 @@ let monoArr = []
       for (let gsl = 0; gsl < glsCnt; gsl++) {
-        let newColor = mainColor
+        let newColor
 
@@ -61,2 +59,3 @@ if (shade === 'dark') {
     }
+    const defaultColors = w.globals.colors.slice()
 
@@ -63,0 +62,0 @@ // if user specfied less colors than no. of series, push the same colors again

src/modules/Toolbar.js

@@ -264,7 +264,7 @@ import Graphics from './Graphics'
     w.config.yaxis.map((yaxe, index) => {
-      yaxis[index].min = undefined
-      yaxis[index].max = undefined
+      yaxis[index].min = w.globals.initialConfig.yaxis[index].min
+      yaxis[index].max = w.globals.initialConfig.yaxis[index].max
     })
-    xaxis.min = undefined
-    xaxis.max = undefined
+    xaxis.min = w.globals.initialConfig.xaxis.min
+    xaxis.max = w.globals.initialConfig.xaxis.max
 
@@ -271,0 +271,0 @@ me.ctx.updateOptionsInternal(w.globals.initialConfig, false, true)

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

Improved metrics

Total package byte prevSize

938688

increased by0.06%

Lines of code

16700

increased by0.02%

No dependency changes