Currently Selected Item cannot be tapped about icarousel

[nicklockwood]

Well spotted. It turns out subviews don't propagate events to their superview in quite the way I thought. This is now fixed (I've switched to using a UITapGestureRecognizer instead of a UIControl to detect the tap).

[andriyadi]

Hi demosthenese,

Thanks for the fix. I confirm that the fix works.

However, related to didTap method. I suggest to change to something like this.

- (void)didTap:(UITapGestureRecognizer *)tapGesture
{
    ...

    if (index == self.currentItemIndex && [delegate respondsToSelector:@selector(carousel:didSelectItemAtIndex:)])
    {
        [delegate carousel:self didSelectItemAtIndex:index];
    }
}

When

centerItemWhenSelected

is enabled, clicking to non-centered item will make it centered and also trigger didSelectItemAtIndex. What I want is that item is only centered, not triggering didSelectItemAtIndex. Or maybe, there should be another flag to enable centering and selecting or just centering.

Thanks.

[nicklockwood]

I think it's more generally useful as it stands because some users may wish to be notified that an item is tapped even when auto-centering is on.

For your purposes you can simply put if (index == carousel.currentItemIndex) {...} in your delegate method to exclude taps on the non-selected items.

[andriyadi]

That's cool. Thanks!