8

My onCreateOptionsMenu works only in my MainActivity and when I try to put another onCreateOptionsMenu in another activity to inflate a different menu it does not display my menu bar (note that I have it setup exactly the same in both activities).

I do not get any errors, it just does not display my menu bar on my secondary activity.

Here is my code:

@Override
protected void onCreate(Bundle savedInstanceState){
    super.onCreate(savedInstanceState);
    setContentView(R.layout.expanded_view_layout);
}

@Override
public boolean onCreateOptionsMenu(Menu menu) {
    getMenuInflater().inflate(R.menu.menu_main, menu);
    return true;
}

@Override
public boolean onOptionsItemSelected(MenuItem item) {
    //noinspection SimplifiableIfStatement
    int id = item.getItemId();
    if (id == R.id.action_back) {
        finish();
    }
    return super.onOptionsItemSelected(item);
}

Also note that I changed getMenuInflater() to super.getMenuInflater() and got same result.

Zoe
  • 27,060
  • 21
  • 118
  • 148
Errol Green
  • 1,367
  • 5
  • 19
  • 32

7 Answers7

9

In case you are using tool bar in your activity

@Override
public void onCreate(Bundle savedInstanceState) {
   super.onCreate(savedInstanceState);
   setContentView(R.layout.activity_map);
   mToolbar = (Toolbar) findViewById(R.id.tool_bar);
   setSupportActionBar(mToolbar);
}
Mr.Q
  • 4,316
  • 3
  • 43
  • 40
6

try adding method of super in your callback onCreateOptionsMenu implementation.

return super.onCreateOptionsMenu(menu);
srv_sud
  • 647
  • 1
  • 9
  • 26
5

The problem in my case is if extend Activity, then onCreateOptionsMenu is not triggered. Did it manually with in onCreate:

Toolbar toolbar = findViewById(R.id.toolbar);
toolbar.inflateMenu(R.menu.menu_main);

Of course, toolbar is defined in CoordinatorLayout

<android.support.design.widget.AppBarLayout
        android:id="@+id/appBarLayout"
        android:layout_width="match_parent"
        android:layout_height="wrap_content">

        <android.support.v7.widget.Toolbar
            android:id="@+id/toolbar"
            android:layout_width="match_parent"
            android:layout_height="?attr/actionBarSize"
            android:background="?attr/colorPrimary"
            app:layout_scrollFlags="scroll|enterAlways"/>
</android.support.design.widget.AppBarLayout>
  • after following this example https://learn.microsoft.com/en-us/xamarin/android/user-interface/controls/tool-bar/replacing-the-action-bar#apply-the-custom-theme I found that your soltuion of inflating the menu on create is the only thing that seems to get the menu items to show. Thanks – AntDC Oct 24 '18 at 10:08
1

don't forget to call parent function like this

@Override
public boolean onCreateOptionsMenu(Menu menu) {
    super.onCreateOptionsMenu(menu);
    ....
}
baroni
  • 176
  • 8
  • are you calling return super.onOptionsItemSelected(item); inside onCreateOptionsMenu() or its just an illustration??? – srv_sud Apr 19 '16 at 14:35
  • this will create a stackoverflow, a infinite loop of calls – Kas Apr 27 '19 at 11:07
0

you should building a new xml menu (Second.xml).

@Override
public boolean onCreateOptionsMenu(Menu menu) {
    // Inflate the menu; this adds items to the action bar if it is present.
    getMenuInflater().inflate(R.menu.second, menu);//Second es your new xml.
    return true;
}
Slinker
  • 379
  • 1
  • 4
  • 14
0

I was having the same problem: - I found the solution that worked for me.

1>Check to what you extend class: - In my case, I have tried my activity with extending AppCompatActivity class and the menu is showing proper and when I changed it to activity class then it stopped displaying.

Not Working: - public class GridViewActivity extends Activity

2>Try to extend your class with AppCompatActivity class instead of activity class.

Working: -

public class GridViewActivity extends AppCompatActivity

Neel Patel
  • 1
  • 1
  • 4
0

Call setHasOptionsMenu(true) from onCreate.

    @Override
    public void onCreate(@Nullable Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setHasOptionsMenu(true);
     }
Jon
  • 9,156
  • 9
  • 56
  • 73
Mohammad Zaer
  • 638
  • 7
  • 9