Spotify app Home Screen using CollectionView#26
Open
luven1992 wants to merge 1 commit intopauldipietro:masterfrom
Open
Spotify app Home Screen using CollectionView#26luven1992 wants to merge 1 commit intopauldipietro:masterfrom
luven1992 wants to merge 1 commit intopauldipietro:masterfrom
Conversation
Owner
|
@luven1992 Thank you for the submission and feedback. Some of these features like refreshing, grouping, and the header/footer are planned. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Screenshots
Original

iOS Screenshot

Android Screenshot

What went well?
CollectionView was quite similar to ListView. I was able to implement item layout easily using CollectionView's ItemLayout property.
What didn't go well?
The code to create spacing between items was a bit tricky.
How is the performance?
I'm quite impressed by the performance so far. For this example I managed to put in three CollectionViews into one ScrollView to follow Spotify's app layout.
Missing or Desired Things:
It would have been better if CollectionView had properties from ListView such as IsRefreshing, CachingStrategy, IsPullToRefeshEnabled, Header, Footer, Grouping, etc...