diff options
| author | Robin Gareus <robin@gareus.org> | 2020-02-27 23:27:42 +0100 |
|---|---|---|
| committer | Robin Gareus <robin@gareus.org> | 2020-02-27 23:27:42 +0100 |
| commit | 8710a2d9432469909eb51d0644f2f25133240dbb (patch) | |
| tree | 5f0c9e1663df050249152ec48f75dfa554ce56f2 /gtk2_ardour/session_import_dialog.cc | |
| parent | e5c819956a95a1dc9dee8f805dc55757dc51233d (diff) | |
Dialog default to cancel (#7915)
There are various ways to cancel a dialog. Only checking for
RESPONSE_CANCEL is not sufficient. e.g. Esc causes a delete-event.
* Gtk::RESPONSE_CLOSE
* Gtk::RESPONSE_REJECT
* Gtk::RESPONSE_DELETE_EVENT
* Gtk::RESPONSE_CANCEL
* Gtk::RESPONSE_NO
Among others this fixes "Clicking session > open,
then hitting ESC opens the currently selected folder and session"
Diffstat (limited to 'gtk2_ardour/session_import_dialog.cc')
| -rw-r--r-- | gtk2_ardour/session_import_dialog.cc | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/gtk2_ardour/session_import_dialog.cc b/gtk2_ardour/session_import_dialog.cc index 00567da2ca..4c0727b57e 100644 --- a/gtk2_ardour/session_import_dialog.cc +++ b/gtk2_ardour/session_import_dialog.cc @@ -236,9 +236,13 @@ SessionImportDialog::update (string path) // Prompt user for verification string txt = _("This will select all elements of this type!"); ArdourMessageDialog msg (txt, false, MESSAGE_QUESTION, BUTTONS_OK_CANCEL, true); - if (msg.run() == RESPONSE_CANCEL) { - (*cell)[sb_cols.queued] = false; - return; + switch (msg.run()) { + case Gtk::RESPONSE_ACCEPT: + case Gtk::RESPONSE_OK: + break; + default: + (*cell)[sb_cols.queued] = false; + return; } } |