Skip to content

Commit 0397bab

Browse files
committed
Rework scatter-chart
- Remove old classes - Use LineOptions and add appropriate comment
1 parent 6203279 commit 0397bab

File tree

5 files changed

+13
-61
lines changed

5 files changed

+13
-61
lines changed

src/ChartJs.Blazor/ChartJS/ScatterChart/GridLineSettings.cs

Lines changed: 0 additions & 7 deletions
This file was deleted.

src/ChartJs.Blazor/ChartJS/ScatterChart/ScatterAxis.cs

Lines changed: 0 additions & 15 deletions
This file was deleted.

src/ChartJs.Blazor/ChartJS/ScatterChart/ScatterConfig.cs

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,19 @@
33

44
namespace ChartJs.Blazor.ChartJS.ScatterChart
55
{
6-
public class ScatterConfig : ConfigBase<ScatterOptions, ScatterData>
6+
/// <summary>
7+
/// Represents the config for a scatter chart.
8+
/// <para>
9+
/// Scatter charts are based on basic line charts with the x axis changed to a
10+
/// <see cref="Common.Axes.LinearCartesianAxis"/> (unless otherwise specified).
11+
/// Therefore, many configuration options are from the line chart.
12+
/// </para>
13+
/// </summary>
14+
public class ScatterConfig : ConfigBase<LineChart.LineOptions, ScatterData>
715
{
16+
/// <summary>
17+
/// Creates a new instance of <see cref="ScatterConfig"/>.
18+
/// </summary>
819
public ScatterConfig() : base(ChartType.Scatter) { }
920
}
10-
}
21+
}

src/ChartJs.Blazor/ChartJS/ScatterChart/ScatterOptions.cs

Lines changed: 0 additions & 14 deletions
This file was deleted.

src/ChartJs.Blazor/ChartJS/ScatterChart/ScatterScales.cs

Lines changed: 0 additions & 23 deletions
This file was deleted.

0 commit comments

Comments
 (0)