Fix close/cancel button accel problems on Mac
This commit is contained in:
parent
b7941345d1
commit
b642784e91
@ -1004,7 +1004,7 @@ ConfigWizard::ConfigWizard(wxWindow *parent, RunReason reason)
|
|||||||
p->btn_prev = new wxButton(this, wxID_ANY, _(L("< &Back")));
|
p->btn_prev = new wxButton(this, wxID_ANY, _(L("< &Back")));
|
||||||
p->btn_next = new wxButton(this, wxID_ANY, _(L("&Next >")));
|
p->btn_next = new wxButton(this, wxID_ANY, _(L("&Next >")));
|
||||||
p->btn_finish = new wxButton(this, wxID_APPLY, _(L("&Finish")));
|
p->btn_finish = new wxButton(this, wxID_APPLY, _(L("&Finish")));
|
||||||
p->btn_cancel = new wxButton(this, wxID_CANCEL);
|
p->btn_cancel = new wxButton(this, wxID_CANCEL, _(L("Cancel"))); // Note: The label needs to be present, otherwise we get accelerator bugs on Mac
|
||||||
p->btnsizer->AddStretchSpacer();
|
p->btnsizer->AddStretchSpacer();
|
||||||
p->btnsizer->Add(p->btn_prev, 0, wxLEFT, BTN_SPACING);
|
p->btnsizer->Add(p->btn_prev, 0, wxLEFT, BTN_SPACING);
|
||||||
p->btnsizer->Add(p->btn_next, 0, wxLEFT, BTN_SPACING);
|
p->btnsizer->Add(p->btn_next, 0, wxLEFT, BTN_SPACING);
|
||||||
|
@ -768,7 +768,7 @@ FirmwareDialog::FirmwareDialog(wxWindow *parent) :
|
|||||||
// Experience says it needs to be 1, otherwise things won't get sized properly.
|
// Experience says it needs to be 1, otherwise things won't get sized properly.
|
||||||
vsizer->Add(p->spoiler, 1, wxEXPAND | wxBOTTOM, SPACING);
|
vsizer->Add(p->spoiler, 1, wxEXPAND | wxBOTTOM, SPACING);
|
||||||
|
|
||||||
p->btn_close = new wxButton(panel, wxID_CLOSE);
|
p->btn_close = new wxButton(panel, wxID_CLOSE, _(L("Close"))); // Note: The label needs to be present, otherwise we get accelerator bugs on Mac
|
||||||
p->btn_flash = new wxButton(panel, wxID_ANY, p->btn_flash_label_ready);
|
p->btn_flash = new wxButton(panel, wxID_ANY, p->btn_flash_label_ready);
|
||||||
p->btn_flash->Disable();
|
p->btn_flash->Disable();
|
||||||
auto *bsizer = new wxBoxSizer(wxHORIZONTAL);
|
auto *bsizer = new wxBoxSizer(wxHORIZONTAL);
|
||||||
|
@ -157,7 +157,7 @@ PrintHostQueueDialog::PrintHostQueueDialog(wxWindow *parent)
|
|||||||
btn_cancel->Disable();
|
btn_cancel->Disable();
|
||||||
btn_error = new wxButton(this, wxID_ANY, _(L("Show error message")));
|
btn_error = new wxButton(this, wxID_ANY, _(L("Show error message")));
|
||||||
btn_error->Disable();
|
btn_error->Disable();
|
||||||
auto *btn_close = new wxButton(this, wxID_CANCEL, _(L("Close")));
|
auto *btn_close = new wxButton(this, wxID_CANCEL, _(L("Close"))); // Note: The label needs to be present, otherwise we get accelerator bugs on Mac
|
||||||
btnsizer->Add(btn_cancel, 0, wxRIGHT, SPACING);
|
btnsizer->Add(btn_cancel, 0, wxRIGHT, SPACING);
|
||||||
btnsizer->Add(btn_error, 0);
|
btnsizer->Add(btn_error, 0);
|
||||||
btnsizer->AddStretchSpacer();
|
btnsizer->AddStretchSpacer();
|
||||||
|
Loading…
Reference in New Issue
Block a user