another redirect dialog fix
[ardour.git] / gtk2_ardour / tempo_dialog.cc
index f0ad928aa550a54f5c36f50db02e7bfbe9c4fc2b..d69acf7ef1c43fea000ad5b8836bdbfc9f62582b 100644 (file)
@@ -1,16 +1,41 @@
+/*
+    Copyright (C) 2000-2007 Paul Davis 
+
+    This program is free software; you can redistribute it and/or modify
+    it under the terms of the GNU General Public License as published by
+    the Free Software Foundation; either version 2 of the License, or
+    (at your option) any later version.
+
+    This program is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    GNU General Public License for more details.
+
+    You should have received a copy of the GNU General Public License
+    along with this program; if not, write to the Free Software
+    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+
+*/
+
 #include <cstdio> // for snprintf, grrr 
 
+#include <gtkmm/stock.h>
 #include <gtkmm2ext/utils.h>
 
 #include "tempo_dialog.h"
+#include "utils.h"
 
 #include "i18n.h"
 
 using namespace Gtk;
+using namespace Gtkmm2ext;
 using namespace ARDOUR;
+using namespace PBD;
 
-TempoDialog::TempoDialog (TempoMap& map, jack_nframes_t frame, string action)
-       : ArdourDialog ("tempo dialog"),
+TempoDialog::TempoDialog (TempoMap& map, nframes_t frame, const string & action)
+       : ArdourDialog (_("edit tempo")),
+         bpm_adjustment (60.0, 1.0, 999.9, 0.1, 1.0, 1.0),
+         bpm_spinner (bpm_adjustment),
          bpm_frame (_("Beats per minute")),
          ok_button (action),
          cancel_button (_("Cancel")),
@@ -26,8 +51,10 @@ TempoDialog::TempoDialog (TempoMap& map, jack_nframes_t frame, string action)
        init (when, tempo.beats_per_minute(), true);
 }
 
-TempoDialog::TempoDialog (TempoSection& section, string action)
+TempoDialog::TempoDialog (TempoSection& section, const string & action)
        : ArdourDialog ("tempo dialog"),
+         bpm_adjustment (60.0, 1.0, 999.9, 0.1, 1.0, 1.0),
+         bpm_spinner (bpm_adjustment),
          bpm_frame (_("Beats per minute")),
          ok_button (action),
          cancel_button (_("Cancel")),
