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

Migrate to FlatList #28

Closed
brh55 opened this issue Jun 21, 2017 · 6 comments
Closed

Migrate to FlatList #28

brh55 opened this issue Jun 21, 2017 · 6 comments

Comments

@brh55
Copy link
Owner

brh55 commented Jun 21, 2017

As title explains, I will try to get to this when I have time (3 - 4 weeks), but open for grabs as well.

@fxfactorial
Copy link

Would need to bump up a min requirement, which I think is fine to do in this fast moving environment.

@brh55
Copy link
Owner Author

brh55 commented Jun 21, 2017

Shouldn't be too bad. 🤞

@srameshr
Copy link
Contributor

Is this done? ListView is daunting.

@brh55
Copy link
Owner Author

brh55 commented Jul 27, 2017

@srameshr Per docs masonry layouts (different heights) are not supported, so if I went with a flatlist + numColumns approach, we would not get a desired masonry effect. Judging by the rendered outcomes as it appears a <view> wraps each element in a horizontal row, and proceeds to stack additional wrapped <view>s.

@srameshr
Copy link
Contributor

srameshr commented Jul 27, 2017

@brh55 Agreed. With the current ListView, will ordering and infinite scroll be implemented?

@brh55
Copy link
Owner Author

brh55 commented Aug 16, 2017

Tied to #33 . Currently in review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants