From 006343460eea8e193d8b25b74d56f11b0b904ebe Mon Sep 17 00:00:00 2001 From: Alan Evans Date: Thu, 5 Dec 2019 09:10:16 -0500 Subject: [PATCH] Fix long press on image thumbnail in detail view. Allow long press to select inside select mode. --- .../securesms/mediaoverview/MediaGalleryAllAdapter.java | 2 +- .../securesms/mediaoverview/MediaOverviewPageFragment.java | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/org/thoughtcrime/securesms/mediaoverview/MediaGalleryAllAdapter.java b/src/org/thoughtcrime/securesms/mediaoverview/MediaGalleryAllAdapter.java index 0fe47921698..45fdd431cca 100644 --- 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 index 042f61e28d3..0e669d48f1f 100644 --- 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(); } }