Skip to content

OnScale listener - #24

Open
Macarse wants to merge 1 commit into
sephiroth74:masterfrom
Macarse:onScaleListener
Open

OnScale listener#24
Macarse wants to merge 1 commit into
sephiroth74:masterfrom
Macarse:onScaleListener

Conversation

@Macarse

@Macarse Macarse commented Mar 18, 2013

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Owner

why do we need that listener?

@Macarse

Macarse commented Mar 18, 2013

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
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

Macarse commented Mar 18, 2013

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown

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