Fix mantis bug #1619; de-selecting Options->Crossfades->Show now hides all crossfades...
[ardour.git] / gtk2_ardour / visual_time_axis.cc
index 71c0da3ebe879cc0b0a8ad81632b386e8e04a5e8..b68597114488e8e95a2992d22476b0f15186e1a6 100644 (file)
@@ -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>
@@ -26,6 +25,7 @@
 
 #include <pbd/error.h>
 #include <pbd/stl_delete.h>
+#include <pbd/whitespace.h>
 
 #include <gtkmm2ext/utils.h>
 #include <gtkmm2ext/selector.h>
@@ -41,7 +41,6 @@
 #include "ardour_ui.h"
 #include "public_editor.h"
 #include "imageframe_time_axis.h"
-#include "canvas-simplerect.h"
 #include "imageframe_time_axis_view.h"
 #include "marker_time_axis_view.h"
 #include "imageframe_view.h"
 #include "utils.h"
 #include "prompter.h"
 #include "rgb_macros.h"
+#include "canvas_impl.h"
 
 #include "i18n.h"
 
 using namespace ARDOUR;
+using namespace PBD;
 using namespace sigc;
 using namespace Gtk;
-
-//XXX should really have a common home...
-static const gchar* small_x_xpm[] = {
-       "11 11 2 1",
-       "       c None",
-       ".      c #000000",
-       "           ",
-       "           ",
-       "  .     .  ",
-       "   .   .   ",
-       "    . .    ",
-       "     .     ",
-       "    . .    ",
-       "   .   .   ",
-       "  .     .  ",
-       "           ",
-       "           "};
-
        
 /**
  * Abstract Constructor for base visual time axis classes
@@ -83,30 +66,28 @@ static const gchar* small_x_xpm[] = {
  * @param sess the current session
  * @param canvas the parent canvas object
  */
