From ee3dcd38a689118360b0f26b0e23029bbcb0c29c Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Mon, 11 May 2026 14:16:37 +0000
Subject: [PATCH] Fix review issues: NO_POSITION and setCurrentItem animate
 param

Agent-Logs-Url: https://github.com/ArcaneChat/android/sessions/de374ae1-3e38-498d-8896-624a25191eb3

Co-authored-by: adbenitez <24558636+adbenitez@users.noreply.github.com>
---
 .../java/org/thoughtcrime/securesms/MediaPreviewActivity.java   | 2 +-
 src/main/java/org/thoughtcrime/securesms/qr/QrActivity.java     | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/thoughtcrime/securesms/MediaPreviewActivity.java b/src/main/java/org/thoughtcrime/securesms/MediaPreviewActivity.java
index 885a3770e..e9d6a0866 100644
--- a/src/main/java/org/thoughtcrime/securesms/MediaPreviewActivity.java
+++ b/src/main/java/org/thoughtcrime/securesms/MediaPreviewActivity.java
@@ -674,7 +674,7 @@ public class MediaPreviewActivity extends PassphraseRequiredActionBarActivity
     public void onViewRecycled(@NonNull MediaViewHolder holder) {
       super.onViewRecycled(holder);
       int pos = holder.getBindingAdapterPosition();
-      if (pos != RecyclerView.NO_ID) {
+      if (pos != RecyclerView.NO_POSITION) {
         mediaViews.remove(pos);
       }
       holder.mediaView.cleanup();
diff --git a/src/main/java/org/thoughtcrime/securesms/qr/QrActivity.java b/src/main/java/org/thoughtcrime/securesms/qr/QrActivity.java
index 563a3a8fb..a8d0ff36c 100644
--- a/src/main/java/org/thoughtcrime/securesms/qr/QrActivity.java
+++ b/src/main/java/org/thoughtcrime/securesms/qr/QrActivity.java
@@ -230,7 +230,7 @@ public class QrActivity extends BaseActionBarActivity implements View.OnClickLis
 
   @Override
   public void onClick(View v) {
-    viewPager.setCurrentItem(TAB_SCAN);
+    viewPager.setCurrentItem(TAB_SCAN, true);
   }
 
   private class ProfilePagerAdapter extends FragmentStateAdapter {