CarouselViewChallenge commit JBR #25
Open
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.
!## Submission description
What went well
Overall, it is a great control, easy to use. The control includes features that saves me writing lot of code (example emptyview).
PeekAreaInsets is a very nice feature that simplifies the code
Flowdirection is a great feature
Stability of the beta release
What didn't go well
Carousel initial loading is going to the first page even with the current item set to 4th element
Missing or desired thingsThe Emptyview idea is great but seems not to work yet:
NOR
<CarouselView.EmptyView>
</CarouselView.EmptyView>
Seem to work
_Refreshcommand property
missing api peakarredocumentation is missing
Anything else
If we missed anything, feel free to mention it here.
Take out a survey for some goodies
Please visit this link to take a survey. If you provide your contact information, we'll send you some swag!