-VisualTimeAxis::VisualTimeAxis(std::string name, PublicEditor& ed, ARDOUR::Session& sess, Widget *canvas)
+VisualTimeAxis::VisualTimeAxis(const string & name, PublicEditor& ed, ARDOUR::Session& sess, Canvas& canvas)
        : AxisView(sess),
          TimeAxisView(sess,ed,(TimeAxisView*) 0, canvas),
          visual_button (_("v")),
          size_button (_("h"))
 {
        time_axis_name = name ;
-       name_prompter = 0 ;
        _color = unique_random_color() ;
        _marked_for_display = true;
        
-       name_entry.activate.connect(slot(*this, &VisualTimeAxis::name_entry_changed)) ;
-       name_entry.focus_out_event.connect(slot(*this, &VisualTimeAxis::name_entry_focus_out_handler)) ;
-       name_entry.button_press_event.connect(slot(*this, &VisualTimeAxis::name_entry_button_press_handler)) ;
-       name_entry.button_release_event.connect(slot(*this, &VisualTimeAxis::name_entry_button_release_handler)) ;
-       name_entry.key_release_event.connect(slot(*this, &VisualTimeAxis::name_entry_key_release_handler)) ;
+       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") ;
-       hide_button.add(*(Gtk::manage(new Gtk::Image(small_x_xpm)))) ;
-       size_button.button_release_event.connect (slot (*this, &VisualTimeAxis::size_click)) ;
-       visual_button.signal_clicked().connect (slot (*this, &VisualTimeAxis::visual_click)) ;
-       hide_button.signal_clicked().connect (slot (*this, &VisualTimeAxis::hide_click)) ;
+       hide_button.add(*(Gtk::manage(new Gtk::Image(get_xpm("small_x.xpm")))));
+       size_button.signal_button_release_event().connect (mem_fun (*this, &VisualTimeAxis::size_click)) ;
+       visual_button.signal_clicked().connect (mem_fun (*this, &VisualTimeAxis::visual_click)) ;
+       hide_button.signal_clicked().connect (mem_fun (*this, &VisualTimeAxis::hide_click)) ;
        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")) ;
@@ -129,11 +110,6 @@ VisualTimeAxis::VisualTimeAxis(std::string name, PublicEditor& ed, ARDOUR::Sessi
  */
 VisualTimeAxis::~VisualTimeAxis()
 {
-       if(name_prompter)
-       {
-               delete name_prompter ;
-               name_prompter = 0 ;
-       }
 }
 
 
@@ -141,7 +117,7 @@ VisualTimeAxis::~VisualTimeAxis()
 // Name/Id Accessors/Mutators
 
 void
-VisualTimeAxis::set_time_axis_name(std::string name, void* src)
+VisualTimeAxis::set_time_axis_name(const string & name, void* src)
 {
        std::string old_name = time_axis_name ;
        
@@ -182,22 +158,22 @@ VisualTimeAxis::set_height(TrackHeight h)
                case Larger:
                case Normal:
                {
-                       name_label.hide() ;
-                       name_entry.show() ;
+                       hide_name_label ();
+                       show_name_entry ();
                        other_button_hbox.show_all() ;
                        break;
                }
                case Smaller:
                {
-                       name_label.hide() ;
-                       name_entry.show() ;
+                       hide_name_label ();
+                       show_name_entry ();
                        other_button_hbox.hide_all() ;
                        break;
                }
                case Small:
                {
-                       name_label.show() ;
-                       name_entry.hide() ;
+                       hide_name_entry ();
+                       show_name_label ();
                        other_button_hbox.hide_all() ;
                }
                break;
@@ -222,7 +198,12 @@ VisualTimeAxis::visual_click()
 void
 VisualTimeAxis::hide_click()
 {
-       editor.unselect_strip_in_display (*this);
+       // LAME fix for hide_button display refresh
+       hide_button.set_sensitive(false);
+       
+       editor.hide_track_in_display (*this);
+       
+       hide_button.set_sensitive(true);
 }
 
 
@@ -247,15 +228,17 @@ bool
 VisualTimeAxis::choose_time_axis_color()
 {
        bool picked ;
-       GdkColor 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 ;
        current[3] = 1.0 ;
-       
-       color = Gtkmm2ext::UI::instance()->get_color(_("ardour: color selection"),picked, current) ;
+
+       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)
        {
@@ -270,13 +253,13 @@ VisualTimeAxis::choose_time_axis_color()
  * @param c the new TimeAxis color
  */
 void
-VisualTimeAxis::set_time_axis_color(Gdk_Color c)
+VisualTimeAxis::set_time_axis_color(Gdk::Color c)
 {
        _color = c ;
 }
 
 void
-VisualTimeAxis::set_selected_regionviews (AudioRegionSelection& regions)
+VisualTimeAxis::set_selected_regionviews (RegionSelection& regions)
 {
        // Not handled by purely visual TimeAxis
 }
@@ -294,25 +277,19 @@ VisualTimeAxis::remove_this_time_axis(void* src)
 {
        vector<string> choices;
 
-       std::string prompt  = compose (_("Do you really want to remove track \"%1\" ?\n(cannot be undone)"), time_axis_name);
+       std::string prompt  = string_compose (_("Do you really want to remove track \"%1\" ?\n(cannot be undone)"), time_axis_name);
 
-       choices.push_back (_("Yes, remove it."));
        choices.push_back (_("No, do nothing."));
+       choices.push_back (_("Yes, remove it."));
 
        Gtkmm2ext::Choice prompter (prompt, choices);
 
-       prompter.chosen.connect (Gtk::Main::quit.slot());
-       prompter.show_all ();
-
-       Gtk::Main::run ();
-
-       if (prompter.get_choice() == 0)
-       {
+       if (prompter.run () == 1) {
                /*
-                  defer to idle loop, otherwise we'll delete this object
-                  while we're still inside this function ...
+                 defer to idle loop, otherwise we'll delete this object
+                 while we're still inside this function ...
                */
-               Gtk::Main::idle.connect(bind(slot(&VisualTimeAxis::idle_remove_this_time_axis), this, src));
+               Glib::signal_idle().connect(bind(sigc::ptr_fun(&VisualTimeAxis::idle_remove_this_time_axis), this, src));
        }
 }
 
@@ -347,18 +324,27 @@ VisualTimeAxis::idle_remove_this_time_axis(VisualTimeAxis* ta, void* src)
 void
 VisualTimeAxis::start_time_axis_rename()
 {
-       if(name_prompter)
-       {
-               delete name_prompter ;
-               name_prompter = 0 ;
+       ArdourPrompter name_prompter;
+
+       name_prompter.set_prompt (_("new name: ")) ;
+       name_prompter.add_button (_("Rename"), Gtk::RESPONSE_ACCEPT);
+       name_prompter.set_response_sensitive (Gtk::RESPONSE_ACCEPT, false);
+       name_prompter.show_all() ;
+
+       switch (name_prompter.run ()) {
+       case Gtk::RESPONSE_ACCEPT:
+         string result;
+         name_prompter.get_result (result);
+         if (result.length()) {
+                 if (editor.get_named_time_axis(result) != 0) {
+                   ARDOUR_UI::instance()->popup_error (_("A track already exists with that name"));
+                   return ;
+                 }
+         
+                 set_time_axis_name(result, this) ;
+         }
        }
-
-       name_prompter = new ArdourPrompter() ;
-
-       name_prompter->set_prompt (_("new name: ")) ;
-       ARDOUR_UI::instance()->allow_focus(true) ;
-       name_prompter->done.connect (slot(*this, &VisualTimeAxis::finish_time_axis_rename)) ;
-       name_prompter->show_all() ;
+       label_view() ;
 }
 
 /**
@@ -366,37 +352,7 @@ VisualTimeAxis::start_time_axis_rename()
  *
  * @see start_time_axis_rename()
  */
-void
-VisualTimeAxis::finish_time_axis_rename()
-{
-       name_prompter->hide_all () ;
-       ARDOUR_UI::instance()->allow_focus (false) ;
-       
-       if (name_prompter->status == Gtkmm2ext::Prompter::cancelled)
-       {
-               return;
-       }
-       
-       string result ;
-       name_prompter->get_result (result) ;
-       //time_axis_name = result ;
-       
-       if (editor.get_named_time_axis(result) != 0) {
-               ARDOUR_UI::instance()->popup_error (_("A track already exists with that name"));
-               return ;
-       }
 
-       set_time_axis_name(result, this) ;
-       
-       delete name_prompter ;
-       name_prompter = 0 ;
-       label_view() ;
-}
-
-/**
- * Handle the (re-)displaying of the TimeAxis name label
- *
- */
 void
 VisualTimeAxis::label_view()
 {
@@ -412,25 +368,23 @@ VisualTimeAxis::label_view()
 void
 VisualTimeAxis::name_entry_changed()
 {
-       ARDOUR_UI::generic_focus_out_event (0);
-
        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);
        }
 }
@@ -450,13 +404,6 @@ VisualTimeAxis::name_entry_button_release_handler(GdkEventButton *ev)
        return FALSE;
 }
 
-gint
-VisualTimeAxis::name_entry_focus_out_handler(GdkEventFocus* ev)
-{
-       name_entry_changed();
-       return TRUE;
-}
-
 gint
 VisualTimeAxis::name_entry_key_release_handler(GdkEventKey* ev)
 {
@@ -477,7 +424,7 @@ 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
 }