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.
Fixes #28044
cc: @plastovicka ... There's a bit of churn on this one! 😵😄 I think this will solve most of the problems. I might need a little more PU feedback, but this is moving in the right direction.
In this follow-up PR ...
index.html
or{_Host|_Layout}.cshtml
files. I'll continue to be 👂 for further PU remarks on this. I might need to do a little more work on this aspect, but let's see how it goes. My take on PU remarks thus far is that JS initializers are the way to go 👍 when you want to guarantee before or after JS tasks, including script loading.then
onBlazor.start
for script loading control in 6.0+ in favor of JS intializers. Until further notice from the PU, we'll keep the Chain to thePromise
that results from a manual start section for all doc versions, which is generic and doesn't focus on script loading. Again, I might need to do a hair more work on this because the docs don't clarify exact use cases for the chaining approach withthen
versus theafterStarted
approach. I've created a separate tracking remark in Blazor project tracking 2023 #28001 to remind myself to investigate and possibly cover this further on a future PR.