Alert dialog on menu item click

Question

Code

public boolean onNavigationItemSelected(MenuItem item) {
    int id = item.getItemId();
    if (id == R.id.name) {
        AlertDialog.Builder alertDialog = new AlertDialog.Builder(Profile.this);
        alertDialog.setTitle("Enter Name");
        final EditText input = new EditText(Profile.this);
        LinearLayout.LayoutParams lp = new LinearLayout.LayoutParams(
                LinearLayout.LayoutParams.MATCH_PARENT,
                LinearLayout.LayoutParams.MATCH_PARENT);
        input.setLayoutParams(lp);
        alertDialog.setView(input);
        alertDialog.setPositiveButton("Save",
                new DialogInterface.OnClickListener() {
                    public void onClick(DialogInterface dialog, int which) {
                        String mName = input.getText().toString();
                        mDatabaseReference.child("UserData").child(UID).child("Name").setValue(mName);
                        dialog.cancel();
                        Toast.makeText(getApplicationContext(),
                                        "Name successfully changed", Toast.LENGTH_SHORT).show();
                        }
                });
        alertDialog.setNegativeButton("Cancel",
                new DialogInterface.OnClickListener() {
                    public void onClick(DialogInterface dialog, int which) {
                        dialog.cancel();
                    }
                });
        alertDialog.show();

    }
    return super.onOptionsItemSelected(item);
  }
}

Never worked with alert dialog which has an EditText so not really sure on why this code isn't working. I want an EditText to be displayed on menu item click.

So what is happening when you are using this code? Do you have an error? — Alex Mamo, Oct 10 '18 at 08:59
Try to create Layout and put editText on it. Here is an example https://stackoverflow.com/a/12876690/6055194 — Eugene Babich, Oct 10 '18 at 09:05
Try to create Alert Dialog inside onCreate() method and just call show() while comparing the IDs based on the selection in onNavigationItemSelected(). @MichelleKinsten — Salman Khan, Oct 10 '18 at 09:08

score 1 · Accepted Answer · answered Oct 10 '18 at 09:10

Try it with switch (item.getItemId()) case instead of if. Try with below mentioned changes and with that i can able to see Dialog as you want.

public boolean onNavigationItemSelected(MenuItem item) {
    switch (item.getItemId()) {
        case R.id.name:
            AlertDialog.Builder alertDialog = new AlertDialog.Builder(Profile.this);
            alertDialog.setTitle("Enter Name");
            final EditText input = new EditText(Profile.this);
            LinearLayout.LayoutParams lp = new LinearLayout.LayoutParams(
                    LinearLayout.LayoutParams.MATCH_PARENT,
                    LinearLayout.LayoutParams.MATCH_PARENT);
            input.setLayoutParams(lp);
            alertDialog.setView(input);
            alertDialog.setPositiveButton("Save",
                    new DialogInterface.OnClickListener() {
                        public void onClick(DialogInterface dialog, int which) {
                            String mName = input.getText().toString();
                            mDatabaseReference.child("UserData").child(UID).child("Name").setValue(mName);
                            dialog.cancel();
                            Toast.makeText(getApplicationContext(),
                                            "Name successfully changed", Toast.LENGTH_SHORT).show();
                            }
                    });
            alertDialog.setNegativeButton("Cancel",
                    new DialogInterface.OnClickListener() {
                        public void onClick(DialogInterface dialog, int which) {
                            dialog.cancel();
                        }
                    });
            alertDialog.show();
            return true;

    }
    return super.onOptionsItemSelected(item);
  }
}

score 1 · Answer 2 · answered Oct 10 '18 at 09:12

1

You are missing below code i guess

alertDialog.create().show();

instead of

alertDialog.show();

answered Oct 10 '18 at 09:12

Uday Ramjiyani

1,407
9
22

score 0 · Answer 3 · answered Mar 13 '22 at 03:24

Well,you only need to add some codes in if-else:(take a try)

@Override
public boolean onContextItemSelected(@NonNull MenuItem item) {
    switch (item.getItemId()) {
        case R.id.menu_delete:
            new AlertDialog.Builder(this)
                    .setTitle("Confirm Deleteing!")
                    .setMessage("Are you sure you want to delete?")
                    .setPositiveButton("Yes", new DialogInterface.OnClickListener() {
                        @Override
                        public void onClick(DialogInterface dialog, int which) {
                            //Delete menu item
                            itemlist.remove(item_id);
                            myAdapter.notifyDataSetChanged();
                        }
                    }).setNegativeButton("No", null).show();
            break;
        case R.id.menu_edit:
            break;}

Alert dialog on menu item click

3 Answers3