Skip to content

Fixed CS8602 warnings in the toolkit charts. - #71

Merged
Indhumathy-Loganathan merged 2 commits into
mainfrom
1052437-phase2
Sep 10, 2026
Merged

Fixed CS8602 warnings in the toolkit charts.#71
Indhumathy-Loganathan merged 2 commits into
mainfrom
1052437-phase2

Conversation

@Yokesh-SF4393

@Yokesh-SF4393 Yokesh-SF4393 commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

Description

  • Fixed CS8602 warnings in the blazor toolkit charts. Resolved 341 warnings and checked the bunit test cases and chart samples in sample browser manually.
  • Updated the mobile mode SB rendering issue. Updated the max-width style property, which caused the SB page not rendering.
  • Updated the program.cs documentation to configure the blazor toolkit charts.

Key Changes

  • Updated MainLayout.razor file to fix the SB page rendering issue in mobile mode.
  • Updated Charts getting started page with proper import syntax.
  • Updated Chart component related files to clear the CS8602 warnings.

Files Affected

Samples

  • samples/Blazor.Toolkit.Samples.Client/Layout/MainLayout.razor‎

Charts Samples

  • samples/Blazor.Toolkit.Samples.Client/Pages/Components/Charts/GettingStarted.razor‎

Charts

  • src/Components/Charts/Chart/Renderer/SeriesRenderers/BaseRenderers/ColumnBase.cs
  • src/Components/Charts/Chart/Renderer/SeriesRenderers/BaseRenderers/SeriesContainer.cs
  • src/Components/Charts/Chart/Renderer/SeriesRenderers/BaseRenderers/SplineBase.cs
  • src/Components/Charts/Chart/Renderer/SeriesRenderers/DatalabelRenderer/DataLabelRenderer.cs
  • src/Components/Charts/Chart/Renderer/SeriesRenderers/LastDatalabelRenderer/LastDataLabelRenderer.cs
  • src/Components/Charts/Chart/Renderer/SeriesRenderers/MarkerRenders/ChartMarkerRenderer.cs
  • src/Components/Charts/Chart/Renderer/SeriesRenderers/SeriesTypeRenderers/AreaSeriesRenderer.cs
  • src/Components/Charts/Chart/Renderer/SeriesRenderers/SeriesTypeRenderers/BubbleSeriesRenderer.cs
  • src/Components/Charts/Chart/Renderer/SeriesRenderers/SeriesTypeRenderers/MultiColoredAreaSeriesRenderer.cs
  • src/Components/Charts/Chart/Renderer/SeriesRenderers/SeriesTypeRenderers/MultiColoredLineSeriesRenderer.cs
  • src/Components/Charts/Chart/Renderer/SeriesRenderers/SeriesTypeRenderers/SplineAreaSeriesRenderer.cs
  • src/Components/Charts/Chart/Renderer/SeriesRenderers/SeriesTypeRenderers/SplineSeriesRenderer.cs
  • src/Components/Charts/Chart/Renderer/SeriesRenderers/SeriesTypeRenderers/StackingAreaSeriesRenderer.cs
  • src/Components/Charts/Chart/Renderer/SeriesRenderers/SeriesTypeRenderers/StackingBarSeriesRenderer.cs
  • src/Components/Charts/Chart/Renderer/SeriesRenderers/SeriesTypeRenderers/StackingColumnSeriesRenderer.cs
  • src/Components/Charts/Chart/Renderer/SeriesRenderers/SeriesTypeRenderers/StackingLineSeriesRenderer.cs
  • src/Components/Charts/Chart/Renderer/SeriesRenderers/SeriesTypeRenderers/StepLineSeriesRenderer.cs
  • src/Components/Charts/Chart/Renderer/StriplineRenderer/ChartStriplineRenderer.cs
  • src/Components/Charts/Chart/Renderer/TrendlineRenderer/TrendlineBase.cs
  • src/Components/Charts/Chart/Renderer/TrendlineRenderer/TrendlineSeriesRenderer/Trendline_SplineSeriesRenderer.cs
  • src/Components/Charts/Chart/Renderer/UserInteractions/DataEditing.cs
  • src/Components/Charts/Chart/Series/ChartSeries.cs
  • src/Components/Charts/Chart/SfChart.razor.JsInterop.cs
  • src/Components/Charts/Chart/UserInteractions/Tooltip/ChartData.cs
  • src/Components/Charts/Common/BaseTooltip/TooltipData.razor.cs
  • src/Components/Charts/Common/ChartUtils/ChartHelper.cs
  • src/Components/Charts/Common/CommonLegend/LegendBase.cs
  • src/Components/Charts/Common/SvgComponents/SvgPattern.razor

Samples & Tests

  • Sample pages under samples/Blazor.Toolkit.Samples.Client/... (Tested for Charts and SB rendering in mobile mode)
  • Playwright samples and specs under tests/Syncfusion.Blazor.Playwright.Test/... (Tested for charts)

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

CI Summary

Job Status
bUnit (.NET 8 / 9 / 10) ✅ Passed
Playwright ✅ Passed
NuGet vulnerability scan ✅ Passed
ESLint security ✅ Passed
XSS / unsafe markup scan ✅ Passed

Overall: ✅ All checks passed

@Yokesh-SF4393 Yokesh-SF4393 changed the title 1052437: Fixed warnings in the toolkit charts. Fixed CS8602 warnings in the toolkit charts. Sep 10, 2026
@Indhumathy-Loganathan
Indhumathy-Loganathan merged commit 2394b94 into main Sep 10, 2026
12 checks passed
@Indhumathy-Loganathan
Indhumathy-Loganathan deleted the 1052437-phase2 branch September 11, 2026 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants