Skip to content

Conversation

@abarth
Copy link
Contributor

@abarth abarth commented Feb 4, 2017

Also, use CustomScrollView in Shrine and fix a bug with one-line grids
not painting properly due to their reporiting zero paintExtent.

@abarth abarth mentioned this pull request Feb 4, 2017
@abarth
Copy link
Contributor Author

abarth commented Feb 4, 2017

@Hixie

Copy link
Contributor

Choose a reason for hiding this comment

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

looks like we got the alignment of the parentheses off a bit here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

Copy link
Contributor

Choose a reason for hiding this comment

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

s/=/:/ unless you have some convention i'm not aware of :-)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ugg. They started out as = and I forgot to convert that one.

Copy link
Contributor

Choose a reason for hiding this comment

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

paings?

@Hixie
Copy link
Contributor

Hixie commented Feb 4, 2017

LGTM

Thanks.

@abarth abarth force-pushed the custom_scroll_view branch from 2634bad to 7246211 Compare February 4, 2017 06:37
Also, use CustomScrollView in Shrine and fix a bug with one-line grids
not painting properly due to their reporiting zero paintExtent.
@abarth abarth force-pushed the custom_scroll_view branch from 7246211 to 99df401 Compare February 4, 2017 06:38
@abarth abarth merged commit ff14f35 into flutter:master Feb 4, 2017
@abarth abarth deleted the custom_scroll_view branch February 4, 2017 06:52
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants