Fix reference of track after free() (#4795).
[ardour.git] / gtk2_ardour / ardour_ui2.cc
index 1858011164482ce5a93b029d918a807fb0eecdaa..05addb98e3addfc936fdf59444f675faef126c55 100644 (file)
@@ -48,6 +48,7 @@
 #include "public_editor.h"
 #include "audio_clock.h"
 #include "actions.h"
+#include "button_joiner.h"
 #include "utils.h"
 #include "theme_manager.h"
 #include "midi_tracer.h"
@@ -129,12 +130,11 @@ ARDOUR_UI::setup_tooltips ()
        set_tip (stop_button, _("Stop playback"));
        set_tip (rec_button, _("Toggle record"));
        set_tip (play_selection_button, _("Play range/selection"));
-       set_tip (join_play_range_button, _("Always play range/selection"));
        set_tip (goto_start_button, _("Go to start of session"));
        set_tip (goto_end_button, _("Go to end of session"));
        set_tip (auto_loop_button, _("Play loop range"));
        set_tip (midi_panic_button, _("MIDI Panic\nSend note off and reset controller messages on all MIDI channels"));
-
+       set_tip (*transport_joiner, _("Always Play Range Selection (if any)"));
        set_tip (auto_return_button, _("Return to last playback start when stopped"));
        set_tip (auto_play_button, _("Start playback after any locate"));
        set_tip (auto_input_button, _("Be sensible about input monitoring"));
@@ -261,7 +261,7 @@ ARDOUR_UI::setup_transport ()
        click_button.set_name ("transport active option button");
        sync_button.set_name ("transport active option button");
 
-       stop_button.set_active_state (Active);
+       stop_button.set_active (true);
 
        goto_start_button.set_image (get_icon (X_("transport_start")));
        goto_end_button.set_image (get_icon (X_("transport_end")));
@@ -270,7 +270,6 @@ ARDOUR_UI::setup_transport ()
        play_selection_button.set_image (get_icon (X_("transport_range")));
        rec_button.set_image (get_icon (X_("transport_record")));
        auto_loop_button.set_image (get_icon (X_("transport_loop")));
-       join_play_range_button.set_image (get_icon (X_("tool_object_range")));
 
        midi_panic_button.set_image (get_icon (X_("midi_panic")));
        /* the icon for this has an odd aspect ratio, so fatten up the button */
@@ -294,8 +293,11 @@ ARDOUR_UI::setup_transport ()
        midi_panic_button.set_related_action (act);
        act = ActionManager::get_action (X_("Transport"), X_("ToggleExternalSync"));
        sync_button.set_related_action (act);
+
+       transport_joiner = manage (new ButtonJoiner ("transport button", play_selection_button, roll_button));
+
        act = ActionManager::get_action (X_("Transport"), X_("AlwaysPlayRange"));
-       join_play_range_button.set_related_action (act);
+       transport_joiner->set_related_action (act);
 
        /* clocks, etc. */
 
@@ -328,49 +330,64 @@ ARDOUR_UI::setup_transport ()
        alert_box.pack_start (auditioning_alert_button, true, false);
        alert_box.pack_start (feedback_alert_button, true, false);
 
+       /* all transport buttons should be the same size vertically and
+        * horizontally 
+        */
+
+       Glib::RefPtr<SizeGroup> transport_button_size_group = SizeGroup::create (SIZE_GROUP_BOTH);
+       transport_button_size_group->add_widget (goto_start_button);
+       transport_button_size_group->add_widget (goto_end_button);
+       transport_button_size_group->add_widget (auto_loop_button);
+       transport_button_size_group->add_widget (rec_button);
+       transport_button_size_group->add_widget (play_selection_button);
+       transport_button_size_group->add_widget (roll_button);
+       transport_button_size_group->add_widget (stop_button);
+
+       HBox* tbox1 = manage (new HBox);
+       HBox* tbox2 = manage (new HBox);
+       HBox* tbox3 = manage (new HBox);
        HBox* tbox = manage (new HBox);
-       tbox->set_spacing (2);
 
-       tbox->pack_start (midi_panic_button, false, false);
-       tbox->pack_start (goto_start_button, false, false);
-       tbox->pack_start (goto_end_button, false, false);
+       VBox* vbox1 = manage (new VBox);
+       VBox* vbox2 = manage (new VBox);
+       VBox* vbox3 = manage (new VBox);
 
-       Glib::RefPtr<SizeGroup> transport_button_size_group1 = SizeGroup::create (SIZE_GROUP_HORIZONTAL);
-       transport_button_size_group1->add_widget (goto_start_button);
-       transport_button_size_group1->add_widget (goto_end_button);
-       transport_button_size_group1->add_widget (auto_loop_button);
-       transport_button_size_group1->add_widget (rec_button);
+       Alignment* a1 = manage (new Alignment);
+       Alignment* a2 = manage (new Alignment);
+       Alignment* a3 = manage (new Alignment);
 
-       if (Profile->get_sae()) {
-               tbox->pack_start (auto_loop_button);
-               tbox->pack_start (roll_button);
-               transport_button_size_group1->add_widget (play_selection_button);
-               transport_button_size_group1->add_widget (roll_button);
+       tbox1->set_spacing (2);
+       tbox2->set_spacing (2);
+       tbox3->set_spacing (2);
+       tbox->set_spacing (2);
 
-       } else {
+       tbox1->pack_start (midi_panic_button, false, false);
+       tbox1->pack_start (goto_start_button, false, false);
+       tbox1->pack_start (goto_end_button, false, false);
+       tbox1->pack_start (auto_loop_button, false, false);
 
-               tbox->pack_start (auto_loop_button, false, false);
+       play_selection_button.set_rounded_corner_mask (0x1); /* upper left only */
+       roll_button.set_rounded_corner_mask (0x2); /* upper right only */
 
-               Frame* jpframe = manage (new Frame);
-               HBox* jpbox = manage (new HBox);
+       tbox2->pack_start (*transport_joiner, false, false);
 
-               jpframe->add (*jpbox);
-               jpframe->set_shadow_type (SHADOW_NONE);
-               
-               jpbox->pack_start (play_selection_button, false, false);
-               jpbox->pack_start (join_play_range_button, false, false);
-               jpbox->pack_start (roll_button, false, false);
+       tbox3->pack_start (stop_button, false, false);
+       tbox3->pack_start (rec_button, false, false, 6);
 
-               tbox->pack_start (*jpframe, false, false);
+       vbox1->pack_start (*tbox1, false, false);
+       vbox2->pack_start (*tbox2, false, false);
+       vbox3->pack_start (*tbox3, false, false);
 
-               Glib::RefPtr<SizeGroup> transport_button_size_group2 = SizeGroup::create (SIZE_GROUP_HORIZONTAL);
-               transport_button_size_group2->add_widget (play_selection_button);
-               transport_button_size_group2->add_widget (join_play_range_button);
-               transport_button_size_group2->add_widget (roll_button);
-       }
+       a1->add (*vbox1);
+       a1->set (0.5, 1.0, 0.0, 0.0);
+       a2->add (*vbox2);
+       a2->set (0.5, 1.0, 0.0, 0.0);
+       a3->add (*vbox3);
+       a3->set (0.5, 1.0, 0.0, 0.0);
 
-       tbox->pack_start (stop_button, false, false);
-       tbox->pack_start (rec_button, false, false, 6);
+       tbox->pack_start (*a1, false, false);
+       tbox->pack_start (*a2, false, false);
+       tbox->pack_start (*a3, false, false);
 
        HBox* clock_box = manage (new HBox);