diff options
author | ByteHamster <info@bytehamster.com> | 2021-10-05 00:22:04 +0200 |
---|---|---|
committer | ByteHamster <info@bytehamster.com> | 2021-10-05 00:22:04 +0200 |
commit | 96fe0917e055aa691f8f3a1b3a52a63ce7e29cbf (patch) | |
tree | 394bc086abc742f8c4c742a9572dc2c372234810 /app/src/main/java | |
parent | 160401ad13778e9f611fde864910e55339a3f921 (diff) | |
download | AntennaPod-96fe0917e055aa691f8f3a1b3a52a63ce7e29cbf.zip |
Fix wrong fragment receiving context menu event
Diffstat (limited to 'app/src/main/java')
-rw-r--r-- | app/src/main/java/de/danoeh/antennapod/fragment/EpisodesListFragment.java | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/app/src/main/java/de/danoeh/antennapod/fragment/EpisodesListFragment.java b/app/src/main/java/de/danoeh/antennapod/fragment/EpisodesListFragment.java index 6d63e4ab2..7ea76bb8d 100644 --- a/app/src/main/java/de/danoeh/antennapod/fragment/EpisodesListFragment.java +++ b/app/src/main/java/de/danoeh/antennapod/fragment/EpisodesListFragment.java @@ -168,10 +168,9 @@ public abstract class EpisodesListFragment extends Fragment { @Override public boolean onContextItemSelected(@NonNull MenuItem item) { Log.d(TAG, "onContextItemSelected() called with: " + "item = [" + item + "]"); - if (!getUserVisibleHint()) { - return false; - } - if (!isVisible()) { + if (!getUserVisibleHint() || !isVisible() || !isMenuVisible()) { + // The method is called on all fragments in a ViewPager, so this needs to be ignored in invisible ones. + // Apparently, none of the visibility check method works reliably on its own, so we just use all. return false; } if (item.getItemId() == R.id.share_item) { |