Skip to content
This repository was archived by the owner on Jun 17, 2023. It is now read-only.

Commit 833a71a

Browse files
author
Amir Khorsandi
committed
Fix changing current page on animated page change
1 parent 83dacfa commit 833a71a

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

Lib/CollectionViewPagingLayout.swift

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -171,10 +171,10 @@ public class CollectionViewPagingLayout: UICollectionViewLayout {
171171

172172
// MARK: Private functions
173173

174-
private func updateCurrentPageIfNeeded(basedOn contentOffset: CGPoint? = nil) {
174+
private func updateCurrentPageIfNeeded() {
175175
var currentPage: Int = 0
176176
if let collectionView = collectionView {
177-
let contentOffset = contentOffset ?? collectionView.contentOffset
177+
let contentOffset = collectionView.contentOffset
178178
let pageSize = scrollDirection == .horizontal ? collectionView.frame.width : collectionView.frame.height
179179
let offset = scrollDirection == .horizontal ?
180180
(contentOffset.x + collectionView.contentInset.left) :
@@ -212,10 +212,7 @@ public class CollectionViewPagingLayout: UICollectionViewLayout {
212212
let contentOffset: CGPoint = scrollDirection == .horizontal ? CGPoint(x: offset, y: 0) : CGPoint(x: 0, y: offset)
213213
CATransaction.begin()
214214
CATransaction.setCompletionBlock { [weak self] in
215-
self?.updateCurrentPageIfNeeded(basedOn: contentOffset)
216-
}
217-
if !animated {
218-
updateCurrentPageIfNeeded(basedOn: contentOffset)
215+
self?.invalidateLayout()
219216
}
220217
collectionView?.setContentOffset(contentOffset, animated: animated)
221218
CATransaction.commit()

0 commit comments

Comments
 (0)