Fixed regression with wrong higher path on arearange series on invert…

…ed chart. Closes #1930.
highcharts · Jun 10, 2013 · 905535d · 905535d
1 parent c14b9d1
commit 905535d
Show file tree

Hide file tree

Showing 5 changed files with 40 additions and 4 deletions.
diff --git a/js/highcharts-more.src.js b/js/highcharts-more.src.js
@@ -697,13 +697,13 @@ seriesTypes.arearange = Highcharts.extendClass(seriesTypes.area, {
 				point.y = null;
 			} else if (low === null) {
 				point.plotLow = point.plotY = null;
-				point.plotHigh = yAxis.toPixels(high, true);
+				point.plotHigh = yAxis.translate(high, 0, 1, 0, 1);
 			} else if (high === null) {
 				point.plotLow = plotY;
 				point.plotHigh = null;
 			} else {
 				point.plotLow = plotY;
-				point.plotHigh = yAxis.toPixels(high, true);
+				point.plotHigh = yAxis.translate(high, 0, 1, 0, 1);
 			}
 		});
 	},

diff --git a/js/parts-more/AreaRangeSeries.js b/js/parts-more/AreaRangeSeries.js
@@ -70,13 +70,13 @@ seriesTypes.arearange = Highcharts.extendClass(seriesTypes.area, {
 				point.y = null;
 			} else if (low === null) {
 				point.plotLow = point.plotY = null;
-				point.plotHigh = yAxis.toPixels(high, true);
+				point.plotHigh = yAxis.translate(high, 0, 1, 0, 1);
 			} else if (high === null) {
 				point.plotLow = plotY;
 				point.plotHigh = null;
 			} else {
 				point.plotLow = plotY;
-				point.plotHigh = yAxis.toPixels(high, true);
+				point.plotHigh = yAxis.translate(high, 0, 1, 0, 1);
 			}
 		});
 	},

diff --git a/samples/highcharts/issues/1930/demo.details b/samples/highcharts/issues/1930/demo.details
@@ -0,0 +1,5 @@
+---
+ name: Highcharts Demo
+ authors:
+   - Torstein Hønsi
+...
diff --git a/samples/highcharts/issues/1930/demo.html b/samples/highcharts/issues/1930/demo.html
@@ -0,0 +1,4 @@
+<script src="http://code.highcharts.com/highcharts.js"></script>
+<script src="http://code.highcharts.com/highcharts-more.js"></script>
+
+<div id="container" style="height: 300px; width: 400px; margin: 0 auto;"></div>
diff --git a/samples/highcharts/issues/1930/demo.js b/samples/highcharts/issues/1930/demo.js
@@ -0,0 +1,27 @@
+$(function () {
+	var chart = new Highcharts.Chart({
+
+	    chart: {
+	        renderTo: 'container',
+	        type: 'arearange',
+	        inverted: true
+	    },
+
+	    xAxis: {
+	        type: 'category'
+	    },
+
+	    yAxis: {
+	        minTickInterval: 1
+	    },
+
+	    series: [{
+	        data: [
+	            ['Ein', 1, 2],
+	            ['To', 2, 3],
+	            ['Tre', 3, 4]
+	        ]
+	    }]
+
+	});
+});