Merge pull request #73 from apexcharts/fix-datapoints-selection

Fixed datapoint selection for mixed charts

Author: joadan

docs/BlazorApexCharts.Docs/Components/Events/DataPointSelection/Basic.razor

@@ -7,21 +7,23 @@
                OnDataPointSelection=DataPointsSelected
                @ref=chart>
 
-        <ApexPointSeries TItem="Order"
+                   <ApexPointSeries TItem="Order"
                          Items="Orders"
-                         Name="Gross Value"
+                         Name="Net Value"
                          SeriesType="SeriesType.Line"
                          XValue="@(e => e.Country)"
-                         YAggregate="@(e => e.Sum(e => e.GrossValue))"
+                         YAggregate="@(e => e.Sum(e => e.NetValue))"
                          OrderByDescending="e=>e.X" />
 
         <ApexPointSeries TItem="Order"
                          Items="Orders"
-                         Name="Net Value"
-                         SeriesType="SeriesType.Line"
+                         Name="Gross Value"
+                         SeriesType="SeriesType.Bar"
                          XValue="@(e => e.Country)"
-                         YAggregate="@(e => e.Sum(e => e.NetValue))"
+                         YAggregate="@(e => e.Sum(e => e.GrossValue))"
                          OrderByDescending="e=>e.X" />
+
+    
     </ApexChart>
 </DemoContainer>
 

src/Blazor-ApexCharts/ApexChart.razor.cs

@@ -39,9 +39,6 @@ public partial class ApexChart<TItem> : IDisposable where TItem : class
         private string chartId;
         public string ChartId => ChartId;
 
-
-
-
         protected override async Task OnAfterRenderAsync(bool firstRender)
         {
             if (firstRender && isReady == false)
@@ -223,13 +220,28 @@ private void UpdateDataForNoAxisCharts()
             {
                 Options.Colors = colors;
             }
-           
+        }
+
+
+        private bool ShouldFixDataSelection()
+        {
+            if (!OnDataPointSelection.HasDelegate || !Options.Series.Any()) { return false; }
+
+            if(Options.Chart?.Type != null && Options.Chart.Type == ChartType.Line || Options.Chart.Type == ChartType.Area || Options.Chart.Type == ChartType.Radar)
+            {
+                return true;
+            }
+
+            if(Options.Series.Any(e=> e.Type == MixedType.Line || e.Type == MixedType.Area)) {
+                return true;
+            }
 
+            return false;
         }
 
         private void FixLineDataSelection()
         {
-            if ((Options.Chart.Type == ChartType.Line || Options.Chart.Type == ChartType.Area || Options.Chart.Type == ChartType.Radar) && OnDataPointSelection.HasDelegate)
+            if (ShouldFixDataSelection())
             {
                 if (Options.Tooltip == null) { Options.Tooltip = new Tooltip(); }
                 if (Options.Markers == null) { Options.Markers = new Markers(); }