Apply patch from mantis 2766 to fix apply sensitivity when editing the meter marker...
[ardour.git] / gtk2_ardour / location_ui.cc
index d10f43cd307bc4a1d48c67ad401f0258dc71ded9..7aeb8b33017695e1dddb4e4472e6e4e341d67bac 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 <cmath>
 
 #include <gtkmm2ext/utils.h>
 #include <gtkmm2ext/stop_signal.h>
+#include <gtkmm2ext/window_title.h>
 
-#include <ardour/utils.h>
-#include <ardour/configuration.h>
-#include <ardour/session.h>
+#include "ardour/utils.h"
+#include "ardour/configuration.h"
+#include "ardour/session.h"
+#include "pbd/memento_command.h"
 
 #include "ardour_ui.h"
 #include "prompter.h"
 
 #include "i18n.h"
 
+using namespace std;
 using namespace ARDOUR;
+using namespace PBD;
 using namespace Gtk;
 using namespace Gtkmm2ext;
 
 LocationEditRow::LocationEditRow(Session * sess, Location * loc, int32_t num)
        : location(0), session(0),
          item_table (1, 7, false),
-         start_set_button (_("Set")),
+         start_set_button (_("Use PH")),
          start_go_button (_("Go")),
-         start_clock (X_("LocationEditRowClock"), true),
-         end_set_button (_("Set")),
+         start_clock (X_("locationstart"), true, X_("LocationEditRowClock"), true),
+         end_set_button (_("Use PH")),
          end_go_button (_("Go")),
-         end_clock (X_("LocationEditRowClock"), true),
-         length_clock (X_("LocationEditRowClock"), true, true),
+         end_clock (X_("locationend"), true, X_("LocationEditRowClock"), true),
+         length_clock (X_("locationlength"), true, X_("LocationEditRowClock"), true, true),
          cd_check_button (_("CD")),
          hide_check_button (_("Hidden")),
          remove_button (_("Remove")),
@@ -80,7 +83,9 @@ LocationEditRow::LocationEditRow(Session * sess, Location * loc, int32_t num)
        composer_label.set_name ("LocationEditNumberLabel");
        composer_entry.set_name ("LocationEditNameEntry");
 
-
+       ARDOUR_UI::instance()->tooltips().set_tip(start_set_button, _("Set value to Playhead"));
+       ARDOUR_UI::instance()->tooltips().set_tip(end_set_button, _("Set value to Playhead"));
        isrc_label.set_text ("ISRC: ");
        isrc_label.set_size_request (30, -1);
        performer_label.set_text ("Performer: ");
@@ -108,17 +113,8 @@ LocationEditRow::LocationEditRow(Session * sess, Location * loc, int32_t num)
        cd_track_details_hbox.pack_start (composer_entry, true, true);
 
        isrc_entry.signal_changed().connect (mem_fun(*this, &LocationEditRow::isrc_entry_changed)); 
-       isrc_entry.signal_focus_in_event().connect (mem_fun(*this, &LocationEditRow::entry_focus_event));
-       isrc_entry.signal_focus_out_event().connect (mem_fun(*this, &LocationEditRow::entry_focus_event));
-
        performer_entry.signal_changed().connect (mem_fun(*this, &LocationEditRow::performer_entry_changed));
-       performer_entry.signal_focus_in_event().connect (mem_fun(*this, &LocationEditRow::entry_focus_event));
-       performer_entry.signal_focus_out_event().connect (mem_fun(*this, &LocationEditRow::entry_focus_event));
-
        composer_entry.signal_changed().connect (mem_fun(*this, &LocationEditRow::composer_entry_changed));
-       composer_entry.signal_focus_in_event().connect (mem_fun(*this, &LocationEditRow::entry_focus_event));
-       composer_entry.signal_focus_out_event().connect (mem_fun(*this, &LocationEditRow::entry_focus_event));
-
        scms_check_button.signal_toggled().connect(mem_fun(*this, &LocationEditRow::scms_toggled));
        preemph_check_button.signal_toggled().connect(mem_fun(*this, &LocationEditRow::preemph_toggled));
 
