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

Visual Challenge - bunq - bank of the free #34

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

Conversation

imuller
Copy link

@imuller imuller commented Apr 26, 2019

Before
Before

After
After

Demo video!
https://raw.githubusercontent.com/imuller/CollectionViewChallenge/79243b70d284e16778b3b8a5f864adff48babba9/CollectionViewChallenge/bunq_after.mov

Experience/General Feedback:
Awesome new feature to implement. Had some issues with the padding of the content on iOS. The CollectionView gets behind the navigation bar. Easily add content on top and bottom of the CollectionView was not clear to me, i added it inside a ScrollView with a StackLayout.

Performance Feedback:
No difference with the ListView. Tested it on iOS device.

Missing or Desired Things:

  • Better Xamarin Previewer support, faster reloading
  • Support for iOS Large Titles :)

@davidortinau
Copy link
Contributor

@imuller here is how you should be able to enable large titles on iOS: https://docs.microsoft.com/en-us/xamarin/xamarin-forms/platform/ios/page-large-title

@imuller
Copy link
Author

imuller commented Apr 28, 2019

@imuller here is how you should be able to enable large titles on iOS: https://docs.microsoft.com/en-us/xamarin/xamarin-forms/platform/ios/page-large-title

How should i use this with Xamarin.Forms Shell? Can i still have a NavigationPage in Shell? Couldn't find it on the docs..

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