packages icon indicating copy to clipboard operation
packages copied to clipboard

[adaptive_scaffold] New and Improved Helper Widgets & Updated Examples and Their Tests

Open rena987 opened this issue 3 years ago • 7 comments

https://github.com/flutter/packages/pull/2406

Added builders and logic to the existing merged pull request above and updated examples and their respective tests.

rena987 avatar Aug 10 '22 20:08 rena987

You will also want to run flutter pub global activate flutter_plugin_tools && flutter pub global run flutter_plugin_tools format once you've fixed the lints to catch the remaining formatting/tab issues.

youssef-attia avatar Aug 10 '22 21:08 youssef-attia

Also you will want to remove the <?code-excerpt ...> s from the README.md and remove the ! from before the for the images.

youssef-attia avatar Aug 10 '22 22:08 youssef-attia

You will want to add this file https://github.com/flutter/plugins/blob/main/packages/local_auth/local_auth/example/build.excerpt.yaml to the example directory. The problem with code-excerpts should be resolved by this document https://github.com/flutter/flutter/wiki/Contributing-to-Plugins-and-Packages#readme-code.

Edit: Actually, this takes a little more effort so I'll chat with you once you're ready.

youssef-attia avatar Aug 11 '22 16:08 youssef-attia

You will also want to add a link to this issue: https://github.com/flutter/flutter/issues/109278 at the bottom of the adaptive_layout_test in the comment regarding why some tests are currently disabled.

youssef-attia avatar Aug 11 '22 16:08 youssef-attia

You will want to add:

dev_dependencies:
  build_runner: ^2.1.10

to your example pubsec.yaml

youssef-attia avatar Aug 11 '22 17:08 youssef-attia

Not sure if this could also be an issue but pointing it out just in case. Do we have permission to use the title/text of the sample emails in the demo?

youssef-attia avatar Aug 12 '22 16:08 youssef-attia

Not sure if this could also be an issue but pointing it out just in case. Do we have permission to use the title/text of the sample emails in the demo?

Depends on the source. If it came from an internal source, it's fine. If it was copied from somewhere else, then we should replace the text with something else.

If you just need filler text, you can generate it here: https://www.lipsum.com/

gspencergoog avatar Aug 12 '22 16:08 gspencergoog

What's the status of this PR? Is it waiting for updates, or for another round of reviews?

stuartmorgan-g avatar Sep 06 '22 20:09 stuartmorgan-g

Sorry, this is no longer relevant. The content was merged in another PR.

gspencergoog avatar Sep 06 '22 22:09 gspencergoog