@@ -138,6 +134,7 @@ LocationEditRow::LocationEditRow(Session * sess, Location * loc, int32_t num)
        start_set_button.signal_clicked().connect(bind (mem_fun (*this, &LocationEditRow::set_button_pressed), LocStart));
        start_go_button.signal_clicked().connect(bind (mem_fun (*this, &LocationEditRow::go_button_pressed), LocStart));
        start_clock.ValueChanged.connect (bind (mem_fun (*this, &LocationEditRow::clock_changed), LocStart));
+       start_clock.ChangeAborted.connect (bind (mem_fun (*this, &LocationEditRow::change_aborted), LocStart));
 
        
        end_hbox.pack_start (end_go_button, false, false);
@@ -149,9 +146,11 @@ LocationEditRow::LocationEditRow(Session * sess, Location * loc, int32_t num)
        end_set_button.signal_clicked().connect(bind (mem_fun (*this, &LocationEditRow::set_button_pressed), LocEnd));
        end_go_button.signal_clicked().connect(bind (mem_fun (*this, &LocationEditRow::go_button_pressed), LocEnd));
        end_clock.ValueChanged.connect (bind (mem_fun (*this, &LocationEditRow::clock_changed), LocEnd));
+       end_clock.ChangeAborted.connect (bind (mem_fun (*this, &LocationEditRow::change_aborted), LocEnd));
        
 //     item_table.attach (length_clock, 3, 4, 0, 1, 0, 0, 4, 0);
        length_clock.ValueChanged.connect (bind ( mem_fun(*this, &LocationEditRow::clock_changed), LocLength));
+       length_clock.ChangeAborted.connect (bind (mem_fun (*this, &LocationEditRow::change_aborted), LocLength));
 
 //     item_table.attach (cd_check_button, 4, 5, 0, 1, 0, Gtk::FILL, 4, 0);
 //     item_table.attach (hide_check_button, 5, 6, 0, 1, 0, Gtk::FILL, 4, 0);
@@ -198,7 +197,7 @@ LocationEditRow::set_number (int num)
        number = num;
 
        if (number >= 0 ) {
-               number_label.set_text (compose ("%1", number));
+               number_label.set_text (string_compose ("%1", number));
        }
 }
 
@@ -222,14 +221,14 @@ LocationEditRow::set_location (Location *loc)
        }
        hide_check_button.set_active (location->is_hidden());
        
