I just noticed that we have two components inside key_pad_menu_item.js file. One for links and one for buttons.

This was an old component that was created before our pattern of being able to pass an href or onClick and the component handles the correct html element. We should remove EuiKeyPadMenuItemButton in favor of just EuiKeyPadMenuItem that can also accept an onClick