Fix initial display of things.
[dcpomatic.git] / src / wx / film_editor.cc
index afd79b08dfbbdacd1d089d04aa535912cd8530bc..7ef18165f25fd70d9f7c8950acb191bf3d7f89eb 100644 (file)
@@ -32,7 +32,6 @@
 #include "lib/film.h"
 #include "lib/transcode_job.h"
 #include "lib/exceptions.h"
-#include "lib/ab_transcode_job.h"
 #include "lib/job_manager.h"
 #include "lib/filter.h"
 #include "lib/ratio.h"
@@ -103,39 +102,44 @@ FilmEditor::make_dcp_panel ()
        _dcp_sizer = new wxBoxSizer (wxVERTICAL);
        _dcp_panel->SetSizer (_dcp_sizer);
 
-       wxGridBagSizer* grid = new wxGridBagSizer (4, 4);
+       wxGridBagSizer* grid = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
        _dcp_sizer->Add (grid, 0, wxEXPAND | wxALL, 8);
 
        int r = 0;
        
-       add_label_to_grid_bag_sizer (grid, _dcp_panel, _("Name"), wxGBPosition (r, 0));
+       add_label_to_grid_bag_sizer (grid, _dcp_panel, _("Name"), true, wxGBPosition (r, 0));
        _name = new wxTextCtrl (_dcp_panel, wxID_ANY);
-       grid->Add (_name, wxGBPosition(r, 1), wxDefaultSpan, wxEXPAND);
+       grid->Add (_name, wxGBPosition(r, 1), wxDefaultSpan, wxEXPAND | wxLEFT | wxRIGHT);
        ++r;
        
-       add_label_to_grid_bag_sizer (grid, _dcp_panel, _("DCP Name"), wxGBPosition (r, 0));
+       add_label_to_grid_bag_sizer (grid, _dcp_panel, _("DCP Name"), true, wxGBPosition (r, 0));
        _dcp_name = new wxStaticText (_dcp_panel, wxID_ANY, wxT (""));
        grid->Add (_dcp_name, wxGBPosition(r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
        ++r;
 
+       int flags = wxALIGN_CENTER_VERTICAL;
+#ifdef __WXOSX__
+       flags |= wxALIGN_RIGHT;
+#endif 
+
        _use_dci_name = new wxCheckBox (_dcp_panel, wxID_ANY, _("Use DCI name"));
-       grid->Add (_use_dci_name, wxGBPosition (r, 0), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
+       grid->Add (_use_dci_name, wxGBPosition (r, 0), wxDefaultSpan, flags);
        _edit_dci_button = new wxButton (_dcp_panel, wxID_ANY, _("Details..."));
        grid->Add (_edit_dci_button, wxGBPosition (r, 1), wxDefaultSpan);
        ++r;
 
-       add_label_to_grid_bag_sizer (grid, _dcp_panel, _("Container"), wxGBPosition (r, 0));
+       add_label_to_grid_bag_sizer (grid, _dcp_panel, _("Container"), true, wxGBPosition (r, 0));
        _container = new wxChoice (_dcp_panel, wxID_ANY);
-       grid->Add (_container, wxGBPosition (r, 1));
+       grid->Add (_container, wxGBPosition (r, 1), wxDefaultSpan, wxEXPAND);
        ++r;
 
-       add_label_to_grid_bag_sizer (grid, _dcp_panel, _("Content Type"), wxGBPosition (r, 0));
+       add_label_to_grid_bag_sizer (grid, _dcp_panel, _("Content Type"), true, wxGBPosition (r, 0));
        _dcp_content_type = new wxChoice (_dcp_panel, wxID_ANY);
        grid->Add (_dcp_content_type, wxGBPosition (r, 1));
        ++r;
 
        {
-               add_label_to_grid_bag_sizer (grid, _dcp_panel, _("DCP Frame Rate"), wxGBPosition (r, 0));
+               add_label_to_grid_bag_sizer (grid, _dcp_panel, _("DCP Frame Rate"), true, wxGBPosition (r, 0));
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _dcp_frame_rate = new wxChoice (_dcp_panel, wxID_ANY);
                s->Add (_dcp_frame_rate, 1, wxALIGN_CENTER_VERTICAL);
@@ -145,19 +149,24 @@ FilmEditor::make_dcp_panel ()
        }
        ++r;
 
+       add_label_to_grid_bag_sizer (grid, _dcp_panel, _("DCP audio channels"), true, wxGBPosition (r, 0));
+       _dcp_audio_channels = new wxSpinCtrl (_dcp_panel, wxID_ANY);
+       grid->Add (_dcp_audio_channels, wxGBPosition (r, 1));
+       ++r;
+
        {
-               add_label_to_grid_bag_sizer (grid, _dcp_panel, _("JPEG2000 bandwidth"), wxGBPosition (r, 0));
+               add_label_to_grid_bag_sizer (grid, _dcp_panel, _("JPEG2000 bandwidth"), true, wxGBPosition (r, 0));
                wxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _j2k_bandwidth = new wxSpinCtrl (_dcp_panel, wxID_ANY);
                s->Add (_j2k_bandwidth, 1);
-               add_label_to_sizer (s, _dcp_panel, _("MBps"));
+               add_label_to_sizer (s, _dcp_panel, _("MBps"), false);
                grid->Add (s, wxGBPosition (r, 1));
        }
        ++r;
 
-       add_label_to_grid_bag_sizer (grid, _dcp_panel, _("Scaler"), wxGBPosition (r, 0));
+       add_label_to_grid_bag_sizer (grid, _dcp_panel, _("Scaler"), true, wxGBPosition (r, 0));
        _scaler = new wxChoice (_dcp_panel, wxID_ANY);
-       grid->Add (_scaler, wxGBPosition (r, 1));
+       grid->Add (_scaler, wxGBPosition (r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
        ++r;
 
        vector<Scaler const *> const sc = Scaler::all ();
@@ -180,6 +189,7 @@ FilmEditor::make_dcp_panel ()
                _dcp_frame_rate->Append (std_to_wx (boost::lexical_cast<string> (*i)));
        }
 
+       _dcp_audio_channels->SetRange (0, MAX_AUDIO_CHANNELS);
        _j2k_bandwidth->SetRange (50, 250);
 }
 
@@ -207,10 +217,10 @@ FilmEditor::connect_to_widgets ()
        _dcp_content_type->Connect       (wxID_ANY, wxEVT_COMMAND_CHOICE_SELECTED,      wxCommandEventHandler (FilmEditor::dcp_content_type_changed), 0, this);
        _dcp_frame_rate->Connect         (wxID_ANY, wxEVT_COMMAND_CHOICE_SELECTED,      wxCommandEventHandler (FilmEditor::dcp_frame_rate_changed), 0, this);
        _best_dcp_frame_rate->Connect    (wxID_ANY, wxEVT_COMMAND_BUTTON_CLICKED,       wxCommandEventHandler (FilmEditor::best_dcp_frame_rate_clicked), 0, this);
+       _dcp_audio_channels->Connect     (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED,     wxCommandEventHandler (FilmEditor::dcp_audio_channels_changed), 0, this);
        _with_subtitles->Connect         (wxID_ANY, wxEVT_COMMAND_CHECKBOX_CLICKED,     wxCommandEventHandler (FilmEditor::with_subtitles_toggled), 0, this);
        _subtitle_offset->Connect        (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED,     wxCommandEventHandler (FilmEditor::subtitle_offset_changed), 0, this);
        _subtitle_scale->Connect         (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED,     wxCommandEventHandler (FilmEditor::subtitle_scale_changed), 0, this);
-       _colour_lut->Connect             (wxID_ANY, wxEVT_COMMAND_CHOICE_SELECTED,      wxCommandEventHandler (FilmEditor::colour_lut_changed), 0, this);
        _j2k_bandwidth->Connect          (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED,     wxCommandEventHandler (FilmEditor::j2k_bandwidth_changed), 0, this);
        _audio_gain->Connect             (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED,     wxCommandEventHandler (FilmEditor::audio_gain_changed), 0, this);
        _audio_gain_calculate_button->Connect (
@@ -232,31 +242,31 @@ FilmEditor::make_video_panel ()
        wxBoxSizer* video_sizer = new wxBoxSizer (wxVERTICAL);
        _video_panel->SetSizer (video_sizer);
        
-       wxGridBagSizer* grid = new wxGridBagSizer (4, 4);
+       wxGridBagSizer* grid = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
        video_sizer->Add (grid, 0, wxALL, 8);
 
        int r = 0;
-       add_label_to_grid_bag_sizer (grid, _video_panel, _("Left crop"), wxGBPosition (r, 0));
+       add_label_to_grid_bag_sizer (grid, _video_panel, _("Left crop"), true, wxGBPosition (r, 0));
        _left_crop = new wxSpinCtrl (_video_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
        grid->Add (_left_crop, wxGBPosition (r, 1));
        ++r;
 
-       add_label_to_grid_bag_sizer (grid, _video_panel, _("Right crop"), wxGBPosition (r, 0));
+       add_label_to_grid_bag_sizer (grid, _video_panel, _("Right crop"), true, wxGBPosition (r, 0));
        _right_crop = new wxSpinCtrl (_video_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
        grid->Add (_right_crop, wxGBPosition (r, 1));
        ++r;
        
-       add_label_to_grid_bag_sizer (grid, _video_panel, _("Top crop"), wxGBPosition (r, 0));
+       add_label_to_grid_bag_sizer (grid, _video_panel, _("Top crop"), true, wxGBPosition (r, 0));
        _top_crop = new wxSpinCtrl (_video_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
        grid->Add (_top_crop, wxGBPosition (r, 1));
        ++r;
        
-       add_label_to_grid_bag_sizer (grid, _video_panel, _("Bottom crop"), wxGBPosition (r, 0));
+       add_label_to_grid_bag_sizer (grid, _video_panel, _("Bottom crop"), true, wxGBPosition (r, 0));
        _bottom_crop = new wxSpinCtrl (_video_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
        grid->Add (_bottom_crop, wxGBPosition (r, 1));
        ++r;
 
-       add_label_to_grid_bag_sizer (grid, _video_panel, _("Scale to"), wxGBPosition (r, 0));
+       add_label_to_grid_bag_sizer (grid, _video_panel, _("Scale to"), true, wxGBPosition (r, 0));
        _ratio = new wxChoice (_video_panel, wxID_ANY);
        grid->Add (_ratio, wxGBPosition (r, 1));
        ++r;
@@ -271,25 +281,16 @@ FilmEditor::make_video_panel ()
 
        /* VIDEO-only stuff */
        {
-               add_label_to_grid_bag_sizer (grid, _video_panel, _("Filters"), wxGBPosition (r, 0));
+               add_label_to_grid_bag_sizer (grid, _video_panel, _("Filters"), true, wxGBPosition (r, 0));
                wxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _filters = new wxStaticText (_video_panel, wxID_ANY, _("None"));
                s->Add (_filters, 1, wxEXPAND | wxALIGN_CENTER_VERTICAL | wxTOP | wxBOTTOM | wxRIGHT, 6);
                _filters_button = new wxButton (_video_panel, wxID_ANY, _("Edit..."));
-               s->Add (_filters_button, 0);
+               s->Add (_filters_button, 0, wxALIGN_CENTER_VERTICAL);
                grid->Add (s, wxGBPosition (r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
        }
        ++r;
 
-       add_label_to_grid_bag_sizer (grid, _video_panel, _("Colour look-up table"), wxGBPosition (r, 0));
-       _colour_lut = new wxChoice (_video_panel, wxID_ANY);
-       for (int i = 0; i < 2; ++i) {
-               _colour_lut->Append (std_to_wx (colour_lut_index_to_name (i)));
-       }
-       _colour_lut->SetSelection (0);
-       grid->Add (_colour_lut, wxGBPosition (r, 1), wxDefaultSpan, wxEXPAND);
-       ++r;
-
        _left_crop->SetRange (0, 1024);
        _top_crop->SetRange (0, 1024);
        _right_crop->SetRange (0, 1024);
@@ -330,7 +331,7 @@ FilmEditor::make_content_panel ()
        h->Add (_loop_content, 0, wxALL, 6);
        _loop_count = new wxSpinCtrl (_content_panel, wxID_ANY);
        h->Add (_loop_count, 0, wxALL, 6);
-       add_label_to_sizer (h, _content_panel, _("times"));
+       add_label_to_sizer (h, _content_panel, _("times"), false);
        _content_sizer->Add (h, 0, wxALL, 6);
 
        _content_notebook = new wxNotebook (_content_panel, wxID_ANY, wxDefaultPosition, wxDefaultSize, wxNB_LEFT);
@@ -355,7 +356,7 @@ FilmEditor::make_audio_panel ()
        wxBoxSizer* audio_sizer = new wxBoxSizer (wxVERTICAL);
        _audio_panel->SetSizer (audio_sizer);
        
-       wxFlexGridSizer* grid = new wxFlexGridSizer (3, 4, 4);
+       wxFlexGridSizer* grid = new wxFlexGridSizer (3, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
        audio_sizer->Add (grid, 0, wxALL, 8);
 
        _show_audio = new wxButton (_audio_panel, wxID_ANY, _("Show Audio..."));
@@ -363,33 +364,33 @@ FilmEditor::make_audio_panel ()
        grid->AddSpacer (0);
        grid->AddSpacer (0);
 
-       add_label_to_sizer (grid, _audio_panel, _("Audio Gain"));
+       add_label_to_sizer (grid, _audio_panel, _("Audio Gain"), true);
        {
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _audio_gain = new wxSpinCtrl (_audio_panel);
                s->Add (_audio_gain, 1);
-               add_label_to_sizer (s, _audio_panel, _("dB"));
+               add_label_to_sizer (s, _audio_panel, _("dB"), false);
                grid->Add (s, 1);
        }
        
        _audio_gain_calculate_button = new wxButton (_audio_panel, wxID_ANY, _("Calculate..."));
        grid->Add (_audio_gain_calculate_button);
 
-       add_label_to_sizer (grid, _audio_panel, _("Audio Delay"));
+       add_label_to_sizer (grid, _audio_panel, _("Audio Delay"), false);
        {
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _audio_delay = new wxSpinCtrl (_audio_panel);
                s->Add (_audio_delay, 1);
                /// TRANSLATORS: this is an abbreviation for milliseconds, the unit of time
-               add_label_to_sizer (s, _audio_panel, _("ms"));
+               add_label_to_sizer (s, _audio_panel, _("ms"), false);
                grid->Add (s);
        }
 
        grid->AddSpacer (0);
 
-       add_label_to_sizer (grid, _audio_panel, _("Audio Stream"));
+       add_label_to_sizer (grid, _audio_panel, _("Audio Stream"), true);
        _audio_stream = new wxChoice (_audio_panel, wxID_ANY);
-       grid->Add (_audio_stream, 1);
+       grid->Add (_audio_stream, 1, wxEXPAND);
        _audio_description = new wxStaticText (_audio_panel, wxID_ANY, wxT (""));
        grid->AddSpacer (0);
        
@@ -410,7 +411,7 @@ FilmEditor::make_subtitle_panel ()
        _subtitle_panel = new wxPanel (_content_notebook);
        wxBoxSizer* subtitle_sizer = new wxBoxSizer (wxVERTICAL);
        _subtitle_panel->SetSizer (subtitle_sizer);
-       wxFlexGridSizer* grid = new wxFlexGridSizer (2, 4, 4);
+       wxFlexGridSizer* grid = new wxFlexGridSizer (2, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
        subtitle_sizer->Add (grid, 0, wxALL, 8);
 
        _with_subtitles = new wxCheckBox (_subtitle_panel, wxID_ANY, _("With Subtitles"));
@@ -418,30 +419,30 @@ FilmEditor::make_subtitle_panel ()
        grid->AddSpacer (0);
        
        {
-               add_label_to_sizer (grid, _subtitle_panel, _("Subtitle Offset"));
+               add_label_to_sizer (grid, _subtitle_panel, _("Subtitle Offset"), true);
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _subtitle_offset = new wxSpinCtrl (_subtitle_panel);
                s->Add (_subtitle_offset);
-               add_label_to_sizer (s, _subtitle_panel, _("pixels"));
+               add_label_to_sizer (s, _subtitle_panel, _("%"), false);
                grid->Add (s);
        }
 
        {
-               add_label_to_sizer (grid, _subtitle_panel, _("Subtitle Scale"));
+               add_label_to_sizer (grid, _subtitle_panel, _("Subtitle Scale"), true);
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _subtitle_scale = new wxSpinCtrl (_subtitle_panel);
                s->Add (_subtitle_scale);
-               add_label_to_sizer (s, _subtitle_panel, _("%"));
+               add_label_to_sizer (s, _subtitle_panel, _("%"), false);
                grid->Add (s);
        }
 
-       add_label_to_sizer (grid, _subtitle_panel, _("Subtitle Stream"));
+       add_label_to_sizer (grid, _subtitle_panel, _("Subtitle Stream"), true);
        _subtitle_stream = new wxChoice (_subtitle_panel, wxID_ANY);
-       grid->Add (_subtitle_stream, 1, wxEXPAND | wxALL, 6);
-       grid->AddSpacer (0);
+       grid->Add (_subtitle_stream, 1, wxEXPAND);
        
-       _subtitle_offset->SetRange (-1024, 1024);
+       _subtitle_offset->SetRange (-100, 100);
        _subtitle_scale->SetRange (1, 1000);
+       _subtitle_scale->SetValue (100);
 }
 
 void
@@ -453,10 +454,10 @@ FilmEditor::make_timing_panel ()
        wxFlexGridSizer* grid = new wxFlexGridSizer (2, 4, 4);
        timing_sizer->Add (grid, 0, wxALL, 8);
 
-       add_label_to_sizer (grid, _timing_panel, _("Start time"));
+       add_label_to_sizer (grid, _timing_panel, _("Start time"), true);
        _start = new Timecode (_timing_panel);
        grid->Add (_start);
-       add_label_to_sizer (grid, _timing_panel, _("Length"));
+       add_label_to_sizer (grid, _timing_panel, _("Length"), true);
        _length = new Timecode (_timing_panel);
        grid->Add (_length);
 }
@@ -524,31 +525,23 @@ FilmEditor::name_changed (wxCommandEvent &)
 void
 FilmEditor::subtitle_offset_changed (wxCommandEvent &)
 {
-       if (!_film) {
+       shared_ptr<SubtitleContent> c = selected_subtitle_content ();
+       if (!c) {
                return;
        }
 
-       _film->set_subtitle_offset (_subtitle_offset->GetValue ());
+       c->set_subtitle_offset (_subtitle_offset->GetValue() / 100.0);
 }
 
 void
 FilmEditor::subtitle_scale_changed (wxCommandEvent &)
 {
-       if (!_film) {
+       shared_ptr<SubtitleContent> c = selected_subtitle_content ();
+       if (!c) {
                return;
        }
 
-       _film->set_subtitle_scale (_subtitle_scale->GetValue() / 100.0);
-}
-
-void
-FilmEditor::colour_lut_changed (wxCommandEvent &)
-{
-       if (!_film) {
-               return;
-       }
-       
-       _film->set_colour_lut (_colour_lut->GetSelection ());
+       c->set_subtitle_scale (_subtitle_scale->GetValue() / 100.0);
 }
 
 void
@@ -575,6 +568,16 @@ FilmEditor::dcp_frame_rate_changed (wxCommandEvent &)
                );
 }
 
+void
+FilmEditor::dcp_audio_channels_changed (wxCommandEvent &)
+{
+       if (!_film) {
+               return;
+       }
+
+       _film->set_dcp_audio_channels (_dcp_audio_channels->GetValue ());
+}
+
 
 /** Called when the metadata stored in the Film object has changed;
  *  so that we can update the GUI.
@@ -596,8 +599,6 @@ FilmEditor::film_changed (Film::Property p)
                break;
        case Film::CONTENT:
                setup_content ();
-               setup_ratios ();
-//             setup_ratio ();
                setup_subtitle_control_sensitivity ();
                setup_show_audio_sensitivity ();
                break;
@@ -625,15 +626,6 @@ FilmEditor::film_changed (Film::Property p)
                setup_subtitle_control_sensitivity ();
                setup_dcp_name ();
                break;
-       case Film::SUBTITLE_OFFSET:
-               checked_set (_subtitle_offset, _film->subtitle_offset ());
-               break;
-       case Film::SUBTITLE_SCALE:
-               checked_set (_subtitle_scale, _film->subtitle_scale() * 100);
-               break;
-       case Film::COLOUR_LUT:
-               checked_set (_colour_lut, _film->colour_lut ());
-               break;
        case Film::J2K_BANDWIDTH:
                checked_set (_j2k_bandwidth, double (_film->j2k_bandwidth()) / 1e6);
                break;
@@ -662,6 +654,10 @@ FilmEditor::film_changed (Film::Property p)
                _best_dcp_frame_rate->Enable (_film->best_dcp_video_frame_rate () != _film->dcp_video_frame_rate ());
                break;
        }
+       case Film::DCP_AUDIO_CHANNELS:
+               _dcp_audio_channels->SetValue (_film->dcp_audio_channels ());
+               _audio_mapping->set_channels (_film->dcp_audio_channels ());
+               break;
        }
 }
 
@@ -678,10 +674,12 @@ FilmEditor::film_content_changed (weak_ptr<Content> weak_content, int property)
        shared_ptr<Content> content = weak_content.lock ();
        shared_ptr<VideoContent> video_content;
        shared_ptr<AudioContent> audio_content;
+       shared_ptr<SubtitleContent> subtitle_content;
        shared_ptr<FFmpegContent> ffmpeg_content;
        if (content) {
                video_content = dynamic_pointer_cast<VideoContent> (content);
                audio_content = dynamic_pointer_cast<AudioContent> (content);
+               subtitle_content = dynamic_pointer_cast<SubtitleContent> (content);
                ffmpeg_content = dynamic_pointer_cast<FFmpegContent> (content);
        }
 
@@ -700,9 +698,9 @@ FilmEditor::film_content_changed (weak_ptr<Content> weak_content, int property)
                        _length->set (0, 24);
                }
        } else if (property == VideoContentProperty::VIDEO_CROP) {
-               checked_set (_left_crop,   video_content ? video_content->crop().left :   0);
-               checked_set (_right_crop,  video_content ? video_content->crop().right  0);
-               checked_set (_top_crop,    video_content ? video_content->crop().top :    0);
+               checked_set (_left_crop,   video_content ? video_content->crop().left   : 0);
+               checked_set (_right_crop,  video_content ? video_content->crop().right  : 0);
+               checked_set (_top_crop,    video_content ? video_content->crop().top    : 0);
                checked_set (_bottom_crop, video_content ? video_content->crop().bottom : 0);
                setup_scaling_description ();
        } else if (property == VideoContentProperty::VIDEO_RATIO) {
@@ -714,7 +712,7 @@ FilmEditor::film_content_changed (weak_ptr<Content> weak_content, int property)
                                ++i;
                                ++n;
                        }
-                       
+
                        if (i == ratios.end()) {
                                checked_set (_ratio, -1);
                        } else {
@@ -723,6 +721,7 @@ FilmEditor::film_content_changed (weak_ptr<Content> weak_content, int property)
                } else {
                        checked_set (_ratio, -1);
                }
+               setup_scaling_description ();
        } else if (property == AudioContentProperty::AUDIO_GAIN) {
                checked_set (_audio_gain, audio_content ? audio_content->audio_gain() : 0);
        } else if (property == AudioContentProperty::AUDIO_DELAY) {
@@ -774,6 +773,10 @@ FilmEditor::film_content_changed (weak_ptr<Content> weak_content, int property)
                        }
                        _dcp_sizer->Layout ();
                }
+       } else if (property == SubtitleContentProperty::SUBTITLE_OFFSET) {
+               checked_set (_subtitle_offset, subtitle_content ? (subtitle_content->subtitle_offset() * 100) : 0);
+       } else if (property == SubtitleContentProperty::SUBTITLE_SCALE) {
+               checked_set (_subtitle_scale, subtitle_content ? (subtitle_content->subtitle_scale() * 100) : 100);
        }
 }
 
@@ -851,10 +854,6 @@ FilmEditor::set_film (shared_ptr<Film> f)
                FileChanged ("");
        }
 
-       if (_audio_dialog) {
-               _audio_dialog->set_film (_film);
-       }
-       
        film_changed (Film::NAME);
        film_changed (Film::USE_DCI_NAME);
        film_changed (Film::CONTENT);
@@ -863,26 +862,17 @@ FilmEditor::set_film (shared_ptr<Film> f)
        film_changed (Film::CONTAINER);
        film_changed (Film::SCALER);
        film_changed (Film::WITH_SUBTITLES);
-       film_changed (Film::SUBTITLE_OFFSET);
-       film_changed (Film::SUBTITLE_SCALE);
-       film_changed (Film::COLOUR_LUT);
        film_changed (Film::J2K_BANDWIDTH);
        film_changed (Film::DCI_METADATA);
        film_changed (Film::DCP_VIDEO_FRAME_RATE);
+       film_changed (Film::DCP_AUDIO_CHANNELS);
 
-       shared_ptr<Content> s = selected_content ();
-       film_content_changed (s, ContentProperty::START);
-       film_content_changed (s, ContentProperty::LENGTH);
-       film_content_changed (s, VideoContentProperty::VIDEO_CROP);
-       film_content_changed (s, VideoContentProperty::VIDEO_RATIO);
-       film_content_changed (s, AudioContentProperty::AUDIO_GAIN);
-       film_content_changed (s, AudioContentProperty::AUDIO_DELAY);
-       film_content_changed (s, AudioContentProperty::AUDIO_MAPPING);
-       film_content_changed (s, FFmpegContentProperty::SUBTITLE_STREAMS);
-       film_content_changed (s, FFmpegContentProperty::SUBTITLE_STREAM);
-       film_content_changed (s, FFmpegContentProperty::AUDIO_STREAMS);
-       film_content_changed (s, FFmpegContentProperty::AUDIO_STREAM);
-       film_content_changed (s, FFmpegContentProperty::FILTERS);
+       if (!_film->content().empty ()) {
+               set_selection (_film->content().front ());
+       }
+
+       wxListEvent ev;
+       content_selection_changed (ev);
 }
 
 /** Updates the sensitivity of lots of widgets to a given value.
@@ -905,9 +895,8 @@ FilmEditor::set_things_sensitive (bool s)
        _filters_button->Enable (s);
        _scaler->Enable (s);
        _dcp_content_type->Enable (s);
-       _best_dcp_frame_rate->Enable (s);
        _dcp_frame_rate->Enable (s);
-       _colour_lut->Enable (s);
+       _dcp_audio_channels->Enable (s);
        _j2k_bandwidth->Enable (s);
        _audio_gain->Enable (s);
        _audio_gain_calculate_button->Enable (s);
@@ -920,6 +909,7 @@ FilmEditor::set_things_sensitive (bool s)
        setup_subtitle_control_sensitivity ();
        setup_show_audio_sensitivity ();
        setup_content_sensitivity ();
+       _best_dcp_frame_rate->Enable (s && _film && _film->best_dcp_video_frame_rate () != _film->dcp_video_frame_rate ());
 }
 
 /** Called when the `Edit filters' button has been clicked */
@@ -1109,10 +1099,20 @@ FilmEditor::show_audio_clicked (wxCommandEvent &)
                _audio_dialog->Destroy ();
                _audio_dialog = 0;
        }
+
+       shared_ptr<Content> c = selected_content ();
+       if (!c) {
+               return;
+       }
+
+       shared_ptr<AudioContent> ac = dynamic_pointer_cast<AudioContent> (c);
+       if (!ac) {
+               return;
+       }
        
        _audio_dialog = new AudioDialog (this);
        _audio_dialog->Show ();
-       _audio_dialog->set_film (_film);
+       _audio_dialog->set_content (ac);
 }
 
 void
@@ -1152,7 +1152,7 @@ FilmEditor::setup_content ()
        }
 
        if (selected_summary.empty () && !content.empty ()) {
-               /* Select the item of content if non was selected before */
+               /* Select the item of content if none was selected before */
                _content->SetItemState (0, wxLIST_STATE_SELECTED, wxLIST_STATE_SELECTED);
        }
 }
@@ -1195,6 +1195,9 @@ FilmEditor::content_remove_clicked (wxCommandEvent &)
        if (c) {
                _film->remove_content (c);
        }
+
+       wxListEvent ev;
+       content_selection_changed (ev);
 }
 
 void
@@ -1202,6 +1205,11 @@ FilmEditor::content_selection_changed (wxListEvent &)
 {
         setup_content_sensitivity ();
        shared_ptr<Content> s = selected_content ();
+       
+       if (_audio_dialog && s && dynamic_pointer_cast<AudioContent> (s)) {
+               _audio_dialog->set_content (dynamic_pointer_cast<AudioContent> (s));
+       }
+
        film_content_changed (s, ContentProperty::START);
        film_content_changed (s, ContentProperty::LENGTH);
        film_content_changed (s, VideoContentProperty::VIDEO_CROP);
@@ -1212,6 +1220,10 @@ FilmEditor::content_selection_changed (wxListEvent &)
        film_content_changed (s, FFmpegContentProperty::AUDIO_STREAM);
        film_content_changed (s, FFmpegContentProperty::AUDIO_STREAMS);
        film_content_changed (s, FFmpegContentProperty::SUBTITLE_STREAM);
+       film_content_changed (s, FFmpegContentProperty::SUBTITLE_STREAMS);
+       film_content_changed (s, FFmpegContentProperty::FILTERS);
+       film_content_changed (s, SubtitleContentProperty::SUBTITLE_OFFSET);
+       film_content_changed (s, SubtitleContentProperty::SUBTITLE_SCALE);
 }
 
 void
@@ -1268,27 +1280,44 @@ FilmEditor::selected_audio_content ()
        return dynamic_pointer_cast<AudioContent> (c);
 }
 
+shared_ptr<SubtitleContent>
+FilmEditor::selected_subtitle_content ()
+{
+       shared_ptr<Content> c = selected_content ();
+       if (!c) {
+               return shared_ptr<SubtitleContent> ();
+       }
+
+       return dynamic_pointer_cast<SubtitleContent> (c);
+}
+
 void
 FilmEditor::setup_scaling_description ()
 {
+       shared_ptr<VideoContent> vc = selected_video_content ();
+       if (!vc) {
+               _scaling_description->SetLabel ("");
+               return;
+       }
+
        wxString d;
 
-#if 0  
-XXX
        int lines = 0;
 
-       if (_film->video_size().width && _film->video_size().height) {
+       if (vc->video_size().width && vc->video_size().height) {
                d << wxString::Format (
                        _("Original video is %dx%d (%.2f:1)\n"),
-                       _film->video_size().width, _film->video_size().height,
-                       float (_film->video_size().width) / _film->video_size().height
+                       vc->video_size().width, vc->video_size().height,
+                       float (vc->video_size().width) / vc->video_size().height
                        );
                ++lines;
        }
 
-       Crop const crop = _film->crop ();
-       if (crop.left || crop.right || crop.top || crop.bottom) {
-               libdcp::Size const cropped = _film->cropped_size (_film->video_size ());
+       Crop const crop = vc->crop ();
+       if ((crop.left || crop.right || crop.top || crop.bottom) && vc->video_size() != libdcp::Size (0, 0)) {
+               libdcp::Size cropped = vc->video_size ();
+               cropped.width -= crop.left + crop.right;
+               cropped.height -= crop.top + crop.bottom;
                d << wxString::Format (
                        _("Cropped to %dx%d (%.2f:1)\n"),
                        cropped.width, cropped.height,
@@ -1297,11 +1326,11 @@ XXX
                ++lines;
        }
 
-       Format const * format = _film->format ();
-       if (format) {
-               int const padding = format->dcp_padding (_film);
-               libdcp::Size scaled = format->dcp_size ();
-               scaled.width -= padding * 2;
+       Ratio const * ratio = vc->ratio ();
+       if (ratio) {
+               libdcp::Size container_size = _film->container()->size (_film->full_frame ());
+               
+               libdcp::Size const scaled = ratio->size (container_size);
                d << wxString::Format (
                        _("Scaled to %dx%d (%.2f:1)\n"),
                        scaled.width, scaled.height,
@@ -1309,11 +1338,11 @@ XXX
                        );
                ++lines;
 
-               if (padding) {
+               if (scaled != container_size) {
                        d << wxString::Format (
                                _("Padded with black to %dx%d (%.2f:1)\n"),
-                               format->dcp_size().width, format->dcp_size().height,
-                               float (format->dcp_size().width) / format->dcp_size().height
+                               container_size.width, container_size.height,
+                               float (container_size.width) / container_size.height
                                );
                        ++lines;
                }
@@ -1323,7 +1352,6 @@ XXX
                d << wxT ("\n ");
        }
 
-#endif 
        _scaling_description->SetLabel (d);
 }