Skip to content

Conversation

@mklim
Copy link
Contributor

@mklim mklim commented Feb 6, 2019

No description provided.

@mklim mklim requested a review from sfshaza2 February 6, 2019 01:35
@googlebot googlebot added the cla: yes Contributor has signed the Contributor License Agreement label Feb 6, 2019
Copy link
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are a few minor changes I'd made, as a writer, but nothing significant. LGTM

@sfshaza2 sfshaza2 merged commit 4763143 into flutter:master Feb 7, 2019
@mklim mklim deleted the androidx_info branch February 7, 2019 21:24
use both sets of libraries.

Some Flutter plugins have been migrated to AndroidX after originally using the
original suport libraries. Flutter apps that aren't compatible with AndroidX
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo in "support"

This requires the latest version of Android Studio.

1. Follow the instructions on [this
page](https://flutter.io/docs/development/tools/android-studio#android-ide)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are a number of "this page" links. We should strongly prefer descriptive links (e.g. "Follow the instructions to import a Flutter app into Android Studio", "Follow the instructions for migrating an app to AndroidX automatically using Android Studio").

One major problem with "This page" (or "here") links is that they're terrible for things that scrape links from a page (such as screen readers) since they'll end up with a bunch of links that all look the same.

@mklim
Copy link
Contributor Author

mklim commented Feb 7, 2019

@jamesderlin I opened a follow up PR at #2360.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Contributor has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants