no longer delete and recreate MonitorSection when it is removed/added to a Session
[ardour.git] / gtk2_ardour / tempo_dialog.cc
index cca9e6f7c5a3fc6e2602697a272e45bd6f651f94..48b54e981f06fba017975689ddff9f7422a195ac 100644 (file)
 #include <cstdio> // for snprintf, grrr
 
 #include <gtkmm/stock.h>
-#include <gtkmm2ext/utils.h>
+
+#include "gtkmm2ext/utils.h"
 
 #include "tempo_dialog.h"
-#include "utils.h"
+#include "ui_config.h"
 
-#include "i18n.h"
+#include "pbd/i18n.h"
 
 using namespace std;
 using namespace Gtk;
@@ -33,109 +34,220 @@ using namespace Gtkmm2ext;
 using namespace ARDOUR;
 using namespace PBD;
 
-TempoDialog::TempoDialog (TempoMap& map, framepos_t frame, const string & action)
-       : ArdourDialog (_("New Tempo")),
-         bpm_adjustment (60.0, 1.0, 999.9, 0.1, 1.0),
-         bpm_spinner (bpm_adjustment),
-         ok_button (action),
-         cancel_button (_("Cancel")),
-         when_bar_label (_("bar:"), ALIGN_LEFT, ALIGN_CENTER),
-         when_beat_label (_("beat:"), ALIGN_LEFT, ALIGN_CENTER)
+TempoDialog::TempoDialog (TempoMap& map, samplepos_t sample, const string&)
+       : ArdourDialog (_("New Tempo"))
+       , _map (&map)
+       , _section (0)
+       , bpm_adjustment (60.0, 1.0, 999.9, 0.1, 1.0)
+       , bpm_spinner (bpm_adjustment)
+       , end_bpm_adjustment (60.0, 1.0, 999.9, 0.1, 1.0)
+       , end_bpm_spinner (end_bpm_adjustment)
+       , _end_bpm_label (_("End Beats per Minute:"), ALIGN_LEFT, ALIGN_CENTER)
+       , when_bar_label (_("bar:"), ALIGN_RIGHT, ALIGN_CENTER)
+       , when_beat_label (_("beat:"), ALIGN_RIGHT, ALIGN_CENTER)
+       , pulse_selector_label (_("Pulse:"), ALIGN_LEFT, ALIGN_CENTER)
+       , tap_tempo_button (_("Tap tempo"))
 {
-       Timecode::BBT_Time when;
-       Tempo tempo (map.tempo_at (frame));
-       map.bbt_time (frame, when);
+       Tempo tempo (map.tempo_at_sample (sample));
+       Timecode::BBT_Time when (map.bbt_at_sample (sample));
 
-       init (when, tempo.beats_per_minute(), tempo.note_type(), true);
+       init (when, tempo.note_types_per_minute(), tempo.end_note_types_per_minute(), tempo.note_type(), TempoSection::Constant, true, MusicTime);
 }
 
-TempoDialog::TempoDialog (TempoSection& section, const string & action)
-       : ArdourDialog ("Edit Tempo"),
-         bpm_adjustment (60.0, 1.0, 999.9, 0.1, 1.0),
-         bpm_spinner (bpm_adjustment),
-         ok_button (action),
-         cancel_button (_("Cancel")),
-         when_bar_label (_("bar:"), ALIGN_LEFT, ALIGN_CENTER),
-         when_beat_label (_("beat:"), ALIGN_LEFT, ALIGN_CENTER)
+TempoDialog::TempoDialog (TempoMap& map, TempoSection& section, const string&)
+       : ArdourDialog (_("Edit Tempo"))
+       , _map (&map)
+       , _section (&section)
+       , bpm_adjustment (60.0, 1.0, 999.9, 0.1, 1.0)
+       , bpm_spinner (bpm_adjustment)
+       , end_bpm_adjustment (60.0, 1.0, 999.9, 0.1, 1.0)
+       , end_bpm_spinner (end_bpm_adjustment)
+       , _end_bpm_label (_("End Beats per Minute:"), ALIGN_LEFT, ALIGN_CENTER)
+       , when_bar_label (_("bar:"), ALIGN_RIGHT, ALIGN_CENTER)
+       , when_beat_label (_("beat:"), ALIGN_RIGHT, ALIGN_CENTER)
+       , pulse_selector_label (_("Pulse:"), ALIGN_LEFT, ALIGN_CENTER)
+       , tap_tempo_button (_("Tap tempo"))
 {
-       init (section.start(), section.beats_per_minute(), section.note_type(), section.movable());
+       Timecode::BBT_Time when (map.bbt_at_sample (section.sample()));
+       init (when, section.note_types_per_minute(), section.end_note_types_per_minute(), section.note_type(), section.type()
+             , section.initial() || section.locked_to_meter(), section.position_lock_style());
 }
 
 void
