LXReorderableCollectionViewFlowLayout icon indicating copy to clipboard operation
LXReorderableCollectionViewFlowLayout copied to clipboard

Content Insets and Merging Items

Open KevinAtJoor opened this issue 12 years ago • 3 comments

Awesome project going on here! Honestly it was SO useful.

I've extended it just a bit and I want to know if it's worth pushing up to github. Essentially, we wanted to merge photo thumbnails into groups, so I added a few delegate methods following the same type of architecture so it can tell if you're finger is over another cell - if you drop there a delegate method will trigger (collectionView:willmergeFromIndex:ToIndex:) of course checking that the original cell can be moved and that the destination cell is kosher for merging.

Was there any consideration of collection view content insets in the auto scrolling

KevinAtJoor avatar Aug 20 '13 20:08 KevinAtJoor

This sounds awesome! I had a branch quite long ago that attempts to does that but I had trouble getting its architecture right. I'm definitely interested in your suggestion though. How about sending me a PR? :)

lxcid avatar Aug 21 '13 09:08 lxcid

The branch in question is https://github.com/lxcid/LXReorderableCollectionViewFlowLayout/tree/feature/experimental-drag-and-drop-into-item.

Don't follow the architecture over there though as it isn't something I was looking for thats why it haven't been merged all this while.

lxcid avatar Aug 21 '13 09:08 lxcid

+1 on contentInsets question. Setting a contentInset on the top of the collection view (common in iOS7) causes handleScroll to fail, because it assumes minY = 0 rather than minY = self.collectionView.contentInset.top, for example.

jverdi avatar Jan 14 '14 17:01 jverdi