search

DavidVollmers / Ignis

The Blazor framework for building modern web applications.
https://ignis.dvolper.dev
MIT License
150 stars 8 forks source link

Changes for .Net 8? #33

Closed vindberg closed 11 months ago

vindberg commented 12 months ago

Hi, I was wondering what changes are coming now that .net 8 has some of the same features as your solution.

I wish I had seen your project earlier, could have saved me a lot of work, especially with double reload of serverprerendered :)

Thanks for doing this open source!

DavidVollmers commented 12 months ago

Thank you very much 😄

I am currently still investigating when it comes to .NET 8, but so far I did not see anything which could be breaking Ignis. In fact Ignis will only profit from the cool new features introduced with .NET 8 (Blazor United for example).

But if you have any specific features in mind please tell me!

vindberg commented 11 months ago

My primary problems was SEO issues related to trying to fix the double event on initial load. Secondly, the reconnect of sockets is a big problem.

I think both issues has been solved for me in .Net 8. I guess your solution mitigated the same issues.

DavidVollmers commented 11 months ago

What I can see so far from .NET 8 is that components rendered using render mode ServerPrerendered will still trigger the events twice. When it comes to sockets I am not sure what exactly you mean.

vindberg commented 11 months ago

The lost socket connection is detailed here: https://github.com/dotnet/aspnetcore/issues/30344#issuecomment-1740463872

The reconnect message is awful, especially for mobile users that occasionally loose internet connection.

I will investigate the double trigger further, thanks 👍

DavidVollmers commented 11 months ago

I see thank you. This issue unfortunately will also happen using Ignis when rendering server side components. Ignis does implement its own components but is still attached to the Blazor circuit.

DavidVollmers commented 11 months ago

I will close this for now. Once .NET 8 is released I will also release an update for Ignis to support it!