Rename 'MIDI' drop down 'Channel' (actually describes what's in it).

Fix packing of MIDI track header so Channel is at the top.
Partially separate min-size-to-show-channel and min-size-to-show-keyboard values from the old hLargest etc. variables (towards sticking more widgets in the header dynamically).

git-svn-id: svn://localhost/ardour2/branches/3.0@3863 d708f5d6-7413-0410-9779-e7cbd77b26cf

diff --git a/gtk2_ardour/midi_time_axis.cc b/gtk2_ardour/midi_time_axis.cc
index 53480f3ba553ee459b781e478c6b4afb26f982fd..ef66779f47401769c217c6db8091ad526e5d21f9 100644 (file)
--- a/gtk2_ardour/midi_time_axis.cc
+++ b/gtk2_ardour/midi_time_axis.cc
@@ -80,7 +80,10 @@ using namespace PBD;
 using namespace Gtk;
 using namespace sigc;
 using namespace Editing;
-
+       
+// Minimum height at which a control is displayed
+static const uint32_t CHANNEL_MIN_HEIGHT = 80;
+static const uint32_t KEYBOARD_MIN_HEIGHT = 140;
 
 MidiTimeAxisView::MidiTimeAxisView (PublicEditor& ed, Session& sess, boost::shared_ptr<Route> rt, Canvas& canvas)
        : AxisView(sess) // virtually inherited
@@ -91,7 +94,7 @@ MidiTimeAxisView::MidiTimeAxisView (PublicEditor& ed, Session& sess, boost::shar
        , _note_mode(Sustained)
        , _note_mode_item(NULL)
        , _percussion_mode_item(NULL)
-       , _midi_expander("MIDI")
+       , _midi_expander("Channel")
 {
        subplugin_menu.set_name ("ArdourContextMenu");
 
@@ -143,7 +146,7 @@ MidiTimeAxisView::MidiTimeAxisView (PublicEditor& ed, Session& sess, boost::shar
        _midi_expander.add(*channel_selector_box);
        _midi_expander.property_expanded().signal_changed().connect(
                        mem_fun(this, &MidiTimeAxisView::channel_selector_toggled));
-       controls_vbox.pack_end(_midi_expander, false, false);
+       controls_vbox.pack_start(_midi_expander, false, false);
        boost::shared_ptr<MidiDiskstream> diskstream = midi_track()->midi_diskstream();
 
        // restore channel selector settings
@@ -175,8 +178,6 @@ MidiTimeAxisView::show_at (double y, int& nth, Gtk::VBox *parent)
        xml_node->add_property ("shown_editor", "yes");
                
        guint32 ret = TimeAxisView::show_at (y, nth, parent);
-       _piano_roll_header->show();
-       _range_scroomer->show();
        return ret;
 }
 
@@ -194,34 +195,18 @@ MidiTimeAxisView::set_height (uint32_t h)
 {
        RouteTimeAxisView::set_height (h);
 
-       static const int EXPANDER_MIN_HEIGHT = 100;
-       static const int KEYBOARD_MIN_HEIGHT = 250;
-
-       if (height >= hLarger) {
-               _midi_expander.show();
-               if (is_track() && _range_scroomer)
-                       _range_scroomer->show();
-               if (is_track() && _piano_roll_header)
-                       _piano_roll_header->show();
-
-       } else if (height >= hLarge) {
+       if (height >= CHANNEL_MIN_HEIGHT) {
                _midi_expander.show();
-               _midi_expander.set_expanded(FALSE);
+       } else {
+               _midi_expander.hide();
+       }
+       
+       if (height >= KEYBOARD_MIN_HEIGHT) {
                if (is_track() && _range_scroomer)
                        _range_scroomer->show();
                if (is_track() && _piano_roll_header)
                        _piano_roll_header->show();
-
-       } else if (height >= hNormal) {
-               _midi_expander.show();
-               _midi_expander.set_expanded(FALSE);
-               if (is_track() && _range_scroomer)
-                       _range_scroomer->hide();
-               if (is_track() && _piano_roll_header)
-                       _piano_roll_header->hide();
        } else {
-               _midi_expander.hide();
-               _midi_expander.set_expanded(FALSE);
                if (is_track() && _range_scroomer)
                        _range_scroomer->hide();
                if (is_track() && _piano_roll_header)
@@ -463,16 +448,12 @@ MidiTimeAxisView::channel_selector_toggled()
 {
        static uint32_t previous_height;
        
-       if(_midi_expander.property_expanded()) {
-
+       if (_midi_expander.property_expanded()) {
                previous_height = current_height();
-
                if (previous_height < TimeAxisView::hLargest) {
                        set_height (TimeAxisView::hLarge);
                }
-
        } else {
-
                set_height (previous_height);
        }
 }