Skip to content

OnScale listener #24

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

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

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.

3 participants