@@ -42,26 +69,18 @@ TempoDialog::TempoDialog (TempoSection& section, string action)
 void
 TempoDialog::init (const BBT_Time& when, double bpm, bool movable)
 {
-       snprintf (buf, sizeof (buf), "%.2f", bpm);
-       bpm_entry.set_text (buf);
-       bpm_entry.select_region (0, -1);
-       
+       bpm_spinner.set_numeric (true);
+       bpm_spinner.set_digits (1);
+       bpm_spinner.set_wrap (true);
+       bpm_spinner.set_value (bpm);
+
        hspacer1.set_border_width (5);
-       hspacer1.pack_start (bpm_entry, false, false);
+       hspacer1.pack_start (bpm_spinner, false, false);
        vspacer1.set_border_width (5);
        vspacer1.pack_start (hspacer1, false, false);
 
        bpm_frame.add (vspacer1);
 
-       button_box.set_border_width (10);
-       button_box.set_spacing (5);
-       button_box.set_homogeneous (true);
-       button_box.pack_start (ok_button); 
-       button_box.pack_start (cancel_button); 
-
-       vpacker.set_border_width (10);
-       vpacker.set_spacing (5);
-
        if (movable) {
                snprintf (buf, sizeof (buf), "%" PRIu32, when.bars);
                when_bar_entry.set_text (buf);
@@ -82,42 +101,57 @@ TempoDialog::init (const BBT_Time& when, double bpm, bool movable)
                when_table.set_col_spacings (2);
                when_table.set_border_width (5);
                
-               when_table.attach (when_bar_label, 0, 1, 0, 1, 0, Gtk::FILL|Gtk::EXPAND);
-               when_table.attach (when_bar_entry, 0, 1, 1, 2, 0, Gtk::FILL|Gtk::EXPAND);
+               when_table.attach (when_bar_label, 0, 1, 0, 1, Gtk::AttachOptions(0), Gtk::FILL|Gtk::EXPAND);
+               when_table.attach (when_bar_entry, 0, 1, 1, 2, Gtk::AttachOptions(0), Gtk::FILL|Gtk::EXPAND);
                
-               when_table.attach (when_beat_label, 1, 2, 0, 1, 0, 0);
-               when_table.attach (when_beat_entry, 1, 2, 1, 2, 0, 0);
+               when_table.attach (when_beat_label, 1, 2, 0, 1, Gtk::AttachOptions(0), Gtk::AttachOptions(0));
+               when_table.attach (when_beat_entry, 1, 2, 1, 2, Gtk::AttachOptions(0), Gtk::AttachOptions(0));
                
                when_frame.set_name ("MetricDialogFrame");
                when_frame.add (when_table);
 
-               vpacker.pack_start (when_frame, false, false);
+               get_vbox()->pack_start (when_frame, false, false);
        }
 
-       vpacker.pack_start (bpm_frame, false, false);
-       vpacker.pack_start (button_box, false, false);
-
        bpm_frame.set_name ("MetricDialogFrame");
-       bpm_entry.set_name ("MetricEntry");
-       ok_button.set_name ("MetricButton");
-       cancel_button.set_name ("MetricButton");
+       bpm_spinner.set_name ("MetricEntry");
+
+       get_vbox()->pack_start (bpm_frame, false, false);
+       
+       add_button (Stock::CANCEL, RESPONSE_CANCEL);
+       add_button (Stock::APPLY, RESPONSE_ACCEPT);
+       set_response_sensitive (Gtk::RESPONSE_ACCEPT, false);
+       set_default_response (RESPONSE_ACCEPT);
+
+       get_vbox()->show_all();
+       bpm_spinner.show();
 
-       add (vpacker);
        set_name ("MetricDialog");
 
-       set_keyboard_input(true);
+       bpm_spinner.signal_activate().connect (bind (mem_fun (*this, &TempoDialog::response), RESPONSE_ACCEPT));
+       bpm_spinner.signal_button_press_event().connect (mem_fun (*this, &TempoDialog::bpm_button_press), false);
+       bpm_spinner.signal_button_release_event().connect (mem_fun (*this, &TempoDialog::bpm_button_release), false);
+}
+
+bool
+TempoDialog::bpm_button_press (GdkEventButton* ev)
+{
+       return false;
+}
+
+bool
+TempoDialog::bpm_button_release (GdkEventButton* ev)
+{      
+       /* the value has been modified, accept should work now */
+
+       set_response_sensitive (Gtk::RESPONSE_ACCEPT, true);
+       return false;
 }
 
 double 
 TempoDialog::get_bpm ()
 {
-       double bpm;
-       
-       if (sscanf (bpm_entry.get_text().c_str(), "%lf", &bpm) != 1) {
-               return 0;
-       }
-
-       return bpm;
+       return bpm_spinner.get_value ();
 }      
 
 bool
@@ -135,7 +169,7 @@ TempoDialog::get_bbt_time (BBT_Time& requested)
 }
 
 
-MeterDialog::MeterDialog (TempoMap& map, jack_nframes_t frame, string action)
+MeterDialog::MeterDialog (TempoMap& map, nframes_t frame, const string & action)
        : ArdourDialog ("meter dialog"),
          note_frame (_("Meter denominator")),
          bpb_frame (_("Beats per bar")),
@@ -153,7 +187,7 @@ MeterDialog::MeterDialog (TempoMap& map, jack_nframes_t frame, string action)
        init (when, meter.beats_per_bar(), meter.note_divisor(), true);
 }
 
-MeterDialog::MeterDialog (MeterSection& section, string action)
+MeterDialog::MeterDialog (MeterSection& section, const string & action)
        : ArdourDialog ("meter dialog"),
          note_frame (_("Meter denominator")),
          bpb_frame (_("Beats per bar")),
