Fix for item offset with cellspacing

3ph · May 22, 2019 · a38bf18 · a38bf18
1 parent f3bb07c
commit a38bf18
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/CollectionPickerView/Classes/CollectionPickerView.swift b/CollectionPickerView/Classes/CollectionPickerView.swift
@@ -214,7 +214,7 @@ public class CollectionPickerView: UIView {
                 collectionView,
                 layout: collectionView.collectionViewLayout,
                 sizeForItemAt: indexPath)
-            offset += (isHorizontal ? cellSize.width : cellSize.height)
+            offset += (isHorizontal ? cellSize.width : cellSize.height) + cellSpacing
         }
 
         let selectedIndexPath = IndexPath(item: index, section: 0)
@@ -423,7 +423,7 @@ extension CollectionPickerView: UICollectionViewDelegateFlowLayout {
     }
 
     public func collectionView(_ collectionView: UICollectionView, layout collectionViewLayout: UICollectionViewLayout, minimumInteritemSpacingForSectionAt section: Int) -> CGFloat {
-        return cellSpacing
+        return 0
     }
 
     public func collectionView(_ collectionView: UICollectionView, layout collectionViewLayout: UICollectionViewLayout, minimumLineSpacingForSectionAt section: Int) -> CGFloat {

diff --git a/CollectionPickerView/Classes/CollectionPickerViewFlowLayout.swift b/CollectionPickerView/Classes/CollectionPickerViewFlowLayout.swift
@@ -138,8 +138,8 @@ public class CollectionPickerViewFlowLayout: UICollectionViewFlowLayout {
     var mostRecentOffset : CGPoint = CGPoint()
 
     public override func targetContentOffset(forProposedContentOffset proposedContentOffset: CGPoint, withScrollingVelocity velocity: CGPoint) -> CGPoint {
-        _ = scrollDirection == .horizontal
-
+//        _ = scrollDirection == .horizontal
+//        
 //        if snapToCenter == false {
 //            return super.targetContentOffset(forProposedContentOffset: proposedContentOffset, withScrollingVelocity: velocity)
 //        }