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
Added sample code for scaffold widget [#21136] #21613
Conversation
/// | ||
/// ```dart | ||
/// Scaffold( | ||
/// appBar: AppBar( |
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.
nit: please indent by 2 rather than 4
/// ), | ||
/// ), | ||
/// | ||
/// bottomNavigationBar: BottomAppBar( |
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.
nit: we should be consistent about whether we want blank lines between each entry in this list or not. I'd probably go with no blank lines to be consistent with other sample code, but I can go either way if you think it's important for this example.
This looks fantastic, thanks! Just a few nits if you wouldn't mind updating the code a little for consistency. Let me know if you've had the opportunity to update the PR and then I can land it! |
@Hixie I just added the fixes you recommended to the Scaffold widget sample code. |
Fix one more formatting issue
LGTM! I fixed one more minor nit there on the spacing at the end. Thanks for your contribution! |
Added sample code for Scaffold widget to the repo.