Skip to content

2d transforms UX improvements#30932

Merged
justinmc merged 5 commits intoflutter:masterfrom
justinmc:2d-transforms-explanation
Apr 15, 2019
Merged

2d transforms UX improvements#30932
justinmc merged 5 commits intoflutter:masterfrom
justinmc:2d-transforms-explanation

Conversation

@justinmc
Copy link
Contributor

This just makes the 2D Transformations demo in the Flutter Gallery a bit more user friendly. I added a title in the app bar and a first-time dialog with a description.

Screen Shot 2019-04-11 at 12 38 58 PM

@justinmc justinmc requested a review from HansMuller April 11, 2019 20:45
@justinmc justinmc self-assigned this Apr 11, 2019
@goderbauer goderbauer added framework flutter/packages/flutter repository. See also f: labels. customer: gallery Relating to flutter/gallery repository. Please transfer non-framework issues there. labels Apr 12, 2019
Copy link
Contributor

@Piinks Piinks left a comment

Choose a reason for hiding this comment

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

LGTM once the analyzer is happy!

@override
Widget build (BuildContext context) {
if (!_seenInstructionDialog) {
Future<void>.delayed(Duration(seconds: 1), () {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Future<void>.delayed(Duration(seconds: 1), () {
Future<void>.delayed(const Duration(seconds: 1), () {

Makes analyzer happy. 😊

Copy link
Contributor Author

Choose a reason for hiding this comment

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

On it. Thanks for the review!

@justinmc justinmc merged commit ec7f3f0 into flutter:master Apr 15, 2019
@justinmc justinmc deleted the 2d-transforms-explanation branch April 15, 2019 22:11
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

customer: gallery Relating to flutter/gallery repository. Please transfer non-framework issues there. framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants