diff options
author | ByteHamster <ByteHamster@users.noreply.github.com> | 2021-10-05 20:53:45 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-10-05 20:53:45 +0200 |
commit | bb36cfef63fc265e6f8fc9ab14775b39a76a3ec0 (patch) | |
tree | 74fdd2ed102dac08135eb52a52aa39a1cf1da597 /app | |
parent | 3f869c7d38726bc5cf72764c2e7dca4db59931c5 (diff) | |
parent | 96fe0917e055aa691f8f3a1b3a52a63ce7e29cbf (diff) | |
download | AntennaPod-bb36cfef63fc265e6f8fc9ab14775b39a76a3ec0.zip |
Merge pull request #5445 from ByteHamster/fix-context-menu
Fix wrong fragment receiving context menu event
Diffstat (limited to 'app')
-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) { |