Move section "update your app to include the article view" earlier#13199
Move section "update your app to include the article view" earlier#13199molulan wants to merge 1 commit intoflutter:mainfrom
Conversation
Move the section up to just after creating the ArticleView widget. This way it's possible to follow the changes visually in the app. Which is more motivating when following along the tutorial
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request refactors the listenable-builder.md tutorial by moving the 'Update your app to include the article view' section to an earlier point. This change improves the tutorial's flow by allowing the user to see visual UI updates sooner, which can be more engaging. The moved content and code snippets are correct in their new context. The change is well-reasoned and improves the learning experience.
|
Visit the preview URL for this PR (updated for commit 53abc90): |
2 participants
Move the section up to just after creating the ArticleView widget. This way it's possible to follow the changes visually in the app, Which is more motivating when following along the tutorial.