1005021: Updated the UG documentations getting started content according to component rendering.#8421
Conversation
…ing to component rendering.
Recommendation: Merge the empty step and the GeoJSON step into a single "Add Syncfusion Blazor Maps component" step, showing the complete ready-to-run markup immediately.
Chart: First shows the data model/list in a separate @code block, then shows the complete Razor markup in a second block. This is redundant since the complete block at the end already contains both.
"map the data fields Month and Sales to the series XName and YName" But the actual property in the model is SalesValue, not Sales. The description says Sales while the code uses SalesValue. This will confuse beginners. Fix needed in getting-started-with-web-app.md (Chart): Change: "map the data fields Month and Sales" Issue — Stock Chart: YName="Close" is redundant for candle series:
Accumulation Chart — "Install packages" is H3 (###) instead of H2 (##) — breaks the visual hierarchy and is inconsistent with the other three. Chart — H1 title mismatch: # Getting Started with Blazor Chart Component in Web App — all others say in Blazor Web App. Missing the word "Blazor". Chart — CLI project name inconsistency: Uses BlazorApp (VS Code tab) and BlazorApp (.NET CLI tab). All other three use BlazorWebApp consistently. Stock Chart — "See also" links are outdated: Points to old blazor-server-side-visual-studio and blazor-server-side-dotnet-cli links. Should match the other three pages which link to blazor-webassembly-app and blazor-web-app. Stock Chart — front matter uses control: key instead of component: used by the other three. Minor but worth aligning.
Maps — the empty component warning breaks flow:
Maps: the final "Bind data source" section does not produce a visible rendered output. It only defines data and explains ShapePropertyPath/ShapeDataPath without showing the final combined markup. The section ends without a complete, runnable code block — the reader is pointed to a separate page. This leaves the Getting Started page incomplete as a standalone guide.
File Issue Suggested Fix1 All four No theme CSS in Chart/Accum/Maps Add Syncfusion.Blazor.Themes NuGet + CSS to match Stock Chart pattern Claude Sonnet 4.6 • 3x |
|
Build Status: INPROGRESS 🔃 |
|
CI Status: SUCCESS ✅ |
|
Build Status: INPROGRESS 🔃 |
|
CI Status: SUCCESS ✅ |
37a4039
into
hotfix/hotfix-v33.2.3
Description
Changes for the Chart , Maps , Stock and accumulation chart getting started documentation.
Fix minor grammar and wording issues across Blazor Chart
Code Studio usage(Mandatory)
Code Studio used in this PR/MR?
If
Yes: Primary use (choose one)Outcome
If “Cost time” explain in short (1 or 2 lines):
Type of Change
Reviewer Checklist (Mandatory)