Skip to content
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

Layout example: Card and Stack #2314

Merged
merged 1 commit into from
Jan 30, 2019
Merged

Layout example: Card and Stack #2314

merged 1 commit into from
Jan 30, 2019

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Jan 30, 2019

Contributes to #2233, #2279

@googlebot googlebot added the cla: yes Contributor has signed the Contributor License Agreement label Jan 30, 2019
@chalin chalin merged commit eefbd4b into master Jan 30, 2019
@chalin chalin deleted the chalin-card-and-stack-0130 branch January 30, 2019 17:43
@@ -105,11 +105,11 @@ class MyApp extends StatelessWidget {
children: [
Icon(icon, color: color),
Container(
margin: const EdgeInsets.only(top: 8.0),
margin: const EdgeInsets.only(top: 8),
Copy link

@mshibanami mshibanami Feb 5, 2019

Choose a reason for hiding this comment

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

Hi @chalin, I think this change makes a compile error that says the type is wrong. The same applies to other changes.

Copy link
Contributor

Choose a reason for hiding this comment

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

Hi @mshibanami, this should not give you a compile error anymore. Which version of Flutter are you using?

Choose a reason for hiding this comment

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

Oops, sorry. As you said, this problem was caused by the old version of flutter/dart. It's fixed after upgrading. Thanks @filiph!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Contributor has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants