Basics of per-channel audio gain.
[dcpomatic.git] / src / wx / audio_panel.cc
index e2ea15f645b49af72984e848572ee1e238e3c118..50a8f709cf3d4e78f08a00064bed6b59b9f497d0 100644 (file)
@@ -40,58 +40,61 @@ AudioPanel::AudioPanel (FilmEditor* e)
        : FilmEditorPanel (e, _("Audio"))
        , _audio_dialog (0)
 {
-       wxFlexGridSizer* grid = new wxFlexGridSizer (3, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
+       wxGridBagSizer* grid = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
        _sizer->Add (grid, 0, wxALL, 8);
 
+       int r = 0;
+
        _show = new wxButton (this, wxID_ANY, _("Show Audio..."));
-       grid->Add (_show, 1);
-       grid->AddSpacer (0);
-       grid->AddSpacer (0);
-
-       add_label_to_sizer (grid, this, _("Audio Gain"), true);
-       {
-               wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
-               _gain = new wxSpinCtrl (this);
-               s->Add (_gain, 1);
-               add_label_to_sizer (s, this, _("dB"), false);
-               grid->Add (s, 1);
-       }
+       grid->Add (_show, wxGBPosition (r, 0));
+       ++r;
+
+       add_label_to_grid_bag_sizer (grid, this, _("Audio Gain"), true, wxGBPosition (r, 0));
+       _gain = new ContentSpinCtrl<AudioContent> (
+               this,
+               new wxSpinCtrl (this),
+               AudioContentProperty::AUDIO_GAIN,
+               boost::mem_fn (&AudioContent::audio_gain),
+               boost::mem_fn (&AudioContent::set_audio_gain)
+               );
        
+       _gain->add (grid, wxGBPosition (r, 1));
+       add_label_to_grid_bag_sizer (grid, this, _("dB"), false, wxGBPosition (r, 2));
        _gain_calculate_button = new wxButton (this, wxID_ANY, _("Calculate..."));
-       grid->Add (_gain_calculate_button);
-
-       add_label_to_sizer (grid, this, _("Audio Delay"), false);
-       {
-               wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
-               _delay = new wxSpinCtrl (this);
-               s->Add (_delay, 1);
-               /// TRANSLATORS: this is an abbreviation for milliseconds, the unit of time
-               add_label_to_sizer (s, this, _("ms"), false);
-               grid->Add (s);
-       }
-
-       grid->AddSpacer (0);
+       grid->Add (_gain_calculate_button, wxGBPosition (r, 3));
+       ++r;
+
+       add_label_to_grid_bag_sizer (grid, this, _("Audio Delay"), false, wxGBPosition (r, 0));
+       _delay = new ContentSpinCtrl<AudioContent> (
+               this,
+               new wxSpinCtrl (this),
+               AudioContentProperty::AUDIO_DELAY,
+               boost::mem_fn (&AudioContent::audio_delay),
+               boost::mem_fn (&AudioContent::set_audio_delay)
+               );
+       
+       _delay->add (grid, wxGBPosition (r,1 ));
+       /// TRANSLATORS: this is an abbreviation for milliseconds, the unit of time
+       add_label_to_grid_bag_sizer (grid, this, _("ms"), false, wxGBPosition (r, 2));
+       ++r;
 
-       add_label_to_sizer (grid, this, _("Audio Stream"), true);
+       add_label_to_grid_bag_sizer (grid, this, _("Audio Stream"), true, wxGBPosition (r, 0));
        _stream = new wxChoice (this, wxID_ANY);
-       grid->Add (_stream, 1, wxEXPAND);
-       _description = new wxStaticText (this, wxID_ANY, wxT (""));
-       grid->AddSpacer (0);
+       grid->Add (_stream, wxGBPosition (r, 1));
+       ++r;
        
-       grid->Add (_description, 1, wxALIGN_CENTER_VERTICAL | wxLEFT, 8);
-       grid->AddSpacer (0);
-       grid->AddSpacer (0);
+       _description = new wxStaticText (this, wxID_ANY, wxT (""));
+       grid->Add (_description, wxGBPosition (r, 0));
+       ++r;
        
        _mapping = new AudioMappingView (this);
        _sizer->Add (_mapping, 1, wxEXPAND | wxALL, 6);
 
-       _gain->SetRange (-60, 60);
-       _delay->SetRange (-1000, 1000);
+       _gain->wrapped()->SetRange (-60, 60);
+       _delay->wrapped()->SetRange (-1000, 1000);
 
-       _delay->Bind                 (wxEVT_COMMAND_SPINCTRL_UPDATED, boost::bind (&AudioPanel::delay_changed, this));
        _stream->Bind                (wxEVT_COMMAND_CHOICE_SELECTED,  boost::bind (&AudioPanel::stream_changed, this));
        _show->Bind                  (wxEVT_COMMAND_BUTTON_CLICKED,   boost::bind (&AudioPanel::show_clicked, this));
-       _gain->Bind                  (wxEVT_COMMAND_SPINCTRL_UPDATED, boost::bind (&AudioPanel::gain_changed, this));
        _gain_calculate_button->Bind (wxEVT_COMMAND_BUTTON_CLICKED,   boost::bind (&AudioPanel::gain_calculate_button_clicked, this));
 
        _mapping->Changed.connect (boost::bind (&AudioPanel::mapping_changed, this, _1));
@@ -105,6 +108,7 @@ AudioPanel::film_changed (Film::Property property)
        case Film::AUDIO_CHANNELS:
                _mapping->set_channels (_editor->film()->audio_channels ());
                _sizer->Layout ();
+               _sizer->Fit (this);
                break;
        default:
                break;
@@ -122,20 +126,13 @@ AudioPanel::film_content_changed (int property)
                fcs = dynamic_pointer_cast<FFmpegContent> (acs);
        }
        
-       if (_audio_dialog && acs) {
-               _audio_dialog->set_content (acs);
-       }
-       
-       if (property == AudioContentProperty::AUDIO_GAIN) {
-               checked_set (_gain, acs ? acs->audio_gain() : 0);
-       } else if (property == AudioContentProperty::AUDIO_DELAY) {
-               checked_set (_delay, acs ? acs->audio_delay() : 0);
-       } else if (property == AudioContentProperty::AUDIO_MAPPING) {
+       if (property == AudioContentProperty::AUDIO_MAPPING) {
                _mapping->set (acs ? acs->audio_mapping () : AudioMapping ());
                _sizer->Layout ();
        } else if (property == FFmpegContentProperty::AUDIO_STREAM) {
                setup_stream_description ();
                _mapping->set (acs ? acs->audio_mapping () : AudioMapping ());
+               _sizer->Layout ();
        } else if (property == FFmpegContentProperty::AUDIO_STREAMS) {
                _stream->Clear ();
                if (fcs) {
@@ -152,28 +149,6 @@ AudioPanel::film_content_changed (int property)
        }
 }
 
-void
-AudioPanel::gain_changed ()
-{
-       AudioContentList ac = _editor->selected_audio_content ();
-       if (ac.size() != 1) {
-               return;
-       }
-
-       ac.front()->set_audio_gain (_gain->GetValue ());
-}
-
-void
-AudioPanel::delay_changed ()
-{
-       AudioContentList ac = _editor->selected_audio_content ();
-       if (ac.size() != 1) {
-               return;
-       }
-
-       ac.front()->set_audio_delay (_delay->GetValue ());
-}
-
 void
 AudioPanel::gain_calculate_button_clicked ()
 {
@@ -185,7 +160,7 @@ AudioPanel::gain_calculate_button_clicked ()
                return;
        }
        
-       _gain->SetValue (
+       _gain->wrapped()->SetValue (
                Config::instance()->sound_processor()->db_for_fader_change (
                        d->wanted_fader (),
                        d->actual_fader ()
@@ -195,7 +170,7 @@ AudioPanel::gain_calculate_button_clicked ()
        /* This appears to be necessary, as the change is not signalled,
           I think.
        */
-       gain_changed ();
+       _gain->update_from_model ();
        
        d->Destroy ();
 }
@@ -282,5 +257,20 @@ AudioPanel::mapping_changed (AudioMapping m)
 void
 AudioPanel::content_selection_changed ()
 {
+       AudioContentList sel = _editor->selected_audio_content ();
+
+       if (_audio_dialog && sel.size() == 1) {
+               _audio_dialog->set_content (sel.front ());
+       }
+       
+       _gain->set_content (sel);
+       _delay->set_content (sel);
+
+       _show->Enable (sel.size() == 1);
+       _stream->Enable (sel.size() == 1);
+       _mapping->Enable (sel.size() == 1);
 
+       film_content_changed (AudioContentProperty::AUDIO_MAPPING);
+       film_content_changed (FFmpegContentProperty::AUDIO_STREAM);
+       film_content_changed (FFmpegContentProperty::AUDIO_STREAMS);
 }