jsakamoto / Toolbelt.Blazor.SpeechSynthesis

SpeechSynthesis API access for your Blazor apps.
https://jsakamoto.github.io/Toolbelt.Blazor.SpeechSynthesis/
Mozilla Public License 2.0
89 stars 19 forks source link

.NET 5.0 is not yet supported? #4

Closed bestpika closed 3 years ago

bestpika commented 3 years ago

Today I tried to update the target framework to .NET 5.0 and then compiled it and it didn't work.

jsakamoto commented 3 years ago

It will be works fine even on .NET 5.

see also my sample site: https://github.com/jsakamoto/Toolbelt.Blazor.SpeechSynthesis/commit/c9128483ed0a96611e89c44a90475f90c75de180

bestpika commented 3 years ago

Oh, sorry. I mean the NuGet's package. So I have to change my usage by the sample. Thank you.

bestpika commented 3 years ago

@jsakamoto I find that changing <app> to <div id="app"> breaks the whole layout of the site, and changing it to <app id="app"> should be a better solution.

jsakamoto commented 3 years ago

@bestpika

I find that changing <app> to <div id="app"> breaks the whole layout of the site,

It is your CSS styling problem, not this library's problem. DO NOT post your own problem here.

bestpika commented 3 years ago

@jsakamoto Okay.

But that's not my CSS styling, that's the styling of the example Blazor created. And your example only uses #app to get the element. If you think my advice is bad, forget it, it's just a suggestion.