smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2025-12-08 15:54:57 +01:00
Code Issues Projects Releases Wiki Activity

move trigger rec-enable button above trigger box, for ergonomic reasons

Browse source
This commit is contained in:
Paul Davis 2025-05-13 19:18:32 -06:00
parent 2b1c1804c5
commit 316947f7dc
2 changed files with 26 additions and 21 deletions
Download patch file Download diff file Expand all files Collapse all files

26
gtk2_ardour/trigger_page.cc
View file

@ -78,22 +78,24 @@ TriggerPage::TriggerPage ()
load_bindings (); load_bindings ();
register_actions (); register_actions ();
/* Match TriggerStrip::input_button height */ /* Spacers to match height of TriggerStrip's input, name and toggle
ArdourButton* spacer = manage (new ArdourButton (ArdourButton::Text)); * rec-enable buttons
spacer->set_name ("mixer strip button"); */
spacer->set_sensitive (false);
spacer->set_text (" ");
/* Match TriggerStrip::_name_button height */ ArdourButton* spacer[3];
ArdourButton* spacer2 = manage (new ArdourButton (ArdourButton::Text));
spacer2->set_name ("mixer strip button"); for (std::size_t n = 0; n < sizeof (spacer) / sizeof (spacer[0]); ++n) {
spacer2->set_sensitive (false); spacer[n] = manage (new ArdourButton (ArdourButton::Text));
spacer2->set_text (" "); spacer[n]->set_name ("mixer strip button");
spacer[n]->set_sensitive (false);
spacer[n]->set_text (" ");
}
/* left-side, fixed-size cue-box */ /* left-side, fixed-size cue-box */
_cue_area_box.set_spacing (2); _cue_area_box.set_spacing (2);
_cue_area_box.pack_start (*spacer, Gtk::PACK_SHRINK); _cue_area_box.pack_start (*spacer[0], Gtk::PACK_SHRINK);
_cue_area_box.pack_start (*spacer2, Gtk::PACK_SHRINK); _cue_area_box.pack_start (*spacer[1], Gtk::PACK_SHRINK);
_cue_area_box.pack_start (*spacer[2], Gtk::PACK_SHRINK);
_cue_area_box.pack_start (_cue_box, Gtk::PACK_SHRINK); _cue_area_box.pack_start (_cue_box, Gtk::PACK_SHRINK);
_cue_area_box.pack_start (_master_widget, Gtk::PACK_SHRINK); _cue_area_box.pack_start (_master_widget, Gtk::PACK_SHRINK);

19
gtk2_ardour/trigger_strip.cc
View file

@ -136,6 +136,17 @@ TriggerStrip::init ()
global_vpacker.set_spacing (2); global_vpacker.set_spacing (2);
global_vpacker.pack_start (input_button, Gtk::PACK_SHRINK); global_vpacker.pack_start (input_button, Gtk::PACK_SHRINK);
global_vpacker.pack_start (_name_button, Gtk::PACK_SHRINK); global_vpacker.pack_start (_name_button, Gtk::PACK_SHRINK);
/* rec toggle below name */
rec_toggle_button = manage (new ArdourButton);
rec_toggle_button->set_name ("record enable button");
rec_toggle_button->set_icon (ArdourIcon::RecButton);
UI::instance()->set_tip (rec_toggle_button, _("Switch controls from cue launching to cue recording"), "");
rec_toggle_button->show ();
rec_toggle_button->signal_button_press_event().connect (sigc::mem_fun(*this, &TriggerStrip::rec_toggle_press), false);
global_vpacker.pack_start (*rec_toggle_button, Gtk::PACK_SHRINK);
global_vpacker.pack_start (_trigger_display, Gtk::PACK_SHRINK); global_vpacker.pack_start (_trigger_display, Gtk::PACK_SHRINK);
global_vpacker.pack_start (_tmaster_widget, Gtk::PACK_SHRINK); global_vpacker.pack_start (_tmaster_widget, Gtk::PACK_SHRINK);
global_vpacker.pack_start (_processor_box, true, true); global_vpacker.pack_start (_processor_box, true, true);
@ -150,14 +161,6 @@ TriggerStrip::init ()
mute_solo_table.attach (*mute_button, 0, 1, 0, 1); mute_solo_table.attach (*mute_button, 0, 1, 0, 1);
mute_solo_table.attach (*solo_button, 1, 2, 0, 1); mute_solo_table.attach (*solo_button, 1, 2, 0, 1);
rec_toggle_button = manage (new ArdourButton);
rec_toggle_button->set_name ("record enable button");
rec_toggle_button->set_icon (ArdourIcon::RecButton);
UI::instance()->set_tip (rec_toggle_button, _("Switch controls from cue launching to cue recording"), "");
rec_toggle_button->show ();
rec_toggle_button->signal_button_press_event().connect (sigc::mem_fun(*this, &TriggerStrip::rec_toggle_press), false);
mute_solo_table.attach (*rec_toggle_button, 0, 2, 1, 2);
volume_table.attach (_level_meter, 0, 1, 0, 1); volume_table.attach (_level_meter, 0, 1, 0, 1);
/*Note: _gain_control is added in set_route */ /*Note: _gain_control is added in set_route */