Add snapToInterval support for Android ScrollView

facebook · Aug 29, 2017 · 0d4ed33 · 0d4ed33
1 parent b11656a
commit 0d4ed33
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 3 deletions.
diff --git a/Libraries/Components/ScrollView/ScrollView.js b/Libraries/Components/ScrollView/ScrollView.js
@@ -335,8 +335,6 @@ const ScrollView = createReactClass({
      * that have lengths smaller than the scroll view. Typically used in
      * combination with `snapToAlignment` and `decelerationRate="fast"`.
      * Overrides less configurable `pagingEnabled` prop.
-     *
-     * @platform ios
      */
     snapToInterval: PropTypes.number,
     /**

diff --git a/ReactAndroid/src/main/java/com/facebook/react/views/scroll/ReactHorizontalScrollView.java b/ReactAndroid/src/main/java/com/facebook/react/views/scroll/ReactHorizontalScrollView.java
@@ -22,6 +22,7 @@
 import android.widget.HorizontalScrollView;
 import com.facebook.infer.annotation.Assertions;
 import com.facebook.react.uimanager.MeasureSpecAssertions;
+import com.facebook.react.uimanager.PixelUtil;
 import com.facebook.react.uimanager.ReactClippingViewGroup;
 import com.facebook.react.uimanager.ReactClippingViewGroupHelper;
 import com.facebook.react.uimanager.events.NativeGestureUtil;
@@ -41,6 +42,7 @@ public class ReactHorizontalScrollView extends HorizontalScrollView implements
   private @Nullable Rect mClippingRect;
   private boolean mDragging;
   private boolean mPagingEnabled = false;
+  private double mSnapToInterval;
   private @Nullable Runnable mPostTouchRunnable;
   private boolean mRemoveClippedSubviews;
   private boolean mScrollEnabled = true;
@@ -94,6 +96,10 @@ public void flashScrollIndicators() {
     awakenScrollBars();
   }
 
+  public void setSnapToInterval(double snapToInterval) {
+    mSnapToInterval = snapToInterval;
+  }
+
   @Override
   protected void onMeasure(int widthMeasureSpec, int heightMeasureSpec) {
     MeasureSpecAssertions.assertExplicitMeasureSpec(widthMeasureSpec, heightMeasureSpec);
@@ -235,6 +241,13 @@ private void disableFpsListener() {
     }
   }
 
+  private int getPageWidth() {
+    if(mSnapToInterval != 0) {
+      return (int) (PixelUtil.toPixelFromDIP(mSnapToInterval) + 0.5);
+    }
+    return getWidth();
+  }
+
   private boolean isScrollPerfLoggingEnabled() {
     return mFpsListener != null && mScrollPerfTag != null && !mScrollPerfTag.isEmpty();
   }
@@ -312,7 +325,7 @@ public void run() {
    * scrolling.
    */
   private void smoothScrollToPage(int velocity) {
-    int width = getWidth();
+    int width = getPageWidth();
     int currentX = getScrollX();
     // TODO (t11123799) - Should we do anything beyond linear accounting of the velocity
     int predictedX = currentX + velocity;

diff --git a/...droid/src/main/java/com/facebook/react/views/scroll/ReactHorizontalScrollViewManager.java b/...droid/src/main/java/com/facebook/react/views/scroll/ReactHorizontalScrollViewManager.java
@@ -105,6 +105,11 @@ public void setPagingEnabled(ReactHorizontalScrollView view, boolean pagingEnabl
     view.setPagingEnabled(pagingEnabled);
   }
 
+  @ReactProp(name = "snapToInterval")
+  public void setSnapToInterval(ReactHorizontalScrollView view, double snapToInterval) {
+    view.setSnapToInterval(snapToInterval);
+  }
+
   /**
    * Controls overScroll behaviour
    */