Move setHasOptionsMenu to onCreateView

Commit 06c40d3 replaced the deprecated onActivityCreated with onCreate.
Apparently this leads to issues like backgrounds turning white on
screen rotation.
Fix: use onCreateView() instead, move to AbsMainActivityFragment and
LibraryFragment respectively because onCreateView gets overridden there

Should fix #167
This commit is contained in:
bgaesslein 2022-02-10 21:36:55 +01:00
commit 67a6732640
4 changed files with 2 additions and 16 deletions

View file

@ -1,7 +1,5 @@
package com.dkanada.gramophone.fragments.library; package com.dkanada.gramophone.fragments.library;
import android.os.Bundle;
import com.dkanada.gramophone.fragments.AbsMusicServiceFragment; import com.dkanada.gramophone.fragments.AbsMusicServiceFragment;
import com.dkanada.gramophone.fragments.main.LibraryFragment; import com.dkanada.gramophone.fragments.main.LibraryFragment;
@ -9,10 +7,4 @@ public class AbsLibraryPagerFragment extends AbsMusicServiceFragment {
public LibraryFragment getLibraryFragment() { public LibraryFragment getLibraryFragment() {
return (LibraryFragment) getParentFragment(); return (LibraryFragment) getParentFragment();
} }
@Override
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setHasOptionsMenu(true);
}
} }

View file

@ -31,6 +31,7 @@ public abstract class AbsLibraryPagerRecyclerViewFragment<A extends RecyclerView
@Override @Override
public View onCreateView(@NonNull LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) { public View onCreateView(@NonNull LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
setHasOptionsMenu(true);
binding = FragmentMainActivityRecyclerViewBinding.inflate(getLayoutInflater(), container, false); binding = FragmentMainActivityRecyclerViewBinding.inflate(getLayoutInflater(), container, false);
return binding.getRoot(); return binding.getRoot();

View file

@ -1,7 +1,5 @@
package com.dkanada.gramophone.fragments.main; package com.dkanada.gramophone.fragments.main;
import android.os.Bundle;
import androidx.fragment.app.Fragment; import androidx.fragment.app.Fragment;
import com.dkanada.gramophone.activities.MainActivity; import com.dkanada.gramophone.activities.MainActivity;
@ -10,10 +8,4 @@ public abstract class AbsMainActivityFragment extends Fragment {
public MainActivity getMainActivity() { public MainActivity getMainActivity() {
return (MainActivity) getActivity(); return (MainActivity) getActivity();
} }
@Override
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setHasOptionsMenu(true);
}
} }

View file

@ -47,6 +47,7 @@ public class LibraryFragment extends AbsMainActivityFragment implements ViewPage
@Override @Override
public View onCreateView(@NonNull LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) { public View onCreateView(@NonNull LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
setHasOptionsMenu(true);
binding = FragmentLibraryBinding.inflate(inflater); binding = FragmentLibraryBinding.inflate(inflater);
return binding.getRoot(); return binding.getRoot();