Conversation
There was a problem hiding this comment.
Code Review
This pull request provides several documentation updates, primarily focused on cleaning up wording, improving formatting for readability, and updating guidance for web development. Key changes include replacing 'see' with 'visit' or 'check out' for external links, reformatting long lines, and adding information about the Jaspr package for building static websites with Dart. My review found a minor grammatical issue where a semicolon was used incorrectly, and I've suggested a fix. Overall, the changes improve the clarity and accuracy of the documentation.
Note: Security Review has been skipped due to the limited scope of the PR.
|
|
||
| Not currently. | ||
| Not currently. Instead, | ||
| consider using the [`os_detect`][] package. |
There was a problem hiding this comment.
I think support exists for kIsWeb - if this is for platform detection would should perhaps update this question here.
There was a problem hiding this comment.
Interesting, @jonasfj asked for this fix. I'll see what he says when we reviews.
|
Visit the preview URL for this PR (updated for commit 193700c): https://flutter-docs-prod--pr13160-os-detect-2ngx3z6o.web.app |
Fixes #13146
Also, fixes formatting and updates guidance around static websites.