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

CollectionView feedback - Eliezer Sant Ana #25

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

er7santana
Copy link

@er7santana er7santana commented Apr 24, 2019

Screenshots

Android

ezgif com-resize

iOS

ezgif com-gif-maker

What went well?

It was very easy and smooth the implementation. The new layout options such as HorizontalList, GridList are amazing and are going to substitute lots of "extra work" that we need to do today.

What didn't go well?

Actually, the only tiny issue that I faced in Android platform was that I was using a Grid with 3 rows, 1 for an Image, and 2 for Labels.
Using ListView, I was able to set the rows heights as "Auto" for the labels and it worked as expected, but with CollectionView I had to specify the Height of the row with a real value, otherwise the rows with the Labels were not visible in Android devices, the first row with the image occupied all the ItemTemplate space.

How is the performance?

I didn't have any problems with performance. I think it's just fine.

Missing or Desired Things:

I think it would be interesting to have a PullToRefresh feature just like ListView.

@pauldipietro
Copy link
Owner

@er7santana Thank you for your submission and feedback. Pull-to-refresh is currently planned in the spec.

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.

2 participants