Closed abitofevrything closed 3 months ago
/gcbrun
Visit the preview URL for this PR (updated for commit 9af242f19e1a651977cdfd758432046ead3a3442):
https://dart-dev--pr5939-flutter-install-warning-qkm6dj4t.web.app
question: This point already exists on this page.
Does this not meet your requirement? (Honestly, just asking. Overuse of admonitions is a concern with the docs.)
Yes, it is mentioned - but only further down the list than the package manager option. I see a lot of linux users on a flutter support group (FlutterDev on Discord) who see the package manager option and immediately jump to it, skipping over the point about installing flutter underneath.
Another option would therefore be to reorder the list to place the Install Flutter option first, but I also think that having the message stand out in some way (in this case, by being in a tip box) early on in the page can help draw more attention to it as most users of Dart are still currently only using Dart because of Flutter - and therefore do not need to be visiting the Get Dart page in the first place.
@abitofevrything : Make sense. Let me check the admonitions quick and see if we can get more content above the edge of the first screen.
Fixes https://github.com/dart-lang/site-www/issues/5938
Contribution guidelines:
- See our [contributor guide](https://github.com/dart-lang/site-www/blob/main/CONTRIBUTING.md) for general expectations for PRs. - Larger or significant changes should be discussed in an issue before creating a PR. - Code changes should generally follow the [Dart style guide](https://dart.dev/effective-dart) and use `dart format`. - Updates to [code excerpts](https://github.com/dart-lang/site-shared/blob/main/doc/code-excerpts.md) indicated by `