Skip to content

Conversation

@kenzieschmoll
Copy link
Member

No description provided.

Copy link
Contributor

@albertusdev albertusdev left a comment

Choose a reason for hiding this comment

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

LGTM!

import '../../flutter/split.dart';
import '../../globals.dart';
import '../../service_extensions.dart' as extensions;
import '../../ui/fake_flutter/_real_flutter.dart';
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: remove this? I think you should be able to remove this

Copy link
Contributor

Choose a reason for hiding this comment

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

import 'package:flutter/foundation.dart'; would be the correct import

Copy link
Member Author

Choose a reason for hiding this comment

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

good catch! replaced with the flutter/foundation import

Copy link
Contributor

@jacob314 jacob314 left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@albertusdev albertusdev left a comment

Choose a reason for hiding this comment

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

image

@kenzieschmoll kenzieschmoll merged commit 6204332 into flutter:master Dec 6, 2019
@kenzieschmoll kenzieschmoll deleted the disableLayoutExplorerOnWeb branch December 6, 2019 18:33
DaveShuckerow pushed a commit to DaveShuckerow/devtools that referenced this pull request Dec 10, 2019
albertusdev pushed a commit to albertusdev/devtools that referenced this pull request Dec 10, 2019
albertusdev pushed a commit that referenced this pull request Dec 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants