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

My Ifood - CollectionViewChallenge #29

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

Conversation

AmorimRob
Copy link

@AmorimRob AmorimRob commented Apr 25, 2019

I reproduced a very popular app of delivery food in Brazil, the IFood app. It's not my app and do not sure there are a ListView version. The IFood app are available on the stores to download.

Screenshots

My Gif

mygif2

-> Original

WhatsApp Image 2019-04-25 at 17 00 48

-> My ScreenShot

myIfood1

-> Original

WhatsApp Image 2019-04-25 at 17 00 48 (1)

-> MyScreenshot

MyIfood2

What went well?

It was easy to build it with CollectionView. I especially liked the ease of not having to work with cells and with ScrollView. The CollectionView is simple to understand, but unfortunately does not yet have some features that would need in this app to hide the scroll bar.

What didn't go well?

Android
I developed the app running on iOS when I was running on Android, for some reason it did not work correctly some CollectionViews did not render as expected.

Performance
I felt a small delay to start scrolling the items horizontally.

Loading
While loading some images from web, the items become disorganized and overlapping.

How is the performance?

The performance was good on iOS, on Android it was a bit slower, the small delay to start the scroll.

Missing or Desired Things:

  • Hide the scroll bar
  • PullToRefresh

@AmorimRob AmorimRob changed the title My Ifood - CollectionViewChallange My Ifood - CollectionViewChallenge Apr 25, 2019
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