Bodge around visual glitch where the ranges & markers list would overlap the editor...
[ardour.git] / gtk2_ardour / visual_time_axis.cc
index c27ed33089bdab3ee2ce10ea3eb5c257ff7f80d2..9574a83c26436caad4dddcf243bbf2b7852997aa 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2003 Paul Davis 
+    Copyright (C) 2003 Paul Davis
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -15,7 +15,6 @@
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
 #include <cstdlib>
@@ -24,9 +23,9 @@
 #include <string>
 #include <vector>
 
-#include <pbd/error.h>
-#include <pbd/stl_delete.h>
-#include <pbd/whitespace.h>
+#include "pbd/error.h"
+#include "pbd/stl_delete.h"
+#include "pbd/whitespace.h"
 
 #include <gtkmm2ext/utils.h>
 #include <gtkmm2ext/selector.h>
 #include <gtkmm2ext/stop_signal.h>
 #include <gtkmm2ext/choice.h>
 
-#include <ardour/session.h>
-#include <ardour/utils.h>
-#include <ardour/insert.h>
-#include <ardour/location.h>
+#include "ardour/session.h"
+#include "ardour/utils.h"
+#include "ardour/processor.h"
+#include "ardour/location.h"
 
 #include "ardour_ui.h"
 #include "public_editor.h"
@@ -58,7 +57,7 @@ using namespace ARDOUR;
 using namespace PBD;
 using namespace sigc;
 using namespace Gtk;
-       
+
 /**
  * Abstract Constructor for base visual time axis classes
  *
@@ -75,13 +74,12 @@ VisualTimeAxis::VisualTimeAxis(const string & name, PublicEditor& ed, ARDOUR::Se
 {
        time_axis_name = name ;
        _color = unique_random_color() ;
-       _marked_for_display = true;
-       
+
        name_entry.signal_activate().connect(mem_fun(*this, &VisualTimeAxis::name_entry_changed)) ;
        name_entry.signal_button_press_event().connect(mem_fun(*this, &VisualTimeAxis::name_entry_button_press_handler)) ;
        name_entry.signal_button_release_event().connect(mem_fun(*this, &VisualTimeAxis::name_entry_button_release_handler)) ;
        name_entry.signal_key_release_event().connect(mem_fun(*this, &VisualTimeAxis::name_entry_key_release_handler)) ;
-       
+
        size_button.set_name("TrackSizeButton") ;
        visual_button.set_name("TrackVisualButton") ;
        hide_button.set_name("TrackRemoveButton") ;
@@ -92,7 +90,7 @@ VisualTimeAxis::VisualTimeAxis(const string & name, PublicEditor& ed, ARDOUR::Se
        ARDOUR_UI::instance()->tooltips().set_tip(size_button,_("Display Height")) ;
        ARDOUR_UI::instance()->tooltips().set_tip(visual_button, _("Visual options")) ;
        ARDOUR_UI::instance()->tooltips().set_tip(hide_button, _("Hide this track")) ;
-               
+
        controls_table.attach (hide_button, 0, 1, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
        controls_table.attach (visual_button, 1, 2, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
        controls_table.attach (size_button, 2, 3, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND);
@@ -101,8 +99,8 @@ VisualTimeAxis::VisualTimeAxis(const string & name, PublicEditor& ed, ARDOUR::Se
        size_button.unset_flags(Gtk::CAN_FOCUS) ;
        hide_button.unset_flags(Gtk::CAN_FOCUS) ;
        visual_button.unset_flags(Gtk::CAN_FOCUS) ;
-       
-       set_height(Normal) ;
+
+       set_height (hNormal) ;
 }
 
 /**
@@ -121,13 +119,13 @@ void
 VisualTimeAxis::set_time_axis_name(const string & name, void* src)
 {
        std::string old_name = time_axis_name ;
-       
+
        if(name != time_axis_name)
        {
                time_axis_name = name ;
                label_view() ;
                editor.route_name_changed(this) ;
-       
+
                 NameChanged(time_axis_name, old_name, src) ; /* EMIT_SIGNAL */
        }
 }
@@ -145,39 +143,25 @@ VisualTimeAxis::name() const
 /**
  * Sets the height of this TrackView to one of the defined TrackHeghts
  *
- * @param h the TrackHeight value to set
+ * @param h
  */
 void
