Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

Generics for CollectionView iOS implementation #7918

Merged
merged 3 commits into from
Oct 10, 2019
Merged

Conversation

hartez
Copy link
Contributor

@hartez hartez commented Oct 10, 2019

Description of Change

These changes move the CollectionView iOS implementation from casting controllers, delegators, and view classes at runtime to using generics.

These changes also split up the delegator classes to make them more readable and maintainable.

API Changes

None

Platforms Affected

  • iOS

Behavioral/Visual Changes

None

Before/After Screenshots

Not applicable

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

@samhouts samhouts added the blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. label Oct 10, 2019
@samhouts samhouts added the approved Has two approvals, no pending reviews, and no changes requested label Oct 10, 2019
@samhouts samhouts merged commit f79080b into 4.3.0 Oct 10, 2019
@samhouts samhouts deleted the cv-ios-generic branch October 10, 2019 23:21
felipebaltazar pushed a commit to felipebaltazar/Xamarin.Forms that referenced this pull request Oct 16, 2019
* Make the ItemsView type generic

* Make controller type generic

* Generic all the things, and split up the delegators

Fix rebase error
felipebaltazar pushed a commit to felipebaltazar/Xamarin.Forms that referenced this pull request Oct 16, 2019
* Make the ItemsView type generic

* Make controller type generic

* Generic all the things, and split up the delegators

Fix rebase error
@samhouts samhouts added this to the 4.3.0 milestone Oct 17, 2019
CliffAgius pushed a commit to CliffAgius/Xamarin.Forms that referenced this pull request Dec 6, 2019
* Make the ItemsView type generic

* Make controller type generic

* Generic all the things, and split up the delegators

Fix rebase error
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/carouselview a/collectionview approved Has two approvals, no pending reviews, and no changes requested blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. p/iOS 🍎
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants