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

Expandable collection view #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

manutdkid77
Copy link

@manutdkid77 manutdkid77 commented Apr 15, 2019

Tried to recreate a collectionview whose items would expand on selected to view hidden content (ex: labels). I had implemented a similar feature for a Listview control on an older client app.

Screenshots:
expandable_collectionview_android

expandable_collectionview_ios

expandable_listview_android

Experience/General Feedback:
As the ListView in xamarin forms made use of ViewCell, we could extend the ViewCell class, intercept the OnTapped event and then use ForceUpdateSize to expand the cell on selection.

Had to try a hacky/bad way of modifying the IsVisible property, removing the item, adding it again to the collection and setting selected item to null. I didn't get much time to see if I could figure something out by creating a custom renderer (maybe by calling notifyDataSetChanged for android).

This is the second time I have used a collectionview as I tried before in the Visual challenge, and the feature I like the most is the horizontal layout ability.

Performance Feedback:
The scroll performance on iOS was much better than on Android. I also tested loading more than 200 items at a time, and there was no lag.

Missing or Desired Things:
Features like PullToRefresh, Grouping, DataTemplate support, but I believe they are already in the works.
Its too early, but would like the feature to reorder items in collectionview in the future.

@pauldipietro
Copy link
Owner

Hi @manutdkid77 -- thanks for your submission. The feedback has been valuable and those missing things like the pull-to-refresh are definitely on the roadmap. We added a survey later on which you may have missed for sending out stickers to participants in the challenge, so if you're interested, please send me an email to [email protected] with where we can send them to. Thanks!

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