-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consider converting C# tree builders to .razor files #168
Comments
Omar says: |
Omar says: There were a couple of reasons we were using C# directly: Daniel says: @keith Melmon Instead of starting with Blazor WebAssembly 3.2, I actually think you should go ahead and start working with Blazor WebAssembly in .NET 5 RC2. .NET 5 will release in about two weeks, and the RC2 bits are really close to the final version. Steve says: => What would you recommend here? Should they give it a try with the latest bits and we’ll see what we can do to help them workaround any SVG issues that come up? Yes. The known SVG issues are more to do with runtime issues, not compile-time ones. So I would not expect that the use of .razor vs .cs would make any difference to those issues. However it’s possible I’m missing something subtle. |
When investigating #165, I asked for some help from Daniel Roth (Blazor guy from DevDiv), who said this:
"It looks like most of the components in this project are written directly in C# instead of using Razor. That complicates things a bit, because we didn’t bother to document many of the API changes in the render tree builder in the various preview releases."
We should consider converting the C# tree builders to .razor files as it will keep us more inline with the docs/samples/guidance.
The text was updated successfully, but these errors were encountered: