updating slideTo... methods

some devices require invalidate to be called immediately following a scroller.startScroll *and* in computeScroll.
moagrius · Sep 1, 2013 · d1e0859 · d1e0859
1 parent 7dd9072
commit d1e0859
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 14 deletions.
diff --git a/src/com/qozix/layouts/ZoomPanLayout.java b/src/com/qozix/layouts/ZoomPanLayout.java
@@ -351,6 +351,7 @@ public void slideToPoint( Point point ) { // TODO:
 		int dx = point.x - startX;
 		int dy = point.y - startY;
 		scroller.startScroll( startX, startY, dx, dy, SLIDE_DURATION );
+		invalidate();  // we're posting invalidate in computeScroll, yet both are required
 	}
 
 	/**
@@ -490,8 +491,8 @@ public void computeScroll() {
 		if ( scroller.computeScrollOffset() ) {
 			Point destination = new Point( scroller.getCurrX(), scroller.getCurrY() );
 			scrollToPoint( destination );
-			postInvalidate();  // should not be necessary but is...
 			dispatchScrollActionNotification();
+			postInvalidate();  // should not be necessary but is...
 		}
 	}
 
@@ -852,4 +853,4 @@ public static interface GestureListener {
 		public void onFlingComplete( Point point );
 	}
 
-}
+}
diff --git a/src/com/qozix/tileview/TileView.java b/src/com/qozix/tileview/TileView.java
@@ -381,9 +381,7 @@ public double unscale( double value ) {
 	 * @param y (double) the relative y position to move to
 	 */
 	public void moveTo( double x, double y ) {
-		Point point = positionManager.translate( x, y );
-		point.x *= getScale();
-		point.y *= getScale();
+		Point point = positionManager.translate( x, y, getScale() );
 		scrollToPoint( point );
 	}
 
@@ -393,9 +391,7 @@ public void moveTo( double x, double y ) {
 	 * @param y (double) the relative y position to move to
 	 */
 	public void moveToAndCenter( double x, double y ) {
-		Point point = positionManager.translate( x, y );
-		point.x *= getScale();
-		point.y *= getScale();
+		Point point = positionManager.translate( x, y, getScale() );
 		scrollToAndCenter( point );
 	}
 
@@ -405,9 +401,7 @@ public void moveToAndCenter( double x, double y ) {
 	 * @param y (double) the relative y position to move to
 	 */
 	public void slideTo( double x, double y ) {
-		Point point = positionManager.translate( x, y );
-		point.x *= getScale();
-		point.y *= getScale();
+		Point point = positionManager.translate( x, y, getScale() );
 		slideToPoint( point );
 	}
 
@@ -417,9 +411,7 @@ public void slideTo( double x, double y ) {
 	 * @param y (double) the relative y position to move to
 	 */
 	public void slideToAndCenter( double x, double y ) {
-		Point point = positionManager.translate( x, y );
-		point.x *= getScale();
-		point.y *= getScale();
+		Point point = positionManager.translate( x, y, getScale() );
 		slideToAndCenter( point );
 	}