Menu.Item should have the onClick param

What problem does this feature solve?

I’d like to have a dropdown where one of the buttons is the logout button. When having a dropdown you can use MenuItemGroup and then populate it with X numbers of Menu.Item. At the moment Menu.Item doesn’t support the onClick param so I can’t attach a function to it. And I can’t just add a Button component inside of MenuItemGroup because it will cause a lot of errors.

If Menu.Item supported the onClick param the development would be must easier and the design would stay consistent.

You could ofc add the onClick on the Menu and then write something like:

const onClick = ({ key }) => {
  // Use the key here
  // If you want to use a specific key you have to do something like:
  if (key === 'valueOfTheKey') {
    console.log('something')
  }
}

But why do that?

What does the proposed API look like?

Add the onClick param to Menu.Item. It would make things a lot easier.

[ddcat1115]

Menu onClick can meet your needs

[channprj]

NOTE: Please click the thumb up (👍) rather than ‘+1’ comment.

<Menu.Item key='activate'>
  <a onClick={(event) => this.props.hadleActivate(event, this.props.index, !this.props.activate)}>
    { !this.props.activate ? 'Activate' : 'Deactivate' }
  </a>
</Menu.Item>

Wrapping with <a> works fine, but I think this way doesn’t look nice. Need your help!