CODEWITHSUNDEEP
androidandroid-studioandroid-fragments
Asma Ahmad
Asma Ahmad

Reputation: 438

Fragment Menuitem Click

   public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) {
    inflater.inflate(R.menu.menu_fragement, menu);
    super.onCreateOptionsMenu(menu, inflater);
}

public boolean onOptionsItemSelected(MenuItem item) {
    Intent intent = new Intent();
    switch (item.getItemId()) {
        case R.id.color_menu:
            intent.setClass(rootView.getContext(), CandleColorActivity.class);
            getActivity().startActivityForResult(intent,COLOR_ACTION);
            break;
    }
    return super.onOptionsItemSelected(item);
}

Using the above code menu item is visible in the fragment but item click is not working. Menu item xml:

<item
android:title="selectColor"
android:icon="@drawable/addcolor"
app:showAsAction="always"
android:id="@+id/color_menu"></item>

main activity menu xml always show on each of the fragment

<item
android:title="menu"
android:icon="@drawable/menu"
app:showAsAction="always"
android:id="@+id/uper_menu"></item>

Main activity menu java code open a dialog box on the item click

   @Override
public boolean onCreateOptionsMenu(Menu menu) {
    MenuInflater inflater = getMenuInflater();
    inflater.inflate(R.menu.menu, menu);
    return true;
}
@Override
public boolean onOptionsItemSelected(MenuItem item) {
    switch (item.getItemId()) {

        case R.id.uper_menu:

            AlertDialog.Builder builder = new AlertDialog.Builder(context);
            builder.setView(R.layout.menu_dialog);
            alertDialog = builder.show();

            alertDialog.getWindow().setGravity(Gravity.BOTTOM);
            alertDialog.getWindow().setLayout(WindowManager.LayoutParams.MATCH_PARENT, WindowManager.LayoutParams.MATCH_PARENT);
            alertDialog.getWindow().getDecorView().setBackgroundResource(android.R.color.transparent);
            viewIds();
            break;
        default:
            break;
    }
    return true;
}

Upvotes: 0

Views: 55

Answers (1)

Ridcully
Ridcully

Reputation: 23665

You should not call the super class' onOptionsItemSelected(), if you handled the event yourself. So change your method to this:

public boolean onOptionsItemSelected(MenuItem item) {
    ...
    switch (item.getItemId()) {
        case R.id.color_menu:
            ...
            return true;
        default:
            return super.onOptionsItemSelected(item);
    }           
}

EDIT

In fragment and activity, only return true, if you handled the event, otherwise return super.onOptionsItemSelected(item);

Reason is, that the system first asks the activity to handle the event, and if the activity says it did handle it (by returning true), the system doesn't ask the fragment anymore.

Upvotes: 2

Related Questions