@@ -174,7 +208,6 @@ MeterDialog::init (const BBT_Time& when, double bpb, double note_type, bool mova
        bpb_entry.select_region (0, -1);
        Gtkmm2ext::set_size_request_to_display_given_text (bpb_entry, "999999g", 5, 5);
 
-       
        strings.push_back (_("whole (1)"));
        strings.push_back (_("second (2)"));
        strings.push_back (_("third (3)"));
@@ -183,27 +216,28 @@ MeterDialog::init (const BBT_Time& when, double bpb, double note_type, bool mova
        strings.push_back (_("sixteenth (16)"));
        strings.push_back (_("thirty-second (32)"));
        
-       note_types.set_popdown_strings (strings);
-       
+       set_popdown_strings (note_types, strings);
+
        if (note_type==1.0f)
-               note_types.get_entry()->set_text(_("whole (1)"));
+               note_types.set_active_text (_("whole (1)"));
        else if (note_type==2.0f)
-               note_types.get_entry()->set_text(_("second (2)"));
+               note_types.set_active_text (_("second (2)"));
        else if (note_type==3.0f)
-               note_types.get_entry()->set_text(_("third (3)"));
+               note_types.set_active_text (_("third (3)"));
        else if (note_type==4.0f)
-               note_types.get_entry()->set_text(_("quarter (4)"));
+               note_types.set_active_text (_("quarter (4)"));
        else if (note_type==8.0f)
-               note_types.get_entry()->set_text(_("eighth (8)"));
+               note_types.set_active_text (_("eighth (8)"));
        else if (note_type==16.0f)
-               note_types.get_entry()->set_text(_("sixteenth (16)"));
+               note_types.set_active_text (_("sixteenth (16)"));
        else if (note_type==32.0f)
-               note_types.get_entry()->set_text(_("thirty-second (32)"));
+               note_types.set_active_text (_("thirty-second (32)"));
        else
-               note_types.get_entry()->set_text(_("quarter (4)"));
+               note_types.set_active_text (_("quarter (4)"));
                
-       /* strings.back() just happens to be the longest one to display */
-       Gtkmm2ext::set_size_request_to_display_given_text (*(note_types.get_entry()), strings.back(), 7, 7);
+       /* the string here needs to be the longest one to display */
+       const guint32 FUDGE = 20; // Combo's are stupid - they steal space from the entry for the button
+        Gtkmm2ext::set_size_request_to_display_given_text (note_types, "thirty-second (32)", 7+FUDGE, 7);
 
        hspacer1.set_border_width (5);
        hspacer1.pack_start (note_types, false, false);
@@ -218,15 +252,6 @@ MeterDialog::init (const BBT_Time& when, double bpb, double note_type, bool mova
        note_frame.add (vspacer1);
        bpb_frame.add (vspacer2);
 
-       button_box.set_border_width (10);
-       button_box.set_spacing (5);
-       button_box.set_homogeneous (true);
-       button_box.pack_start (ok_button); 
-       button_box.pack_start (cancel_button); 
-
-       vpacker.set_border_width (10);
-       vpacker.set_spacing (5);
-       
        if (movable) {
                snprintf (buf, sizeof (buf), "%" PRIu32, when.bars);
                when_bar_entry.set_text (buf);
@@ -247,33 +272,102 @@ MeterDialog::init (const BBT_Time& when, double bpb, double note_type, bool mova
                when_table.set_col_spacings (2);
                when_table.set_border_width (5);
                
-               when_table.attach (when_bar_label, 0, 1, 0, 1, 0, Gtk::FILL|Gtk::EXPAND);
-               when_table.attach (when_bar_entry, 0, 1, 1, 2, 0, Gtk::FILL|Gtk::EXPAND);
+               when_table.attach (when_bar_label, 0, 1, 0, 1, Gtk::AttachOptions(0), Gtk::FILL|Gtk::EXPAND);
+               when_table.attach (when_bar_entry, 0, 1, 1, 2, Gtk::AttachOptions(0), Gtk::FILL|Gtk::EXPAND);
                
-               when_table.attach (when_beat_label, 1, 2, 0, 1, 0, 0);
-               when_table.attach (when_beat_entry, 1, 2, 1, 2, 0, 0);
+               when_table.attach (when_beat_label, 1, 2, 0, 1, Gtk::AttachOptions(0), Gtk::AttachOptions(0));
+               when_table.attach (when_beat_entry, 1, 2, 1, 2, Gtk::AttachOptions(0), Gtk::AttachOptions(0));
                
                when_frame.set_name ("MetricDialogFrame");
                when_frame.add (when_table);
                
-               vpacker.pack_start (when_frame, false, false);
+               get_vbox()->pack_start (when_frame, false, false);
        }
-
-       vpacker.pack_start (bpb_frame, false, false);
-       vpacker.pack_start (note_frame, false, false);
-       vpacker.pack_start (button_box, false, false);
+       get_vbox()->pack_start (bpb_frame, false, false);
+       get_vbox()->pack_start (note_frame, false, false);
        
        bpb_frame.set_name ("MetricDialogFrame");
        note_frame.set_name ("MetricDialogFrame");
-       note_types.get_entry()->set_name ("MetricEntry");
        bpb_entry.set_name ("MetricEntry");
-       ok_button.set_name ("MetricButton");
-       cancel_button.set_name ("MetricButton");
 
-       add (vpacker);
+       add_button (Stock::CANCEL, RESPONSE_CANCEL);
+       add_button (Stock::APPLY, RESPONSE_ACCEPT);
+       set_response_sensitive (RESPONSE_ACCEPT, false);
+       set_default_response (RESPONSE_ACCEPT);
+
+       get_vbox()->show_all ();
+       bpb_entry.show ();
+
        set_name ("MetricDialog");
+       bpb_entry.signal_activate().connect (bind (mem_fun (*this, &MeterDialog::response), RESPONSE_ACCEPT));
+       bpb_entry.signal_key_press_event().connect (mem_fun (*this, &MeterDialog::bpb_key_press), false);
+       bpb_entry.signal_key_release_event().connect (mem_fun (*this, &MeterDialog::bpb_key_release));
+       note_types.signal_changed().connect (mem_fun (*this, &MeterDialog::note_types_change));
+}
 
-       set_keyboard_input(true);
+bool
+MeterDialog::bpb_key_press (GdkEventKey* ev)
+{
+
+switch (ev->keyval) { 
+
+ case GDK_0:
+ case GDK_1:
+ case GDK_2:
+ case GDK_3:
+ case GDK_4:
+ case GDK_5:
+ case GDK_6:
+ case GDK_7:
+ case GDK_8:
+ case GDK_9:
+ case GDK_KP_0:
+ case GDK_KP_1:
+ case GDK_KP_2:
+ case GDK_KP_3:
+ case GDK_KP_4:
+ case GDK_KP_5:
+ case GDK_KP_6:
+ case GDK_KP_7:
+ case GDK_KP_8:
+ case GDK_KP_9:
+ case GDK_period:
+ case GDK_comma:
+ case  GDK_KP_Delete:
+ case  GDK_KP_Enter:
+ case  GDK_Delete:
+ case  GDK_BackSpace:
+ case  GDK_Escape:
+ case  GDK_Return:
+ case  GDK_Home:
+ case  GDK_End:
+ case  GDK_Left:
+ case  GDK_Right:
+ case  GDK_Num_Lock:
+ case  GDK_Tab:
+    return FALSE;
+ default:
+      break;
+ }
+
+   return TRUE;
+}
+
+bool
+MeterDialog::bpb_key_release (GdkEventKey* ev)
+{
+        if (bpb_entry.get_text() != "") {
+               set_response_sensitive (RESPONSE_ACCEPT, true);
+       } else {
+               set_response_sensitive (RESPONSE_ACCEPT, false);
+       }
+       return false;
+}
+
+void
+MeterDialog::note_types_change ()
+{
+        set_response_sensitive (Gtk::RESPONSE_ACCEPT, true);
 }
 
 double
@@ -292,13 +386,13 @@ double
 MeterDialog::get_note_type ()
 {
        double note_type = 0;
-       vector<const gchar *>::iterator i;
-       string text = note_types.get_entry()->get_text();
+       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 << compose(_("garbaged note type entry (%1)"), text) << endmsg;
+                               error << string_compose(_("garbaged note type entry (%1)"), text) << endmsg;
                                return 0;
                        } else {
                                break;
@@ -308,7 +402,7 @@ MeterDialog::get_note_type ()
        
        if (i == strings.end()) {
                if (sscanf (text.c_str(), "%lf", &note_type) != 1) {
-                       error << compose(_("incomprehensible note type entry (%1)"), text) << endmsg;
+                       error << string_compose(_("incomprehensible note type entry (%1)"), text) << endmsg;
                        return 0;
                }
        }