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/utils_videotl.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/utils_videotl.cc')
| -rw-r--r-- | gtk2_ardour/utils_videotl.cc | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/gtk2_ardour/utils_videotl.cc b/gtk2_ardour/utils_videotl.cc index 36f10b8410..894eaaea40 100644 --- a/gtk2_ardour/utils_videotl.cc +++ b/gtk2_ardour/utils_videotl.cc @@ -70,7 +70,12 @@ VideoUtils::confirm_video_outfn (Gtk::Window& parent, std::string outfn, std::st confirm.add_button (Gtk::Stock::CANCEL, Gtk::RESPONSE_CANCEL); confirm.add_button (_("Continue"), Gtk::RESPONSE_ACCEPT); confirm.show_all (); - if (confirm.run() == RESPONSE_CANCEL) { return false; } + switch (confirm.run ()) { + case Gtk::RESPONSE_ACCEPT: + break; + default: + return false; + } } if (Glib::file_test(outfn, Glib::FILE_TEST_EXISTS)) { |