-
Notifications
You must be signed in to change notification settings - Fork 7.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename Navigator 2.0 to Router #712
Conversation
This aligns better with the documentation for this new API, and doesn't imply that apps need to be upgraded.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left a comment on the wording. the rest LGTM
navigation_and_routing/README.md
Outdated
|
||
## Samples | ||
|
||
**Navigator 1.0 samples** | ||
**Introductory samples** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Introductory samples sounds a bit weird, Maybe just Navigator samples
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that makes sense, I was concerned that it might be misleading to call these Navigator samples, since Router
works alongside Navigator
too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have no strong opinion on this maybe you can add something like existing navigator sample
or imperative navigator example
but then you will need to explain what imperative is....
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Me neither, I'll change this to navigator
* update sample to use new versions (#702) * Fixed Typo in add-to-app/README.md (#706) * Fix cookbook scraper bugs and re-run (#697) * fix path separator bug in cookbook scraper * update lockfiles * add category to image name to avoid conflicts e.g. testing/integration/introduction testing/widget/introduction * fix form_app symlink * remove webimages directory * remove vision_challenge vision_challenge doesn't build on the dev channel * update pubspec.yaml in _tool and form_app * Added sample code for multiple flutters on iOS (#670) * Update link (#709) * Adjust pubspecs (#713) * Update file header date in contributing.md (#714) * Added Focusnodes in form sample (#707) * Rename Navigator 2.0 to Router (#712) * Rename Navigator 2.0 to Router This aligns better with the documentation for this new API, and doesn't imply that apps need to be upgraded. * introduction -> navigator * [animations] added test for focus image (#711) * Removed hardcoded NDK version for android_fullscreen (#717) Co-authored-by: Ayush Bherwani <ayush.bherwani1998@gmail.com> Co-authored-by: Swayam221 <68892315+Swayam221@users.noreply.github.com> Co-authored-by: John Ryan <ryjohn@google.com> Co-authored-by: gaaclarke <30870216+gaaclarke@users.noreply.github.com> Co-authored-by: Utkarsh Shendge <utkarshcpark@gmail.com> Co-authored-by: Michael Thomsen <mit@google.com> Co-authored-by: Daniel Etuk <77677996+0xdanny@users.noreply.github.com> Co-authored-by: Alabhya <singhalabhya32@gmail.com> Co-authored-by: Arteev Raina <arteevraina@gmail.com>
This aligns better with the documentation for this new API, and doesn't imply that apps need to be upgraded.
cc: @chunhtai