Consider the enabled property in our touch data (#2010)

* Consider the `enabled` property in LineTouchData, BarTouchData, PieTouchData, ScatterTouchData, RadarTouchData and CandlestickTouchData, #1676

* Update tests

Author: imaNNeo

CHANGELOG.md

@@ -1,3 +1,6 @@
+## newVersion
+* **BUGFIX** (by @imaNNeo) Consider the `enabled` property in [LineTouchData](https://github.com/imaNNeo/fl_chart/blob/main/repo_files/documentations/line_chart.md#linetouchdata-read-about-touch-handling), [BarTouchData](https://github.com/imaNNeo/fl_chart/blob/main/repo_files/documentations/bar_chart.md#bartouchdata-read-about-touch-handling), [PieTouchData](https://github.com/imaNNeo/fl_chart/blob/main/repo_files/documentations/pie_chart.md#pietouchdata-read-about-touch-handling), [ScatterTouchData](https://github.com/imaNNeo/fl_chart/blob/main/repo_files/documentations/scatter_chart.md#scattertouchdata-read-about-touch-handling), [RadarTouchData](https://github.com/imaNNeo/fl_chart/blob/main/repo_files/documentations/radar_chart.md#radartouchdata-read-about-touch-handling) and [CandlestickTouchData](https://github.com/imaNNeo/fl_chart/blob/main/repo_files/documentations/candlestick_chart.md#candlesticktouchdata-read-about-touch-handling), #1676
+
 ## 1.1.1
 * **IMPROVEMENT** (by @imaNNeo) Upgrade `vector_math` dependency to `2.2.0`, #1985
 * **IMPROVEMENT** (by @imaNNeo) Upgrade `build_runner` (dev) dependency to `2.8.0`

example/lib/presentation/samples/bar/bar_chart_sample1.dart

@@ -161,6 +161,7 @@ class BarChartSample1State extends State<BarChartSample1> {
   BarChartData mainBarData() {
     return BarChartData(
       barTouchData: BarTouchData(
+        enabled: true,
         touchTooltipData: BarTouchTooltipData(
           getTooltipColor: (_) => Colors.blueGrey,
           tooltipHorizontalAlignment: FLHorizontalAlignment.right,

lib/src/chart/bar_chart/bar_chart_renderer.dart

@@ -120,6 +120,14 @@ class RenderBarChart extends RenderBaseChart<BarTouchResponse> {
     canvas.restore();
   }
 
+  @override
+  bool hitTestSelf(Offset position) {
+    if (!targetData.barTouchData.enabled) {
+      return false;
+    }
+    return super.hitTestSelf(position);
+  }
+
   @override
   BarTouchResponse getResponseAtLocation(Offset localPosition) {
     final chartSize = mockTestSize ?? size;

lib/src/chart/candlestick_chart/candlestick_chart_renderer.dart

@@ -128,6 +128,14 @@ class RenderCandlestickChart extends RenderBaseChart<CandlestickTouchResponse> {
     canvas.restore();
   }
 
+  @override
+  bool hitTestSelf(Offset position) {
+    if (!targetData.candlestickTouchData.enabled) {
+      return false;
+    }
+    return super.hitTestSelf(position);
+  }
+
   @override
   CandlestickTouchResponse getResponseAtLocation(Offset localPosition) {
     final chartSize = mockTestSize ?? size;

lib/src/chart/line_chart/line_chart_renderer.dart

@@ -121,6 +121,14 @@ class RenderLineChart extends RenderBaseChart<LineTouchResponse> {
     canvas.restore();
   }
 
+  @override
+  bool hitTestSelf(Offset position) {
+    if (!targetData.lineTouchData.enabled) {
+      return false;
+    }
+    return super.hitTestSelf(position);
+  }
+
   @override
   LineTouchResponse getResponseAtLocation(Offset localPosition) {
     final chartSize = mockTestSize ?? size;

lib/src/chart/pie_chart/pie_chart_renderer.dart

@@ -159,6 +159,14 @@ class RenderPieChart extends RenderBaseChart<PieTouchResponse>
     }
   }
 
+  @override
+  bool hitTestSelf(Offset position) {
+    if (!targetData.pieTouchData.enabled) {
+      return false;
+    }
+    return super.hitTestSelf(position);
+  }
+
   @override
   PieTouchResponse getResponseAtLocation(Offset localPosition) {
     return PieTouchResponse(

lib/src/chart/radar_chart/radar_chart_renderer.dart

@@ -100,6 +100,14 @@ class RenderRadarChart extends RenderBaseChart<RadarTouchResponse> {
     canvas.restore();
   }
 
+  @override
+  bool hitTestSelf(Offset position) {
+    if (!targetData.radarTouchData.enabled) {
+      return false;
+    }
+    return super.hitTestSelf(position);
+  }
+
   @override
   RadarTouchResponse getResponseAtLocation(Offset localPosition) {
     return RadarTouchResponse(

lib/src/chart/scatter_chart/scatter_chart_renderer.dart

@@ -124,6 +124,14 @@ class RenderScatterChart extends RenderBaseChart<ScatterTouchResponse> {
     canvas.restore();
   }
 
+  @override
+  bool hitTestSelf(Offset position) {
+    if (!targetData.scatterTouchData.enabled) {
+      return false;
+    }
+    return super.hitTestSelf(position);
+  }
+
   @override
   ScatterTouchResponse getResponseAtLocation(Offset localPosition) {
     final chartSize = mockTestSize ?? size;

test/chart/bar_chart/bar_chart_renderer_test.dart

@@ -38,6 +38,7 @@ void main() {
         topTitles: AxisTitles(),
         bottomTitles: AxisTitles(),
       ),
+      barTouchData: const BarTouchData(enabled: false),
     );
 
     const textScaler = TextScaler.linear(4);
@@ -69,6 +70,7 @@ void main() {
       expect(renderBarChart.paintHolder.data == data, true);
       expect(renderBarChart.paintHolder.targetData == targetData, true);
       expect(renderBarChart.paintHolder.textScaler == textScaler, true);
+      expect(renderBarChart.hitTestSelf(Offset.zero), false);
     });
 
     test('test 2 check paint function', () {

test/chart/candlestick_chart/candlestick_chart_renderer_test.dart

@@ -20,6 +20,7 @@ void main() {
 
     final targetData = CandlestickChartData(
       candlestickSpots: [candlestickSpot3],
+      candlestickTouchData: CandlestickTouchData(enabled: false),
     );
 
     const textScaler = TextScaler.linear(4);
@@ -51,6 +52,7 @@ void main() {
       expect(renderCandlestickChart.paintHolder.data == data, true);
       expect(renderCandlestickChart.paintHolder.targetData == targetData, true);
       expect(renderCandlestickChart.paintHolder.textScaler == textScaler, true);
+      expect(renderCandlestickChart.hitTestSelf(Offset.zero), false);
     });
 
     test('test 2 check paint function', () {