[wxOSX_COCOA] Key&board accelerators for button labels

[joostn]

Sample app:

#include "wx/wx.h"

class MyFrame : public wxFrame
{
    DECLARE_CLASS( MyFrame )
    DECLARE_EVENT_TABLE()
public:
    MyFrame()  
      : wxFrame(NULL,wxID_ANY,wxT("Frame"))
    {
      wxButton *btn1=new wxButton(this,1100,wxT("Button &A"),wxPoint(0,10));
      wxButton *btn2=new wxButton(this,1101,wxT("Button &B"),wxPoint(0,100));
    }
    void OnButtonAClick(wxCommandEvent &event)
    {
      wxMessageBox(wxT("A"));
    }
    void OnButtonBClick(wxCommandEvent &event)
    {
      wxMessageBox(wxT("B"));
    }
};


IMPLEMENT_CLASS( MyFrame, wxFrame )
BEGIN_EVENT_TABLE( MyFrame, wxFrame )
    EVT_BUTTON( 1100, MyFrame::OnButtonAClick )
    EVT_BUTTON( 1101, MyFrame::OnButtonBClick )
END_EVENT_TABLE()


class MyApp : public wxApp
{
public:
    bool OnInit()
    {
      wxApp::OnInit();
      MyFrame *frame=new MyFrame();
      frame->Show();
      return true;
    }
};

DECLARE_APP(MyApp)
IMPLEMENT_APP(MyApp)

Pressing Cmd+A should invoke button A and Cmd+B should invoke button B, just like the Alt shortcuts under windows. This is accomplished by the attached patch.

Keyboard shortcut is ignored if the button is the default button, since I haven't found a way to attach two keys to the same button.

[joostn]

The patch

[vadz]

Looks reasonable to me. Stefan, any objections to applying this?

[csomor]

apart from the fact that it should be #if and not #ifdef for wxOSX_USE_COCOA and that it should have the same brace around the call for MacSetAcceleratorFromLabel in _osx.cpp it looks fine to me

[VZ]

(In [72809]) Implement support for button mnemonics in wxOSX/Cocoa.

Set the mnemonic as "key equivalent".

Closes #12917.