Skip to content

Conversation

@Hixie
Copy link
Contributor

@Hixie Hixie commented Feb 4, 2017

Depends on #7872

This patch as written shows a bug whereby the grid doesn't show the first row of items until they all fit on the screen.

config.products
.where((Product product) => product != config.order.product)
.map((Product product) {
return new RepaintBoundary(
Copy link
Contributor

Choose a reason for hiding this comment

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

This repaint boundary isn't needed. SliverGrid adds one itself.

product.imageAsset,
fit: ImageFit.contain,
)
)
Copy link
Contributor

Choose a reason for hiding this comment

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

,,,,,,,,,,,

@abarth
Copy link
Contributor

abarth commented Feb 4, 2017

I fixed the issue with grid and added some tests in #7881

Closing in favor of #7881

@abarth abarth closed this Feb 4, 2017
@Hixie Hixie deleted the customscrollview branch February 5, 2017 21:04
@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