search

ashfurrow / ARCollectionViewMasonryLayout

MIT License
185 stars 16 forks source link

mainItemMargin #5

Closed dkhamsing closed 10 years ago

dkhamsing commented 10 years ago

Is this line correct? Always returning height https://github.com/AshFurrow/ARCollectionViewMasonryLayout/blob/master/ARCollectionViewTest/ARCollectionViewMasonryLayout.m#L367

ashfurrow commented 10 years ago

Hmm. Good catch. @orta – any recommendation here besides just replacing that line?

orta commented 10 years ago

Agreed, it doesn't look correct to me. Especially considering the alternateItemMargin switches between the two. It's likely that this just never cropped up in the artsy app.