diff options
author | Ebrahim Byagowi <ebrahim@gnu.org> | 2020-04-12 12:04:16 +0430 |
---|---|---|
committer | Ebrahim Byagowi <ebrahim@gnu.org> | 2020-04-21 19:30:17 +0430 |
commit | 430e0d403e5f9c88dc4fc045077cad546220def8 (patch) | |
tree | b224237bf6c71afa8a0f319267ffafea772341fc /app/src/main | |
parent | c88336f17fc3e234319405bb70841ce5237d0626 (diff) | |
download | AntennaPod-430e0d403e5f9c88dc4fc045077cad546220def8.zip |
Fix menu items duplication in episodes and downloads fragment
As ViewPager2 different lifecycle apparently setHasOptionsMenu
needs to be called from onResume() from now on.
Diffstat (limited to 'app/src/main')
3 files changed, 13 insertions, 4 deletions
diff --git a/app/src/main/java/de/danoeh/antennapod/fragment/CompletedDownloadsFragment.java b/app/src/main/java/de/danoeh/antennapod/fragment/CompletedDownloadsFragment.java index a3c07721a..6629d74e2 100644 --- a/app/src/main/java/de/danoeh/antennapod/fragment/CompletedDownloadsFragment.java +++ b/app/src/main/java/de/danoeh/antennapod/fragment/CompletedDownloadsFragment.java @@ -86,11 +86,16 @@ public class CompletedDownloadsFragment extends Fragment { @Override public void onStart() { super.onStart(); - setHasOptionsMenu(true); loadItems(); } @Override + public void onResume() { + super.onResume(); + setHasOptionsMenu(true); + } + + @Override public void onStop() { super.onStop(); if (disposable != null) { diff --git a/app/src/main/java/de/danoeh/antennapod/fragment/DownloadLogFragment.java b/app/src/main/java/de/danoeh/antennapod/fragment/DownloadLogFragment.java index 0305a7d48..a2fea9f24 100644 --- a/app/src/main/java/de/danoeh/antennapod/fragment/DownloadLogFragment.java +++ b/app/src/main/java/de/danoeh/antennapod/fragment/DownloadLogFragment.java @@ -51,6 +51,12 @@ public class DownloadLogFragment extends ListFragment { } @Override + public void onResume() { + super.onResume(); + setHasOptionsMenu(true); + } + + @Override public void onStop() { super.onStop(); if (disposable != null) { @@ -61,8 +67,6 @@ public class DownloadLogFragment extends ListFragment { @Override public void onViewCreated(View view, Bundle savedInstanceState) { super.onViewCreated(view, savedInstanceState); - setHasOptionsMenu(true); - // add padding final ListView lv = getListView(); lv.setClipToPadding(false); 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 189e026a1..4f283a9ab 100644 --- a/app/src/main/java/de/danoeh/antennapod/fragment/EpisodesListFragment.java +++ b/app/src/main/java/de/danoeh/antennapod/fragment/EpisodesListFragment.java @@ -83,7 +83,6 @@ public abstract class EpisodesListFragment extends Fragment { @Override public void onStart() { super.onStart(); - setHasOptionsMenu(true); EventBus.getDefault().register(this); loadItems(); } @@ -91,6 +90,7 @@ public abstract class EpisodesListFragment extends Fragment { @Override public void onResume() { super.onResume(); + setHasOptionsMenu(true); registerForContextMenu(recyclerView); } |