Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 1bf45112 authored by Nikita Dubrovsky's avatar Nikita Dubrovsky Committed by Automerger Merge Worker
Browse files

Merge "Change cursor drag threshold from 30 to 45 degrees from vertical" into... 

Merge "Change cursor drag threshold from 30 to 45 degrees from vertical" into rvc-dev am: f44a60bd am: cf4e0dc4 am: c84cbdb1

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/12052997

Change-Id: I1af31595de6c6558218b3a524f1bbfae09c01356
parents e145c998 c84cbdb1

core/java/android/widget/EditorTouchState.java

+3 −6
Original line number Diff line number Diff line
@@ -174,12 +174,9 @@ public class EditorTouchState { 
                int touchSlop = config.getScaledTouchSlop();

                mMovedEnoughForDrag = distanceSquared > touchSlop * touchSlop;

                if (mMovedEnoughForDrag) {

                    // If the direction of the swipe motion is within 30 degrees of vertical, it is

                    // considered a vertical drag. We don't actually have to compute the angle to

                    // implement the check though. When the angle is exactly 30 degrees from

                    // vertical, 2*deltaX = distance. When the angle is less than 30 degrees from

                    // vertical, 2*deltaX < distance.

                    mIsDragCloseToVertical = (4 * deltaXSquared) <= distanceSquared;

                    // If the direction of the swipe motion is within 45 degrees of vertical, it is

                    // considered a vertical drag.

                    mIsDragCloseToVertical = Math.abs(deltaX) <= Math.abs(deltaY);

                }

            }

        } else if (action == MotionEvent.ACTION_CANCEL) {

core/tests/coretests/src/android/widget/EditorCursorDragTest.java

+2 −2
Original line number Diff line number Diff line
@@ -145,7 +145,7 @@ public class EditorCursorDragTest { 
        // Swipe along a diagonal path. This should drag the cursor. Because we snap the finger to

        // the handle as the touch moves downwards (and because we have some slop to avoid jumping

        // across lines), the cursor position will end up higher than the finger position.

        onView(withId(R.id.textview)).perform(dragOnText(text.indexOf("line1"), text.indexOf("3")));

        onView(withId(R.id.textview)).perform(dragOnText(text.indexOf("line1"), text.indexOf("2")));

        onView(withId(R.id.textview)).check(hasInsertionPointerAtIndex(text.indexOf("1")));



        // Swipe right-down along a very steep diagonal path. This should not drag the cursor.
@@ -181,7 +181,7 @@ public class EditorCursorDragTest { 
        // Swipe along a diagonal path. This should drag the cursor. Because we snap the finger to

        // the handle as the touch moves downwards (and because we have some slop to avoid jumping

        // across lines), the cursor position will end up higher than the finger position.

        onView(withId(R.id.textview)).perform(dragOnText(text.indexOf("line1"), text.indexOf("3")));

        onView(withId(R.id.textview)).perform(dragOnText(text.indexOf("line1"), text.indexOf("2")));

        onView(withId(R.id.textview)).check(hasInsertionPointerAtIndex(text.indexOf("1")));



        // Swipe right-down along a very steep diagonal path. This should not drag the cursor.

core/tests/coretests/src/android/widget/EditorTouchStateTest.java

+2 −2
Original line number Diff line number Diff line
@@ -326,9 +326,9 @@ public class EditorTouchStateTest { 
        mTouchState.update(event1, mConfig);

        assertSingleTap(mTouchState, 0f, 0f, 0, 0);



        // Simulate an ACTION_MOVE event that is > 30 deg from vertical.

        // Simulate an ACTION_MOVE event that is > 45 deg from vertical.

        long event2Time = 1002;

        MotionEvent event2 = moveEvent(event1Time, event2Time, 100f, 173f);

        MotionEvent event2 = moveEvent(event1Time, event2Time, 100f, 90f);

        mTouchState.update(event2, mConfig);

        assertDrag(mTouchState, 0f, 0f, 0, 0, false);