-       if (location->is_auto_loop() || location->is_auto_punch()) {
+       if (location->is_auto_loop() || location-> is_auto_punch()) {
                // use label instead of entry
 
                name_label.set_text (location->name());
                name_label.set_size_request (80, -1);
 
                if (!name_label.get_parent()) {
-                       item_table.attach (name_label, 1, 2, 0, 1, FILL, Gtk::FILL, 4, 0);
+                       item_table.attach (name_label, 1, 2, 0, 1, FILL, FILL, 4, 0);
                }
                
                name_label.show();
@@ -240,8 +239,6 @@ LocationEditRow::set_location (Location *loc)
                name_entry.set_size_request (100, -1);
                name_entry.set_editable (true);
                name_entry.signal_changed().connect (mem_fun(*this, &LocationEditRow::name_entry_changed));  
-               name_entry.signal_focus_in_event().connect (mem_fun(*this, &LocationEditRow::entry_focus_event));
-               name_entry.signal_focus_out_event().connect (mem_fun(*this, &LocationEditRow::entry_focus_event));
 
                if (!name_entry.get_parent()) {
                        item_table.attach (name_entry, 1, 2, 0, 1, FILL | EXPAND, FILL, 4, 0);
@@ -249,27 +246,34 @@ LocationEditRow::set_location (Location *loc)
                name_entry.show();
 
                if (!cd_check_button.get_parent()) {
-                       item_table.attach (cd_check_button, 5, 6, 0, 1, FILL, Gtk::FILL, 4, 0);
+                       item_table.attach (cd_check_button, 5, 6, 0, 1, FILL, FILL, 4, 0);
                }
                if (!remove_button.get_parent()) {
-                       item_table.attach (remove_button, 7, 8, 0, 1, FILL, Gtk::FILL, 4, 0);
+                       item_table.attach (remove_button, 7, 8, 0, 1, FILL, FILL, 4, 0);
                }
 
                /* XXX i can't find a way to hide the button without messing up 
                   the row spacing, so make it insensitive (paul).
                */
 
-               if (location->is_end()) {
+               if (location->is_end() || location->is_start()) {
                        remove_button.set_sensitive (false);
                }
 
                cd_check_button.set_active (location->is_cd_marker());
                cd_check_button.show();
+
+               if (location->start() == session->current_start_frame()) {
+                       cd_check_button.set_sensitive (false);
+               } else {
+                       cd_check_button.set_sensitive (true);
+               }
+
                hide_check_button.show();
        }
 
        start_clock.set (location->start(), true);
-       
+
 
        if (!location->is_mark()) {
                if (!end_hbox.get_parent()) {
@@ -286,34 +290,27 @@ LocationEditRow::set_location (Location *loc)
                end_go_button.show();
                end_clock.show();
                length_clock.show();
-       }
-       else {
+
+       } else {
+
                end_set_button.hide();
                end_go_button.hide();
                end_clock.hide();
                length_clock.hide();
+
        }
 
+       start_clock.set_sensitive (!location->locked());
+       end_clock.set_sensitive (!location->locked());
+       length_clock.set_sensitive (!location->locked());
+
        start_changed_connection = location->start_changed.connect (mem_fun(*this, &LocationEditRow::start_changed));
        end_changed_connection = location->end_changed.connect (mem_fun(*this, &LocationEditRow::end_changed));
        name_changed_connection = location->name_changed.connect (mem_fun(*this, &LocationEditRow::name_changed));
        changed_connection = location->changed.connect (mem_fun(*this, &LocationEditRow::location_changed));
        flags_changed_connection = location->FlagsChanged.connect (mem_fun(*this, &LocationEditRow::flags_changed));
-       
 }
 
-gint
-LocationEditRow::entry_focus_event (GdkEventFocus* ev)
-{
-       if (ev->in) {
-               ARDOUR_UI::instance()->allow_focus (true);
-       } else {
-               ARDOUR_UI::instance()->allow_focus (false);
-       }
-       return TRUE;
-}
-
-
 void
 LocationEditRow::name_entry_changed ()
 {
@@ -423,47 +420,67 @@ LocationEditRow::clock_changed (LocationPart part)
 
 }
 
+void
+LocationEditRow::change_aborted (LocationPart /*part*/)
+{
+       if (i_am_the_modifier || !location) return;
+       
+       set_location(location);
+}
+
 void
 LocationEditRow::cd_toggled ()
 {
+       if (i_am_the_modifier || !location) {
+               return;
+       }
+       
+       //if (cd_check_button.get_active() == location->is_cd_marker()) {
+       //      return;
+       //}
+
+       if (cd_check_button.get_active()) {
+               if (location->start() <= session->current_start_frame()) {
+                       error << _("You cannot put a CD marker at the start of the session") << endmsg;
+                       cd_check_button.set_active (false);
+                       return;
+               }
+       }
 
-       if (i_am_the_modifier || !location) return;
        location->set_cd (cd_check_button.get_active(), this);
 
        if (location->is_cd_marker() && !(location->is_mark())) {
 
-         if (location->cd_info.find("isrc") != location->cd_info.end()) {
-           isrc_entry.set_text(location->cd_info["isrc"]);
-         }
-         if (location->cd_info.find("performer") != location->cd_info.end()) {
-           performer_entry.set_text(location->cd_info["performer"]);
-         }
-         if (location->cd_info.find("composer") != location->cd_info.end()) {
-           composer_entry.set_text(location->cd_info["composer"]);
-         }
-         if (location->cd_info.find("scms") != location->cd_info.end()) {
-           scms_check_button.set_active(true);
-         }
-         if (location->cd_info.find("preemph") != location->cd_info.end()) {
-           preemph_check_button.set_active(true);
-         }
-         
-         if(!cd_track_details_hbox.get_parent()) {
-           item_table.attach (cd_track_details_hbox, 1, 8, 1, 2, FILL | EXPAND, FILL, 4, 0);
-         }
-         // item_table.resize(2, 7);
-         cd_track_details_hbox.show_all();
-
+               if (location->cd_info.find("isrc") != location->cd_info.end()) {
+                       isrc_entry.set_text(location->cd_info["isrc"]);
+               }
+               if (location->cd_info.find("performer") != location->cd_info.end()) {
+                       performer_entry.set_text(location->cd_info["performer"]);
+               }
+               if (location->cd_info.find("composer") != location->cd_info.end()) {
+                       composer_entry.set_text(location->cd_info["composer"]);
+               }
+               if (location->cd_info.find("scms") != location->cd_info.end()) {
+                       scms_check_button.set_active(true);
+               }
+               if (location->cd_info.find("preemph") != location->cd_info.end()) {
+                       preemph_check_button.set_active(true);
+               }
+               
+               if (!cd_track_details_hbox.get_parent()) {
+                       item_table.attach (cd_track_details_hbox, 1, 8, 1, 2, FILL | EXPAND, FILL, 4, 0);
+               }
+               // item_table.resize(2, 7);
+               cd_track_details_hbox.show_all();
+               
        } else if (cd_track_details_hbox.get_parent()){
-
-           item_table.remove (cd_track_details_hbox);  
-           //    item_table.resize(1, 7);
-           redraw_ranges(); /*         EMIT_SIGNAL */
+               
+               item_table.remove (cd_track_details_hbox);      
+               //        item_table.resize(1, 7);
+               redraw_ranges(); /*     EMIT_SIGNAL */
        }
-
 }
 
-
 void
 LocationEditRow::hide_toggled ()
 {
@@ -527,13 +544,19 @@ void
 LocationEditRow::start_changed (ARDOUR::Location *loc)
 {
        ENSURE_GUI_THREAD(bind (mem_fun(*this, &LocationEditRow::start_changed), loc));
-       
+
        if (!location) return;
        
        // update end and length
        i_am_the_modifier++;
 
        start_clock.set (location->start());
+
+       if (location->start() == session->current_start_frame()) {
+               cd_check_button.set_sensitive (false);
+       } else {
+               cd_check_button.set_sensitive (true);
+       }
        
        i_am_the_modifier--;
 }
@@ -568,6 +591,10 @@ LocationEditRow::location_changed (ARDOUR::Location *loc)
        end_clock.set (location->end());
        length_clock.set (location->length());
 
+       start_clock.set_sensitive (!location->locked());
+       end_clock.set_sensitive (!location->locked());
+       length_clock.set_sensitive (!location->locked());
+
        i_am_the_modifier--;
 
 }
@@ -587,34 +614,41 @@ LocationEditRow::flags_changed (ARDOUR::Location *loc, void *src)
        i_am_the_modifier--;
 }
 
+void
+LocationEditRow::focus_name() {
+       name_entry.grab_focus();
+}
+
+
 LocationUI::LocationUI ()
-       : ArdourDialog ("location dialog"),
+       : ArdourDialog ("locations dialog"),
          add_location_button (_("Add New Location")),
          add_range_button (_("Add New Range"))
 {
        i_am_the_modifier = 0;
+       
+       WindowTitle title(Glib::get_application_name());
+       title += _("Locations");
 
-       set_title(_("ardour: locations"));
-       set_wmclass(_("ardour_locations"), "Ardour");
+       set_title(title.get_string());
+       set_wmclass(X_("ardour_locations"), "Ardour");
 
        set_name ("LocationWindow");
-       signal_delete_event().connect (bind (ptr_fun (just_hide_it), static_cast<Window*>(this)));
 
-       add (location_hpacker);
+       get_vbox()->pack_start (location_hpacker);
 
-
-       location_vpacker.set_border_width (10);
        location_vpacker.set_spacing (5);
 
-
        location_vpacker.pack_start (loop_edit_row, false, false);
        location_vpacker.pack_start (punch_edit_row, false, false);
 
        location_rows.set_name("LocationLocRows");
-       location_rows_scroller.add_with_viewport (location_rows);
+       location_rows_scroller.add (location_rows);
        location_rows_scroller.set_name ("LocationLocRowsScroller");
        location_rows_scroller.set_policy (Gtk::POLICY_NEVER, Gtk::POLICY_AUTOMATIC);
        location_rows_scroller.set_size_request (-1, 130);
+       
+       newest_location = 0;
 
        loc_frame_box.set_spacing (5);
        loc_frame_box.set_border_width (5);
@@ -626,13 +660,13 @@ LocationUI::LocationUI ()
        loc_frame_box.pack_start (add_location_button, false, false);
 
        loc_frame.set_name ("LocationLocEditorFrame");
-       loc_frame.set_label (_("Location (CD Index) Markers"));
+       loc_frame.set_label (_("Location (CD index) markers"));
        loc_frame.add (loc_frame_box);
        loc_range_panes.pack1(loc_frame, true, false);
 
        
        range_rows.set_name("LocationRangeRows");
-       range_rows_scroller.add_with_viewport (range_rows);
+       range_rows_scroller.add (range_rows);
        range_rows_scroller.set_name ("LocationRangeRowsScroller");
        range_rows_scroller.set_policy (Gtk::POLICY_NEVER, Gtk::POLICY_AUTOMATIC);
        range_rows_scroller.set_size_request (-1, 130);
@@ -646,7 +680,7 @@ LocationUI::LocationUI ()
        range_frame_box.pack_start (add_range_button, false, false);
 
        range_frame.set_name ("LocationRangeEditorFrame");
-       range_frame.set_label (_("Range (CD Track) Markers"));
+       range_frame.set_label (_("Range (CD track) markers"));
        range_frame.add (range_frame_box);
        loc_range_panes.pack2(range_frame, true, false);
        location_vpacker.pack_start (loc_range_panes, true, true);
@@ -665,6 +699,11 @@ LocationUI::~LocationUI()
 {
 }
 
+void LocationUI::on_show()
+{
+       ArdourDialog::on_show();
+       refresh_location_list();
+}
 
 
 gint LocationUI::do_location_remove (ARDOUR::Location *loc)
@@ -679,9 +718,10 @@ gint LocationUI::do_location_remove (ARDOUR::Location *loc)
        }
 
        session->begin_reversible_command (_("remove marker"));
-       session->add_undo (session->locations()->get_memento());
+       XMLNode &before = session->locations()->get_state();
        session->locations()->remove (loc);
-       session->add_redo_no_execute (session->locations()->get_memento());
+       XMLNode &after = session->locations()->get_state();
+       session->add_command(new MementoCommand<Locations>(*(session->locations()), &before, &after));
        session->commit_reversible_command ();
 
        return FALSE;
@@ -770,14 +810,20 @@ LocationUI::map_locations (Locations::LocationList& locations)
                        erow->remove_requested.connect (mem_fun(*this, &LocationUI::location_remove_requested));
                        erow->redraw_ranges.connect (mem_fun(*this, &LocationUI::location_redraw_ranges));
                        loc_children.push_back(Box_Helpers::Element(*erow, PACK_SHRINK, 1, PACK_START));
+                       if (location == newest_location) {
+                               newest_location = 0;
+                               erow->focus_name();
+                       }
                }
                else if (location->is_auto_punch()) {
                        punch_edit_row.set_session (session);
                        punch_edit_row.set_location (location);
+                       punch_edit_row.show_all();
                }
                else if (location->is_auto_loop()) {
                        loop_edit_row.set_session (session);
                        loop_edit_row.set_location (location);
+                       loop_edit_row.show_all();
                }
                else {
                        erow = manage (new LocationEditRow(session, location));
@@ -793,13 +839,20 @@ LocationUI::map_locations (Locations::LocationList& locations)
 void
 LocationUI::add_new_location()
 {
+       string markername;
+
        if (session) {
-               jack_nframes_t where = session->audible_frame();
-               Location *location = new Location (where, where, "mark", Location::IsMark);
+               nframes_t where = session->audible_frame();
+               session->locations()->next_available_name(markername,"mark");
+               Location *location = new Location (where, where, markername, Location::IsMark);
+               if (Config->get_name_new_markers()) {
+                       newest_location = location;                     
+               }
                session->begin_reversible_command (_("add marker"));
-               session->add_undo (session->locations()->get_memento());
+               XMLNode &before = session->locations()->get_state();
                session->locations()->add (location, true);
-               session->add_redo_no_execute (session->locations()->get_memento());
+               XMLNode &after = session->locations()->get_state();
+               session->add_command (new MementoCommand<Locations>(*(session->locations()), &before, &after));
                session->commit_reversible_command ();
        }
        
@@ -808,13 +861,17 @@ LocationUI::add_new_location()
 void
 LocationUI::add_new_range()
 {
+       string rangename;
+
        if (session) {
-               jack_nframes_t where = session->audible_frame();
-               Location *location = new Location (where, where, "unnamed");
+               nframes_t where = session->audible_frame();
+               session->locations()->next_available_name(rangename,"unnamed");
+               Location *location = new Location (where, where, rangename, Location::IsRangeMarker);
                session->begin_reversible_command (_("add range marker"));
-               session->add_undo (session->locations()->get_memento());
+               XMLNode &before = session->locations()->get_state();
                session->locations()->add (location, true);
-               session->add_redo_no_execute (session->locations()->get_memento());
+               XMLNode &after = session->locations()->get_state();
+               session->add_command (new MementoCommand<Locations>(*(session->locations()), &before, &after));
                session->commit_reversible_command ();
        }
 }
@@ -834,6 +891,9 @@ LocationUI::refresh_location_list ()
        ENSURE_GUI_THREAD(mem_fun(*this, &LocationUI::refresh_location_list));
        using namespace Box_Helpers;
 
+       // this is just too expensive to do when window is not shown
+       if (!is_visible()) return;
+
        BoxList & loc_children = location_rows.children();
        BoxList & range_children = range_rows.children();
 
@@ -856,7 +916,7 @@ LocationUI::set_session(ARDOUR::Session* sess)
                session->locations()->StateChanged.connect (mem_fun(*this, &LocationUI::refresh_location_list_s));
                session->locations()->added.connect (mem_fun(*this, &LocationUI::location_added));
                session->locations()->removed.connect (mem_fun(*this, &LocationUI::location_removed));
-               session->going_away.connect (mem_fun(*this, &LocationUI::session_gone));
+               session->GoingAway.connect (mem_fun(*this, &LocationUI::session_gone));
        }
        refresh_location_list ();
 }
@@ -880,7 +940,13 @@ LocationUI::session_gone()
 
        punch_edit_row.set_session (0);
        punch_edit_row.set_location (0);
-       
-       ArdourDialog::session_gone();
+
+       ArdourDialog::session_gone ();
 }
 
+bool
+LocationUI::on_delete_event (GdkEventAny*)
+{
+       hide ();
+       return true;
+}