Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

OnScale listener #24

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

Conversation

Macarse
Copy link
Contributor

@Macarse Macarse commented Mar 18, 2013

This is something I need in my app to track when user is scaling the image.
I just place this listener to check that.

@sephiroth74
Copy link
Owner

why do we need that listener?

@Macarse
Copy link
Contributor Author

Macarse commented Mar 18, 2013

My app tracks how the user zooms the image.

My Activity holding the ImageViewTouch has something like:

@Override
public void onDoubleTap() {
    logger.logUserInteraction("", Constants.TrackingValues.TAP_TO_ZOOM);
}

@Override
public void onScale() {
    logger.logUserInteraction("", Constants.TrackingValues.PINCH_TO_ZOOM);
}

@sephiroth74
Copy link
Owner

ok but in that case probably we need also a listener for scroll and fling and maybe we should provide more info about the event

@Macarse
Copy link
Contributor Author

Macarse commented Mar 18, 2013

I was about to send the event but I think it would confuse the library user

@ghost ghost assigned sephiroth74 Mar 19, 2013
@Malachiasz
Copy link

listener is good idea, I personally had to modify code and add it for my purpose which was to turn on/of paging when zooming is active or not

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants