Various fixes to make audio analysis sort-of work.
[dcpomatic.git] / src / wx / film_editor.cc
index 85c4f71f866631f4018e635d0b8cb5d3dab3256d..27803fb9a7e2464778a2eae26f0fb76c811d8d7d 100644 (file)
 #include <boost/thread.hpp>
 #include <boost/filesystem.hpp>
 #include <boost/lexical_cast.hpp>
-#include "lib/format.h"
 #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"
 #include "lib/config.h"
-#include "lib/ffmpeg_decoder.h"
 #include "lib/imagemagick_content.h"
 #include "lib/sndfile_content.h"
 #include "lib/dcp_content_type.h"
@@ -50,8 +48,9 @@
 #include "scaler.h"
 #include "audio_dialog.h"
 #include "imagemagick_content_dialog.h"
-#include "ffmpeg_content_dialog.h"
+#include "timeline_dialog.h"
 #include "audio_mapping_view.h"
+#include "timecode.h"
 
 using std::string;
 using std::cout;
@@ -72,21 +71,19 @@ FilmEditor::FilmEditor (shared_ptr<Film> f, wxWindow* parent)
        : wxPanel (parent)
        , _generally_sensitive (true)
        , _audio_dialog (0)
+       , _timeline_dialog (0)
 {
        wxBoxSizer* s = new wxBoxSizer (wxVERTICAL);
-       _notebook = new wxNotebook (this, wxID_ANY);
-       s->Add (_notebook, 1);
 
-       make_film_panel ();
-       _notebook->AddPage (_film_panel, _("Film"), true);
+       _main_notebook = new wxNotebook (this, wxID_ANY);
+       s->Add (_main_notebook, 1);
+
        make_content_panel ();
-       _notebook->AddPage (_content_panel, _("Content"), false);
-       make_video_panel ();
-       _notebook->AddPage (_video_panel, _("Video"), false);
-       make_audio_panel ();
-       _notebook->AddPage (_audio_panel, _("Audio"), false);
-       make_subtitle_panel ();
-       _notebook->AddPage (_subtitle_panel, _("Subtitles"), false);
+       _main_notebook->AddPage (_content_panel, _("Content"), true);
+       make_dcp_panel ();
+       _main_notebook->AddPage (_dcp_panel, _("DCP"), false);
+       
+       setup_ratios ();
 
        set_film (f);
        connect_to_widgets ();
@@ -95,95 +92,82 @@ FilmEditor::FilmEditor (shared_ptr<Film> f, wxWindow* parent)
                bind (&FilmEditor::active_jobs_changed, this, _1)
                );
        
-       setup_formats ();
-
        SetSizerAndFit (s);
 }
 
 void
-FilmEditor::make_film_panel ()
+FilmEditor::make_dcp_panel ()
 {
-       _film_panel = new wxPanel (_notebook);
-       _film_sizer = new wxBoxSizer (wxVERTICAL);
-       _film_panel->SetSizer (_film_sizer);
+       _dcp_panel = new wxPanel (_main_notebook);
+       _dcp_sizer = new wxBoxSizer (wxVERTICAL);
+       _dcp_panel->SetSizer (_dcp_sizer);
 
        wxGridBagSizer* grid = new wxGridBagSizer (4, 4);
-       _film_sizer->Add (grid, 0, wxALL, 8);
+       _dcp_sizer->Add (grid, 0, wxEXPAND | wxALL, 8);
 
        int r = 0;
        
-       add_label_to_grid_bag_sizer (grid, _film_panel, _("Name"), wxGBPosition (r, 0));
-       _name = new wxTextCtrl (_film_panel, wxID_ANY);
+       add_label_to_grid_bag_sizer (grid, _dcp_panel, _("Name"), wxGBPosition (r, 0));
+       _name = new wxTextCtrl (_dcp_panel, wxID_ANY);
        grid->Add (_name, wxGBPosition(r, 1), wxDefaultSpan, wxEXPAND);
        ++r;
        
-       add_label_to_grid_bag_sizer (grid, _film_panel, _("DCP Name"), wxGBPosition (r, 0));
-       _dcp_name = new wxStaticText (_film_panel, wxID_ANY, wxT (""));
+       add_label_to_grid_bag_sizer (grid, _dcp_panel, _("DCP Name"), wxGBPosition (r, 0));
+       _dcp_name = new wxStaticText (_dcp_panel, wxID_ANY, wxT (""));
        grid->Add (_dcp_name, wxGBPosition(r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
        ++r;
 
-       _use_dci_name = new wxCheckBox (_film_panel, wxID_ANY, _("Use DCI name"));
+       _use_dci_name = new wxCheckBox (_dcp_panel, wxID_ANY, _("Use DCI name"));
        grid->Add (_use_dci_name, wxGBPosition (r, 0), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
-       _edit_dci_button = new wxButton (_film_panel, wxID_ANY, _("Details..."));
+       _edit_dci_button = new wxButton (_dcp_panel, wxID_ANY, _("Details..."));
        grid->Add (_edit_dci_button, wxGBPosition (r, 1), wxDefaultSpan);
        ++r;
 
-       _trust_content_headers = new wxCheckBox (_film_panel, wxID_ANY, _("Trust content's header"));
-       grid->Add (_trust_content_headers, wxGBPosition (r, 0), wxGBSpan(1, 2));
+       add_label_to_grid_bag_sizer (grid, _dcp_panel, _("Container"), wxGBPosition (r, 0));
+       _container = new wxChoice (_dcp_panel, wxID_ANY);
+       grid->Add (_container, wxGBPosition (r, 1));
        ++r;
 
-       add_label_to_grid_bag_sizer (grid, _film_panel, _("Content Type"), wxGBPosition (r, 0));
-       _dcp_content_type = new wxChoice (_film_panel, wxID_ANY);
+       add_label_to_grid_bag_sizer (grid, _dcp_panel, _("Content Type"), 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, _film_panel, _("DCP Frame Rate"), wxGBPosition (r, 0));
+               add_label_to_grid_bag_sizer (grid, _dcp_panel, _("DCP Frame Rate"), wxGBPosition (r, 0));
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
-               _dcp_frame_rate = new wxChoice (_film_panel, wxID_ANY);
+               _dcp_frame_rate = new wxChoice (_dcp_panel, wxID_ANY);
                s->Add (_dcp_frame_rate, 1, wxALIGN_CENTER_VERTICAL);
-               _best_dcp_frame_rate = new wxButton (_film_panel, wxID_ANY, _("Use best"));
-               s->Add (_best_dcp_frame_rate, 1, wxALIGN_CENTER_VERTICAL | wxALL | wxEXPAND, 6);
+               _best_dcp_frame_rate = new wxButton (_dcp_panel, wxID_ANY, _("Use best"));
+               s->Add (_best_dcp_frame_rate, 1, wxALIGN_CENTER_VERTICAL | wxEXPAND);
                grid->Add (s, wxGBPosition (r, 1));
        }
        ++r;
 
-       _frame_rate_description = new wxStaticText (_film_panel, wxID_ANY, wxT ("\n \n "), wxDefaultPosition, wxDefaultSize);
-       grid->Add (_frame_rate_description, wxGBPosition (r, 0), wxGBSpan (1, 2), wxEXPAND | wxALIGN_CENTER_VERTICAL | wxALL, 6);
-       wxFont font = _frame_rate_description->GetFont();
-       font.SetStyle(wxFONTSTYLE_ITALIC);
-       font.SetPointSize(font.GetPointSize() - 1);
-       _frame_rate_description->SetFont(font);
-       ++r;
-       
-       add_label_to_grid_bag_sizer (grid, _film_panel, _("Length"), wxGBPosition (r, 0));
-       _length = new wxStaticText (_film_panel, wxID_ANY, wxT (""));
-       grid->Add (_length, wxGBPosition (r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
-       ++r;
-
-
        {
-               add_label_to_grid_bag_sizer (grid, _film_panel, _("Trim frames"), wxGBPosition (r, 0));
-               wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
-               add_label_to_sizer (s, _film_panel, _("Start"));
-               _trim_start = new wxSpinCtrl (_film_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
-               s->Add (_trim_start);
-               add_label_to_sizer (s, _film_panel, _("End"));
-               _trim_end = new wxSpinCtrl (_film_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
-               s->Add (_trim_end);
-
+               add_label_to_grid_bag_sizer (grid, _dcp_panel, _("JPEG2000 bandwidth"), 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"));
                grid->Add (s, wxGBPosition (r, 1));
        }
        ++r;
 
-       add_label_to_grid_bag_sizer (grid, _film_panel, _("Trim method"), wxGBPosition (r, 0));
-       _trim_type = new wxChoice (_film_panel, wxID_ANY);
-       grid->Add (_trim_type, wxGBPosition (r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
+       add_label_to_grid_bag_sizer (grid, _dcp_panel, _("Scaler"), wxGBPosition (r, 0));
+       _scaler = new wxChoice (_dcp_panel, wxID_ANY);
+       grid->Add (_scaler, wxGBPosition (r, 1));
        ++r;
 
-       _ab = new wxCheckBox (_film_panel, wxID_ANY, _("A/B"));
-       grid->Add (_ab, wxGBPosition (r, 0));
-       ++r;
+       vector<Scaler const *> const sc = Scaler::all ();
+       for (vector<Scaler const *>::const_iterator i = sc.begin(); i != sc.end(); ++i) {
+               _scaler->Append (std_to_wx ((*i)->name()));
+       }
+
+       vector<Ratio const *> const ratio = Ratio::all ();
+       for (vector<Ratio const *>::const_iterator i = ratio.begin(); i != ratio.end(); ++i) {
+               _container->Append (std_to_wx ((*i)->nickname ()));
+       }
 
        vector<DCPContentType const *> const ct = DCPContentType::all ();
        for (vector<DCPContentType const *>::const_iterator i = ct.begin(); i != ct.end(); ++i) {
@@ -195,8 +179,7 @@ FilmEditor::make_film_panel ()
                _dcp_frame_rate->Append (std_to_wx (boost::lexical_cast<string> (*i)));
        }
 
-       _trim_type->Append (_("encode all frames and play the subset"));
-       _trim_type->Append (_("encode only the subset"));
+       _j2k_bandwidth->SetRange (50, 250);
 }
 
 void
@@ -205,16 +188,13 @@ FilmEditor::connect_to_widgets ()
        _name->Connect                   (wxID_ANY, wxEVT_COMMAND_TEXT_UPDATED,         wxCommandEventHandler (FilmEditor::name_changed), 0, this);
        _use_dci_name->Connect           (wxID_ANY, wxEVT_COMMAND_CHECKBOX_CLICKED,     wxCommandEventHandler (FilmEditor::use_dci_name_toggled), 0, this);
        _edit_dci_button->Connect        (wxID_ANY, wxEVT_COMMAND_BUTTON_CLICKED,       wxCommandEventHandler (FilmEditor::edit_dci_button_clicked), 0, this);
-       _format->Connect                 (wxID_ANY, wxEVT_COMMAND_CHOICE_SELECTED,      wxCommandEventHandler (FilmEditor::format_changed), 0, this);
-       _trust_content_headers->Connect  (wxID_ANY, wxEVT_COMMAND_CHECKBOX_CLICKED,     wxCommandEventHandler (FilmEditor::trust_content_headers_changed), 0, this);
+       _container->Connect              (wxID_ANY, wxEVT_COMMAND_CHOICE_SELECTED,      wxCommandEventHandler (FilmEditor::container_changed), 0, this);
+       _ratio->Connect                  (wxID_ANY, wxEVT_COMMAND_CHOICE_SELECTED,      wxCommandEventHandler (FilmEditor::ratio_changed), 0, this);
        _content->Connect                (wxID_ANY, wxEVT_COMMAND_LIST_ITEM_SELECTED,   wxListEventHandler    (FilmEditor::content_selection_changed), 0, this);
        _content->Connect                (wxID_ANY, wxEVT_COMMAND_LIST_ITEM_DESELECTED, wxListEventHandler    (FilmEditor::content_selection_changed), 0, this);
-       _content->Connect                (wxID_ANY, wxEVT_COMMAND_LIST_ITEM_ACTIVATED,  wxListEventHandler    (FilmEditor::content_activated), 0, this);
        _content_add->Connect            (wxID_ANY, wxEVT_COMMAND_BUTTON_CLICKED,       wxCommandEventHandler (FilmEditor::content_add_clicked), 0, this);
        _content_remove->Connect         (wxID_ANY, wxEVT_COMMAND_BUTTON_CLICKED,       wxCommandEventHandler (FilmEditor::content_remove_clicked), 0, this);
-       _content_edit->Connect           (wxID_ANY, wxEVT_COMMAND_BUTTON_CLICKED,       wxCommandEventHandler (FilmEditor::content_edit_clicked), 0, this);
-       _content_earlier->Connect        (wxID_ANY, wxEVT_COMMAND_BUTTON_CLICKED,       wxCommandEventHandler (FilmEditor::content_earlier_clicked), 0, this);
-       _content_later->Connect          (wxID_ANY, wxEVT_COMMAND_BUTTON_CLICKED,       wxCommandEventHandler (FilmEditor::content_later_clicked), 0, this);
+       _content_timeline->Connect       (wxID_ANY, wxEVT_COMMAND_BUTTON_CLICKED,       wxCommandEventHandler (FilmEditor::content_timeline_clicked), 0, this);
        _loop_content->Connect           (wxID_ANY, wxEVT_COMMAND_CHECKBOX_CLICKED,     wxCommandEventHandler (FilmEditor::loop_content_toggled), 0, this);
        _loop_count->Connect             (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED,     wxCommandEventHandler (FilmEditor::loop_count_changed), 0, this);
        _left_crop->Connect              (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED,     wxCommandEventHandler (FilmEditor::left_crop_changed), 0, this);
@@ -226,10 +206,6 @@ 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);
-       _ab->Connect                     (wxID_ANY, wxEVT_COMMAND_CHECKBOX_CLICKED,     wxCommandEventHandler (FilmEditor::ab_toggled), 0, this);
-       _trim_start->Connect             (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED,     wxCommandEventHandler (FilmEditor::trim_start_changed), 0, this);
-       _trim_end->Connect               (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED,     wxCommandEventHandler (FilmEditor::trim_end_changed), 0, this);
-       _trim_type->Connect              (wxID_ANY, wxEVT_COMMAND_CHOICE_SELECTED,      wxCommandEventHandler (FilmEditor::trim_type_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);
@@ -241,24 +217,24 @@ FilmEditor::connect_to_widgets ()
                );
        _show_audio->Connect             (wxID_ANY, wxEVT_COMMAND_BUTTON_CLICKED,       wxCommandEventHandler (FilmEditor::show_audio_clicked), 0, this);
        _audio_delay->Connect            (wxID_ANY, wxEVT_COMMAND_SPINCTRL_UPDATED,     wxCommandEventHandler (FilmEditor::audio_delay_changed), 0, this);
+       _audio_stream->Connect           (wxID_ANY, wxEVT_COMMAND_CHOICE_SELECTED,      wxCommandEventHandler (FilmEditor::audio_stream_changed), 0, this);
+       _subtitle_stream->Connect        (wxID_ANY, wxEVT_COMMAND_CHOICE_SELECTED,      wxCommandEventHandler (FilmEditor::subtitle_stream_changed), 0, this);
+       _audio_mapping->Changed.connect  (boost::bind (&FilmEditor::audio_mapping_changed, this, _1));
+       _start->Changed.connect          (boost::bind (&FilmEditor::start_changed, this));
+       _length->Changed.connect         (boost::bind (&FilmEditor::length_changed, this));
 }
 
 void
 FilmEditor::make_video_panel ()
 {
-       _video_panel = new wxPanel (_notebook);
-       _video_sizer = new wxBoxSizer (wxVERTICAL);
-       _video_panel->SetSizer (_video_sizer);
+       _video_panel = new wxPanel (_content_notebook);
+       wxBoxSizer* video_sizer = new wxBoxSizer (wxVERTICAL);
+       _video_panel->SetSizer (video_sizer);
        
        wxGridBagSizer* grid = new wxGridBagSizer (4, 4);
-       _video_sizer->Add (grid, 0, wxALL, 8);
+       video_sizer->Add (grid, 0, wxALL, 8);
 
        int r = 0;
-       add_label_to_grid_bag_sizer (grid, _video_panel, _("Format"), wxGBPosition (r, 0));
-       _format = new wxChoice (_video_panel, wxID_ANY);
-       grid->Add (_format, wxGBPosition (r, 1));
-       ++r;
-
        add_label_to_grid_bag_sizer (grid, _video_panel, _("Left crop"), wxGBPosition (r, 0));
        _left_crop = new wxSpinCtrl (_video_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1));
        grid->Add (_left_crop, wxGBPosition (r, 1));
@@ -279,6 +255,11 @@ FilmEditor::make_video_panel ()
        grid->Add (_bottom_crop, wxGBPosition (r, 1));
        ++r;
 
+       add_label_to_grid_bag_sizer (grid, _video_panel, _("Scale to"), wxGBPosition (r, 0));
+       _ratio = new wxChoice (_video_panel, wxID_ANY);
+       grid->Add (_ratio, wxGBPosition (r, 1));
+       ++r;
+
        _scaling_description = new wxStaticText (_video_panel, wxID_ANY, wxT ("\n \n \n \n"), wxDefaultPosition, wxDefaultSize);
        grid->Add (_scaling_description, wxGBPosition (r, 0), wxGBSpan (1, 2), wxEXPAND | wxALIGN_CENTER_VERTICAL | wxALL, 6);
        wxFont font = _scaling_description->GetFont();
@@ -299,16 +280,6 @@ FilmEditor::make_video_panel ()
        }
        ++r;
 
-       add_label_to_grid_bag_sizer (grid, _video_panel, _("Scaler"), wxGBPosition (r, 0));
-       _scaler = new wxChoice (_video_panel, wxID_ANY);
-       grid->Add (_scaler, wxGBPosition (r, 1));
-       ++r;
-
-       vector<Scaler const *> const sc = Scaler::all ();
-       for (vector<Scaler const *>::const_iterator i = sc.begin(); i != sc.end(); ++i) {
-               _scaler->Append (std_to_wx ((*i)->name()));
-       }
-
        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) {
@@ -318,32 +289,19 @@ FilmEditor::make_video_panel ()
        grid->Add (_colour_lut, wxGBPosition (r, 1), wxDefaultSpan, wxEXPAND);
        ++r;
 
-       {
-               add_label_to_grid_bag_sizer (grid, _video_panel, _("JPEG2000 bandwidth"), wxGBPosition (r, 0));
-               wxSizer* s = new wxBoxSizer (wxHORIZONTAL);
-               _j2k_bandwidth = new wxSpinCtrl (_video_panel, wxID_ANY);
-               s->Add (_j2k_bandwidth, 1);
-               add_label_to_sizer (s, _video_panel, _("MBps"));
-               grid->Add (s, wxGBPosition (r, 1));
-       }
-       ++r;
-
        _left_crop->SetRange (0, 1024);
        _top_crop->SetRange (0, 1024);
        _right_crop->SetRange (0, 1024);
        _bottom_crop->SetRange (0, 1024);
-       _trim_start->SetRange (0, 100);
-       _trim_end->SetRange (0, 100);
-       _j2k_bandwidth->SetRange (50, 250);
 }
 
 void
 FilmEditor::make_content_panel ()
 {
-       _content_panel = new wxPanel (_notebook);
+       _content_panel = new wxPanel (_main_notebook);
        _content_sizer = new wxBoxSizer (wxVERTICAL);
        _content_panel->SetSizer (_content_sizer);
-       
+
         {
                 wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                 
@@ -355,19 +313,15 @@ FilmEditor::make_content_panel ()
 
                 wxBoxSizer* b = new wxBoxSizer (wxVERTICAL);
                 _content_add = new wxButton (_content_panel, wxID_ANY, _("Add..."));
-                b->Add (_content_add);
+                b->Add (_content_add, 1, wxEXPAND | wxLEFT | wxRIGHT);
                 _content_remove = new wxButton (_content_panel, wxID_ANY, _("Remove"));
-                b->Add (_content_remove);
-                _content_edit = new wxButton (_content_panel, wxID_ANY, _("Edit..."));
-                b->Add (_content_edit);
-                _content_earlier = new wxButton (_content_panel, wxID_ANY, _("Earlier"));
-                b->Add (_content_earlier);
-                _content_later = new wxButton (_content_panel, wxID_ANY, _("Later"));
-                b->Add (_content_later);
+                b->Add (_content_remove, 1, wxEXPAND | wxLEFT | wxRIGHT);
+               _content_timeline = new wxButton (_content_panel, wxID_ANY, _("Timeline..."));
+               b->Add (_content_timeline, 1, wxEXPAND | wxLEFT | wxRIGHT);
 
                 s->Add (b, 0, wxALL, 4);
 
-                _content_sizer->Add (s, 1, wxEXPAND | wxALL, 6);
+                _content_sizer->Add (s, 0.75, wxEXPAND | wxALL, 6);
         }
 
        wxBoxSizer* h = new wxBoxSizer (wxHORIZONTAL);
@@ -378,8 +332,17 @@ FilmEditor::make_content_panel ()
        add_label_to_sizer (h, _content_panel, _("times"));
        _content_sizer->Add (h, 0, wxALL, 6);
 
-       _content_information = new wxTextCtrl (_content_panel, wxID_ANY, wxT ("\n\n\n\n"), wxDefaultPosition, wxDefaultSize, wxTE_READONLY | wxTE_MULTILINE);
-       _content_sizer->Add (_content_information, 1, wxEXPAND | wxALL, 6);
+       _content_notebook = new wxNotebook (_content_panel, wxID_ANY, wxDefaultPosition, wxDefaultSize, wxNB_LEFT);
+       _content_sizer->Add (_content_notebook, 1, wxEXPAND | wxTOP, 6);
+
+       make_video_panel ();
+       _content_notebook->AddPage (_video_panel, _("Video"), false);
+       make_audio_panel ();
+       _content_notebook->AddPage (_audio_panel, _("Audio"), false);
+       make_subtitle_panel ();
+       _content_notebook->AddPage (_subtitle_panel, _("Subtitles"), false);
+       make_timing_panel ();
+       _content_notebook->AddPage (_timing_panel, _("Timing"), false);
 
        _loop_count->SetRange (2, 1024);
 }
@@ -387,30 +350,32 @@ FilmEditor::make_content_panel ()
 void
 FilmEditor::make_audio_panel ()
 {
-       _audio_panel = new wxPanel (_notebook);
-       _audio_sizer = new wxBoxSizer (wxVERTICAL);
-       _audio_panel->SetSizer (_audio_sizer);
+       _audio_panel = new wxPanel (_content_notebook);
+       wxBoxSizer* audio_sizer = new wxBoxSizer (wxVERTICAL);
+       _audio_panel->SetSizer (audio_sizer);
        
-       wxFlexGridSizer* grid = new wxFlexGridSizer (2, 4, 4);
-       _audio_sizer->Add (grid, 0, wxALL, 8);
+       wxFlexGridSizer* grid = new wxFlexGridSizer (3, 4, 4);
+       audio_sizer->Add (grid, 0, wxALL, 8);
 
        _show_audio = new wxButton (_audio_panel, wxID_ANY, _("Show Audio..."));
        grid->Add (_show_audio, 1);
        grid->AddSpacer (0);
+       grid->AddSpacer (0);
 
+       add_label_to_sizer (grid, _audio_panel, _("Audio Gain"));
        {
-               add_label_to_sizer (grid, _audio_panel, _("Audio Gain"));
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _audio_gain = new wxSpinCtrl (_audio_panel);
                s->Add (_audio_gain, 1);
                add_label_to_sizer (s, _audio_panel, _("dB"));
-               _audio_gain_calculate_button = new wxButton (_audio_panel, wxID_ANY, _("Calculate..."));
-               s->Add (_audio_gain_calculate_button, 1, wxEXPAND);
-               grid->Add (s);
+               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"));
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _audio_delay = new wxSpinCtrl (_audio_panel);
                s->Add (_audio_delay, 1);
@@ -419,9 +384,21 @@ FilmEditor::make_audio_panel ()
                grid->Add (s);
        }
 
-       _audio_mapping = new AudioMappingView (_audio_panel);
-       _audio_sizer->Add (_audio_mapping, 1, wxEXPAND | wxALL, 6);
+       grid->AddSpacer (0);
+
+       add_label_to_sizer (grid, _audio_panel, _("Audio Stream"));
+       _audio_stream = new wxChoice (_audio_panel, wxID_ANY);
+       grid->Add (_audio_stream, 1);
+       _audio_description = new wxStaticText (_audio_panel, wxID_ANY, wxT (""));
+       grid->AddSpacer (0);
        
+       grid->Add (_audio_description, 1, wxALIGN_CENTER_VERTICAL | wxLEFT, 8);
+       grid->AddSpacer (0);
+       grid->AddSpacer (0);
+       
+       _audio_mapping = new AudioMappingView (_audio_panel);
+       audio_sizer->Add (_audio_mapping, 1, wxEXPAND | wxALL, 6);
+
        _audio_gain->SetRange (-60, 60);
        _audio_delay->SetRange (-1000, 1000);
 }
@@ -429,11 +406,11 @@ FilmEditor::make_audio_panel ()
 void
 FilmEditor::make_subtitle_panel ()
 {
-       _subtitle_panel = new wxPanel (_notebook);
-       _subtitle_sizer = new wxBoxSizer (wxVERTICAL);
-       _subtitle_panel->SetSizer (_subtitle_sizer);
+       _subtitle_panel = new wxPanel (_content_notebook);
+       wxBoxSizer* subtitle_sizer = new wxBoxSizer (wxVERTICAL);
+       _subtitle_panel->SetSizer (subtitle_sizer);
        wxFlexGridSizer* grid = new wxFlexGridSizer (2, 4, 4);
-       _subtitle_sizer->Add (grid, 0, wxALL, 8);
+       subtitle_sizer->Add (grid, 0, wxALL, 8);
 
        _with_subtitles = new wxCheckBox (_subtitle_panel, wxID_ANY, _("With Subtitles"));
        grid->Add (_with_subtitles, 1);
@@ -457,73 +434,79 @@ FilmEditor::make_subtitle_panel ()
                grid->Add (s);
        }
 
+       add_label_to_sizer (grid, _subtitle_panel, _("Subtitle Stream"));
+       _subtitle_stream = new wxChoice (_subtitle_panel, wxID_ANY);
+       grid->Add (_subtitle_stream, 1, wxEXPAND | wxALL, 6);
+       grid->AddSpacer (0);
+       
        _subtitle_offset->SetRange (-1024, 1024);
        _subtitle_scale->SetRange (1, 1000);
 }
 
+void
+FilmEditor::make_timing_panel ()
+{
+       _timing_panel = new wxPanel (_content_notebook);
+       wxBoxSizer* timing_sizer = new wxBoxSizer (wxVERTICAL);
+       _timing_panel->SetSizer (timing_sizer);
+       wxFlexGridSizer* grid = new wxFlexGridSizer (2, 4, 4);
+       timing_sizer->Add (grid, 0, wxALL, 8);
+
+       add_label_to_sizer (grid, _timing_panel, _("Start time"));
+       _start = new Timecode (_timing_panel);
+       grid->Add (_start);
+       add_label_to_sizer (grid, _timing_panel, _("Length"));
+       _length = new Timecode (_timing_panel);
+       grid->Add (_length);
+}
+
+
 /** Called when the left crop widget has been changed */
 void
 FilmEditor::left_crop_changed (wxCommandEvent &)
 {
-       if (!_film) {
+       shared_ptr<VideoContent> c = selected_video_content ();
+       if (!c) {
                return;
        }
 
-       _film->set_left_crop (_left_crop->GetValue ());
+       c->set_left_crop (_left_crop->GetValue ());
 }
 
 /** Called when the right crop widget has been changed */
 void
 FilmEditor::right_crop_changed (wxCommandEvent &)
 {
-       if (!_film) {
+       shared_ptr<VideoContent> c = selected_video_content ();
+       if (!c) {
                return;
        }
 
-       _film->set_right_crop (_right_crop->GetValue ());
+       c->set_right_crop (_right_crop->GetValue ());
 }
 
 /** Called when the top crop widget has been changed */
 void
 FilmEditor::top_crop_changed (wxCommandEvent &)
 {
-       if (!_film) {
+       shared_ptr<VideoContent> c = selected_video_content ();
+       if (!c) {
                return;
        }
 
-       _film->set_top_crop (_top_crop->GetValue ());
+       c->set_top_crop (_top_crop->GetValue ());
 }
 
 /** Called when the bottom crop value has been changed */
 void
 FilmEditor::bottom_crop_changed (wxCommandEvent &)
 {
-       if (!_film) {
-               return;
-       }
-
-       _film->set_bottom_crop (_bottom_crop->GetValue ());
-}
-
-void
-FilmEditor::trust_content_headers_changed (wxCommandEvent &)
-{
-       if (!_film) {
+       shared_ptr<VideoContent> c = selected_video_content ();
+       if (!c) {
                return;
        }
 
-       _film->set_trust_content_headers (_trust_content_headers->GetValue ());
-}
-
-/** Called when the DCP A/B switch has been toggled */
-void
-FilmEditor::ab_toggled (wxCommandEvent &)
-{
-       if (!_film) {
-               return;
-       }
-       
-       _film->set_ab (_ab->GetValue ());
+       c->set_bottom_crop (_bottom_crop->GetValue ());
 }
 
 /** Called when the name widget has been changed */
@@ -584,7 +567,7 @@ FilmEditor::dcp_frame_rate_changed (wxCommandEvent &)
                return;
        }
 
-       _film->set_dcp_frame_rate (
+       _film->set_dcp_video_frame_rate (
                boost::lexical_cast<int> (
                        wx_to_std (_dcp_frame_rate->GetString (_dcp_frame_rate->GetSelection ()))
                        )
@@ -612,42 +595,17 @@ FilmEditor::film_changed (Film::Property p)
                break;
        case Film::CONTENT:
                setup_content ();
-               setup_formats ();
-               setup_format ();
                setup_subtitle_control_sensitivity ();
                setup_show_audio_sensitivity ();
-               setup_length ();
                break;
        case Film::LOOP:
                checked_set (_loop_content, _film->loop() > 1);
                checked_set (_loop_count, _film->loop());
                setup_loop_sensitivity ();
                break;
-       case Film::TRUST_CONTENT_HEADERS:
-               checked_set (_trust_content_headers, _film->trust_content_headers ());
-               break;
-       case Film::FORMAT:
-               setup_format ();
-               break;
-       case Film::CROP:
-               checked_set (_left_crop, _film->crop().left);
-               checked_set (_right_crop, _film->crop().right);
-               checked_set (_top_crop, _film->crop().top);
-               checked_set (_bottom_crop, _film->crop().bottom);
-               setup_scaling_description ();
-               break;
-       case Film::FILTERS:
-       {
-               pair<string, string> p = Filter::ffmpeg_strings (_film->filters ());
-               if (p.first.empty () && p.second.empty ()) {
-                       _filters->SetLabel (_("None"));
-               } else {
-                       string const b = p.first + " " + p.second;
-                       _filters->SetLabel (std_to_wx (b));
-               }
-               _film_sizer->Layout ();
+       case Film::CONTAINER:
+               setup_container ();
                break;
-       }
        case Film::NAME:
                checked_set (_name, _film->name());
                setup_dcp_name ();
@@ -656,27 +614,9 @@ FilmEditor::film_changed (Film::Property p)
                checked_set (_dcp_content_type, DCPContentType::as_index (_film->dcp_content_type ()));
                setup_dcp_name ();
                break;
-       case Film::AB:
-               checked_set (_ab, _film->ab ());
-               break;
        case Film::SCALER:
                checked_set (_scaler, Scaler::as_index (_film->scaler ()));
                break;
-       case Film::TRIM_START:
-               checked_set (_trim_start, _film->trim_start());
-               break;
-       case Film::TRIM_END:
-               checked_set (_trim_end, _film->trim_end());
-               break;
-       case Film::TRIM_TYPE:
-               checked_set (_trim_type, _film->trim_type() == Film::CPL ? 0 : 1);
-               break;
-       case Film::AUDIO_GAIN:
-               checked_set (_audio_gain, _film->audio_gain ());
-               break;
-       case Film::AUDIO_DELAY:
-               checked_set (_audio_delay, _film->audio_delay ());
-               break;
        case Film::WITH_SUBTITLES:
                checked_set (_with_subtitles, _film->with_subtitles ());
                setup_subtitle_control_sensitivity ();
@@ -701,11 +641,11 @@ FilmEditor::film_changed (Film::Property p)
        case Film::DCI_METADATA:
                setup_dcp_name ();
                break;
-       case Film::DCP_FRAME_RATE:
+       case Film::DCP_VIDEO_FRAME_RATE:
        {
                bool done = false;
                for (unsigned int i = 0; i < _dcp_frame_rate->GetCount(); ++i) {
-                       if (wx_to_std (_dcp_frame_rate->GetString(i)) == boost::lexical_cast<string> (_film->dcp_frame_rate())) {
+                       if (wx_to_std (_dcp_frame_rate->GetString(i)) == boost::lexical_cast<string> (_film->dcp_video_frame_rate())) {
                                checked_set (_dcp_frame_rate, i);
                                done = true;
                                break;
@@ -716,22 +656,14 @@ FilmEditor::film_changed (Film::Property p)
                        checked_set (_dcp_frame_rate, -1);
                }
 
-               if (_film->video_frame_rate()) {
-                       _best_dcp_frame_rate->Enable (best_dcp_frame_rate (_film->video_frame_rate ()) != _film->dcp_frame_rate ());
-               } else {
-                       _best_dcp_frame_rate->Disable ();
-               }
-               setup_frame_rate_description ();
+               _best_dcp_frame_rate->Enable (_film->best_dcp_video_frame_rate () != _film->dcp_video_frame_rate ());
                break;
        }
-       case Film::AUDIO_MAPPING:
-               _audio_mapping->set_mapping (_film->audio_mapping ());
-               break;
        }
 }
 
 void
-FilmEditor::film_content_changed (weak_ptr<Content> content, int property)
+FilmEditor::film_content_changed (weak_ptr<Content> weak_content, int property)
 {
        if (!_film) {
                /* We call this method ourselves (as well as using it as a signal handler)
@@ -739,103 +671,143 @@ FilmEditor::film_content_changed (weak_ptr<Content> content, int property)
                */
                return;
        }
-               
-       if (property == FFmpegContentProperty::SUBTITLE_STREAMS) {
+
+       shared_ptr<Content> content = weak_content.lock ();
+       shared_ptr<VideoContent> video_content;
+       shared_ptr<AudioContent> audio_content;
+       shared_ptr<FFmpegContent> ffmpeg_content;
+       if (content) {
+               video_content = dynamic_pointer_cast<VideoContent> (content);
+               audio_content = dynamic_pointer_cast<AudioContent> (content);
+               ffmpeg_content = dynamic_pointer_cast<FFmpegContent> (content);
+       }
+
+       /* We can't use case {} here */
+       
+       if (property == ContentProperty::START) {
+               if (content) {
+                       _start->set (content->start (), _film->dcp_video_frame_rate ());
+               } else {
+                       _start->set (0, 24);
+               }
+       } else if (property == ContentProperty::LENGTH) {
+               if (content) {
+                       _length->set (content->length (), _film->dcp_video_frame_rate ());
+               } else {
+                       _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 (_bottom_crop, video_content ? video_content->crop().bottom : 0);
+               setup_scaling_description ();
+       } else if (property == VideoContentProperty::VIDEO_RATIO) {
+               if (video_content) {
+                       int n = 0;
+                       vector<Ratio const *> ratios = Ratio::all ();
+                       vector<Ratio const *>::iterator i = ratios.begin ();
+                       while (i != ratios.end() && *i != video_content->ratio()) {
+                               ++i;
+                               ++n;
+                       }
+
+                       if (i == ratios.end()) {
+                               checked_set (_ratio, -1);
+                       } else {
+                               checked_set (_ratio, n);
+                       }
+               } else {
+                       checked_set (_ratio, -1);
+               }
+       } else if (property == AudioContentProperty::AUDIO_GAIN) {
+               checked_set (_audio_gain, audio_content ? audio_content->audio_gain() : 0);
+       } else if (property == AudioContentProperty::AUDIO_DELAY) {
+               checked_set (_audio_delay, audio_content ? audio_content->audio_delay() : 0);
+       } else if (property == AudioContentProperty::AUDIO_MAPPING) {
+               _audio_mapping->set (audio_content ? audio_content->audio_mapping () : AudioMapping ());
+       } else if (property == FFmpegContentProperty::SUBTITLE_STREAMS) {
+               _subtitle_stream->Clear ();
+               if (ffmpeg_content) {
+                       vector<shared_ptr<FFmpegSubtitleStream> > s = ffmpeg_content->subtitle_streams ();
+                       if (s.empty ()) {
+                               _subtitle_stream->Enable (false);
+                       }
+                       for (vector<shared_ptr<FFmpegSubtitleStream> >::iterator i = s.begin(); i != s.end(); ++i) {
+                               _subtitle_stream->Append (std_to_wx ((*i)->name), new wxStringClientData (std_to_wx (lexical_cast<string> ((*i)->id))));
+                       }
+                       
+                       if (ffmpeg_content->subtitle_stream()) {
+                               checked_set (_subtitle_stream, lexical_cast<string> (ffmpeg_content->subtitle_stream()->id));
+                       } else {
+                               _subtitle_stream->SetSelection (wxNOT_FOUND);
+                       }
+               }
                setup_subtitle_control_sensitivity ();
        } else if (property == FFmpegContentProperty::AUDIO_STREAMS) {
-               setup_show_audio_sensitivity ();
-       } else if (property == VideoContentProperty::VIDEO_LENGTH || property == AudioContentProperty::AUDIO_LENGTH) {
-               setup_length ();
-               boost::shared_ptr<Content> c = content.lock ();
-               if (c && c == selected_content()) {
-                       setup_content_information ();
+               _audio_stream->Clear ();
+               if (ffmpeg_content) {
+                       vector<shared_ptr<FFmpegAudioStream> > a = ffmpeg_content->audio_streams ();
+                       for (vector<shared_ptr<FFmpegAudioStream> >::iterator i = a.begin(); i != a.end(); ++i) {
+                               _audio_stream->Append (std_to_wx ((*i)->name), new wxStringClientData (std_to_wx (lexical_cast<string> ((*i)->id))));
+                       }
+                       
+                       if (ffmpeg_content->audio_stream()) {
+                               checked_set (_audio_stream, lexical_cast<string> (ffmpeg_content->audio_stream()->id));
+                       }
                }
+               setup_show_audio_sensitivity ();
        } else if (property == FFmpegContentProperty::AUDIO_STREAM) {
                setup_dcp_name ();
                setup_show_audio_sensitivity ();
+       } else if (property == FFmpegContentProperty::FILTERS) {
+               if (ffmpeg_content) {
+                       pair<string, string> p = Filter::ffmpeg_strings (ffmpeg_content->filters ());
+                       if (p.first.empty () && p.second.empty ()) {
+                               _filters->SetLabel (_("None"));
+                       } else {
+                               string const b = p.first + " " + p.second;
+                               _filters->SetLabel (std_to_wx (b));
+                       }
+                       _dcp_sizer->Layout ();
+               }
        }
 }
 
 void
-FilmEditor::setup_format ()
+FilmEditor::setup_container ()
 {
        int n = 0;
-       vector<Format const *>::iterator i = _formats.begin ();
-       while (i != _formats.end() && *i != _film->format ()) {
+       vector<Ratio const *> ratios = Ratio::all ();
+       vector<Ratio const *>::iterator i = ratios.begin ();
+       while (i != ratios.end() && *i != _film->container ()) {
                ++i;
                ++n;
        }
        
-       if (i == _formats.end()) {
-               checked_set (_format, -1);
+       if (i == ratios.end()) {
+               checked_set (_container, -1);
        } else {
-               checked_set (_format, n);
+               checked_set (_container, n);
        }
        
        setup_dcp_name ();
        setup_scaling_description ();
 }      
 
+/** Called when the container widget has been changed */
 void
-FilmEditor::setup_length ()
-{
-       stringstream s;
-       ContentVideoFrame const frames = _film->content_length ();
-       
-       if (frames && _film->video_frame_rate()) {
-               s << frames << " " << wx_to_std (_("frames")) << "; " << seconds_to_hms (frames / _film->video_frame_rate());
-       } else if (frames) {
-               s << frames << " " << wx_to_std (_("frames"));
-       }
-
-       _length->SetLabel (std_to_wx (s.str ()));
-       
-       if (frames) {
-               _trim_start->SetRange (0, frames);
-               _trim_end->SetRange (0, frames);
-       }
-}      
-
-void
-FilmEditor::setup_frame_rate_description ()
-{
-       wxString d;
-       int lines = 0;
-       
-       if (_film->video_frame_rate()) {
-               d << std_to_wx (FrameRateConversion (_film->video_frame_rate(), _film->dcp_frame_rate()).description);
-               ++lines;
-#ifdef HAVE_SWRESAMPLE
-               if (_film->audio_frame_rate() && _film->audio_frame_rate() != _film->target_audio_sample_rate ()) {
-                       d << wxString::Format (
-                               _("Audio will be resampled from %dHz to %dHz\n"),
-                               _film->audio_frame_rate(),
-                               _film->target_audio_sample_rate()
-                               );
-                       ++lines;
-               }
-#endif         
-       }
-
-       for (int i = lines; i < 2; ++i) {
-               d << wxT ("\n ");
-       }
-
-       _frame_rate_description->SetLabel (d);
-}
-
-/** Called when the format widget has been changed */
-void
-FilmEditor::format_changed (wxCommandEvent &)
+FilmEditor::container_changed (wxCommandEvent &)
 {
        if (!_film) {
                return;
        }
 
-       int const n = _format->GetSelection ();
+       int const n = _container->GetSelection ();
        if (n >= 0) {
-               assert (n < int (_formats.size()));
-               _film->set_format (_formats[n]);
+               vector<Ratio const *> ratios = Ratio::all ();
+               assert (n < int (ratios.size()));
+               _film->set_container (ratios[n]);
        }
 }
 
@@ -876,41 +848,23 @@ 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);
        film_changed (Film::LOOP);
-       film_changed (Film::TRUST_CONTENT_HEADERS);
        film_changed (Film::DCP_CONTENT_TYPE);
-       film_changed (Film::FORMAT);
-       film_changed (Film::CROP);
-       film_changed (Film::FILTERS);
+       film_changed (Film::CONTAINER);
        film_changed (Film::SCALER);
-       film_changed (Film::TRIM_START);
-       film_changed (Film::TRIM_END);
-       film_changed (Film::AB);
-       film_changed (Film::TRIM_TYPE);
-       film_changed (Film::AUDIO_GAIN);
-       film_changed (Film::AUDIO_DELAY);
        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_FRAME_RATE);
-       film_changed (Film::AUDIO_MAPPING);
+       film_changed (Film::DCP_VIDEO_FRAME_RATE);
 
-       film_content_changed (boost::shared_ptr<Content> (), FFmpegContentProperty::SUBTITLE_STREAMS);
-       film_content_changed (boost::shared_ptr<Content> (), FFmpegContentProperty::SUBTITLE_STREAM);
-       film_content_changed (boost::shared_ptr<Content> (), FFmpegContentProperty::AUDIO_STREAMS);
-       film_content_changed (boost::shared_ptr<Content> (), FFmpegContentProperty::AUDIO_STREAM);
-
-       setup_content_information ();
+       wxListEvent ev;
+       content_selection_changed (ev);
 }
 
 /** Updates the sensitivity of lots of widgets to a given value.
@@ -924,9 +878,7 @@ FilmEditor::set_things_sensitive (bool s)
        _name->Enable (s);
        _use_dci_name->Enable (s);
        _edit_dci_button->Enable (s);
-       _format->Enable (s);
-       _content->Enable (s);
-       _trust_content_headers->Enable (s);
+       _ratio->Enable (s);
        _content->Enable (s);
        _left_crop->Enable (s);
        _right_crop->Enable (s);
@@ -937,28 +889,37 @@ FilmEditor::set_things_sensitive (bool s)
        _dcp_content_type->Enable (s);
        _best_dcp_frame_rate->Enable (s);
        _dcp_frame_rate->Enable (s);
-       _trim_start->Enable (s);
-       _trim_end->Enable (s);
-       _ab->Enable (s);
-       _trim_type->Enable (s);
        _colour_lut->Enable (s);
        _j2k_bandwidth->Enable (s);
        _audio_gain->Enable (s);
        _audio_gain_calculate_button->Enable (s);
        _show_audio->Enable (s);
        _audio_delay->Enable (s);
+       _container->Enable (s);
+       _loop_content->Enable (s);
+       _loop_count->Enable (s);
 
        setup_subtitle_control_sensitivity ();
        setup_show_audio_sensitivity ();
-       setup_content_button_sensitivity ();
+       setup_content_sensitivity ();
 }
 
 /** Called when the `Edit filters' button has been clicked */
 void
 FilmEditor::edit_filters_clicked (wxCommandEvent &)
 {
-       FilterDialog* d = new FilterDialog (this, _film->filters());
-       d->ActiveChanged.connect (bind (&Film::set_filters, _film, _1));
+       shared_ptr<Content> c = selected_content ();
+       if (!c) {
+               return;
+       }
+
+       shared_ptr<FFmpegContent> fc = dynamic_pointer_cast<FFmpegContent> (c);
+       if (!fc) {
+               return;
+       }
+       
+       FilterDialog* d = new FilterDialog (this, fc->filters());
+       d->ActiveChanged.connect (bind (&FFmpegContent::set_filters, fc, _1));
        d->ShowModal ();
        d->Destroy ();
 }
@@ -980,59 +941,37 @@ FilmEditor::scaler_changed (wxCommandEvent &)
 void
 FilmEditor::audio_gain_changed (wxCommandEvent &)
 {
-       if (!_film) {
+       shared_ptr<AudioContent> ac = selected_audio_content ();
+       if (!ac) {
                return;
        }
 
-       _film->set_audio_gain (_audio_gain->GetValue ());
+       ac->set_audio_gain (_audio_gain->GetValue ());
 }
 
 void
 FilmEditor::audio_delay_changed (wxCommandEvent &)
 {
-       if (!_film) {
+       shared_ptr<AudioContent> ac = selected_audio_content ();
+       if (!ac) {
                return;
        }
 
-       _film->set_audio_delay (_audio_delay->GetValue ());
+       ac->set_audio_delay (_audio_delay->GetValue ());
 }
 
 void
-FilmEditor::setup_notebook_size ()
+FilmEditor::setup_main_notebook_size ()
 {
-       _notebook->InvalidateBestSize ();
-       
-       _film_sizer->Layout ();
-       _film_sizer->SetSizeHints (_film_panel);
-       _video_sizer->Layout ();
-       _video_sizer->SetSizeHints (_video_panel);
-       _audio_sizer->Layout ();
-       _audio_sizer->SetSizeHints (_audio_panel);
-       _subtitle_sizer->Layout ();
-       _subtitle_sizer->SetSizeHints (_subtitle_panel);
-
-       _notebook->Fit ();
-       Fit ();
-}
+       _main_notebook->InvalidateBestSize ();
 
-void
-FilmEditor::trim_start_changed (wxCommandEvent &)
-{
-       if (!_film) {
-               return;
-       }
+       _content_sizer->Layout ();
+       _content_sizer->SetSizeHints (_content_panel);
+       _dcp_sizer->Layout ();
+       _dcp_sizer->SetSizeHints (_dcp_panel);
 
-       _film->set_trim_start (_trim_start->GetValue ());
-}
-
-void
-FilmEditor::trim_end_changed (wxCommandEvent &)
-{
-       if (!_film) {
-               return;
-       }
-
-       _film->set_trim_end (_trim_end->GetValue ());
+       _main_notebook->Fit ();
+       Fit ();
 }
 
 void
@@ -1063,16 +1002,16 @@ FilmEditor::audio_gain_calculate_button_clicked (wxCommandEvent &)
 }
 
 void
-FilmEditor::setup_formats ()
+FilmEditor::setup_ratios ()
 {
-       _formats = Format::all ();
+       _ratios = Ratio::all ();
 
-       _format->Clear ();
-       for (vector<Format const *>::iterator i = _formats.begin(); i != _formats.end(); ++i) {
-               _format->Append (std_to_wx ((*i)->name ()));
+       _ratio->Clear ();
+       for (vector<Ratio const *>::iterator i = _ratios.begin(); i != _ratios.end(); ++i) {
+               _ratio->Append (std_to_wx ((*i)->nickname ()));
        }
 
-       _film_sizer->Layout ();
+       _dcp_sizer->Layout ();
 }
 
 void
@@ -1090,7 +1029,7 @@ FilmEditor::setup_subtitle_control_sensitivity ()
 {
        bool h = false;
        if (_generally_sensitive && _film) {
-               h = !_film->has_subtitles ();
+               h = _film->has_subtitles ();
        }
        
        _with_subtitles->Enable (h);
@@ -1152,10 +1091,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
@@ -1165,13 +1114,13 @@ FilmEditor::best_dcp_frame_rate_clicked (wxCommandEvent &)
                return;
        }
        
-       _film->set_dcp_frame_rate (best_dcp_frame_rate (_film->video_frame_rate ()));
+       _film->set_dcp_video_frame_rate (_film->best_dcp_video_frame_rate ());
 }
 
 void
 FilmEditor::setup_show_audio_sensitivity ()
 {
-       _show_audio->Enable (_film && _film->has_audio ());
+       _show_audio->Enable (_film);
 }
 
 void
@@ -1185,8 +1134,8 @@ FilmEditor::setup_content ()
        
        _content->DeleteAllItems ();
 
-       ContentList content = _film->content ();
-       for (ContentList::iterator i = content.begin(); i != content.end(); ++i) {
+       Playlist::ContentList content = _film->content ();
+       for (Playlist::ContentList::iterator i = content.begin(); i != content.end(); ++i) {
                int const t = _content->GetItemCount ();
                _content->InsertItem (t, std_to_wx ((*i)->summary ()));
                if ((*i)->summary() == selected_summary) {
@@ -1195,7 +1144,7 @@ FilmEditor::setup_content ()
        }
 
        if (selected_summary.empty () && !content.empty ()) {
-               /* Select the first 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);
        }
 }
@@ -1217,13 +1166,17 @@ FilmEditor::content_add_clicked (wxCommandEvent &)
        for (unsigned int i = 0; i < paths.GetCount(); ++i) {
                boost::filesystem::path p (wx_to_std (paths[i]));
 
+               shared_ptr<Content> c;
+
                if (ImageMagickContent::valid_file (p)) {
-                       _film->add_content (shared_ptr<ImageMagickContent> (new ImageMagickContent (p)));
+                       c.reset (new ImageMagickContent (_film, p));
                } else if (SndfileContent::valid_file (p)) {
-                       _film->add_content (shared_ptr<SndfileContent> (new SndfileContent (p)));
+                       c.reset (new SndfileContent (_film, p));
                } else {
-                       _film->add_content (shared_ptr<FFmpegContent> (new FFmpegContent (p)));
+                       c.reset (new FFmpegContent (_film, p));
                }
+
+               _film->examine_and_add_content (c);
        }
 }
 
@@ -1236,96 +1189,42 @@ FilmEditor::content_remove_clicked (wxCommandEvent &)
        }
 }
 
-void
-FilmEditor::content_activated (wxListEvent& ev)
-{
-       ContentList c = _film->content ();
-       assert (ev.GetIndex() >= 0 && size_t (ev.GetIndex()) < c.size ());
-
-       edit_content (c[ev.GetIndex()]);
-}
-
-void
-FilmEditor::content_edit_clicked (wxCommandEvent &)
-{
-       shared_ptr<Content> c = selected_content ();
-       if (!c) {
-               return;
-       }
-
-       edit_content (c);
-}
-
-void
-FilmEditor::edit_content (shared_ptr<Content> c)
-{
-       shared_ptr<ImageMagickContent> im = dynamic_pointer_cast<ImageMagickContent> (c);
-       if (im) {
-               ImageMagickContentDialog* d = new ImageMagickContentDialog (this, im);
-               d->ShowModal ();
-               d->Destroy ();
-       }
-
-       shared_ptr<FFmpegContent> ff = dynamic_pointer_cast<FFmpegContent> (c);
-       if (ff) {
-               FFmpegContentDialog* d = new FFmpegContentDialog (this, ff);
-               d->ShowModal ();
-               d->Destroy ();
-       }
-}
-
-void
-FilmEditor::content_earlier_clicked (wxCommandEvent &)
-{
-       shared_ptr<Content> c = selected_content ();
-       if (c) {
-               _film->move_content_earlier (c);
-       }
-}
-
-void
-FilmEditor::content_later_clicked (wxCommandEvent &)
-{
-       shared_ptr<Content> c = selected_content ();
-       if (c) {
-               _film->move_content_later (c);
-       }
-}
-
 void
 FilmEditor::content_selection_changed (wxListEvent &)
 {
-        setup_content_button_sensitivity ();
-       setup_content_information ();
-}
-
-void
-FilmEditor::setup_content_information ()
-{
-       shared_ptr<Content> c = selected_content ();
-       if (!c) {
-               _content_information->SetValue (wxT (""));
-               return;
+        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));
        }
-
-       _content_information->SetValue (std_to_wx (c->information ()));
+       
+       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::AUDIO_STREAM);
+       film_content_changed (s, FFmpegContentProperty::AUDIO_STREAMS);
+       film_content_changed (s, FFmpegContentProperty::SUBTITLE_STREAM);
 }
 
 void
-FilmEditor::setup_content_button_sensitivity ()
+FilmEditor::setup_content_sensitivity ()
 {
         _content_add->Enable (_generally_sensitive);
 
        shared_ptr<Content> selection = selected_content ();
 
-        _content_edit->Enable (
-               selection && _generally_sensitive &&
-               (dynamic_pointer_cast<ImageMagickContent> (selection) || dynamic_pointer_cast<FFmpegContent> (selection))
-               );
-       
         _content_remove->Enable (selection && _generally_sensitive);
-        _content_earlier->Enable (selection && _generally_sensitive);
-        _content_later->Enable (selection && _generally_sensitive);
+       _content_timeline->Enable (_generally_sensitive);
+
+       _video_panel->Enable    (selection && dynamic_pointer_cast<VideoContent>  (selection) && _generally_sensitive);
+       _audio_panel->Enable    (selection && dynamic_pointer_cast<AudioContent>  (selection) && _generally_sensitive);
+       _subtitle_panel->Enable (selection && dynamic_pointer_cast<FFmpegContent> (selection) && _generally_sensitive);
+       _timing_panel->Enable   (selection && _generally_sensitive);
 }
 
 shared_ptr<Content>
@@ -1336,7 +1235,7 @@ FilmEditor::selected_content ()
                return shared_ptr<Content> ();
        }
 
-       ContentList c = _film->content ();
+       Playlist::ContentList c = _film->content ();
        if (s < 0 || size_t (s) >= c.size ()) {
                return shared_ptr<Content> ();
        }
@@ -1344,11 +1243,35 @@ FilmEditor::selected_content ()
        return c[s];
 }
 
+shared_ptr<VideoContent>
+FilmEditor::selected_video_content ()
+{
+       shared_ptr<Content> c = selected_content ();
+       if (!c) {
+               return shared_ptr<VideoContent> ();
+       }
+
+       return dynamic_pointer_cast<VideoContent> (c);
+}
+
+shared_ptr<AudioContent>
+FilmEditor::selected_audio_content ()
+{
+       shared_ptr<Content> c = selected_content ();
+       if (!c) {
+               return shared_ptr<AudioContent> ();
+       }
+
+       return dynamic_pointer_cast<AudioContent> (c);
+}
+
 void
 FilmEditor::setup_scaling_description ()
 {
        wxString d;
 
+#if 0  
+XXX
        int lines = 0;
 
        if (_film->video_size().width && _film->video_size().height) {
@@ -1397,15 +1320,10 @@ FilmEditor::setup_scaling_description ()
                d << wxT ("\n ");
        }
 
+#endif 
        _scaling_description->SetLabel (d);
 }
 
-void
-FilmEditor::trim_type_changed (wxCommandEvent &)
-{
-       _film->set_trim_type (_trim_type->GetSelection () == 0 ? Film::CPL : Film::ENCODE);
-}
-
 void
 FilmEditor::loop_content_toggled (wxCommandEvent &)
 {
@@ -1429,3 +1347,159 @@ FilmEditor::setup_loop_sensitivity ()
 {
        _loop_count->Enable (_loop_content->GetValue ());
 }
+
+void
+FilmEditor::content_timeline_clicked (wxCommandEvent &)
+{
+       if (_timeline_dialog) {
+               _timeline_dialog->Destroy ();
+               _timeline_dialog = 0;
+       }
+       
+       _timeline_dialog = new TimelineDialog (this, _film);
+       _timeline_dialog->Show ();
+}
+
+void
+FilmEditor::audio_stream_changed (wxCommandEvent &)
+{
+       shared_ptr<Content> c = selected_content ();
+       if (!c) {
+               return;
+       }
+       
+       shared_ptr<FFmpegContent> fc = dynamic_pointer_cast<FFmpegContent> (c);
+       if (!fc) {
+               return;
+       }
+       
+       vector<shared_ptr<FFmpegAudioStream> > a = fc->audio_streams ();
+       vector<shared_ptr<FFmpegAudioStream> >::iterator i = a.begin ();
+       string const s = string_client_data (_audio_stream->GetClientObject (_audio_stream->GetSelection ()));
+       while (i != a.end() && lexical_cast<string> ((*i)->id) != s) {
+               ++i;
+       }
+
+       if (i != a.end ()) {
+               fc->set_audio_stream (*i);
+       }
+
+       if (!fc->audio_stream ()) {
+               _audio_description->SetLabel (wxT (""));
+       } else {
+               wxString s;
+               if (fc->audio_channels() == 1) {
+                       s << _("1 channel");
+               } else {
+                       s << fc->audio_channels() << wxT (" ") << _("channels");
+               }
+               s << wxT (", ") << fc->content_audio_frame_rate() << _("Hz");
+               _audio_description->SetLabel (s);
+       }
+}
+
+
+
+void
+FilmEditor::subtitle_stream_changed (wxCommandEvent &)
+{
+       shared_ptr<Content> c = selected_content ();
+       if (!c) {
+               return;
+       }
+       
+       shared_ptr<FFmpegContent> fc = dynamic_pointer_cast<FFmpegContent> (c);
+       if (!fc) {
+               return;
+       }
+       
+       vector<shared_ptr<FFmpegSubtitleStream> > a = fc->subtitle_streams ();
+       vector<shared_ptr<FFmpegSubtitleStream> >::iterator i = a.begin ();
+       string const s = string_client_data (_subtitle_stream->GetClientObject (_subtitle_stream->GetSelection ()));
+       while (i != a.end() && lexical_cast<string> ((*i)->id) != s) {
+               ++i;
+       }
+
+       if (i != a.end ()) {
+               fc->set_subtitle_stream (*i);
+       }
+}
+
+void
+FilmEditor::audio_mapping_changed (AudioMapping m)
+{
+       shared_ptr<Content> c = selected_content ();
+       if (!c) {
+               return;
+       }
+       
+       shared_ptr<AudioContent> ac = dynamic_pointer_cast<AudioContent> (c);
+       if (!ac) {
+               return;
+       }
+
+       ac->set_audio_mapping (m);
+}
+
+void
+FilmEditor::start_changed ()
+{
+       shared_ptr<Content> c = selected_content ();
+       if (!c) {
+               return;
+       }
+
+       c->set_start (_start->get (_film->dcp_video_frame_rate ()));
+}
+
+void
+FilmEditor::length_changed ()
+{
+       shared_ptr<Content> c = selected_content ();
+       if (!c) {
+               return;
+       }
+
+       shared_ptr<ImageMagickContent> ic = dynamic_pointer_cast<ImageMagickContent> (c);
+       if (ic) {
+               ic->set_video_length (_length->get(_film->dcp_video_frame_rate()) * ic->video_frame_rate() / TIME_HZ);
+       }
+}
+
+void
+FilmEditor::set_selection (weak_ptr<Content> wc)
+{
+       Playlist::ContentList content = _film->content ();
+       for (size_t i = 0; i < content.size(); ++i) {
+               if (content[i] == wc.lock ()) {
+                       _content->SetItemState (i, wxLIST_STATE_SELECTED, wxLIST_STATE_SELECTED);
+               } else {
+                       _content->SetItemState (i, 0, wxLIST_STATE_SELECTED | wxLIST_STATE_FOCUSED);
+               }
+       }
+}
+
+void
+FilmEditor::ratio_changed (wxCommandEvent &)
+{
+       if (!_film) {
+               return;
+       }
+
+       shared_ptr<Content> c = selected_content ();
+       if (!c) {
+               return;
+       }
+
+       shared_ptr<VideoContent> vc = dynamic_pointer_cast<VideoContent> (c);
+       if (!vc) {
+               return;
+       }
+       
+       int const n = _ratio->GetSelection ();
+       if (n >= 0) {
+               vector<Ratio const *> ratios = Ratio::all ();
+               assert (n < int (ratios.size()));
+               vc->set_ratio (ratios[n]);
+       }
+}