Navigation Drawer lag on Android

Question

I'm having a problem with Navigation Drawer , it is too slow, the solution I'm looking for is to close the drawer first and then show the activity, but It is not working, certainly I'm missing something.

private class DrawerItemClickListener implements ListView.OnItemClickListener {         @Override         public void onItemClick(AdapterView<?> parent, View view, int posicao, long id) {             setLastPosition(posicao);             setFragmentList(lastPosition);             layoutDrawer.closeDrawer(linearDrawer);         }     }      private OnClickListener userOnClick = new OnClickListener() {         @Override         public void onClick(View v) {             layoutDrawer.closeDrawer(linearDrawer);         }     };      private void setFragmentList(int posicao) {          FragmentManager fragmentManager = getSupportFragmentManager();         Fragment fragment = new FragmentViagens();          switch (posicao) {              case 0:                 fragmentManager.beginTransaction().replace(R.id.content_frame, fragment).commit();                 break;             case 1:                 fragmentManager.beginTransaction().replace(R.id.content_frame, new FragmentPedidos()).commit();                 break;             case 2:                 fragmentManager.beginTransaction().replace(R.id.content_frame, new FragmentClientes()).commit();                 break;          }          navigationAdapter.setChecked(posicao, true);         setTitleFragments(lastPosition);         navigationAdapter.resetarCheck();         layoutDrawer.closeDrawer(linearDrawer);      } 

Answer 1

You can do It this way to avoid drawer lag, change your onItemClick:

layoutDrawer.closeDrawer(linearDrawer); setLastPosition(posicao); new Handler().postDelayed(new Runnable() {         @Override         public void run() {             setFragmentList(lastPosition);         }     }, 200); 

Edit: prefered way should be setting DrawerListener on DrawerLayout and setting your fragment in onDrawerClosed like this:

Fragment mFragmentToSet = null;  @Override public boolean onNavigationItemSelected(@NonNull MenuItem item) {     // Handle navigation view item clicks here.     switch (item.getItemId()) {         case R.id.nav_home:             mFragmentToSet = HomeFragment.newInstance();              break;     }      mDrawerLayout.closeDrawer(GravityCompat.START);     return true; } 

---

mDrawerLayout.addDrawerListener(new DrawerLayout.DrawerListener() {         @Override public void onDrawerSlide(View drawerView, float slideOffset) {}         @Override public void onDrawerOpened(View drawerView) {}         @Override public void onDrawerStateChanged(int newState) {}          @Override         public void onDrawerClosed(View drawerView) {           //Set your new fragment here           if (mFragmentToSet != null) {             getSupportFragmentManager()                   .beginTransaction()                   .replace(FRAGMENT_CONTAINER_ID, mFragmentToSet)                   .commit();             mFragmentToSet = null;           }         }     }); 

Answer 2

Rather than do transaction in onClick, why not do it in onDrawerClosed from DrawerLayout.DrawerListener?