Fix long press on image thumbnail in detail view.

Allow long press to select inside select mode.
signalapp · Dec 5, 2019 · 0063434 · 0063434
1 parent 1cc8634
commit 0063434
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/org/thoughtcrime/securesms/mediaoverview/MediaGalleryAllAdapter.java b/src/org/thoughtcrime/securesms/mediaoverview/MediaGalleryAllAdapter.java
@@ -283,7 +283,6 @@ public void bind(@NonNull Context context, @NonNull MediaDatabase.MediaRecord me
       }
 
       thumbnailView.setImageResource(glideRequests, slide, false, false);
-
       thumbnailView.setOnClickListener(view -> itemClickListener.onMediaClicked(mediaRecord));
       thumbnailView.setOnLongClickListener(view -> onLongClick());
     }
@@ -466,6 +465,7 @@ public void bind(@NonNull Context context, @NonNull MediaDatabase.MediaRecord me
 
       thumbnailView.setImageResource(glideRequests, slide, false, false);
       thumbnailView.setOnClickListener(view -> itemClickListener.onMediaClicked(mediaRecord));
+      thumbnailView.setOnLongClickListener(view -> onLongClick());
     }
 
     @Override

diff --git a/src/org/thoughtcrime/securesms/mediaoverview/MediaOverviewPageFragment.java b/src/org/thoughtcrime/securesms/mediaoverview/MediaOverviewPageFragment.java
@@ -253,10 +253,10 @@ private static void showFileExternally(@NonNull Context context, @NonNull MediaD
 
   @Override
   public void onMediaLongClicked(MediaDatabase.MediaRecord mediaRecord) {
-    if (actionMode == null) {
-      ((MediaGalleryAllAdapter) recyclerView.getAdapter()).toggleSelection(mediaRecord);
-      recyclerView.getAdapter().notifyDataSetChanged();
+    ((MediaGalleryAllAdapter) recyclerView.getAdapter()).toggleSelection(mediaRecord);
+    recyclerView.getAdapter().notifyDataSetChanged();
 
+    if (actionMode == null) {
       enterMultiSelect();
     }
   }