docs: clarify FlutterLoader.loadEntrypoint deprecation - #13858
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the web initialization documentation to include a new section addressing the deprecation of FlutterLoader.loadEntrypoint in favor of FlutterLoader.load(). The reviewer suggested adding a simpler alternative for users who have not customized their web/index.html file, which is to replace the old inline script block with the standard <script src="flutter_bootstrap.js" async></script> tag.
|
Staged preview of the updated docs.flutter.dev site (updated for commit 00d76ae): https://flutter-docs-prod--docs-pr13858-fix-12184-goxi9oet.web.app |
|
Staged preview of the updated flutter.dev site (updated for commit 00d76ae): https://flutter-dev-230821--www-pr13858-fix-12184-5g50g4so.web.app |
conooi
left a comment
There was a problem hiding this comment.
One suggestion but otherwise lgtm!
| ## Common warnings {:#common-warnings} | ||
|
|
||
| ### Service worker version deprecation |
There was a problem hiding this comment.
Per the Google dev style guide and Flutter doc guidelines, avoid stacking headings without intervening text!
| ## Common warnings {:#common-warnings} | |
| ### Service worker version deprecation | |
| ## Common warnings {:#common-warnings} | |
| This section describes common web build and initialization warnings | |
| and how to resolve them. | |
| ### Service worker version deprecation |
Fixes #12184
Clarifies the deprecation of
FlutterLoader.loadEntrypointin favor of_flutter.loader.load()under a new subsection in the Common warnings section of the web initialization documentation.Presubmit checklist
of 80 characters or fewer.