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

Fixed bug that was causing the didSelectCell call to not be called when it should have been #35

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

helzapps
Copy link

... rect through CGRectIntegral because the rect created and passed into the indexToRectMap sometimes would not match the actual rect created for the view when the NSStringFromCGRect call was subsequently made in the - (BOOL)gestureRecognizer:(UIGestureRecognizer *)gestureRecognizer shouldReceiveTouch:(UITouch *)touch and - (void)didSelectView:(UITapGestureRecognizer *)gestureRecognizer due to rounding that sometimes occurred. This ultimately ended up in the - (void)collectionView:(PSCollectionView *)collectionView didSelectCell:(PSCollectionViewCell *)cell atIndex:(NSInteger)index not being called when a cell actually had been tapped.

…ted rect through CGRectIntegral because the rect created and passed into the indexToRectMap sometimes would not match the actual rect created for the view when the NSStringFromCGRect call was subsequently made in the - (BOOL)gestureRecognizer:(UIGestureRecognizer *)gestureRecognizer shouldReceiveTouch:(UITouch *)touch and - (void)didSelectView:(UITapGestureRecognizer *)gestureRecognizer due to rounding that sometimes occurred. This ultimately ended up in the - (void)collectionView:(PSCollectionView *)collectionView didSelectCell:(PSCollectionViewCell *)cell atIndex:(NSInteger)index not being called when a cell actually had been tapped.
@hyq9527
Copy link

hyq9527 commented Jan 6, 2015

Great!

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.

2 participants