From e6e077ff40a7a9516bba5984264fe7c82f1b4f7d Mon Sep 17 00:00:00 2001 From: Torsten Grote Date: Tue, 11 Dec 2018 16:09:36 -0200 Subject: [PATCH] [android] enable image shared element transition for API 21+22 There's an Android framework bug (#224270) on these APIs that causes a NPE when the shared element is not visible anymore when returning. Since we know restore the list position, the shared element should be visible and thus not produce NPEs anymore. --- .../android/conversation/ConversationActivity.java | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/briar-android/src/main/java/org/briarproject/briar/android/conversation/ConversationActivity.java b/briar-android/src/main/java/org/briarproject/briar/android/conversation/ConversationActivity.java index 6f0a9bd1d..7d8d28f86 100644 --- a/briar-android/src/main/java/org/briarproject/briar/android/conversation/ConversationActivity.java +++ b/briar-android/src/main/java/org/briarproject/briar/android/conversation/ConversationActivity.java @@ -844,15 +844,11 @@ public class ConversationActivity extends BriarActivity i.putExtra(ATTACHMENT_POSITION, attachments.indexOf(item)); i.putExtra(NAME, name); i.putExtra(DATE, messageItem.getTime()); - if (SDK_INT >= 23) { - String transitionName = item.getTransitionName(); - ActivityOptionsCompat options = - makeSceneTransitionAnimation(this, view, transitionName); - ActivityCompat.startActivity(this, i, options.toBundle()); - } else { - // work-around for android bug #224270 - startActivity(i); - } + // restoring list position should not trigger android bug #224270 + String transitionName = item.getTransitionName(); + ActivityOptionsCompat options = + makeSceneTransitionAnimation(this, view, transitionName); + ActivityCompat.startActivity(this, i, options.toBundle()); } @DatabaseExecutor