From 637d99167a337583cc2f1db56cdb06039a9ced7a Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Thu, 13 Nov 2025 09:56:24 -0700 Subject: [PATCH] SessionDialog: remove explicit button press handling This is not needed since ArdourButton + related action takes care of it --- gtk2_ardour/session_dialog.cc | 24 ------------------------ gtk2_ardour/session_dialog.h | 3 --- 2 files changed, 27 deletions(-) diff --git a/gtk2_ardour/session_dialog.cc b/gtk2_ardour/session_dialog.cc index 19ab94976e..8598543208 100644 --- a/gtk2_ardour/session_dialog.cc +++ b/gtk2_ardour/session_dialog.cc @@ -126,21 +126,18 @@ SessionDialog::SessionDialog (DialogTab initial_tab, const std::string& session_ new_button.set_text("NEW"); new_button.set_name ("tab button"); - new_button.signal_button_press_event().connect (sigc::mem_fun (*this, &SessionDialog::new_button_pressed), false); new_button.set_tweaks(ArdourButton::Tweaks(ArdourButton::ForceFlat)); new_button.set_can_focus (true); new_button.set_related_action (new_session_action); recent_button.set_text("RECENT"); recent_button.set_name ("tab button"); - recent_button.signal_button_press_event().connect (sigc::mem_fun (*this, &SessionDialog::recent_button_pressed), false); recent_button.set_tweaks(ArdourButton::Tweaks(ArdourButton::ForceFlat)); recent_button.set_can_focus (true); recent_button.set_related_action (recent_session_action); existing_button.set_text("OPEN"); existing_button.set_name ("tab button"); - existing_button.signal_button_press_event().connect (sigc::mem_fun (*this, &SessionDialog::existing_button_pressed), false); existing_button.set_tweaks(ArdourButton::Tweaks(ArdourButton::ForceFlat)); existing_button.set_can_focus (true); existing_button.set_related_action (existing_session_action); @@ -617,27 +614,6 @@ SessionDialog::existing_button_choice_action () _tabs.set_current_page(2); } -bool -SessionDialog::new_button_pressed (GdkEventButton*) -{ - new_button_choice_action (); - return true; -} - -bool -SessionDialog::recent_button_pressed (GdkEventButton*) -{ - recent_button_choice_action (); - return true; -} - -bool -SessionDialog::existing_button_pressed (GdkEventButton*) -{ - existing_button_choice_action (); - return true; -} - bool SessionDialog::prefs_button_pressed (GdkEventButton*) { diff --git a/gtk2_ardour/session_dialog.h b/gtk2_ardour/session_dialog.h index 61e5af12fe..78d15ad7eb 100644 --- a/gtk2_ardour/session_dialog.h +++ b/gtk2_ardour/session_dialog.h @@ -100,9 +100,6 @@ private: Gtk::ComboBoxText timebase_chooser; - bool new_button_pressed (GdkEventButton*); - bool recent_button_pressed (GdkEventButton*); - bool existing_button_pressed (GdkEventButton*); bool prefs_button_pressed (GdkEventButton*); void new_button_choice_action ();