app/src/main/java/de/jrpie/android/launcher/ui/TouchGestureDetector.kt

@@ -87,40 +87,24 @@ class TouchGestureDetector(
     }
 
     private var paths = HashMap<Int, PointerPath>()
-
-    /* Set when
-     *  - the longPressHandler has detected this gesture as a long press
-     *  - the gesture was cancelled by MotionEvent.ACTION_CANCEL
-     * In any case, the current gesture should be ignored by further detection logic.
-     */
-    private var cancelled = false
+    private var gestureIsLongClick = false
 
     private var lastTappedTime = 0L
     private var lastTappedLocation: Vector? = null
 
     fun onTouchEvent(event: MotionEvent) {
-
-        if (event.actionMasked == MotionEvent.ACTION_CANCEL) {
-            synchronized(this@TouchGestureDetector) {
-                cancelled = true
-            }
-        }
-
         val pointerIdToIndex =
             (0..<event.pointerCount).associateBy { event.getPointerId(it) }
 
         if (event.actionMasked == MotionEvent.ACTION_DOWN) {
             synchronized(this@TouchGestureDetector) {
                 paths = HashMap()
-                cancelled = false
+                gestureIsLongClick = false
             }
             longPressHandler.postDelayed({
                 synchronized(this@TouchGestureDetector) {
-                    if (cancelled) {
-                        return@postDelayed
-                    }
                     if (paths.entries.size == 1 && paths.entries.firstOrNull()?.value?.isTap() == true) {
-                        cancelled = true
+                        gestureIsLongClick = true
                         Gesture.LONG_CLICK.invoke(context)
                     }
                 }
@@ -158,7 +142,7 @@ class TouchGestureDetector(
                 // if the long press handler is still running, kill it
                 longPressHandler.removeCallbacksAndMessages(null)
                 // if the gesture was already detected as a long click, there is nothing to do
-                if (cancelled) {
+                if (gestureIsLongClick) {
                     return
                 }
             }

app/src/main/res/layout/list_apps_row.xml

@@ -15,21 +15,18 @@
         app:layout_constraintBottom_toBottomOf="parent"
         app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintTop_toTopOf="parent"
-        tools:src="@mipmap/ic_launcher_round"
         tools:ignore="ContentDescription" />
 
     <TextView
         android:id="@+id/list_apps_row_name"
         android:layout_width="0dp"
         android:layout_height="wrap_content"
-        android:layout_marginStart="20sp"
+        android:layout_marginStart="60sp"
         android:gravity="start"
         android:text=""
         android:textSize="20sp"
-        tools:text="@string/app_name"
         app:layout_constraintBottom_toBottomOf="parent"
-        app:layout_constraintStart_toEndOf="@id/list_apps_row_icon"
-        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintTop_toTopOf="parent" />
 
 </androidx.constraintlayout.widget.ConstraintLayout>

app/src/main/res/layout/list_apps_row_variant_grid.xml

@@ -15,7 +15,6 @@
         app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintTop_toTopOf="parent"
         app:layout_constraintEnd_toEndOf="parent"
-        tools:src="@mipmap/ic_launcher_round"
         tools:ignore="ContentDescription" />
 
     <TextView
@@ -26,7 +25,7 @@
         android:paddingTop="5dp"
         android:text=""
         android:textSize="11sp"
-        tools:text="@string/app_name"
+        tools:text="some app"
         app:layout_constraintTop_toBottomOf="@id/list_apps_row_icon"
         app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintEnd_toEndOf="parent"

app/src/main/res/layout/list_apps_row_variant_text.xml

@@ -32,6 +32,6 @@
         app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintTop_toTopOf="parent"
-        tools:text="@string/app_name" />
+        tools:text="some app" />
 
 </androidx.constraintlayout.widget.ConstraintLayout>