fix(android): keep onPostScroll under detekt ReturnCount limit

The dismissed-latch added a third early return to onPostScroll — detekt's ReturnCount ceiling is 2 per the project rule. Fold the NestedScrollSource.UserInput guard into the existing if/else if/else chain that branches on the drag direction. Behavior is identical; the source check just becomes the first arm of the expression rather than an early bail.
2026-06-02 08:04:21 -04:00
parent 6ac36dd334
commit 1004b61159
1 changed files with 6 additions and 3 deletions
@@ -205,10 +205,13 @@ private fun rememberDragDismissConnection(
        ): Offset {
            // After dismissal, eat all remaining drag so the
            // scrollable doesn't paint over-scroll deltas during the
-            // pop transition.
+            // pop transition. Source guard folded into the `else if`
+            // chain so the function stays under detekt's ReturnCount
+            // ceiling of 2.
            if (dismissed) return available
-            if (source != NestedScrollSource.UserInput) return Offset.Zero
-            return if (available.y > 0f) {
+            return if (source != NestedScrollSource.UserInput) {
+                Offset.Zero
+            } else if (available.y > 0f) {
                accumulated += available.y
                if (accumulated >= thresholdPx) {
                    dismissed = true