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

Support for UICollectionViewDelegateFlowLayout #350

Closed
wants to merge 1 commit into from

Conversation

cd5z
Copy link

@cd5z cd5z commented Dec 25, 2019

add configureCellSize for custom cell size

usage:

        let dataSource = RxCollectionViewSectionedReloadDataSource<<#SectionModelType#>>(
            configureCell: { (datasource, collectionView, indexPath, item) in
                return UICollectionViewCell()
            },
            configureCellSize: { (dataSource, collectionView, indexPath, item) in
                return CGSize(width: 100, height: 100)
            }
        )

add `configureCellSize` for custom cell size

usage:

```
        let dataSource = RxCollectionViewSectionedReloadDataSource<<#SectionModelType#>>(
            configureCell: { (datasource, collectionView, indexPath, item) in
                return UICollectionViewCell()
            },
            configureCellSize: { (dataSource, collectionView, indexPath, item) in
                return CGSize(width: 100, height: 100)
            }
        )
```
@rxswiftcommunity
Copy link

Warnings
⚠️

It looks like code was changed without adding anything to the Changelog. If this is a trivial PR that doesn't need a changelog, add #trivial to the PR title or body.

Generated by 🚫 dangerJS

@freemansion
Copy link

@kzaher any plans to merge this PR in the nearest time?

@lordzsolt
Copy link
Member

@freemansion Shouldn't this go into RxCocoa?

@cd5z cd5z requested a review from kzaher January 19, 2020 07:36
@freak4pc freak4pc closed this Jan 2, 2021
@freak4pc freak4pc deleted the branch RxSwiftCommunity:master January 2, 2021 09:40
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.

4 participants