Neither onNavigationItemSelected nor onOptionsItemSelected is called

Question

I have a double-drawer layout set up and am trying to handle selecting items inside the drawers. The way I have it set up though, I can see that neither onNavigationItemSelected nor onOptionsItemSelected is getting called when I tap something on the menu (I put a log statement right inside the functions). I'm also not totally clear which of those functions should be called.

EDIT: It looks like onOptionsItemSelected() is being called after all, but whichever item I click on, I always get the same id. So maybe it's only allowing me to click some other layer?

There are a lot of other questions similar to this, but none of the answers have been helpful to me. I'd appreciate any insight you may have.

Here is the relevant code inside my onCreate() function in MainActivity.java:

    // Adding Toolbar to Main screen
    Toolbar toolbar = (Toolbar) findViewById(R.id.toolbar);
    setSupportActionBar(toolbar);


    drawer = (DrawerLayout) findViewById(R.id.drawer_layout);
    ActionBarDrawerToggle toggle = new ActionBarDrawerToggle(this, drawer, toolbar, R.string.drawer_open, R.string.drawer_close);
    drawer.addDrawerListener(toggle);
    toggle.syncState();

    NavigationView leftNavigationView = (NavigationView) findViewById(R.id.notifications_view);
    leftNavigationView.setNavigationItemSelectedListener(new NavigationView.OnNavigationItemSelectedListener() {
        @Override
        public boolean onNavigationItemSelected(MenuItem item) {
            Log.i("left", "left"); // this is never called
            // Handle left navigation view item clicks here
            int id = item.getItemId();

            switch(id) {
                case R.id.emerg_con_menuitem:
                    break;
                case R.id.ride_hist_menuitem:
                    Log.i("I'm hit", "I'm hit");
                    intent = new Intent(MainActivity.this, loc_report.class); // The action I want to happen when this menu item is tapped
                    MainActivity.this.startActivity(intent);
                    break;
                case R.id.settings_menuitem:
                    break;
            }

            drawer.closeDrawer(GravityCompat.START);
            return true;
        }
    });

    NavigationView rightNavigationView = (NavigationView) findViewById(R.id.nav_view);
    rightNavigationView.setNavigationItemSelectedListener(new NavigationView.OnNavigationItemSelectedListener() {
        @Override
        public boolean onNavigationItemSelected(MenuItem item) {
            Log.i("right", "right"); // this is also never called
            // Handle Right navigation view item clicks here.
            int id = item.getItemId();

            Log.i("I am firing", "I am firing");

            switch(id) {
                case R.id.emerg_con_menuitem:
                    break;
                case R.id.ride_hist_menuitem:
                    Log.i("I'm hit navigation", "I'm hit navigation");
                    intent = new Intent(MainActivity.this, loc_report.class);
                    MainActivity.this.startActivity(intent);
                    break;
                case R.id.settings_menuitem:
                    break;
            }

            drawer.closeDrawer(GravityCompat.END);
            return true;
        }
    });

    // Adding menu icon to Toolbar
    ActionBar supportActionBar = getSupportActionBar();
    if (supportActionBar != null) {
        supportActionBar.setHomeAsUpIndicator(R.drawable.ic_notifications);
        supportActionBar.setDisplayHomeAsUpEnabled(true);
        supportActionBar.setDisplayShowTitleEnabled(false);
    }

And the onOptionsItemSelected() that is being called, although not on the correct menu:

@Override
public boolean onCreateOptionsMenu(Menu menu) {
    // Inflate the menu; this adds items to the action bar if it is present.
    getMenuInflater().inflate(R.menu.main_menu, menu);
    return true;
}

@Override
public boolean onOptionsItemSelected(MenuItem item) {
    // Handle action bar item clicks here.
    Log.i("I'm hit options", "I'm hit options"); // also never called...what is going on??
    int id = item.getItemId();

    switch(id) {
        case R.id.emerg_con_menuitem:
            break;
        case R.id.ride_hist_menuitem:
            Log.i("I'm hit options", "I'm hit options");
            Intent intent = new Intent(MainActivity.this, loc_report.class);
            this.startActivity(intent);
            break;
        case R.id.settings_menuitem:
            break;
        case R.id.menu_navigation:
            drawer.openDrawer(GravityCompat.END); /*Opens the Right Drawer*/
            return true;
    }

    return super.onOptionsItemSelected(item);
}

Here is the main view activity_mail.xml:

...

The main menu with the clickable icon to open the drawer:

and the stuff inside the drawer (the right-hand one):

Kaitlyn · Accepted Answer

Ok, I think I figured this out. I'm still having trouble with the right-drawer links, but I got the left-drawer links to work. It was a layering problem--so I was clicking on the main_menu item (which contains the clickable icon that makes the drawer open) instead of the menu items underneath. To fix this, I added this line of code to fix the order:

NavigationView leftNavigationView = (NavigationView) findViewById(R.id.nav_view);
leftNavigationView.bringToFront(); // <- added this line

And then the OnNavigationItemSelectedListener actually fires as expected.

Neither onNavigationItemSelected nor onOptionsItemSelected is called

Answers (1)

Related Questions