diff options
author | Paul Davis <paul@linuxaudiosystems.com> | 2017-03-01 22:29:54 +0100 |
---|---|---|
committer | Paul Davis <paul@linuxaudiosystems.com> | 2017-03-01 22:30:20 +0100 |
commit | 05b010266898b9f1585bb6924cb00f11e87169c2 (patch) | |
tree | 05bdbd0eaa803f427d9ac1d9881127ccbaefcecc /gtk2_ardour/engine_dialog.cc | |
parent | ac9bc1976234f6cf8b732e237192834ad43c8e81 (diff) |
ArdourButton: if _act_on_release is true, only trigger actions/signal clicks on key release, and vice versa
This should really be split in separate semantics for key and button events.
Fixes a subtle but nasty bug in the EngineDialog where the change of the default focus
widget from a Gtk::Button (acts on press) to an ArdourButton (acts on release) caused
events occuring after a dialog has grabbed focus to trigger button clicks
Diffstat (limited to 'gtk2_ardour/engine_dialog.cc')
-rw-r--r-- | gtk2_ardour/engine_dialog.cc | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/gtk2_ardour/engine_dialog.cc b/gtk2_ardour/engine_dialog.cc index 4dc806763f..57a30abc99 100644 --- a/gtk2_ardour/engine_dialog.cc +++ b/gtk2_ardour/engine_dialog.cc @@ -280,6 +280,7 @@ EngineControl::EngineControl () start_stop_button.set_name ("generic button"); start_stop_button.set_can_focus(true); start_stop_button.set_can_default(true); + start_stop_button.set_act_on_release (false); update_devices_button.signal_clicked.connect (mem_fun (*this, &EngineControl::update_devices_button_clicked)); update_devices_button.set_sensitive (false); |