1052437: Fixed blazor toolkit chart rendering issue in SSR - #66
Open
Yokesh-SF4393 wants to merge 2 commits into
Open
1052437: Fixed blazor toolkit chart rendering issue in SSR#66Yokesh-SF4393 wants to merge 2 commits into
Yokesh-SF4393 wants to merge 2 commits into
Conversation
CI Summary
Overall: ✅ All checks passed |
CI Summary
Overall: ✅ All checks passed |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bug description
With Blazor interactivity set to none (Static SSR), Syncfusion charts did not render correctly or at all. Interactive modes were fine.
Root cause
Static SSR has no OnAfterRenderAsync or reliable DOM measurement. Layout and child renderers depended on post-render hooks and JS, so the first (and only) server pass never finished size, series, axis, or decoration rendering.
Solution description
Detect SSR via IsStaticServerRendering(), set default SVG size (e.g. 600×450), run layout through SetDefaultRendererValues / HandleChartSizeChange from BuildRenderTree / Prerender, and keep all interactive-only behavior behind SSR checks.
Review Changes:
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):
Impact assessment
Reason for not identifying earlier
Previously the SSR support is given, but it was not working properly, now fixed.
Areas tested against this fix
Breaking changes
breaking-issue)If yes, provide breaking commit details link and migration guidance.
Regression testing
Action taken to prevent recurrence
Automation status
Cross-platform verification
Related issues
Is this issue present in EJ2 or other components?
needs-attention-coreteam)Output screenshots
API changes
Performance verification
Reviewer Checklist