fix: only call refresh when necessary

farfromrefug · farfromrefug · commit ece31f066fd6 · 2020-01-17T15:26:44.000+01:00
diff --git a/src/charting/listener/BarLineChartTouchListener.ts b/src/charting/listener/BarLineChartTouchListener.ts
@@ -180,7 +180,6 @@ export class BarLineChartTouchListener extends ChartTouchListener<BarLineChartBa
         // this.longpressGestureHandler.detachFromView(chart);
     }
     init() {
-        const chart = this.mChart;
         super.init();
 
         if (this.mChart.isDoubleTapToZoomEnabled()) {
@@ -408,10 +407,10 @@ export class BarLineChartTouchListener extends ChartTouchListener<BarLineChartBa
                 }
                 // }
             }
+            this.mMatrix = this.mChart.getViewPortHandler().refresh(this.mMatrix, this.mChart, true);
             // }
         }
 
-        this.mMatrix = this.mChart.getViewPortHandler().refresh(this.mMatrix, this.mChart, true);
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -180,7 +180,6 @@ export class BarLineChartTouchListener extends ChartTouchListener<BarLineChartBa`
`180`	`180`	`// this.longpressGestureHandler.detachFromView(chart);`
`181`	`181`	`}`
`182`	`182`	`init() {`
`183`		`- const chart = this.mChart;`
`184`	`183`	`super.init();`
`185`	`184`
`186`	`185`	`if (this.mChart.isDoubleTapToZoomEnabled()) {`
`@@ -408,10 +407,10 @@ export class BarLineChartTouchListener extends ChartTouchListener<BarLineChartBa`
`408`	`407`	`}`
`409`	`408`	`// }`
`410`	`409`	`}`
	`410`	`+ this.mMatrix = this.mChart.getViewPortHandler().refresh(this.mMatrix, this.mChart, true);`
`411`	`411`	`// }`
`412`	`412`	`}`
`413`	`413`
`414`		`- this.mMatrix = this.mChart.getViewPortHandler().refresh(this.mMatrix, this.mChart, true);`
`415`	`414`	`}`
`416`	`415`
`417`	`416`	`/**`