In my app, i have an Activity, which has a FrameLayout in it. In this FrameLayout, there is a fragment, containing a ToolBar and a RecyclerView.
In this toolbar, i have a search button, which should start an Activity on item click. However, when i try to use onOptionsItemSelected, the apps gets built and installed succesfully, but when i try to tap that button in question, nothing happens. The Logcat, doesnt say anything either.
Can some points me what im doing wrong? Are there simpler or other easy ways to manage on ToolBar item clicks?
Fragment.kt
class FragmentTrack : Fragment() {
...
override fun onCreate(savedInstanceState: Bundle?) {
super.onCreate(savedInstanceState)
setHasOptionsMenu(true)
}
override fun onCreateView(
inflater: LayoutInflater,
container: ViewGroup?,
savedInstanceState: Bundle?
): View? = inflater.inflate(R.layout.fragment_track, container, false)
override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
super.onViewCreated(view, savedInstanceState)
...
topToolbar.setNavigationOnClickListener {
val dialog = FragmentBottomSheetDrawer()
dialog.show(childFragmentManager, dialog.tag)
}
}
override fun onCreateOptionsMenu(menu: Menu, inflater: MenuInflater) {
inflater.inflate(R.menu.menu_toolbar, menu)
super.onCreateOptionsMenu(menu, inflater)
}
override fun onOptionsItemSelected(item: MenuItem): Boolean {
when(item.itemId) {
R.id.fsvSearch -> Toast.makeText(context, "Clicked search button", Toast.LENGTH_SHORT).show()
}
return true
}
}
fragment.xml
<?xml version="1.0" encoding="utf-8"?>
<androidx.coordinatorlayout.widget.CoordinatorLayout xmlns:android="http://schemas.android.com/apk/res/android"
android:layout_width="match_parent"
android:layout_height="match_parent"
xmlns:app="http://schemas.android.com/apk/res-auto">
<com.google.android.material.appbar.AppBarLayout
android:id="@+id/appbar"
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:orientation="vertical"
android:theme="@style/Theme.Design.NoActionBar">
<androidx.appcompat.widget.Toolbar
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:id="@+id/topToolbar"
android:background="@color/colorPrimaryDark"
app:navigationIcon="@drawable/ic_outline_menu_24"
app:popupTheme="@style/popupMenuThemeDark"
app:titleTextColor="@android:color/white"
app:title="Revo"
app:menu="@menu/menu_toolbar"
app:layout_scrollFlags="scroll|enterAlways" />
</com.google.android.material.appbar.AppBarLayout>
<androidx.recyclerview.widget.RecyclerView
android:id="@+id/rvTracks"
android:layout_height="match_parent"
android:layout_width="match_parent"
android:clipToPadding="false"
app:layout_behavior="@string/appbar_scrolling_view_behavior"
/>
</androidx.coordinatorlayout.widget.CoordinatorLayout>
toolbar_menu.xml
<?xml version="1.0" encoding="utf-8"?>
<menu xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:app="http://schemas.android.com/apk/res-auto">
<item
android:id="@+id/fsvSearch"
android:icon="@drawable/ic_search_24px"
android:title="@string/search"
app:showAsAction="always"/>
<item
android:id="@+id/fsvOrder"
android:icon="@drawable/ic_sort_24px"
android:title="@string/sort"
app:showAsAction="ifRoom">
<menu>
<group android:id="@+id/sortMenu" android:checkableBehavior="single">
<item
android:id="@+id/sortIncrease"
android:title="@string/increasing"/>
<item
android:id="@+id/sortDecrease"
android:title="@string/decreasing"/>
<item
android:id="@+id/sortMArtist"
android:title="@string/artist"/>
<item
android:id="@+id/sortAlbum"
android:title="@string/albums"/>
<item
android:id="@+id/sortYear"
android:title="@string/year"/>
<item
android:id="@+id/sortDate"
android:title="@string/date"/>
</group>
</menu>
</item>
<item
android:id="@+id/fsvGrid"
android:icon="@drawable/ic_grid_on_24px"
android:title="@string/grid"
app:showAsAction="ifRoom">
<menu>
<group android:id="@+id/gridMenu" android:checkableBehavior="single">
<item
android:id="@+id/gridOne"
android:title="1"/>
<item
android:id="@+id/gridTwo"
android:title="2"/>
<item
android:id="@+id/gridThree"
android:title="3"/>
<item
android:id="@+id/gridFour"
android:title="4"/>
</group>
</menu>
</item>
</menu>
I found a solution thanks to some external help. Its possible to work on the Toolbars item in an easier way.
In the onViewCreated
method, we must add:
topToolbar.inflateMenu(R.menu.menu_toolbar)
topToolbar.setOnMenuItemClickListener {
when(it.itemId) {
R.id.fsvSearch -> //your code
}
true
}
Also, if the menu gets duplicated, remove the app:menu
tag in the Toolbars xml