Fixes IsVisible and stuck geometries in canvas

src/LiveChartsCore/CartesianSeries.cs

@@ -289,8 +289,6 @@ public override void SoftDeleteOrDispose(IChartView chart)
         }
 
         foreach (var item in deleted) _ = everFetched.Remove(item);
-
-        OnVisibilityChanged();
     }
 
     /// <summary>

src/LiveChartsCore/CorePieSeries.cs

@@ -646,8 +646,6 @@ public override void SoftDeleteOrDispose(IChartView chart)
         }
 
         foreach (var item in toDelete) _ = everFetched.Remove(item);
-
-        OnVisibilityChanged();
     }
 
     private void AlignLabel(TLabel label, double start, double initialRotation, double sweep)

src/LiveChartsCore/CorePolarLineSeries.cs

@@ -770,8 +770,6 @@ public override void SoftDeleteOrDispose(IChartView chart)
 
         if (GeometryFill is not null) canvas.RemovePaintTask(GeometryFill);
         if (GeometryStroke is not null) canvas.RemovePaintTask(GeometryStroke);
-
-        OnVisibilityChanged();
     }
 
     /// <inheritdoc cref="ChartElement{TDrawingContext}.GetPaintTasks"/>

src/LiveChartsCore/ISeries.cs

@@ -144,11 +144,6 @@ public interface ISeries
     /// </value>
     Func<float, float>? EasingFunction { get; set; }
 
-    /// <summary>
-    /// Occurs when the series <see cref="IsVisible"/> property changes.
-    /// </summary>
-    event Action<ISeries>? VisibilityChanged;
-
     /// <summary>
     /// Gets the tool tip text for a give chart point.
     /// </summary>

src/LiveChartsCore/Series.cs

@@ -239,8 +239,7 @@ public Func<ChartPoint<TModel, TVisual, TLabel>, string> DataLabelsFormatter
     public bool IsVisible
     {
         get => _isVisible;
-        set
-        => SetProperty(ref _isVisible, value);
+        set => SetProperty(ref _isVisible, value);
     }
 
     /// <inheritdoc cref="ISeries.IsHoverable" />
@@ -304,9 +303,6 @@ public Sketch<TDrawingContext> CanvasSchedule
         protected set => SetProperty(ref _miniatureSketch, value);
     }
 
-    /// <inheritdoc cref="ISeries.VisibilityChanged"/>
-    public event Action<ISeries>? VisibilityChanged;
-
     /// <inheritdoc cref="IChartSeries{TDrawingContext}.GetStackGroup"/>
     public virtual int GetStackGroup()
     {
@@ -462,14 +458,6 @@ protected internal virtual void OnPointCreated(ChartPoint chartPoint)
     /// <returns></returns>
     protected abstract void SetDefaultPointTransitions(ChartPoint chartPoint);
 
-    /// <summary>
-    /// Called when the visibility changes.
-    /// </summary>
-    protected virtual void OnVisibilityChanged()
-    {
-        VisibilityChanged?.Invoke(this);
-    }
-
     /// <summary>
     /// Called when the pointer enters a point.
     /// </summary>

tests/LiveChartsCore.UnitTesting/SeriesTests/_SharedPropertiesTests.cs

@@ -29,7 +29,7 @@
         var changes = 0;
 
         values.CollectionChanged +=
             (object? sender, NotifyCollectionChangedEventArgs e) =>
                 changes++;
 
         values.Add(new(1));
@@ -52,8 +52,11 @@
     [TestMethod]
     public void VisbilityAndPaintsTasks()
     {
-        var values = new ObservableCollection<DateTimePoint>() { new() };
-        var lineSeries = new ColumnSeries<DateTimePoint>(values);
+        // based on:
+        // https://github.com/beto-rodriguez/LiveCharts2/issues/1164
+
+        var values = new ObservableCollection<ObservableValue>() { new(1) };
+        var lineSeries = new ColumnSeries<ObservableValue>(values);
         var chart = new SKCartesianChart
         {
             Series = [lineSeries],
@@ -76,5 +79,11 @@
         Assert.IsTrue(
            drawables == canvas.DrawablesCount &&
            geometries == canvas.CountGeometries());
+
+        values.Clear();
+
+        Assert.IsTrue(
+           drawables == canvas.DrawablesCount &&
+           geometries < canvas.CountGeometries());
     }
 }