Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

OnMenuItemSelected isn't called when layout is set for menu item

I have a menu which is inflated from main_menu.xml:

<menu xmlns:android="http://schemas.android.com/apk/res/android">
    <item 
        android:id="@+id/act_sync" 
        android:showAsAction="always"
        android:actionLayout="@layout/sync_action"
        android:icon="@android:drawable/ic_popup_sync" />
</menu>

and here is the code in the activity:

@Override
public boolean onCreateOptionsMenu(Menu menu) {
    getMenuInflater().inflate(R.menu.main_menu, menu);
    return true;
}
@Override
public boolean onOptionsItemSelected(MenuItem item) {
    MyMessageHandler.debug("menu item selected");
    switch(item.getItemId()){
    case R.id.act_sync:
        sync();
        return true;
    }
    return super.onOptionsItemSelected(item);
}

But onOptionsItemSelected is not called when I touch the menu item. When I remove the actionLayout attribute of the menu item, it works fine. How can I fix this?
Thanks.

like image 256
Alireza Mirian Avatar asked Jul 20 '13 17:07

Alireza Mirian


2 Answers

you should use below snippet ( Just for reference )

@Override
    public boolean onCreateOptionsMenu(Menu menu) {
        MenuInflater inflater = getMenuInflater();
        inflater.inflate(R.menu.main_menu, menu);
        final Menu m = menu;
        final MenuItem item = menu.findItem(R.id.ActionConnection);
        item.getActionView().setOnClickListener(new OnClickListener() {

            @Override
            public void onClick(View v) {   
                sync();
            }
        });
        return true;
    }
like image 146
Vipul Avatar answered Sep 19 '22 15:09

Vipul


In addition to the answer provided by Vipul Shah.

Make sure you disable the clickable option of all items in your action layout:

android:clickable="false"

Otherwise, it may steal the click so that you still cant receive onClick call back.

like image 36
Alvin Avatar answered Sep 18 '22 15:09

Alvin