Fixes to Linux Install from @wednesdei #9442
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit introduces several significant enhancements to the Linux installation guide for Flutter. Key changes include: 1. Expanded 'System Requirements' section with clearer, more concise language. 2. Added a new 'Understanding Linux Commands' section to familiarize users with common Linux commands used throughout the installation process. 3. Included a new section on 'Installing Snap on Ubuntu,' guiding users through the process of installing and using Snap on their Ubuntu systems. 4. Revamped the 'Flutter Installation', 'Android Studio Setup', and 'Chrome Setup' sections to provide more detailed, step-by-step instructions. 5. Incorporated a comprehensive 'Glossary' at the end of the document, providing definitions for key terms and commands. The goal of these changes is to make the installation guide more accessible, user-friendly, and informative for users of all experience levels.
This was referenced Sep 22, 2023
|
Visit the preview URL for this PR (updated for commit c7c59c4): https://flutter-docs-prod--pr9442-patch-1-ym8g17ow.web.app (expires Fri, 13 Oct 2023 04:42:42 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: d5ba327eec813901cac8396c4f458b02288624ab |
domesticmouse
requested review from
domesticmouse and
MaryaBelanger
as code owners
domesticmouse
approved these changes
Oct 5, 2023
| @@ -2,129 +2,98 @@ | |||
|
|
|||
| {% include docs/china-notice.md %} | |||
|
|
|||
| On Linux, you have two ways you can install Flutter. | |||
| Install Flutter on Linux using one of the following two methods. | |||
There was a problem hiding this comment.
Flutter on Linux sounds odd to my ears. Flutter for Linux?
|
|
||
| [(loading...)](#){:.download-latest-link-{{os}}.btn.btn-primary} | ||
| If you don't have or can't use `snapd`, follow these steps to install Flutter. |
There was a problem hiding this comment.
If you aren't using snapd ?
parlough
reviewed
Oct 5, 2023
| Once you have snapd, you can | ||
| [install Flutter using the Snap Store][], | ||
| or at the command line: | ||
| After you install `snapd`, install Flutter from the Snap Store or |
There was a problem hiding this comment.
The link to the snap store listing still seems useful here.
| Run the following command to see if there are any | ||
| dependencies you need to install to complete the setup | ||
| (for verbose output, add the `-v` flag): | ||
| For verbose output, add the `-v` flag: |
There was a problem hiding this comment.
Nit: This doesn't provide enough information to the reader why they would want to do this. Will it help them debug the listed issues, etc?
| Once you have installed any missing dependencies, | ||
| run the `flutter doctor` command again to verify | ||
| that you've set everything up correctly. | ||
| After installing any missing dependencies, |
There was a problem hiding this comment.
The previous text also discussed "tasks to perform", but the changes lose that. It would be good to still add that clarification since flutter doctor isn't just about dependencies.
parlough
added
st.RFM.%
Co-authored-by: Parker Lougheed <parlough@gmail.com>
atsansone
added a commit
to atsansone/website
that referenced
this pull request
Oct 9, 2023
_Description of what this PR is changing or adding, and why:_ This replaces * flutter#9082 * flutter#9089 Fixes flutter#8025 Fixes flutter#2131 Fixes flutter#6499 ## Presubmit checklist - [ ] This PR doesn’t contain automatically generated corrections (Grammarly or similar). - [ ] This PR follows the [Google Developer Documentation Style Guidelines](https://developers.google.com/style) — for example, it doesn’t use _i.e._ or _e.g._, and it avoids _I_ and _we_ (first person). - [ ] This PR uses [semantic line breaks](https://github.com/dart-lang/site-shared/blob/main/doc/writing-for-dart-and-flutter-websites.md#semantic-line-breaks) of 80 characters or fewer. --------- Co-authored-by: Chris M <44120439+wednesdei@users.noreply.github.com> Co-authored-by: Brett Morgan <brettmorgan@google.com> Co-authored-by: Parker Lougheed <parlough@gmail.com>
atsansone
added a commit
to atsansone/website
that referenced
this pull request
Oct 13, 2023
_Description of what this PR is changing or adding, and why:_ This replaces * flutter#9082 * flutter#9089 Fixes flutter#8025 Fixes flutter#2131 Fixes flutter#6499 ## Presubmit checklist - [ ] This PR doesn’t contain automatically generated corrections (Grammarly or similar). - [ ] This PR follows the [Google Developer Documentation Style Guidelines](https://developers.google.com/style) — for example, it doesn’t use _i.e._ or _e.g._, and it avoids _I_ and _we_ (first person). - [ ] This PR uses [semantic line breaks](https://github.com/dart-lang/site-shared/blob/main/doc/writing-for-dart-and-flutter-websites.md#semantic-line-breaks) of 80 characters or fewer. --------- Co-authored-by: Chris M <44120439+wednesdei@users.noreply.github.com> Co-authored-by: Brett Morgan <brettmorgan@google.com> Co-authored-by: Parker Lougheed <parlough@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description of what this PR is changing or adding, and why:
This replaces
Fixes #8025
Fixes #2131
Fixes #6499
Presubmit checklist