Merge pull request #222 from AllanWang/swipe-enhancement

Swipe enhancement

6 files changed, 47 insertions, 29 deletions

diff --git a/README.md b/README.md
index ae78c87..2c51db6 100644
--- a/README.md
+++ b/README.md
@@ -48,7 +48,9 @@ dependencies {
     implementation "ca.allanwang.kau:core-ui:$KAU"
     
     implementation "ca.allanwang.kau:about:$KAU"
+    implementation "ca.allanwang.kau:adapter:$KAU"
     implementation "ca.allanwang.kau:colorpicker:$KAU"
+    implementation "ca.allanwang.kau:fastadapter:$KAU"
     implementation "ca.allanwang.kau:kpref-activity:$KAU"
     implementation "ca.allanwang.kau:mediapicker:$KAU"
     implementation "ca.allanwang.kau:searchview:$KAU"
diff --git a/core/src/main/kotlin/ca/allanwang/kau/swipe/SwipeBackHelper.kt b/core/src/main/kotlin/ca/allanwang/kau/swipe/SwipeBackHelper.kt
index 0cb6935..a8c71a4 100644
--- a/core/src/main/kotlin/ca/allanwang/kau/swipe/SwipeBackHelper.kt
+++ b/core/src/main/kotlin/ca/allanwang/kau/swipe/SwipeBackHelper.kt
@@ -34,8 +34,12 @@ internal object SwipeBackHelper {
     private operator fun get(activity: Activity): SwipeBackPage? =
         pageStack.firstOrNull { it.activityRef.get() === activity }
 
-    fun onCreate(activity: Activity, builder: SwipeBackContract.() -> Unit = {}) {
-        val page = this[activity] ?: pageStack.push(SwipeBackPage(activity).apply { builder() })
+    fun onCreate(
+        activity: Activity,
+        builder: SwipeBackContract.() -> Unit = {}
+    ): SwipeBackContract {
+        val swipeBackPage = SwipeBackPage(activity).apply(builder)
+        val page = this[activity] ?: pageStack.push(swipeBackPage)
         val startAnimation: Int = when (page.edgeFlag) {
             SWIPE_EDGE_LEFT -> R.anim.kau_slide_in_right
             SWIPE_EDGE_RIGHT -> R.anim.kau_slide_in_left
@@ -45,6 +49,7 @@ internal object SwipeBackHelper {
         activity.overridePendingTransition(startAnimation, 0)
         page.onPostCreate()
         KL.v { "KauSwipe onCreate ${activity.localClassName}" }
+        return swipeBackPage
     }
 
     fun onDestroy(activity: Activity) {
@@ -73,13 +78,17 @@ internal object SwipeBackHelper {
  *
  * Lastly, don't forget to call [kauSwipeOnDestroy] as well when the activity is destroyed.
  */
-fun Activity.kauSwipeOnCreate(builder: SwipeBackContract.() -> Unit = {}) = SwipeBackHelper.onCreate(this, builder)
+fun Activity.kauSwipeOnCreate(builder: SwipeBackContract.() -> Unit = {}): SwipeBackContract =
+    SwipeBackHelper.onCreate(this, builder)
 
 /**
  * Deprecated as onPostCreate seems unreliable.
  * We will instead initialize everything during [kauSwipeOnCreate]
  */
-@Deprecated(level = DeprecationLevel.WARNING, message = "All functionality has been moved to kauSwipeOnCreate")
+@Deprecated(
+    level = DeprecationLevel.WARNING,
+    message = "All functionality has been moved to kauSwipeOnCreate"
+)
 fun Activity.kauSwipeOnPostCreate() {
 }
 
diff --git a/core/src/main/kotlin/ca/allanwang/kau/swipe/SwipeBackLayout.kt b/core/src/main/kotlin/ca/allanwang/kau/swipe/SwipeBackLayout.kt
index 921cd1f..bf6d09d 100644
--- a/core/src/main/kotlin/ca/allanwang/kau/swipe/SwipeBackLayout.kt
+++ b/core/src/main/kotlin/ca/allanwang/kau/swipe/SwipeBackLayout.kt
@@ -30,6 +30,9 @@ import ca.allanwang.kau.utils.adjustAlpha
 import ca.allanwang.kau.utils.navigationBarColor
 import ca.allanwang.kau.utils.statusBarColor
 import java.lang.ref.WeakReference
+import kotlin.math.abs
+import kotlin.math.max
+import kotlin.math.min
 
 /**
  * The layout that handles all the touch events
@@ -165,12 +168,8 @@ internal class SwipeBackLayout @JvmOverloads constructor(
 
     init {
         dragHelper = ViewDragHelper.create(this, ViewDragCallback())
-        val density = resources.displayMetrics.density
-        val minVel = MIN_FLING_VELOCITY * density
         // allow touch from anywhere on the screen
-        edgeSize = Math.max(resources.displayMetrics.widthPixels, resources.displayMetrics.heightPixels)
-        minVelocity = minVel
-//        maxVelocity = 2.5f * minVel
+        edgeSize = max(resources.displayMetrics.widthPixels, resources.displayMetrics.heightPixels)
         edgeFlag = edgeFlag
         sensitivity = 0.3f
         addListener(chromeFadeListener)
@@ -244,19 +243,18 @@ internal class SwipeBackLayout @JvmOverloads constructor(
     }
 
     override fun onInterceptTouchEvent(event: MotionEvent): Boolean {
-        if (!swipeEnabled || disallowIntercept) {
-            return false
-        }
+        if (!swipeEnabled || disallowIntercept) return false
         return try {
             dragHelper.shouldInterceptTouchEvent(event)
         } catch (e: Exception) {
             false
+
         }
     }
 
     @SuppressLint("ClickableViewAccessibility")
     override fun onTouchEvent(event: MotionEvent): Boolean {
-        if (!swipeEnabled) return false
+        if (!swipeEnabled || disallowIntercept) return false
         try {
             dragHelper.processTouchEvent(event)
         } catch (e: Exception) {
@@ -372,10 +370,8 @@ internal class SwipeBackLayout @JvmOverloads constructor(
             val contentView = contentViewRef.get()
                 ?: return KL.e { "KauSwipe cannot change view position as contentView is null; is onPostCreate called?" }
             // make sure that we are using the proper axis
-            scrollPercent = Math.abs(
-                if (horizontal) left.toFloat() / contentView.width
-                else (top.toFloat() / contentView.height)
-            )
+            scrollPercent = abs(if (horizontal) left.toFloat() / contentView.width
+                else (top.toFloat() / contentView.height))
             contentOffset = if (horizontal) left else top
             invalidate()
             if (scrollPercent < scrollThreshold && !isScrollOverValid)
@@ -400,10 +396,10 @@ internal class SwipeBackLayout @JvmOverloads constructor(
             var result = Pair(0, 0)
             if (scrollPercent <= scrollThreshold) {
                 // threshold not met; check velocities
-                if ((edgeFlag == SWIPE_EDGE_LEFT && xvel > MIN_FLING_VELOCITY) ||
-                    (edgeFlag == SWIPE_EDGE_RIGHT && xvel < -MIN_FLING_VELOCITY) ||
-                    (edgeFlag == SWIPE_EDGE_TOP && yvel > MIN_FLING_VELOCITY) ||
-                    (edgeFlag == SWIPE_EDGE_BOTTOM && yvel < -MIN_FLING_VELOCITY)
+                if ((edgeFlag == SWIPE_EDGE_LEFT && xvel > minVelocity) ||
+                    (edgeFlag == SWIPE_EDGE_RIGHT && xvel < -minVelocity) ||
+                    (edgeFlag == SWIPE_EDGE_TOP && yvel > minVelocity) ||
+                    (edgeFlag == SWIPE_EDGE_BOTTOM && yvel < -minVelocity)
                 )
                     result = exitCaptureOffsets(edgeFlag, releasedChild)
             } else {
@@ -428,26 +424,22 @@ internal class SwipeBackLayout @JvmOverloads constructor(
 
         override fun clampViewPositionHorizontal(child: View, left: Int, dx: Int): Int {
             return when (edgeFlag) {
-                SWIPE_EDGE_RIGHT -> Math.min(0, Math.max(left, -child.width))
-                SWIPE_EDGE_LEFT -> Math.min(child.width, Math.max(left, 0))
+                SWIPE_EDGE_RIGHT -> min(0, max(left, -child.width))
+                SWIPE_EDGE_LEFT -> min(child.width, max(left, 0))
                 else -> 0
             }
         }
 
         override fun clampViewPositionVertical(child: View, top: Int, dy: Int): Int {
             return when (edgeFlag) {
-                SWIPE_EDGE_BOTTOM -> Math.min(0, Math.max(top, -child.height))
-                SWIPE_EDGE_TOP -> Math.min(child.height, Math.max(top, 0))
+                SWIPE_EDGE_BOTTOM -> min(0, max(top, -child.height))
+                SWIPE_EDGE_TOP -> min(child.height, max(top, 0))
                 else -> 0
             }
         }
     }
 
     companion object {
-        /**
-         * Minimum velocity that will be detected as a fling
-         */
-        const val MIN_FLING_VELOCITY = 400 // dips per second
 
         const val DEFAULT_SCRIM_COLOR = 0x99000000.toInt()
 
diff --git a/core/src/main/kotlin/ca/allanwang/kau/swipe/SwipeBackPage.kt b/core/src/main/kotlin/ca/allanwang/kau/swipe/SwipeBackPage.kt
index bc5b459..c85625a 100644
--- a/core/src/main/kotlin/ca/allanwang/kau/swipe/SwipeBackPage.kt
+++ b/core/src/main/kotlin/ca/allanwang/kau/swipe/SwipeBackPage.kt
@@ -75,6 +75,7 @@ internal interface SwipeBackContractInternal : SwipeBackContract {
 interface SwipeBackContract {
     /**
      * Toggle main touch intercept
+     * Disabling will remove the swipe back handler
      */
     var swipeEnabled: Boolean
     /**
@@ -91,6 +92,10 @@ interface SwipeBackContract {
      * Set the scrolling threshold for wish a page is deemed closing
      */
     var scrollThreshold: Float
+    /**
+     * Toggle main touch intercept
+     * Disabling will just ignore touch events, but the handler will remain attached
+     */
     var disallowIntercept: Boolean
     /**
      * Set the min velocity of the view drag helper
diff --git a/docs/Migration.md b/docs/Migration.md
index a7162fa..fb29f7b 100644
--- a/docs/Migration.md
+++ b/docs/Migration.md
@@ -2,6 +2,13 @@
 
 Below are some highlights on major refactoring/breaking changes
 
+# v5.1.0
+
+## KPref
+
+You'll notice some errors with kpref imports. You can simply remove (optimize) them.
+Extension functions were converted to interfaces to allow for mocking, so imports are no longer necessary.
+
 # v5.0.0
 
 ## Material Dialog Update
diff --git a/sample/src/main/res/xml/kau_changelog.xml b/sample/src/main/res/xml/kau_changelog.xml
index 5fee3cb..7baecd8 100644
--- a/sample/src/main/res/xml/kau_changelog.xml
+++ b/sample/src/main/res/xml/kau_changelog.xml
@@ -15,6 +15,9 @@
     <item text=":adapter: Make NoAnimatorChange an object; previously a class" />
     <item text=":core: KPref now supports in memory only variants for testing; pass KPrefBuilderInMemory to KPref constructor" />
     <item text=":core: KPref initializer takes in SharedPreferences so user can configure it" />
+    <item text="" />
+    <item text="" />
+    <item text="" />
 
     <version title="v5.0.0" />
     <item text="Update Android SDK to 29 and Kotlin to 1.3.31" />