X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Flocation_ui.cc;h=7aeced896e9fdeb42586d69eb8f7e51c8a3cb884;hb=8d8666d041f0d0336a48389176ec0631c45cc6ff;hp=004bf9e7658edda79b0d1262223b347367db3b80;hpb=0938a42440cc82ce8d0cb064840c258c863714ab;p=ardour.git diff --git a/gtk2_ardour/location_ui.cc b/gtk2_ardour/location_ui.cc index 004bf9e765..7aeced896e 100644 --- a/gtk2_ardour/location_ui.cc +++ b/gtk2_ardour/location_ui.cc @@ -22,174 +22,181 @@ #include -#include "ardour/utils.h" -#include "ardour/configuration.h" #include "ardour/session.h" #include "pbd/memento_command.h" #include "ardour_ui.h" #include "clock_group.h" +#include "main_clock.h" #include "gui_thread.h" #include "keyboard.h" #include "location_ui.h" #include "prompter.h" #include "utils.h" +#include "public_editor.h" +#include "tooltips.h" +#include "ui_config.h" -#include "i18n.h" +#include "pbd/i18n.h" using namespace std; using namespace ARDOUR; +using namespace ARDOUR_UI_UTILS; using namespace PBD; using namespace Gtk; using namespace Gtkmm2ext; LocationEditRow::LocationEditRow(Session * sess, Location * loc, int32_t num) : SessionHandlePtr (0) /* explicitly set below */ - , location(0) - , item_table (1, 6, false) - , start_clock (X_("locationstart"), true, X_("LocationEditRowClock"), true, false) - , end_clock (X_("locationend"), true, X_("LocationEditRowClock"), true, false) - , length_clock (X_("locationlength"), true, X_("LocationEditRowClock"), true, false, true) - , cd_check_button (_("CD")) - , hide_check_button (_("Hide")) - , lock_check_button (_("Lock")) - , glue_check_button (_("Glue")) - , scms_check_button (_("SCMS")) - , preemph_check_button (_("Pre-Emphasis")) - , _clock_group (0) - { - i_am_the_modifier = 0; - - start_go_button.set_image (*manage (new Image (Stock::JUMP_TO, Gtk::ICON_SIZE_SMALL_TOOLBAR))); - end_go_button.set_image (*manage (new Image (Stock::JUMP_TO, Gtk::ICON_SIZE_SMALL_TOOLBAR))); - remove_button.set_image (*manage (new Image (Stock::REMOVE, Gtk::ICON_SIZE_SMALL_TOOLBAR))); - - number_label.set_name ("LocationEditNumberLabel"); - name_label.set_name ("LocationEditNameLabel"); - name_entry.set_name ("LocationEditNameEntry"); - start_go_button.set_name ("LocationEditGoButton"); - end_go_button.set_name ("LocationEditGoButton"); - cd_check_button.set_name ("LocationEditCdButton"); - hide_check_button.set_name ("LocationEditHideButton"); - lock_check_button.set_name ("LocationEditLockButton"); - glue_check_button.set_name ("LocationEditGlueButton"); - remove_button.set_name ("LocationEditRemoveButton"); - isrc_label.set_name ("LocationEditNumberLabel"); - isrc_entry.set_name ("LocationEditNameEntry"); - scms_check_button.set_name ("LocationEditCdButton"); - preemph_check_button.set_name ("LocationEditCdButton"); - performer_label.set_name ("LocationEditNumberLabel"); - performer_entry.set_name ("LocationEditNameEntry"); - composer_label.set_name ("LocationEditNumberLabel"); - composer_entry.set_name ("LocationEditNameEntry"); - - Gtk::Button* start_to_playhead_button = manage (new Button (_("Use PH"))); - Gtk::Button* end_to_playhead_button = manage (new Button (_("Use PH"))); - - ARDOUR_UI::instance()->tooltips().set_tip (*start_to_playhead_button, _("Set value to playhead")); - ARDOUR_UI::instance()->tooltips().set_tip (*end_to_playhead_button, _("Set value to playhead")); - - isrc_label.set_text ("ISRC: "); - isrc_label.set_size_request (30, -1); - performer_label.set_text ("Performer: "); - performer_label.set_size_request (60, -1); - composer_label.set_text ("Composer: "); - composer_label.set_size_request (60, -1); - - isrc_entry.set_size_request (112, -1); - isrc_entry.set_max_length(12); - isrc_entry.set_editable (true); - - performer_entry.set_size_request (100, -1); - performer_entry.set_editable (true); - - composer_entry.set_size_request (100, -1); - composer_entry.set_editable (true); - - name_label.set_alignment (0, 0.5); - - cd_track_details_hbox.pack_start (isrc_label, false, false); - cd_track_details_hbox.pack_start (isrc_entry, false, false); - cd_track_details_hbox.pack_start (scms_check_button, false, false); - cd_track_details_hbox.pack_start (preemph_check_button, false, false); - cd_track_details_hbox.pack_start (performer_label, false, false); - cd_track_details_hbox.pack_start (performer_entry, true, true); - cd_track_details_hbox.pack_start (composer_label, false, false); - cd_track_details_hbox.pack_start (composer_entry, true, true); - - isrc_entry.signal_changed().connect (sigc::mem_fun(*this, &LocationEditRow::isrc_entry_changed)); - performer_entry.signal_changed().connect (sigc::mem_fun(*this, &LocationEditRow::performer_entry_changed)); - composer_entry.signal_changed().connect (sigc::mem_fun(*this, &LocationEditRow::composer_entry_changed)); - scms_check_button.signal_toggled().connect(sigc::mem_fun(*this, &LocationEditRow::scms_toggled)); - preemph_check_button.signal_toggled().connect(sigc::mem_fun(*this, &LocationEditRow::preemph_toggled)); - - set_session (sess); - - start_hbox.pack_start (start_go_button, false, false); - start_hbox.pack_start (start_clock, false, false); - start_hbox.pack_start (*start_to_playhead_button, false, false); - - /* this is always in this location, no matter what the location is */ - - item_table.attach (start_hbox, 1, 2, 0, 1, FILL, FILL, 4, 0); - - start_go_button.signal_clicked().connect(sigc::bind (sigc::mem_fun (*this, &LocationEditRow::go_button_pressed), LocStart)); - start_to_playhead_button->signal_clicked().connect (sigc::bind (sigc::mem_fun (*this, &LocationEditRow::to_playhead_button_pressed), LocStart)); - start_clock.ValueChanged.connect (sigc::bind (sigc::mem_fun (*this, &LocationEditRow::clock_changed), LocStart)); - start_clock.ChangeAborted.connect (sigc::bind (sigc::mem_fun (*this, &LocationEditRow::change_aborted), LocStart)); - - end_hbox.pack_start (end_go_button, false, false); - end_hbox.pack_start (end_clock, false, false); - end_hbox.pack_start (*end_to_playhead_button, false, false); - - end_go_button.signal_clicked().connect(sigc::bind (sigc::mem_fun (*this, &LocationEditRow::go_button_pressed), LocEnd)); - end_to_playhead_button->signal_clicked().connect (sigc::bind (sigc::mem_fun (*this, &LocationEditRow::to_playhead_button_pressed), LocEnd)); - end_clock.ValueChanged.connect (sigc::bind (sigc::mem_fun (*this, &LocationEditRow::clock_changed), LocEnd)); - end_clock.ChangeAborted.connect (sigc::bind (sigc::mem_fun (*this, &LocationEditRow::change_aborted), LocEnd)); - - length_clock.ValueChanged.connect (sigc::bind ( sigc::mem_fun(*this, &LocationEditRow::clock_changed), LocLength)); - length_clock.ChangeAborted.connect (sigc::bind (sigc::mem_fun (*this, &LocationEditRow::change_aborted), LocLength)); - - cd_check_button.signal_toggled().connect(sigc::mem_fun(*this, &LocationEditRow::cd_toggled)); - hide_check_button.signal_toggled().connect(sigc::mem_fun(*this, &LocationEditRow::hide_toggled)); - lock_check_button.signal_toggled().connect(sigc::mem_fun(*this, &LocationEditRow::lock_toggled)); - glue_check_button.signal_toggled().connect(sigc::mem_fun(*this, &LocationEditRow::glue_toggled)); - - remove_button.signal_clicked().connect(sigc::mem_fun(*this, &LocationEditRow::remove_button_pressed)); - - pack_start(item_table, true, true); - - set_location (loc); - set_number (num); - } - - LocationEditRow::~LocationEditRow() - { - if (location) { - connections.drop_connections (); - } - - if (_clock_group) { - _clock_group->remove (start_clock); - _clock_group->remove (end_clock); - _clock_group->remove (length_clock); - } - } - - void - LocationEditRow::set_clock_group (ClockGroup& cg) - { - if (_clock_group) { - _clock_group->remove (start_clock); - _clock_group->remove (end_clock); - _clock_group->remove (length_clock); - } - - _clock_group = &cg; - - _clock_group->add (start_clock); - _clock_group->add (end_clock); - _clock_group->add (length_clock); + , location(0) + , item_table (1, 6, false) + , start_clock (X_("locationstart"), true, "", true, false) + , start_to_playhead_button (_("Use PH")) + , locate_to_start_button (_("Goto")) + , end_clock (X_("locationend"), true, "", true, false) + , end_to_playhead_button (_("Use PH")) + , locate_to_end_button (_("Goto")) + , length_clock (X_("locationlength"), true, "", true, false, true) + , cd_check_button (_("CD")) + , hide_check_button (_("Hide")) + , lock_check_button (_("Lock")) + , glue_check_button (_("Glue")) + , _clock_group (0) +{ + i_am_the_modifier = 0; + + remove_button.set_icon (ArdourIcon::CloseCross); + remove_button.set_events (remove_button.get_events() & ~(Gdk::ENTER_NOTIFY_MASK|Gdk::LEAVE_NOTIFY_MASK)); + + number_label.set_name ("LocationEditNumberLabel"); + name_label.set_name ("LocationEditNameLabel"); + name_entry.set_name ("LocationEditNameEntry"); + cd_check_button.set_name ("LocationEditCdButton"); + hide_check_button.set_name ("LocationEditHideButton"); + lock_check_button.set_name ("LocationEditLockButton"); + glue_check_button.set_name ("LocationEditGlueButton"); + isrc_label.set_name ("LocationEditNumberLabel"); + isrc_entry.set_name ("LocationEditNameEntry"); + scms_check_button.set_name ("LocationEditCdButton"); + preemph_check_button.set_name ("LocationEditCdButton"); + performer_label.set_name ("LocationEditNumberLabel"); + performer_entry.set_name ("LocationEditNameEntry"); + composer_label.set_name ("LocationEditNumberLabel"); + composer_entry.set_name ("LocationEditNameEntry"); + + isrc_label.set_text (X_("ISRC:")); + performer_label.set_text (_("Performer:")); + composer_label.set_text (_("Composer:")); + scms_label.set_text (X_("SCMS")); + preemph_label.set_text (_("Pre-Emphasis")); + + isrc_entry.set_size_request (112, -1); + isrc_entry.set_max_length(12); + isrc_entry.set_editable (true); + + performer_entry.set_size_request (100, -1); + performer_entry.set_editable (true); + + composer_entry.set_size_request (100, -1); + composer_entry.set_editable (true); + + name_label.set_alignment (0, 0.5); + + Gtk::HBox* front_spacing = manage (new HBox); + front_spacing->set_size_request (20, -1); + Gtk::HBox* mid_spacing = manage (new HBox); + mid_spacing->set_size_request (20, -1); + + cd_track_details_hbox.set_spacing (4); + cd_track_details_hbox.pack_start (*front_spacing, false, false); + cd_track_details_hbox.pack_start (isrc_label, false, false); + cd_track_details_hbox.pack_start (isrc_entry, false, false); + cd_track_details_hbox.pack_start (performer_label, false, false); + cd_track_details_hbox.pack_start (performer_entry, true, true); + cd_track_details_hbox.pack_start (composer_label, false, false); + cd_track_details_hbox.pack_start (composer_entry, true, true); + cd_track_details_hbox.pack_start (*mid_spacing, false, false); + cd_track_details_hbox.pack_start (scms_label, false, false); + cd_track_details_hbox.pack_start (scms_check_button, false, false); + cd_track_details_hbox.pack_start (preemph_label, false, false); + cd_track_details_hbox.pack_start (preemph_check_button, false, false); + + isrc_entry.signal_changed().connect (sigc::mem_fun(*this, &LocationEditRow::isrc_entry_changed)); + performer_entry.signal_changed().connect (sigc::mem_fun(*this, &LocationEditRow::performer_entry_changed)); + composer_entry.signal_changed().connect (sigc::mem_fun(*this, &LocationEditRow::composer_entry_changed)); + scms_check_button.signal_toggled().connect(sigc::mem_fun(*this, &LocationEditRow::scms_toggled)); + preemph_check_button.signal_toggled().connect(sigc::mem_fun(*this, &LocationEditRow::preemph_toggled)); + + set_session (sess); + + start_hbox.set_spacing (2); + start_hbox.pack_start (locate_to_start_button, false, false); + start_hbox.pack_start (start_clock, false, false); + start_hbox.pack_start (start_to_playhead_button, false, false); + + /* this is always in this location, no matter what the location is */ + + item_table.attach (remove_button, 8, 9, 0, 1, SHRINK, SHRINK, 4, 1); + item_table.attach (start_hbox, 0, 1, 0, 1, FILL, Gtk::AttachOptions(0), 4, 0); + + start_to_playhead_button.signal_clicked.connect (sigc::bind (sigc::mem_fun (*this, &LocationEditRow::to_playhead_button_pressed), LocStart)); + locate_to_start_button.signal_clicked.connect (sigc::bind (sigc::mem_fun (*this, &LocationEditRow::locate_button_pressed), LocStart)); + start_clock.ValueChanged.connect (sigc::bind (sigc::mem_fun (*this, &LocationEditRow::clock_changed), LocStart)); + start_clock.signal_button_press_event().connect (sigc::bind (sigc::mem_fun (*this, &LocationEditRow::locate_to_clock), &start_clock), false); + + end_hbox.set_spacing (2); + end_hbox.pack_start (locate_to_end_button, false, false); + end_hbox.pack_start (end_clock, false, false); + end_hbox.pack_start (end_to_playhead_button, false, false); + + end_to_playhead_button.signal_clicked.connect (sigc::bind (sigc::mem_fun (*this, &LocationEditRow::to_playhead_button_pressed), LocEnd)); + locate_to_end_button.signal_clicked.connect (sigc::bind (sigc::mem_fun (*this, &LocationEditRow::locate_button_pressed), LocEnd)); + end_clock.ValueChanged.connect (sigc::bind (sigc::mem_fun (*this, &LocationEditRow::clock_changed), LocEnd)); + end_clock.signal_button_press_event().connect (sigc::bind (sigc::mem_fun (*this, &LocationEditRow::locate_to_clock), &end_clock), false); + + length_clock.ValueChanged.connect (sigc::bind ( sigc::mem_fun(*this, &LocationEditRow::clock_changed), LocLength)); + + cd_check_button.signal_toggled().connect(sigc::mem_fun(*this, &LocationEditRow::cd_toggled)); + hide_check_button.signal_toggled().connect(sigc::mem_fun(*this, &LocationEditRow::hide_toggled)); + lock_check_button.signal_toggled().connect(sigc::mem_fun(*this, &LocationEditRow::lock_toggled)); + glue_check_button.signal_toggled().connect(sigc::mem_fun(*this, &LocationEditRow::glue_toggled)); + + remove_button.signal_clicked.connect(sigc::mem_fun(*this, &LocationEditRow::remove_button_pressed)); + + pack_start(item_table, true, true); + + set_location (loc); + set_number (num); + cd_toggled(); // show/hide cd-track details +} + +LocationEditRow::~LocationEditRow() +{ + if (location) { + connections.drop_connections (); + } + + if (_clock_group) { + _clock_group->remove (start_clock); + _clock_group->remove (end_clock); + _clock_group->remove (length_clock); + } +} + +void +LocationEditRow::set_clock_group (ClockGroup& cg) +{ + if (_clock_group) { + _clock_group->remove (start_clock); + _clock_group->remove (end_clock); + _clock_group->remove (length_clock); + } + + _clock_group = &cg; + + _clock_group->add (start_clock); + _clock_group->add (end_clock); + _clock_group->add (length_clock); } void @@ -204,7 +211,6 @@ LocationEditRow::set_session (Session *sess) start_clock.set_session (_session); end_clock.set_session (_session); length_clock.set_session (_session); - } void @@ -247,8 +253,10 @@ LocationEditRow::set_location (Location *loc) name_label.set_text (location->name()); name_label.set_size_request (80, -1); + remove_button.hide (); + if (!name_label.get_parent()) { - item_table.attach (name_label, 0, 1, 0, 1, FILL, FILL, 4, 0); + item_table.attach (name_label, 2, 3, 0, 1, EXPAND|FILL, FILL, 4, 0); } name_label.show(); @@ -261,15 +269,12 @@ LocationEditRow::set_location (Location *loc) name_entry.signal_changed().connect (sigc::mem_fun(*this, &LocationEditRow::name_entry_changed)); if (!name_entry.get_parent()) { - item_table.attach (name_entry, 0, 1, 0, 1, FILL | EXPAND, FILL, 4, 0); + item_table.attach (name_entry, 2, 3, 0, 1, FILL | EXPAND, FILL, 4, 0); } name_entry.show(); if (!cd_check_button.get_parent()) { - item_table.attach (cd_check_button, 4, 5, 0, 1, FILL, FILL, 4, 0); - } - if (!remove_button.get_parent()) { - item_table.attach (remove_button, 8, 9, 0, 1, FILL, FILL, 4, 0); + item_table.attach (cd_check_button, 4, 5, 0, 1, FILL, Gtk::AttachOptions (0), 4, 0); } if (location->is_session_range()) { @@ -295,54 +300,60 @@ LocationEditRow::set_location (Location *loc) if (!location->is_mark()) { if (!end_hbox.get_parent()) { - item_table.attach (end_hbox, 2, 3, 0, 1, FILL, FILL, 4, 0); + item_table.attach (end_hbox, 1, 2, 0, 1, FILL, Gtk::AttachOptions (0), 4, 0); } if (!length_clock.get_parent()) { - item_table.attach (length_clock, 3, 4, 0, 1, FILL, FILL, 4, 0); + end_hbox.pack_start (length_clock, false, false, 4); } end_clock.set (location->end(), true); length_clock.set (location->length(), true); - end_go_button.show(); end_clock.show(); length_clock.show(); - ARDOUR_UI::instance()->set_tip (end_go_button, _("Jump to the end of this range")); - ARDOUR_UI::instance()->set_tip (start_go_button, _("Jump to the start of this range")); - ARDOUR_UI::instance()->set_tip (remove_button, _("Forget this range")); - ARDOUR_UI::instance()->set_tip (start_clock, _("Start time")); - ARDOUR_UI::instance()->set_tip (end_clock, _("End time")); - ARDOUR_UI::instance()->set_tip (length_clock, _("Length")); + if (location->is_cd_marker()) { + show_cd_track_details (); + } + + set_tooltip (&remove_button, _("Remove this range")); + set_tooltip (start_clock, _("Start time - middle click to locate here")); + set_tooltip (end_clock, _("End time - middle click to locate here")); + set_tooltip (length_clock, _("Length")); + + set_tooltip (&start_to_playhead_button, _("Set range start from playhead location")); + set_tooltip (&end_to_playhead_button, _("Set range end from playhead location")); } else { - ARDOUR_UI::instance()->set_tip (start_go_button, _("Jump to this marker")); - ARDOUR_UI::instance()->set_tip (remove_button, _("Forget this marker")); - ARDOUR_UI::instance()->set_tip (start_clock, _("Position")); + set_tooltip (&remove_button, _("Remove this marker")); + set_tooltip (start_clock, _("Position - middle click to locate here")); + + set_tooltip (&start_to_playhead_button, _("Set marker time from playhead location")); - end_go_button.hide(); end_clock.hide(); length_clock.hide(); } - set_clock_sensitivity (); + set_clock_editable_status (); --i_am_the_modifier; - location->start_changed.connect (connections, invalidator (*this), ui_bind (&LocationEditRow::start_changed, this, _1), gui_context()); - location->end_changed.connect (connections, invalidator (*this), ui_bind (&LocationEditRow::end_changed, this, _1), gui_context()); - location->name_changed.connect (connections, invalidator (*this), ui_bind (&LocationEditRow::name_changed, this, _1), gui_context()); - location->changed.connect (connections, invalidator (*this), ui_bind (&LocationEditRow::location_changed, this, _1), gui_context()); - location->FlagsChanged.connect (connections, invalidator (*this), ui_bind (&LocationEditRow::flags_changed, this, _1, _2), gui_context()); - location->LockChanged.connect (connections, invalidator (*this), ui_bind (&LocationEditRow::lock_changed, this, _1), gui_context()); - location->PositionLockStyleChanged.connect (connections, invalidator (*this), ui_bind (&LocationEditRow::position_lock_style_changed, this, _1), gui_context()); + /* connect to per-location signals, since this row only cares about this location */ + + location->NameChanged.connect (connections, invalidator (*this), boost::bind (&LocationEditRow::name_changed, this), gui_context()); + location->StartChanged.connect (connections, invalidator (*this), boost::bind (&LocationEditRow::start_changed, this), gui_context()); + location->EndChanged.connect (connections, invalidator (*this), boost::bind (&LocationEditRow::end_changed, this), gui_context()); + location->Changed.connect (connections, invalidator (*this), boost::bind (&LocationEditRow::location_changed, this), gui_context()); + location->FlagsChanged.connect (connections, invalidator (*this), boost::bind (&LocationEditRow::flags_changed, this), gui_context()); + location->LockChanged.connect (connections, invalidator (*this), boost::bind (&LocationEditRow::lock_changed, this), gui_context()); + location->PositionLockStyleChanged.connect (connections, invalidator (*this), boost::bind (&LocationEditRow::position_lock_style_changed, this), gui_context()); } void LocationEditRow::name_entry_changed () { - ENSURE_GUI_THREAD (*this, &LocationEditRow::name_entry_changed) + ENSURE_GUI_THREAD (*this, &LocationEditRow::name_entry_changed); if (i_am_the_modifier || !location) { return; @@ -355,84 +366,94 @@ LocationEditRow::name_entry_changed () void LocationEditRow::isrc_entry_changed () { - ENSURE_GUI_THREAD (*this, &LocationEditRow::isrc_entry_changed) + ENSURE_GUI_THREAD (*this, &LocationEditRow::isrc_entry_changed); if (i_am_the_modifier || !location) return; if (isrc_entry.get_text() != "" ) { - location->cd_info["isrc"] = isrc_entry.get_text(); + location->cd_info["isrc"] = isrc_entry.get_text(); } else { - location->cd_info.erase("isrc"); + location->cd_info.erase("isrc"); } } void LocationEditRow::performer_entry_changed () { - ENSURE_GUI_THREAD (*this, &LocationEditRow::performer_entry_changed) + ENSURE_GUI_THREAD (*this, &LocationEditRow::performer_entry_changed); if (i_am_the_modifier || !location) return; if (performer_entry.get_text() != "") { - location->cd_info["performer"] = performer_entry.get_text(); + location->cd_info["performer"] = performer_entry.get_text(); } else { - location->cd_info.erase("performer"); + location->cd_info.erase("performer"); } } void LocationEditRow::composer_entry_changed () { - ENSURE_GUI_THREAD (*this, &LocationEditRow::composer_entry_changed) + ENSURE_GUI_THREAD (*this, &LocationEditRow::composer_entry_changed); if (i_am_the_modifier || !location) return; if (composer_entry.get_text() != "") { - location->cd_info["composer"] = composer_entry.get_text(); + location->cd_info["composer"] = composer_entry.get_text(); } else { - location->cd_info.erase("composer"); + location->cd_info.erase("composer"); } } - void -LocationEditRow::go_button_pressed (LocationPart part) +LocationEditRow::to_playhead_button_pressed (LocationPart part) { if (!location) { return; } + const int32_t divisions = PublicEditor::instance().get_grid_music_divisions (0); + switch (part) { - case LocStart: - ARDOUR_UI::instance()->do_transport_locate (location->start(), _session->transport_rolling ()); - break; - case LocEnd: - ARDOUR_UI::instance()->do_transport_locate (location->end(), _session->transport_rolling ()); - break; - default: - break; + case LocStart: + location->set_start (_session->transport_frame (), false, true, divisions); + break; + case LocEnd: + location->set_end (_session->transport_frame (), false, true,divisions); + if (location->is_session_range()) { + _session->set_end_is_free (false); + } + break; + default: + break; } } void -LocationEditRow::to_playhead_button_pressed (LocationPart part) +LocationEditRow::locate_button_pressed (LocationPart part) { - if (!location) { - return; + switch (part) { + case LocStart: + _session->request_locate (start_clock.current_time()); + break; + case LocEnd: + _session->request_locate (end_clock.current_time()); + break; + default: + break; } +} - switch (part) { - case LocStart: - location->set_start (_session->transport_frame ()); - break; - case LocEnd: - location->set_end (_session->transport_frame ()); - break; - default: - break; +bool +LocationEditRow::locate_to_clock (GdkEventButton* ev, AudioClock* clock) +{ + if (Keyboard::is_button2_event (ev)) { + _session->request_locate (clock->current_time()); + return true; } + return false; } void @@ -442,26 +463,53 @@ LocationEditRow::clock_changed (LocationPart part) return; } + const int32_t divisions = PublicEditor::instance().get_grid_music_divisions (0); + switch (part) { - case LocStart: - location->set_start (start_clock.current_time()); - break; - case LocEnd: - location->set_end (end_clock.current_time()); - break; - case LocLength: - location->set_end (location->start() + length_clock.current_duration()); - default: - break; + case LocStart: + location->set_start (start_clock.current_time(), false, true, divisions); + break; + case LocEnd: + location->set_end (end_clock.current_time(), false, true, divisions); + if (location->is_session_range()) { + _session->set_end_is_free (false); + } + break; + case LocLength: + location->set_end (location->start() + length_clock.current_duration(), false, true, divisions); + if (location->is_session_range()) { + _session->set_end_is_free (false); + } + default: + break; } } void -LocationEditRow::change_aborted (LocationPart /*part*/) +LocationEditRow::show_cd_track_details () { - if (i_am_the_modifier || !location) return; + 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); + } + - set_location(location); + if (!cd_track_details_hbox.get_parent()) { + item_table.attach (cd_track_details_hbox, 0, 7, 1, 2, FILL | EXPAND, FILL, 4, 0); + } + // item_table.resize(2, 7); + cd_track_details_hbox.show_all(); } void @@ -485,29 +533,9 @@ LocationEditRow::cd_toggled () 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 (location->is_cd_marker()) { - if (!cd_track_details_hbox.get_parent()) { - item_table.attach (cd_track_details_hbox, 0, 7, 1, 2, FILL | EXPAND, FILL, 4, 0); - } - // item_table.resize(2, 7); - cd_track_details_hbox.show_all(); + show_cd_track_details (); } else if (cd_track_details_hbox.get_parent()){ @@ -593,7 +621,7 @@ LocationEditRow::preemph_toggled () } void -LocationEditRow::end_changed (ARDOUR::Location *) +LocationEditRow::end_changed () { ENSURE_GUI_THREAD (*this, &LocationEditRow::end_changed, loc) @@ -609,7 +637,7 @@ LocationEditRow::end_changed (ARDOUR::Location *) } void -LocationEditRow::start_changed (ARDOUR::Location*) +LocationEditRow::start_changed () { if (!location) return; @@ -628,7 +656,7 @@ LocationEditRow::start_changed (ARDOUR::Location*) } void -LocationEditRow::name_changed (ARDOUR::Location *) +LocationEditRow::name_changed () { if (!location) return; @@ -643,7 +671,7 @@ LocationEditRow::name_changed (ARDOUR::Location *) } void -LocationEditRow::location_changed (ARDOUR::Location*) +LocationEditRow::location_changed () { if (!location) return; @@ -654,14 +682,14 @@ LocationEditRow::location_changed (ARDOUR::Location*) end_clock.set (location->end()); length_clock.set (location->length()); - set_clock_sensitivity (); + set_clock_editable_status (); i_am_the_modifier--; } void -LocationEditRow::flags_changed (ARDOUR::Location*, void *) +LocationEditRow::flags_changed () { if (!location) { return; @@ -677,7 +705,7 @@ LocationEditRow::flags_changed (ARDOUR::Location*, void *) } void -LocationEditRow::lock_changed (ARDOUR::Location*) +LocationEditRow::lock_changed () { if (!location) { return; @@ -687,13 +715,13 @@ LocationEditRow::lock_changed (ARDOUR::Location*) lock_check_button.set_active (location->locked()); - set_clock_sensitivity (); + set_clock_editable_status (); i_am_the_modifier--; } void -LocationEditRow::position_lock_style_changed (ARDOUR::Location*) +LocationEditRow::position_lock_style_changed () { if (!location) { return; @@ -707,33 +735,36 @@ LocationEditRow::position_lock_style_changed (ARDOUR::Location*) } void -LocationEditRow::focus_name() { - name_entry.grab_focus(); +LocationEditRow::focus_name() +{ + name_entry.grab_focus (); } void -LocationEditRow::set_clock_sensitivity () +LocationEditRow::set_clock_editable_status () { - start_clock.set_sensitive (!location->locked()); - end_clock.set_sensitive (!location->locked()); - length_clock.set_sensitive (!location->locked()); + start_clock.set_editable (!location->locked()); + end_clock.set_editable (!location->locked()); + length_clock.set_editable (!location->locked()); } /*------------------------------------------------------------------------*/ -LocationUI::LocationUI () +LocationUI::LocationUI (std::string state_node_name) : add_location_button (_("New Marker")) , add_range_button (_("New Range")) + , _mode (AudioClock::Frames) + , _mode_set (false) + , _state_node_name (state_node_name) { i_am_the_modifier = 0; - _clock_group = new ClockGroup; - _clock_group->set_clock_mode (AudioClock::Frames); + _clock_group = new ClockGroup; VBox* vbox = manage (new VBox); Table* table = manage (new Table (2, 2)); - table->set_spacings (4); + table->set_spacings (2); table->set_col_spacing (0, 32); int table_row = 0; @@ -743,9 +774,10 @@ LocationUI::LocationUI () table->attach (*l, 0, 2, table_row, table_row + 1); ++table_row; - loop_edit_row.set_clock_group (*_clock_group); - punch_edit_row.set_clock_group (*_clock_group); + loop_edit_row.set_clock_group (*_clock_group); + punch_edit_row.set_clock_group (*_clock_group); + loop_punch_box.set_border_width (6); // 5 + 1 px framebox-border loop_punch_box.pack_start (loop_edit_row, false, false); loop_punch_box.pack_start (punch_edit_row, false, false); @@ -754,8 +786,8 @@ LocationUI::LocationUI () vbox->pack_start (*table, false, false); - table = manage (new Table (3, 2)); - table->set_spacings (4); + table = manage (new Table (3, 2)); + table->set_spacings (2); table->set_col_spacing (0, 32); table_row = 0; @@ -784,13 +816,13 @@ LocationUI::LocationUI () add_location_button.set_name ("LocationAddLocationButton"); - table->attach (loc_frame_box, 1, 2, table_row, table_row + 1); + table->attach (loc_frame_box, 0, 2, table_row, table_row + 1); ++table_row; - loc_range_panes.pack1 (*table, true, false); + loc_range_panes.add (*table); - table = manage (new Table (3, 2)); - table->set_spacings (4); + table = manage (new Table (3, 2)); + table->set_spacings (2); table->set_col_spacing (0, 32); table_row = 0; @@ -816,10 +848,10 @@ LocationUI::LocationUI () add_range_button.set_name ("LocationAddRangeButton"); - table->attach (range_frame_box, 1, 2, table_row, table_row + 1); + table->attach (range_frame_box, 0, 2, table_row, table_row + 1); ++table_row; - loc_range_panes.pack2 (*table, true, false); + loc_range_panes.add (*table); HBox* add_button_box = manage (new HBox); add_button_box->pack_start (add_location_button, true, true); @@ -834,11 +866,15 @@ LocationUI::LocationUI () add_range_button.signal_clicked().connect (sigc::mem_fun(*this, &LocationUI::add_new_range)); show_all (); + + signal_map().connect (sigc::mem_fun (*this, &LocationUI::refresh_location_list)); } LocationUI::~LocationUI() { - delete _clock_group; + loop_edit_row.unset_clock_group (); + punch_edit_row.unset_clock_group (); + delete _clock_group; } gint @@ -853,12 +889,12 @@ LocationUI::do_location_remove (ARDOUR::Location *loc) return FALSE; } - _session->begin_reversible_command (_("remove marker")); + PublicEditor::instance().begin_reversible_command (_("remove marker")); XMLNode &before = _session->locations()->get_state(); _session->locations()->remove (loc); XMLNode &after = _session->locations()->get_state(); _session->add_command(new MementoCommand(*(_session->locations()), &before, &after)); - _session->commit_reversible_command (); + PublicEditor::instance().commit_reversible_command (); return FALSE; } @@ -881,9 +917,9 @@ LocationUI::location_redraw_ranges () } struct LocationSortByStart { - bool operator() (Location *a, Location *b) { - return a->start() < b->start(); - } + bool operator() (Location *a, Location *b) { + return a->start() < b->start(); + } }; void @@ -899,7 +935,7 @@ LocationUI::location_added (Location* location) LocationEditRow* erow = manage (new LocationEditRow (_session, location)); - erow->set_clock_group (*_clock_group); + erow->set_clock_group (*_clock_group); erow->remove_requested.connect (sigc::mem_fun (*this, &LocationUI::location_remove_requested)); Box_Helpers::BoxList & children = location->is_range_marker() ? range_rows.children () : location_rows.children (); @@ -929,6 +965,11 @@ LocationUI::location_added (Location* location) range_rows.show_all (); location_rows.show_all (); + + if (location == newest_location) { + newest_location = 0; + erow->focus_name(); + } } } @@ -954,7 +995,7 @@ LocationUI::location_removed (Location* location) } void -LocationUI::map_locations (Locations::LocationList& locations) +LocationUI::map_locations (const Locations::LocationList& locations) { Locations::LocationList::iterator i; gint n; @@ -963,9 +1004,8 @@ LocationUI::map_locations (Locations::LocationList& locations) LocationSortByStart cmp; temp.sort (cmp); - locations = temp; - for (n = 0, i = locations.begin(); i != locations.end(); ++n, ++i) { + for (n = 0, i = temp.begin(); i != temp.end(); ++n, ++i) { Location* location = *i; @@ -976,12 +1016,8 @@ LocationUI::map_locations (Locations::LocationList& locations) erow->remove_requested.connect (sigc::mem_fun(*this, &LocationUI::location_remove_requested)); erow->redraw_ranges.connect (sigc::mem_fun(*this, &LocationUI::location_redraw_ranges)); - Box_Helpers::BoxList & loc_children = location_rows.children(); + Box_Helpers::BoxList & loc_children = location_rows.children(); 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); @@ -993,7 +1029,7 @@ LocationUI::map_locations (Locations::LocationList& locations) } else { LocationEditRow* erow = manage (new LocationEditRow(_session, location)); - erow->set_clock_group (*_clock_group); + erow->set_clock_group (*_clock_group); erow->remove_requested.connect (sigc::mem_fun(*this, &LocationUI::location_remove_requested)); Box_Helpers::BoxList & range_children = range_rows.children(); @@ -1014,15 +1050,15 @@ LocationUI::add_new_location() framepos_t where = _session->audible_frame(); _session->locations()->next_available_name(markername,"mark"); Location *location = new Location (*_session, where, where, markername, Location::IsMark); - if (Config->get_name_new_markers()) { + if (UIConfiguration::instance().get_name_new_markers()) { newest_location = location; } - _session->begin_reversible_command (_("add marker")); + PublicEditor::instance().begin_reversible_command (_("add marker")); XMLNode &before = _session->locations()->get_state(); _session->locations()->add (location, true); XMLNode &after = _session->locations()->get_state(); _session->add_command (new MementoCommand(*(_session->locations()), &before, &after)); - _session->commit_reversible_command (); + PublicEditor::instance().commit_reversible_command (); } } @@ -1036,12 +1072,12 @@ LocationUI::add_new_range() framepos_t where = _session->audible_frame(); _session->locations()->next_available_name(rangename,"unnamed"); Location *location = new Location (*_session, where, where, rangename, Location::IsRangeMarker); - _session->begin_reversible_command (_("add range marker")); + PublicEditor::instance().begin_reversible_command (_("add range marker")); XMLNode &before = _session->locations()->get_state(); _session->locations()->add (location, true); XMLNode &after = _session->locations()->get_state(); _session->add_command (new MementoCommand(*(_session->locations()), &before, &after)); - _session->commit_reversible_command (); + PublicEditor::instance().commit_reversible_command (); } } @@ -1073,10 +1109,13 @@ LocationUI::set_session(ARDOUR::Session* s) SessionHandlePtr::set_session (s); if (_session) { - _session->locations()->changed.connect (_session_connections, invalidator (*this), boost::bind (&LocationUI::locations_changed, this, _1), gui_context()); - _session->locations()->StateChanged.connect (_session_connections, invalidator (*this), boost::bind (&LocationUI::refresh_location_list, this), gui_context()); - _session->locations()->added.connect (_session_connections, invalidator (*this), ui_bind (&LocationUI::location_added, this, _1), gui_context()); - _session->locations()->removed.connect (_session_connections, invalidator (*this), ui_bind (&LocationUI::location_removed, this, _1), gui_context()); + _session->locations()->added.connect (_session_connections, invalidator (*this), boost::bind (&LocationUI::location_added, this, _1), gui_context()); + _session->locations()->removed.connect (_session_connections, invalidator (*this), boost::bind (&LocationUI::location_removed, this, _1), gui_context()); + _session->locations()->changed.connect (_session_connections, invalidator (*this), boost::bind (&LocationUI::refresh_location_list, this), gui_context()); + + _clock_group->set_clock_mode (clock_mode_from_session_instant_xml ()); + } else { + _mode_set = false; } loop_edit_row.set_session (s); @@ -1085,17 +1124,6 @@ LocationUI::set_session(ARDOUR::Session* s) refresh_location_list (); } -void -LocationUI::locations_changed (Locations::Change c) -{ - /* removal is signalled by both a removed and a changed signal emission from Locations, - so we don't need to refresh the list on a removal - */ - if (c != Locations::REMOVAL) { - refresh_location_list (); - } -} - void LocationUI::session_going_away() { @@ -1114,19 +1142,73 @@ LocationUI::session_going_away() punch_edit_row.set_session (0); punch_edit_row.set_location (0); + _mode_set = false; + SessionHandlePtr::session_going_away (); } +XMLNode & +LocationUI::get_state () const +{ + XMLNode* node = new XMLNode (_state_node_name); + if (_mode_set) { + node->add_property (X_("clock-mode"), enum_2_string (_mode)); + } else { + node->add_property (X_("clock-mode"), enum_2_string (_clock_group->clock_mode ())); + } + return *node; +} + +int +LocationUI::set_state (const XMLNode& node) +{ + if (node.name() != _state_node_name) { + return -1; + } + XMLProperty const* p = node.property (X_("clock-mode")); + if (!p) { + return -1; + } + _mode = (AudioClock::Mode) string_2_enum (p->value (), AudioClock::Mode); + _mode_set = true; + if (_clock_group) { + _clock_group->set_clock_mode (_mode); + } + return 0; +} + +AudioClock::Mode +LocationUI::clock_mode_from_session_instant_xml () +{ + if (_mode_set) { + return _mode; + } + + XMLNode* node = _session->instant_xml (_state_node_name); + if (!node) { + return AudioClock::Frames; + } + + XMLProperty const * p = node->property (X_("clock-mode")); + if (!p) { + return ARDOUR_UI::instance()->secondary_clock->mode(); + } + + _mode = (AudioClock::Mode) string_2_enum (p->value (), AudioClock::Mode); + _mode_set = true; + return _mode; +} + /*------------------------*/ LocationUIWindow::LocationUIWindow () - : ArdourDialog (_("Locations")) + : ArdourWindow (S_("Ranges|Locations")) { set_wmclass(X_("ardour_locations"), PROGRAM_NAME); set_name ("LocationWindow"); - get_vbox()->pack_start (_ui); + add (_ui); } LocationUIWindow::~LocationUIWindow() @@ -1136,27 +1218,27 @@ LocationUIWindow::~LocationUIWindow() void LocationUIWindow::on_map () { - ArdourDialog::on_map (); + ArdourWindow::on_map (); _ui.refresh_location_list(); } bool LocationUIWindow::on_delete_event (GdkEventAny*) { - hide (); - return true; + return false; } void LocationUIWindow::set_session (Session *s) { - ArdourDialog::set_session (s); + ArdourWindow::set_session (s); _ui.set_session (s); + _ui.show_all (); } void LocationUIWindow::session_going_away () { - ArdourDialog::session_going_away (); + ArdourWindow::session_going_away (); hide_all(); }