-TempoDialog::init (const Timecode::BBT_Time& when, double bpm, double note_type, bool movable)
+TempoDialog::init (const Timecode::BBT_Time& when, double bpm, double end_bpm, double note_type, TempoSection::Type type, bool initial, PositionLockStyle style)
 {
+       vector<string> strings;
+       NoteTypes::iterator x;
+
        bpm_spinner.set_numeric (true);
-       bpm_spinner.set_digits (2);
+       bpm_spinner.set_digits (3);
        bpm_spinner.set_wrap (true);
        bpm_spinner.set_value (bpm);
+       bpm_spinner.set_alignment (1.0);
+
+       end_bpm_spinner.set_numeric (true);
+       end_bpm_spinner.set_digits (3);
+       end_bpm_spinner.set_wrap (true);
+       end_bpm_spinner.set_value (end_bpm);
+       end_bpm_spinner.set_alignment (1.0);
+
+       Gtkmm2ext::set_size_request_to_display_given_text (pulse_selector, _("one-hundred-twenty-eighth"), 3, 6);
+
+       note_types.insert (make_pair (_("whole"), 1.0));
+       strings.push_back (_("whole"));
+       note_types.insert (make_pair (_("second"), 2.0));
+       strings.push_back (_("second"));
+       note_types.insert (make_pair (_("third"), 3.0));
+       strings.push_back (_("third"));
+       note_types.insert (make_pair (_("quarter"), 4.0));
+       strings.push_back (_("quarter"));
+       note_types.insert (make_pair (_("eighth"), 8.0));
+       strings.push_back (_("eighth"));
+       note_types.insert (make_pair (_("sixteenth"), 16.0));
+       strings.push_back (_("sixteenth"));
+       note_types.insert (make_pair (_("thirty-second"), 32.0));
+       strings.push_back (_("thirty-second"));
+       note_types.insert (make_pair (_("sixty-fourth"), 64.0));
+       strings.push_back (_("sixty-fourth"));
+       note_types.insert (make_pair (_("one-hundred-twenty-eighth"), 128.0));
+       strings.push_back (_("one-hundred-twenty-eighth"));
+
+       set_popdown_strings (pulse_selector, strings);
+
+       for (x = note_types.begin(); x != note_types.end(); ++x) {
+               if (x->second == note_type) {
+                       pulse_selector.set_active_text (x->first);
+                       break;
+               }
+       }
+
+       if (x == note_types.end()) {
+               pulse_selector.set_active_text (strings[3]); // "quarter"
+       }
+
+       strings.clear();
+
+       tempo_types.insert (make_pair (_("ramped"), TempoSection::Ramp));
+       strings.push_back (_("ramped"));
+       tempo_types.insert (make_pair (_("constant"), TempoSection::Constant));
+       strings.push_back (_("constant"));
+       set_popdown_strings (tempo_type, strings);
+       TempoTypes::iterator tt;
+       for (tt = tempo_types.begin(); tt != tempo_types.end(); ++tt) {
+               if (tt->second == type) {
+                       tempo_type.set_active_text (tt->first);
+                       break;
+               }
+       }
+       if (tt == tempo_types.end()) {
+               tempo_type.set_active_text (strings[1]); // "constant"
+       }
+
+       strings.clear();
+
+       lock_styles.insert (make_pair (_("music"), MusicTime));
+       strings.push_back (_("music"));
+       lock_styles.insert (make_pair (_("audio"), AudioTime));
+       strings.push_back (_("audio"));
+       set_popdown_strings (lock_style, strings);
+       LockStyles::iterator ls;
+       for (ls = lock_styles.begin(); ls != lock_styles.end(); ++ls) {
+               if (ls->second == style) {
+                       lock_style.set_active_text (ls->first);
+                       break;
+               }
+       }
+       if (ls == lock_styles.end()) {
+               lock_style.set_active_text (strings[0]); // "music"
+       }
 
-       strings.push_back (_("whole (1)"));
-       strings.push_back (_("second (2)"));
-       strings.push_back (_("third (3)"));
-       strings.push_back (_("quarter (4)"));
-       strings.push_back (_("eighth (8)"));
-       strings.push_back (_("sixteenth (16)"));
-       strings.push_back (_("thirty-second (32)"));
-
-       set_popdown_strings (note_types, strings, true);
-
-       if (note_type == 1.0f) {
-               note_types.set_active_text (_("whole (1)"));
-       } else if (note_type == 2.0f) {
-               note_types.set_active_text (_("second (2)"));
-       } else if (note_type == 3.0f) {
-               note_types.set_active_text (_("third (3)"));
-       } else if (note_type == 4.0f) {
-               note_types.set_active_text (_("quarter (4)"));
-       } else if (note_type == 8.0f) {
-               note_types.set_active_text (_("eighth (8)"));
-       } else if (note_type == 16.0f) {
-               note_types.set_active_text (_("sixteenth (16)"));
-       } else if (note_type == 32.0f) {
-               note_types.set_active_text (_("thirty-second (32)"));
+       Table* table;
+
+       if (UIConfiguration::instance().get_allow_non_quarter_pulse()) {
+               table = manage (new Table (5, 7));
        } else {
-               note_types.set_active_text (_("quarter (4)"));
+               table = manage (new Table (5, 6));
        }
 
-       Table* table = manage (new Table (3, 3));
        table->set_spacings (6);
+       table->set_homogeneous (false);
 
-       Label* bpm_label = manage (new Label(_("Beats per minute:"), ALIGN_LEFT, ALIGN_CENTER));
-       table->attach (*bpm_label, 0, 2, 0, 1);
-       table->attach (bpm_spinner, 2, 3, 0, 1);
+       int row = 0;
 
-       if (movable) {
-               snprintf (buf, sizeof (buf), "%" PRIu32, when.bars);
-               when_bar_entry.set_text (buf);
-               snprintf (buf, sizeof (buf), "%" PRIu32, when.beats);
-               when_beat_entry.set_text (buf);
+       if (UIConfiguration::instance().get_allow_non_quarter_pulse()) {
+               table->attach (pulse_selector_label, 0, 1, row, row + 1);
+               table->attach (pulse_selector, 1, 5, row, row + 1);
 
-               when_bar_entry.set_name ("MetricEntry");
-               when_beat_entry.set_name ("MetricEntry");
+               ++row;
+       }
+
+       Label* bpm_label = manage (new Label(_("Start Beats per Minute:"), ALIGN_LEFT, ALIGN_CENTER));
+       table->attach (*bpm_label, 0, 1, row, row + 1);
+       table->attach (bpm_spinner, 1, 5, row, row + 1);
+       ++row;
+
+       table->attach (_end_bpm_label, 0, 1, row, row + 1);
+       table->attach (end_bpm_spinner, 1, 5, row, row + 1);
+       ++row;
+
+       Label* tempo_type_label = manage (new Label(_("Tempo Type:"), ALIGN_LEFT, ALIGN_CENTER));
+       table->attach (*tempo_type_label, 0, 1, row, row + 1);
+       table->attach (tempo_type, 1, 5, row, row + 1);
+
+       ++row;
+
+       char buf[64];
+
+       snprintf (buf, sizeof (buf), "%" PRIu32, when.bars);
+       when_bar_entry.set_text (buf);
+       snprintf (buf, sizeof (buf), "%" PRIu32, when.beats);
+       when_beat_entry.set_text (buf);
+
+       if (!initial) {
+               when_bar_entry.set_width_chars(4);
+               when_beat_entry.set_width_chars (4);
+               when_bar_entry.set_alignment (1.0);
+               when_beat_entry.set_alignment (1.0);
 
                when_bar_label.set_name ("MetricLabel");
                when_beat_label.set_name ("MetricLabel");
 
-               table->attach (when_bar_label, 1, 2, 2, 3);
-               table->attach (when_bar_entry, 2, 3, 2, 3);
+               table->attach (when_bar_label, 1, 2, row, row+1, Gtk::AttachOptions(0), Gtk::AttachOptions(0));
+               table->attach (when_bar_entry, 2, 3, row, row+1, Gtk::AttachOptions(0), Gtk::AttachOptions(0));
 
-               table->attach (when_beat_label, 1, 2, 1, 2);
-               table->attach (when_beat_entry, 2, 3, 1, 2);
+               table->attach (when_beat_label, 3, 4, row, row+1, Gtk::AttachOptions(0), Gtk::AttachOptions(0));
+               table->attach (when_beat_entry, 4, 5, row, row+1, Gtk::AttachOptions(0), Gtk::AttachOptions(0));
 
                Label* when_label = manage (new Label(_("Tempo begins at"), ALIGN_LEFT, ALIGN_CENTER));
-               table->attach (*when_label, 0, 1, 1, 2);
+               table->attach (*when_label, 0, 1, row, row+1);
+
+               ++row;
+               ++row;
+
+               Label* lock_style_label = manage (new Label(_("Lock Style:"), ALIGN_LEFT, ALIGN_CENTER));
+               table->attach (*lock_style_label, 0, 1, row, row + 1);
+               table->attach (lock_style, 1, 5, row, row + 1);
+
+               --row;
        }
 
        get_vbox()->set_border_width (12);
        get_vbox()->pack_end (*table);
+
        table->show_all ();
 
        add_button (Stock::CANCEL, RESPONSE_CANCEL);
        add_button (Stock::APPLY, RESPONSE_ACCEPT);
-       set_response_sensitive (RESPONSE_ACCEPT, false);
+       set_response_sensitive (RESPONSE_ACCEPT, true);
        set_default_response (RESPONSE_ACCEPT);
 
        bpm_spinner.show ();
+       end_bpm_spinner.show ();
+       tap_tempo_button.show ();
+       get_vbox()->set_spacing (6);
+       get_vbox()->pack_end (tap_tempo_button);
+       tap_tempo_button.can_focus ();
+       tap_tempo_button.grab_focus ();
 
        set_name ("MetricDialog");
 
@@ -143,17 +255,35 @@ TempoDialog::init (const Timecode::BBT_Time& when, double bpm, double note_type,
        bpm_spinner.signal_button_press_event().connect (sigc::mem_fun (*this, &TempoDialog::bpm_button_press), false);
        bpm_spinner.signal_button_release_event().connect (sigc::mem_fun (*this, &TempoDialog::bpm_button_release), false);
        bpm_spinner.signal_changed().connect (sigc::mem_fun (*this, &TempoDialog::bpm_changed));
+       end_bpm_spinner.signal_changed().connect (sigc::mem_fun (*this, &TempoDialog::bpm_changed));
        when_bar_entry.signal_activate().connect (sigc::bind (sigc::mem_fun (*this, &TempoDialog::response), RESPONSE_ACCEPT));
        when_bar_entry.signal_key_release_event().connect (sigc::mem_fun (*this, &TempoDialog::entry_key_release), false);
        when_beat_entry.signal_activate().connect (sigc::bind (sigc::mem_fun (*this, &TempoDialog::response), RESPONSE_ACCEPT));
        when_beat_entry.signal_key_release_event().connect (sigc::mem_fun (*this, &TempoDialog::entry_key_release), false);
-       note_types.signal_changed().connect (sigc::mem_fun (*this, &TempoDialog::note_types_change));
+       pulse_selector.signal_changed().connect (sigc::mem_fun (*this, &TempoDialog::pulse_change));
+       tempo_type.signal_changed().connect (sigc::mem_fun (*this, &TempoDialog::tempo_type_change));
+       lock_style.signal_changed().connect (sigc::mem_fun (*this, &TempoDialog::lock_style_change));
+       tap_tempo_button.signal_button_press_event().connect (sigc::mem_fun (*this, &TempoDialog::tap_tempo_button_press), false);
+       tap_tempo_button.signal_key_press_event().connect (sigc::mem_fun (*this, &TempoDialog::tap_tempo_key_press), false);
+       tap_tempo_button.signal_focus_out_event().connect (sigc::mem_fun (*this, &TempoDialog::tap_tempo_focus_out));
+
+       tempo_type_change();
+
+       tapped = false;
+}
+
+bool
+TempoDialog::is_user_input_valid() const
+{
+       return (when_beat_entry.get_text() != "")
+               && (when_bar_entry.get_text() != "")
+               && (when_bar_entry.get_text() != "0");
 }
 
 void
 TempoDialog::bpm_changed ()
 {
-       set_response_sensitive (RESPONSE_ACCEPT, true);
+       set_response_sensitive (RESPONSE_ACCEPT, is_user_input_valid());
 }
 
 bool
@@ -167,18 +297,22 @@ TempoDialog::bpm_button_release (GdkEventButton*)
 {
        /* the value has been modified, accept should work now */
 
-       set_response_sensitive (RESPONSE_ACCEPT, true);
+       set_response_sensitive (RESPONSE_ACCEPT, is_user_input_valid());
        return false;
 }
 
 bool
 TempoDialog::entry_key_release (GdkEventKey*)
 {
-       if (when_beat_entry.get_text() != "" && when_bar_entry.get_text() != "") {
-               set_response_sensitive (RESPONSE_ACCEPT, true);
+       Timecode::BBT_Time bbt;
+       get_bbt_time (bbt);
+
+       if (_section && is_user_input_valid()) {
+               set_response_sensitive (RESPONSE_ACCEPT, _map->can_solve_bbt (_section, bbt));
        } else {
-               set_response_sensitive (RESPONSE_ACCEPT, false);
+               set_response_sensitive (RESPONSE_ACCEPT, is_user_input_valid());
        }
+
        return false;
 }
 
@@ -188,6 +322,16 @@ TempoDialog::get_bpm ()
        return bpm_spinner.get_value ();
 }
 
+double
+TempoDialog::get_end_bpm ()
+{
+       if (get_tempo_type() == TempoSection::Constant) {
+               return bpm_spinner.get_value ();
+       }
+
+       return end_bpm_spinner.get_value ();
+}
+
 bool
 TempoDialog::get_bbt_time (Timecode::BBT_Time& requested)
 {
@@ -207,125 +351,243 @@ TempoDialog::get_bbt_time (Timecode::BBT_Time& requested)
 double
 TempoDialog::get_note_type ()
 {
-       double note_type = 0;
-       vector<string>::iterator i;
-       string text = note_types.get_active_text();
-
-       for (i = strings.begin(); i != strings.end(); ++i) {
-               if (text == *i) {
-                       if (sscanf (text.c_str(), "%*[^0-9]%lf", &note_type) != 1) {
-                               error << string_compose(_("garbaged note type entry (%1)"), text) << endmsg;
-                               return 0;
-                       } else {
-                               break;
-                       }
-               }
+       NoteTypes::iterator x = note_types.find (pulse_selector.get_active_text());
+
+       if (x == note_types.end()) {
+               error << string_compose(_("incomprehensible pulse note type (%1)"), pulse_selector.get_active_text()) << endmsg;
+               return 0;
        }
 
-       if (i == strings.end()) {
-               if (sscanf (text.c_str(), "%lf", &note_type) != 1) {
-                       error << string_compose(_("incomprehensible note type entry (%1)"), text) << endmsg;
-                       return 0;
-               }
+       return x->second;
+}
+
+TempoSection::Type
+TempoDialog::get_tempo_type ()
+{
+       TempoTypes::iterator x = tempo_types.find (tempo_type.get_active_text());
+
+       if (x == tempo_types.end()) {
+               error << string_compose(_("incomprehensible tempo type (%1)"), tempo_type.get_active_text()) << endmsg;
+               return TempoSection::Constant;
        }
 
-       return note_type;
+       return x->second;
+}
+
+PositionLockStyle
+TempoDialog::get_lock_style ()
+{
+       LockStyles::iterator x = lock_styles.find (lock_style.get_active_text());
+
+       if (x == lock_styles.end()) {
+               error << string_compose(_("incomprehensible lock style (%1)"), lock_style.get_active_text()) << endmsg;
+               return MusicTime;
+       }
+
+       return x->second;
 }
 
 void
-TempoDialog::note_types_change ()
+TempoDialog::pulse_change ()
 {
-        set_response_sensitive (RESPONSE_ACCEPT, true);
+       set_response_sensitive (RESPONSE_ACCEPT, is_user_input_valid());
 }
 
+void
+TempoDialog::tempo_type_change ()
+{
+       if (get_tempo_type() == TempoSection::Constant) {
+               end_bpm_spinner.hide ();
+               _end_bpm_label.hide();
+       } else {
+               end_bpm_spinner.show ();
+               _end_bpm_label.show();
+       }
 
-MeterDialog::MeterDialog (TempoMap& map, framepos_t frame, const string & action)
-       : ArdourDialog ("New Meter"),
-         ok_button (action),
-         cancel_button (_("Cancel"))
+       set_response_sensitive (RESPONSE_ACCEPT, is_user_input_valid());
+}
+
+void
+TempoDialog::lock_style_change ()
 {
-       Timecode::BBT_Time when;
-       frame = map.round_to_bar(frame,0);
-       Meter meter (map.meter_at(frame));
+       set_response_sensitive (RESPONSE_ACCEPT, is_user_input_valid());
+}
 
-       map.bbt_time (frame, when);
-       init (when, meter.beats_per_bar(), meter.note_divisor(), true);
+bool
+TempoDialog::tap_tempo_key_press (GdkEventKey*)
+{
+       tap_tempo ();
+       return false;
 }
 
-MeterDialog::MeterDialog (MeterSection& section, const string & action)
-       : ArdourDialog ("Edit Meter"),
-         ok_button (action),
-         cancel_button (_("Cancel"))
+bool
+TempoDialog::tap_tempo_button_press (GdkEventButton* ev)
 {
-       init (section.start(), section.beats_per_bar(), section.note_divisor(), section.movable());
+       if (ev->type == GDK_2BUTTON_PRESS || ev->type == GDK_3BUTTON_PRESS) {
+               return true;
+       }
+       if (ev->button != 1) {
+               return true;
+       }
+       tap_tempo ();
+       return false; // grab focus
 }
 
 void
-MeterDialog::init (const Timecode::BBT_Time& when, double bpb, double note_type, bool movable)
+TempoDialog::tap_tempo ()
 {
+       double t;
+
+       // Linear least-squares regression
+       if (tapped) {
+               t = 1e-6 * (g_get_monotonic_time () - first_t); // Subtract first_t to avoid precision problems
+
+               double n = tap_count;
+               sum_y += t;
+               sum_x += n;
+               sum_xy += n * t;
+               sum_xx += n * n;
+               double T = (sum_xy/n - sum_x/n * sum_y/n) / (sum_xx/n - sum_x/n * sum_x/n);
+
+               if (t - last_t < T / 1.2 || t - last_t > T * 1.2) {
+                       tapped = false;
+               } else {
+                       bpm_spinner.set_value (60.0 / T);
+               }
+       }
+       if (!tapped) {
+               first_t = g_get_monotonic_time ();
+               t = 0.0;
+               sum_y = 0.0;
+               sum_x = 1.0;
+               sum_xy = 0.0;
+               sum_xx = 1.0;
+               tap_count = 1.0;
+
+               tapped = true;
+       }
+       tap_count++;
+       last_t = t;
+}
+
+bool
+TempoDialog::tap_tempo_focus_out (GdkEventFocus* )
+{
+       tapped = false;
+       return false;
+}
+
+MeterDialog::MeterDialog (TempoMap& map, samplepos_t sample, const string&)
+       : ArdourDialog (_("New Meter"))
+{
+       sample = map.round_to_bar(sample, RoundNearest).sample;
+       Timecode::BBT_Time when (map.bbt_at_sample (sample));
+       Meter meter (map.meter_at_sample (sample));
+
+       init (when, meter.divisions_per_bar(), meter.note_divisor(), false, MusicTime);
+}
+
+MeterDialog::MeterDialog (TempoMap& map, MeterSection& section, const string&)
+       : ArdourDialog (_("Edit Meter"))
+{
+       Timecode::BBT_Time when (map.bbt_at_sample (section.sample()));
+
+       init (when, section.divisions_per_bar(), section.note_divisor(), section.initial(), section.position_lock_style());
+}
+
+void
+MeterDialog::init (const Timecode::BBT_Time& when, double bpb, double divisor, bool initial, PositionLockStyle style)
+{
+       char buf[64];
+       vector<string> strings;
+       NoteTypes::iterator x;
+
        snprintf (buf, sizeof (buf), "%.2f", bpb);
        bpb_entry.set_text (buf);
        bpb_entry.select_region (0, -1);
+       bpb_entry.set_alignment (1.0);
+
+       note_types.insert (make_pair (_("whole"), 1.0));
+       strings.push_back (_("whole"));
+       note_types.insert (make_pair (_("second"), 2.0));
+       strings.push_back (_("second"));
+       note_types.insert (make_pair (_("third"), 3.0));
+       strings.push_back (_("third"));
+       note_types.insert (make_pair (_("quarter"), 4.0));
+       strings.push_back (_("quarter"));
+       note_types.insert (make_pair (_("eighth"), 8.0));
+       strings.push_back (_("eighth"));
+       note_types.insert (make_pair (_("sixteenth"), 16.0));
+       strings.push_back (_("sixteenth"));
+       note_types.insert (make_pair (_("thirty-second"), 32.0));
+       strings.push_back (_("thirty-second"));
+       note_types.insert (make_pair (_("sixty-fourth"), 64.0));
+       strings.push_back (_("sixty-fourth"));
+       note_types.insert (make_pair (_("one-hundred-twenty-eighth"), 128.0));
+       strings.push_back (_("one-hundred-twenty-eighth"));
+
+       set_popdown_strings (note_type, strings);
+
+       for (x = note_types.begin(); x != note_types.end(); ++x) {
+               if (x->second == divisor) {
+                       note_type.set_active_text (x->first);
+                       break;
+               }
+       }
 
-       strings.push_back (_("whole (1)"));
-       strings.push_back (_("second (2)"));
-       strings.push_back (_("third (3)"));
-       strings.push_back (_("quarter (4)"));
-       strings.push_back (_("eighth (8)"));
-       strings.push_back (_("sixteenth (16)"));
-       strings.push_back (_("thirty-second (32)"));
-
-       set_popdown_strings (note_types, strings, true);
-
-       if (note_type == 1.0f) {
-               note_types.set_active_text (_("whole (1)"));
-       } else if (note_type == 2.0f) {
-               note_types.set_active_text (_("second (2)"));
-       } else if (note_type == 3.0f) {
-               note_types.set_active_text (_("third (3)"));
-       } else if (note_type == 4.0f) {
-               note_types.set_active_text (_("quarter (4)"));
-       } else if (note_type == 8.0f) {
-               note_types.set_active_text (_("eighth (8)"));
-       } else if (note_type == 16.0f) {
-               note_types.set_active_text (_("sixteenth (16)"));
-       } else if (note_type == 32.0f) {
-               note_types.set_active_text (_("thirty-second (32)"));
-       } else {
-               note_types.set_active_text (_("quarter (4)"));
+       if (x == note_types.end()) {
+               note_type.set_active_text (strings[3]); // "quarter"
+       }
+
+       strings.clear();
+
+       lock_styles.insert (make_pair (_("music"), MusicTime));
+       strings.push_back (_("music"));
+       lock_styles.insert (make_pair (_("audio"), AudioTime));
+       strings.push_back (_("audio"));
+       set_popdown_strings (lock_style, strings);
+       LockStyles::iterator ls;
+       for (ls = lock_styles.begin(); ls != lock_styles.end(); ++ls) {
+               if (ls->second == style) {
+                       lock_style.set_active_text (ls->first);
+                       break;
+               }
+       }
+       if (ls == lock_styles.end()) {
+               lock_style.set_active_text (strings[0]); // "music"
        }
 
-       Label* note_label = manage (new Label (_("Note value:"), ALIGN_LEFT, ALIGN_CENTER));
-       Label* bpb_label = manage (new Label (_("Beats per bar:"), ALIGN_LEFT, ALIGN_CENTER));
-       Table* table = manage (new Table (3, 2));
+       Label* note_label = manage (new Label (_("Note value:"), ALIGN_RIGHT, ALIGN_CENTER));
+       Label* lock_label = manage (new Label (_("Lock style:"), ALIGN_RIGHT, ALIGN_CENTER));
+       Label* bpb_label = manage (new Label (_("Beats per bar:"), ALIGN_RIGHT, ALIGN_CENTER));
+       Table* table = manage (new Table (3, 3));
        table->set_spacings (6);
 
        table->attach (*bpb_label, 0, 1, 0, 1, FILL|EXPAND, FILL|EXPAND);
        table->attach (bpb_entry, 1, 2, 0, 1, FILL|EXPAND, FILL|EXPAND);
        table->attach (*note_label, 0, 1, 1, 2, FILL|EXPAND, FILL|EXPAND);
-       table->attach (note_types, 1, 2, 1, 2, FILL|EXPAND, SHRINK);
+       table->attach (note_type, 1, 2, 1, 2, FILL|EXPAND, FILL|EXPAND);
 
-       if (movable) {
-               snprintf (buf, sizeof (buf), "%" PRIu32, when.bars);
-               when_bar_entry.set_text (buf);
-               when_bar_entry.set_name ("MetricEntry");
+       snprintf (buf, sizeof (buf), "%" PRIu32, when.bars);
+       when_bar_entry.set_text (buf);
+       when_bar_entry.set_alignment (1.0);
 
+       if (!initial) {
                Label* when_label = manage (new Label(_("Meter begins at bar:"), ALIGN_LEFT, ALIGN_CENTER));
 
                table->attach (*when_label, 0, 1, 2, 3, FILL | EXPAND, FILL | EXPAND);
                table->attach (when_bar_entry, 1, 2, 2, 3, FILL | EXPAND, FILL | EXPAND);
-       } else {
-               when_bar_entry.set_text ("0");
+
+               table->attach (*lock_label, 0, 1, 3, 4, FILL|EXPAND, FILL|EXPAND);
+               table->attach (lock_style, 1, 2, 3, 4, FILL|EXPAND, SHRINK);
        }
 
        get_vbox()->set_border_width (12);
        get_vbox()->pack_start (*table, false, false);
 
-       bpb_entry.set_name ("MetricEntry");
-
        add_button (Stock::CANCEL, RESPONSE_CANCEL);
        add_button (Stock::APPLY, RESPONSE_ACCEPT);
-       set_response_sensitive (RESPONSE_ACCEPT, false);
+       set_response_sensitive (RESPONSE_ACCEPT, true);
        set_default_response (RESPONSE_ACCEPT);
 
        get_vbox()->show_all ();
@@ -337,8 +599,17 @@ MeterDialog::init (const Timecode::BBT_Time& when, double bpb, double note_type,
        when_bar_entry.signal_activate().connect (sigc::bind (sigc::mem_fun (*this, &MeterDialog::response), RESPONSE_ACCEPT));
        when_bar_entry.signal_key_press_event().connect (sigc::mem_fun (*this, &MeterDialog::entry_key_press), false);
        when_bar_entry.signal_key_release_event().connect (sigc::mem_fun (*this, &MeterDialog::entry_key_release));
+       note_type.signal_changed().connect (sigc::mem_fun (*this, &MeterDialog::note_type_change));
+       lock_style.signal_changed().connect (sigc::mem_fun (*this, &MeterDialog::lock_style_change));
 
-       note_types.signal_changed().connect (sigc::mem_fun (*this, &MeterDialog::note_types_change));
+}
+
+bool
+MeterDialog::is_user_input_valid() const
+{
+       return (when_bar_entry.get_text() != "")
+               && (when_bar_entry.get_text() != "0")
+               && (bpb_entry.get_text() != "");
 }
 
 bool
@@ -392,18 +663,20 @@ MeterDialog::entry_key_press (GdkEventKey* ev)
 bool
 MeterDialog::entry_key_release (GdkEventKey*)
 {
-        if (when_bar_entry.get_text() != "" && bpb_entry.get_text() != "") {
-               set_response_sensitive (RESPONSE_ACCEPT, true);
-       } else {
-               set_response_sensitive (RESPONSE_ACCEPT, false);
-       }
+       set_response_sensitive (RESPONSE_ACCEPT, is_user_input_valid());
        return false;
 }
 
 void
-MeterDialog::note_types_change ()
+MeterDialog::note_type_change ()
+{
+       set_response_sensitive (RESPONSE_ACCEPT, is_user_input_valid());
+}
+
+void
+MeterDialog::lock_style_change ()
 {
-        set_response_sensitive (RESPONSE_ACCEPT, true);
+       set_response_sensitive (RESPONSE_ACCEPT, is_user_input_valid());
 }
 
 double
@@ -421,29 +694,27 @@ MeterDialog::get_bpb ()
 double
 MeterDialog::get_note_type ()
 {
-       double note_type = 0;
-       vector<string>::iterator i;
-       string text = note_types.get_active_text();
-
-       for (i = strings.begin(); i != strings.end(); ++i) {
-               if (text == *i) {
-                       if (sscanf (text.c_str(), "%*[^0-9]%lf", &note_type) != 1) {
-                               error << string_compose(_("garbaged note type entry (%1)"), text) << endmsg;
-                               return 0;
-                       } else {
-                               break;
-                       }
-               }
+       NoteTypes::iterator x = note_types.find (note_type.get_active_text());
+
+       if (x == note_types.end()) {
+               error << string_compose(_("incomprehensible meter note type (%1)"), note_type.get_active_text()) << endmsg;
+               return 0;
        }
 
-       if (i == strings.end()) {
-               if (sscanf (text.c_str(), "%lf", &note_type) != 1) {
-                       error << string_compose(_("incomprehensible note type entry (%1)"), text) << endmsg;
-                       return 0;
-               }
+       return x->second;
+}
+
+PositionLockStyle
+MeterDialog::get_lock_style ()
+{
+       LockStyles::iterator x = lock_styles.find (lock_style.get_active_text());
+
+       if (x == lock_styles.end()) {
+               error << string_compose(_("incomprehensible meter lock style (%1)"), lock_style.get_active_text()) << endmsg;
+               return MusicTime;
        }
 
-       return note_type;
+       return x->second;
 }
 
 bool
@@ -454,7 +725,6 @@ MeterDialog::get_bbt_time (Timecode::BBT_Time& requested)
        }
 
        requested.beats = 1;
-
        requested.ticks = 0;
 
        return true;