-VisualTimeAxis::set_height(TrackHeight h)
+VisualTimeAxis::set_height(uint32_t h)
 {
-       TimeAxisView::set_height(h) ;
-       
-       switch (height)
-       {
-               case Largest:
-               case Large:
-               case Larger:
-               case Normal:
-               {
-                       hide_name_label ();
-                       show_name_entry ();
-                       other_button_hbox.show_all() ;
-                       break;
-               }
-               case Smaller:
-               {
-                       hide_name_label ();
-                       show_name_entry ();
-                       other_button_hbox.hide_all() ;
-                       break;
-               }
-               case Small:
-               {
-                       hide_name_entry ();
-                       show_name_label ();
-                       other_button_hbox.hide_all() ;
-               }
-               break;
+       TimeAxisView::set_height(h);
+
+       if (h >= hNormal) {
+               hide_name_label ();
+               show_name_entry ();
+               other_button_hbox.show_all() ;
+       } else if (h >= hSmaller) {
+               hide_name_label ();
+               show_name_entry ();
+               other_button_hbox.hide_all() ;
+       } else if (h >= hSmall) {
+               hide_name_entry ();
+               show_name_label ();
+               other_button_hbox.hide_all() ;
        }
 }
 
@@ -199,7 +183,12 @@ VisualTimeAxis::visual_click()
 void
 VisualTimeAxis::hide_click()
 {
+       // LAME fix for hide_button display refresh
+       hide_button.set_sensitive(false);
+
        editor.hide_track_in_display (*this);
+
+       hide_button.set_sensitive(true);
 }
 
 
@@ -227,7 +216,7 @@ VisualTimeAxis::choose_time_axis_color()
        Gdk::Color color ;
        gdouble current[4] ;
        Gdk::Color current_color ;
-       
+
        current[0] = _color.get_red() / 65535.0 ;
        current[1] = _color.get_green() / 65535.0 ;
        current[2] = _color.get_blue() / 65535.0 ;
@@ -235,7 +224,7 @@ VisualTimeAxis::choose_time_axis_color()
 
        current_color.set_rgb_p (current[0],current[1],current[2]);
        color = Gtkmm2ext::UI::instance()->get_color(_("ardour: color selection"),picked, &current_color) ;
-       
+
        if (picked)
        {
                set_time_axis_color(color) ;
@@ -255,7 +244,7 @@ VisualTimeAxis::set_time_axis_color(Gdk::Color c)
 }
 
 void
-VisualTimeAxis::set_selected_regionviews (AudioRegionSelection& regions)
+VisualTimeAxis::set_selected_regionviews (RegionSelection& regions)
 {
        // Not handled by purely visual TimeAxis
 }
@@ -311,7 +300,7 @@ VisualTimeAxis::idle_remove_this_time_axis(VisualTimeAxis* ta, void* src)
 
 //---------------------------------------------------------------------------------------//
 // Handle TimeAxis rename
-               
+
 /**
  * Construct a new prompt to receive a new name for this TimeAxis
  *
@@ -336,7 +325,7 @@ VisualTimeAxis::start_time_axis_rename()
                    ARDOUR_UI::instance()->popup_error (_("A track already exists with that name"));
                    return ;
                  }
-         
+
                  set_time_axis_name(result, this) ;
          }
        }
@@ -359,33 +348,33 @@ VisualTimeAxis::label_view()
 
 
 //---------------------------------------------------------------------------------------//
-// Handle name entry signals 
+// Handle name entry signals
 
 void
 VisualTimeAxis::name_entry_changed()
 {
        string x = name_entry.get_text ();
-       
+
        if (x == time_axis_name) {
                return;
        }
 
+       strip_whitespace_edges(x);
+
        if (x.length() == 0) {
                name_entry.set_text (time_axis_name);
                return;
        }
 
-       strip_whitespace_edges(x);
-
        if (!editor.get_named_time_axis(x)) {
                set_time_axis_name(x, this);
        } else {
-               ARDOUR_UI::instance()->popup_error (_("a track already exists with that name"));
+               ARDOUR_UI::instance()->popup_error (_("A track already exists with that name"));
                name_entry.set_text(time_axis_name);
        }
 }
 
-gint 
+gint
 VisualTimeAxis::name_entry_button_press_handler(GdkEventButton *ev)
 {
        if (ev->button == 3) {
@@ -394,7 +383,7 @@ VisualTimeAxis::name_entry_button_press_handler(GdkEventButton *ev)
        return FALSE;
 }
 
-gint 
+gint
 VisualTimeAxis::name_entry_button_release_handler(GdkEventButton *ev)
 {
        return FALSE;
@@ -418,9 +407,9 @@ VisualTimeAxis::name_entry_key_release_handler(GdkEventKey* ev)
 
 //---------------------------------------------------------------------------------------//
 // Super class methods not handled by VisualTimeAxis
-               
+
 void
-VisualTimeAxis::show_timestretch (jack_nframes_t start, jack_nframes_t end)
+VisualTimeAxis::show_timestretch (nframes_t start, nframes_t end)
 {
   // Not handled by purely visual TimeAxis
 }