X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Fvideo_panel.cc;h=8a9136008b42eab613e1e3f255bf0a197c00fc7d;hb=15a83d720780d58f905d40f8493cdcb86596eaee;hp=2fadfab336ebb8320929472c29f4a3d668fffb8a;hpb=5302f2cda0afd16d714ab898549c7cd278196473;p=dcpomatic.git diff --git a/src/wx/video_panel.cc b/src/wx/video_panel.cc index 2fadfab33..8a9136008 100644 --- a/src/wx/video_panel.cc +++ b/src/wx/video_panel.cc @@ -1,5 +1,5 @@ /* - Copyright (C) 2012-2016 Carl Hetherington + Copyright (C) 2012-2020 Carl Hetherington This file is part of DCP-o-matic. @@ -24,6 +24,10 @@ #include "content_colour_conversion_dialog.h" #include "content_widget.h" #include "content_panel.h" +#include "static_text.h" +#include "check_box.h" +#include "custom_scale_dialog.h" +#include "dcpomatic_button.h" #include "lib/filter.h" #include "lib/ffmpeg_content.h" #include "lib/colour_conversion.h" @@ -35,6 +39,8 @@ #include "lib/video_content.h" #include #include +#include +#include #include #include @@ -48,40 +54,22 @@ using boost::shared_ptr; using boost::dynamic_pointer_cast; using boost::bind; using boost::optional; +using namespace dcpomatic; -static VideoContentScale -index_to_scale (int n) -{ - vector scales = VideoContentScale::all (); - DCPOMATIC_ASSERT (n >= 0); - DCPOMATIC_ASSERT (n < int (scales.size ())); - return scales[n]; -} - -static int -scale_to_index (VideoContentScale scale) -{ - vector scales = VideoContentScale::all (); - for (size_t i = 0; i < scales.size(); ++i) { - if (scales[i] == scale) { - return i; - } - } - - DCPOMATIC_ASSERT (false); -} VideoPanel::VideoPanel (ContentPanel* p) : ContentSubPanel (p, _("Video")) { - _reference = new wxCheckBox (this, wxID_ANY, _("Use this DCP's video as OV and make VF")); - _reference_note = new wxStaticText (this, wxID_ANY, _("")); + _reference = new CheckBox (this, _("Use this DCP's video as OV and make VF")); + _reference_note = new StaticText (this, wxT("")); _reference_note->Wrap (200); wxFont font = _reference_note->GetFont(); font.SetStyle(wxFONTSTYLE_ITALIC); font.SetPointSize(font.GetPointSize() - 1); _reference_note->SetFont(font); + _use = new wxCheckBox (this, wxID_ANY, _("Use")); + _type_label = create_label (this, _("Type"), true); _frame_type = new ContentChoice ( this, @@ -94,7 +82,9 @@ VideoPanel::VideoPanel (ContentPanel* p) &caster ); - _left_crop_label = create_label (this, _("Left crop"), true); + _crop_label = create_label (this, _("Crop"), true); + + _left_crop_label = create_label (this, _("Left"), true); _left_crop = new ContentSpinCtrl ( this, new wxSpinCtrl (this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1)), @@ -104,7 +94,7 @@ VideoPanel::VideoPanel (ContentPanel* p) boost::mem_fn (&VideoContent::set_left_crop) ); - _right_crop_label = create_label (this, _("Right crop"), true); + _right_crop_label = create_label (this, _("Right"), true); _right_crop = new ContentSpinCtrl ( this, new wxSpinCtrl (this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1)), @@ -114,7 +104,7 @@ VideoPanel::VideoPanel (ContentPanel* p) boost::mem_fn (&VideoContent::set_right_crop) ); - _top_crop_label = create_label (this, _("Top crop"), true); + _top_crop_label = create_label (this, _("Top"), true); _top_crop = new ContentSpinCtrl ( this, new wxSpinCtrl (this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1)), @@ -124,7 +114,7 @@ VideoPanel::VideoPanel (ContentPanel* p) boost::mem_fn (&VideoContent::set_top_crop) ); - _bottom_crop_label = create_label (this, _("Bottom crop"), true); + _bottom_crop_label = create_label (this, _("Bottom"), true); _bottom_crop = new ContentSpinCtrl ( this, new wxSpinCtrl (this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (64, -1)), @@ -140,25 +130,18 @@ VideoPanel::VideoPanel (ContentPanel* p) _fade_out_label = create_label (this, _("Fade out"), true); _fade_out = new Timecode (this); - _scale_to_label = create_label (this, _("Scale to"), true); - _scale = new ContentChoice ( - this, - new wxChoice (this, wxID_ANY), - VideoContentProperty::SCALE, - &Content::video, - boost::mem_fn (&VideoContent::scale), - boost::mem_fn (&VideoContent::set_scale), - &index_to_scale, - &scale_to_index - ); - wxClientDC dc (this); wxSize size = dc.GetTextExtent (wxT ("A quite long name")); size.SetHeight (-1); + _scale_label = create_label (this, _("Scale"), true); + _scale_fit = new wxRadioButton (this, wxID_ANY, _("to fit DCP")); + _scale_custom = new wxRadioButton (this, wxID_ANY, _("custom"), wxDefaultPosition, size); + _scale_custom_edit = new Button (this, _("Edit...")); + _filters_label = create_label (this, _("Filters"), true); - _filters = new wxStaticText (this, wxID_ANY, _("None"), wxDefaultPosition, size); - _filters_button = new wxButton (this, wxID_ANY, _("Edit...")); + _filters = new StaticText (this, _("None"), wxDefaultPosition, size); + _filters_button = new Button (this, _("Edit...")); _colour_conversion_label = create_label (this, _("Colour conversion"), true); _colour_conversion = new wxChoice (this, wxID_ANY, wxDefaultPosition, size); @@ -169,20 +152,20 @@ VideoPanel::VideoPanel (ContentPanel* p) /// TRANSLATORS: translate the word "Custom" here; do not include the "Colour|" prefix _colour_conversion->Append (S_("Colour|Custom")); - _edit_colour_conversion_button = new wxButton (this, wxID_ANY, _("Edit...")); + _edit_colour_conversion_button = new Button (this, _("Edit...")); - _description = new wxStaticText (this, wxID_ANY, wxT ("\n \n \n \n \n"), wxDefaultPosition, wxDefaultSize); - _description->SetFont(font); + _range_label = create_label (this, _("Range"), true); + _range = new wxChoice (this, wxID_ANY); + _range->Append (_("Full (JPEG, 0-255)")); + _range->Append (_("Video (MPEG, 16-235)")); - _left_crop->wrapped()->SetRange (0, 1024); - _top_crop->wrapped()->SetRange (0, 1024); - _right_crop->wrapped()->SetRange (0, 1024); - _bottom_crop->wrapped()->SetRange (0, 1024); + _description = new StaticText (this, wxT ("\n \n \n \n \n"), wxDefaultPosition, wxDefaultSize); + _description->SetFont(font); - _scale->wrapped()->Clear (); - BOOST_FOREACH (VideoContentScale const & i, VideoContentScale::all ()) { - _scale->wrapped()->Append (std_to_wx (i.name ())); - } + _left_crop->wrapped()->SetRange (0, 4096); + _top_crop->wrapped()->SetRange (0, 4096); + _right_crop->wrapped()->SetRange (0, 4096); + _bottom_crop->wrapped()->SetRange (0, 4096); _frame_type->wrapped()->Append (_("2D")); _frame_type->wrapped()->Append (_("3D")); @@ -197,9 +180,14 @@ VideoPanel::VideoPanel (ContentPanel* p) _fade_in->Changed.connect (boost::bind (&VideoPanel::fade_in_changed, this)); _fade_out->Changed.connect (boost::bind (&VideoPanel::fade_out_changed, this)); + _use->Bind (wxEVT_CHECKBOX, boost::bind (&VideoPanel::use_clicked, this)); _reference->Bind (wxEVT_CHECKBOX, boost::bind (&VideoPanel::reference_clicked, this)); _filters_button->Bind (wxEVT_BUTTON, boost::bind (&VideoPanel::edit_filters_clicked, this)); + _scale_fit->Bind (wxEVT_RADIOBUTTON, boost::bind (&VideoPanel::scale_fit_clicked, this)); + _scale_custom->Bind (wxEVT_RADIOBUTTON, boost::bind (&VideoPanel::scale_custom_clicked, this)); + _scale_custom_edit->Bind (wxEVT_BUTTON, boost::bind (&VideoPanel::scale_custom_edit_clicked, this)); _colour_conversion->Bind (wxEVT_CHOICE, boost::bind (&VideoPanel::colour_conversion_changed, this)); + _range->Bind (wxEVT_CHOICE, boost::bind (&VideoPanel::range_changed, this)); _edit_colour_conversion_button->Bind (wxEVT_BUTTON, boost::bind (&VideoPanel::edit_colour_conversion_clicked, this)); add_to_grid (); @@ -213,7 +201,7 @@ VideoPanel::add_to_grid () int r = 0; _reference->Show (full); - _reference_note->Show (full); + _reference_note->Show (full && !_reference_note->GetLabel().IsEmpty()); if (full) { wxBoxSizer* reference_sizer = new wxBoxSizer (wxVERTICAL); @@ -223,6 +211,10 @@ VideoPanel::add_to_grid () ++r; } + _use->Show (full); + _grid->Add (_use, wxGBPosition(r, 0), wxGBSpan(1, 2)); + ++r; + add_label_to_sizer (_grid, _type_label, true, wxGBPosition(r, 0)); _frame_type->add (_grid, wxGBPosition(r, 1), wxGBSpan(1, 2)); ++r; @@ -238,33 +230,42 @@ VideoPanel::add_to_grid () _top_crop->add (crop, wxGBPosition (cr, 1)); add_label_to_sizer (crop, _bottom_crop_label, true, wxGBPosition (cr, 2)); _bottom_crop->add (crop, wxGBPosition (cr, 3)); - _grid->Add (crop, wxGBPosition (r, 0), wxGBSpan (2, 4)); - r += 2; - - _fade_in_label->Show (full); - _fade_in->Show (full); - _fade_out_label->Show (full); - _fade_out->Show (full); - _scale_to_label->Show (full); - _scale->show (full); + add_label_to_sizer (_grid, _crop_label, true, wxGBPosition(r, 0)); + _grid->Add (crop, wxGBPosition(r, 1)); + ++r; + + _scale_label->Show (full); + _scale_fit->Show (full); + _scale_custom->Show (full); + _scale_custom_edit->Show (full); _filters_label->Show (full); _filters->Show (full); _filters_button->Show (full); _colour_conversion_label->Show (full); _colour_conversion->Show (full); _edit_colour_conversion_button->Show (full); + _range_label->Show (full); + _range->Show (full); - if (full) { - add_label_to_sizer (_grid, _fade_in_label, true, wxGBPosition (r, 0)); - _grid->Add (_fade_in, wxGBPosition (r, 1), wxGBSpan (1, 3)); - ++r; + add_label_to_sizer (_grid, _fade_in_label, true, wxGBPosition (r, 0)); + _grid->Add (_fade_in, wxGBPosition (r, 1), wxGBSpan (1, 3)); + ++r; - add_label_to_sizer (_grid, _fade_out_label, true, wxGBPosition (r, 0)); - _grid->Add (_fade_out, wxGBPosition (r, 1), wxGBSpan (1, 3)); - ++r; + add_label_to_sizer (_grid, _fade_out_label, true, wxGBPosition (r, 0)); + _grid->Add (_fade_out, wxGBPosition (r, 1), wxGBSpan (1, 3)); + ++r; - add_label_to_sizer (_grid, _scale_to_label, true, wxGBPosition (r, 0)); - _scale->add (_grid, wxGBPosition (r, 1), wxGBSpan (1, 2)); + if (full) { + add_label_to_sizer (_grid, _scale_label, true, wxGBPosition (r, 0)); + { + wxSizer* v = new wxBoxSizer (wxVERTICAL); + v->Add (_scale_fit, 0, wxBOTTOM, 4); + wxSizer* h = new wxBoxSizer (wxHORIZONTAL); + h->Add (_scale_custom, 1, wxRIGHT, 6); + h->Add (_scale_custom_edit, 0); + v->Add (h, 0); + _grid->Add (v, wxGBPosition(r, 1)); + } ++r; add_label_to_sizer (_grid, _filters_label, true, wxGBPosition (r, 0)); @@ -284,12 +285,36 @@ VideoPanel::add_to_grid () _grid->Add (s, wxGBPosition (r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL); } ++r; + + add_label_to_sizer (_grid, _range_label, true, wxGBPosition(r, 0)); + _grid->Add (_range, wxGBPosition(r, 1), wxGBSpan(1, 2), wxALIGN_CENTER_VERTICAL); + ++r; } _grid->Add (_description, wxGBPosition (r, 0), wxGBSpan (1, 4), wxEXPAND | wxALIGN_CENTER_VERTICAL, 6); ++r; } +void +VideoPanel::range_changed () +{ + ContentList vc = _parent->selected_video (); + if (vc.size() != 1) { + return; + } + + switch (_range->GetSelection()) { + case 0: + vc.front()->video->set_range (VIDEO_RANGE_FULL); + break; + case 1: + vc.front()->video->set_range (VIDEO_RANGE_VIDEO); + break; + default: + DCPOMATIC_ASSERT (false); + } +} + void VideoPanel::film_changed (Film::Property property) @@ -310,6 +335,17 @@ VideoPanel::film_changed (Film::Property property) } } +std::size_t +hash_value (boost::optional const & c) +{ + boost::hash hasher; + if (!c) { + return hasher ("none"); + } + return hasher (c->identifier()); +} + + void VideoPanel::film_content_changed (int property) { @@ -327,16 +363,36 @@ VideoPanel::film_content_changed (int property) property == VideoContentProperty::SCALE) { setup_description (); } else if (property == VideoContentProperty::COLOUR_CONVERSION) { - if (vcs && vcs->video->colour_conversion ()) { - optional preset = vcs->video->colour_conversion().get().preset (); - vector cc = PresetColourConversion::all (); - if (preset) { - checked_set (_colour_conversion, preset.get() + 1); + boost::unordered_set > check; + BOOST_FOREACH (shared_ptr i, vc) { + check.insert (i->video->colour_conversion()); + } + + /* Remove any "Many" entry that we might have added previously. There should + * be entries for each preset plus one for "None" and one for "Custom". + */ + vector cc = PresetColourConversion::all (); + if (_colour_conversion->GetCount() > cc.size() + 2) { + _colour_conversion->Delete (_colour_conversion->GetCount() - 1); + } + + if (check.size() == 1) { + if (vcs && vcs->video->colour_conversion ()) { + optional preset = vcs->video->colour_conversion().get().preset (); + if (preset) { + checked_set (_colour_conversion, preset.get() + 1); + } else { + checked_set (_colour_conversion, cc.size() + 1); + } } else { - checked_set (_colour_conversion, cc.size() + 1); + checked_set (_colour_conversion, 0); } - } else { - checked_set (_colour_conversion, 0); + } else if (check.size() > 1) { + /* Add a "many" entry and select it as an indication that multiple different + * colour conversions are present in the selection. + */ + _colour_conversion->Append (_("Many")); + checked_set (_colour_conversion, _colour_conversion->GetCount() - 1); } setup_sensitivity (); @@ -353,6 +409,19 @@ VideoPanel::film_content_changed (int property) checked_set (_filters, p); } } + } else if (property == VideoContentProperty::USE) { + set check; + BOOST_FOREACH (shared_ptr i, vc) { + check.insert (i->video->use()); + } + + if (check.size() == 1) { + checked_set (_use, vc.front()->video->use()); + } else { + checked_set (_use, false); + } + + setup_sensitivity (); } else if (property == VideoContentProperty::FADE_IN) { set check; BOOST_FOREACH (shared_ptr i, vc) { @@ -361,8 +430,8 @@ VideoPanel::film_content_changed (int property) if (check.size() == 1) { _fade_in->set ( - ContentTime::from_frames (vc.front()->video->fade_in (), vc.front()->active_video_frame_rate ()), - vc.front()->active_video_frame_rate () + ContentTime::from_frames (vc.front()->video->fade_in(), vc.front()->active_video_frame_rate(_parent->film())), + vc.front()->active_video_frame_rate(_parent->film()) ); } else { _fade_in->clear (); @@ -375,8 +444,8 @@ VideoPanel::film_content_changed (int property) if (check.size() == 1) { _fade_out->set ( - ContentTime::from_frames (vc.front()->video->fade_out (), vc.front()->active_video_frame_rate ()), - vc.front()->active_video_frame_rate () + ContentTime::from_frames (vc.front()->video->fade_out(), vc.front()->active_video_frame_rate(_parent->film())), + vc.front()->active_video_frame_rate(_parent->film()) ); } else { _fade_out->clear (); @@ -389,6 +458,28 @@ VideoPanel::film_content_changed (int property) checked_set (_reference, false); } + setup_sensitivity (); + } else if (property == VideoContentProperty::RANGE) { + if (vcs) { + checked_set (_range, vcs->video->range() == VIDEO_RANGE_FULL ? 0 : 1); + } else { + checked_set (_range, 0); + } + + setup_sensitivity (); + } else if (property == VideoContentProperty::CUSTOM_RATIO || property == VideoContentProperty::CUSTOM_SIZE) { + set check; + BOOST_FOREACH (shared_ptr i, vc) { + check.insert (i->video->custom_ratio() || i->video->custom_size()); + } + + if (check.size() == 1) { + checked_set (_scale_fit, !vc.front()->video->custom_ratio() && !vc.front()->video->custom_size()); + checked_set (_scale_custom, vc.front()->video->custom_ratio() || vc.front()->video->custom_size()); + } else { + checked_set (_scale_fit, true); + checked_set (_scale_custom, false); + } setup_sensitivity (); } } @@ -420,7 +511,7 @@ VideoPanel::setup_description () return; } - string d = vc.front()->video->processing_description (); + string d = vc.front()->video->processing_description (_parent->film()); size_t lines = count (d.begin(), d.end(), '\n'); for (int i = lines; i < 6; ++i) { @@ -435,19 +526,20 @@ void VideoPanel::colour_conversion_changed () { ContentList vc = _parent->selected_video (); - if (vc.size() != 1) { - return; - } int const s = _colour_conversion->GetSelection (); vector all = PresetColourConversion::all (); - if (s == 0) { - vc.front()->video->unset_colour_conversion (); - } else if (s == int (all.size() + 1)) { + if (s == int(all.size() + 1)) { edit_colour_conversion_clicked (); } else { - vc.front()->video->set_colour_conversion (all[s - 1].conversion); + BOOST_FOREACH (shared_ptr i, _parent->selected_video()) { + if (s == 0) { + i->video->unset_colour_conversion (); + } else if (s != int(all.size() + 2)) { + i->video->set_colour_conversion (all[s - 1].conversion); + } + } } } @@ -455,14 +547,13 @@ void VideoPanel::edit_colour_conversion_clicked () { ContentList vc = _parent->selected_video (); - if (vc.size() != 1) { - return; - } ContentColourConversionDialog* d = new ContentColourConversionDialog (this, vc.front()->video->yuv ()); d->set (vc.front()->video->colour_conversion().get_value_or (PresetColourConversion::all().front().conversion)); if (d->ShowModal() == wxID_OK) { - vc.front()->video->set_colour_conversion (d->get ()); + BOOST_FOREACH (shared_ptr i, vc) { + i->video->set_colour_conversion (d->get ()); + } } else { /* Reset the colour conversion choice */ film_content_changed (VideoContentProperty::COLOUR_CONVERSION); @@ -480,13 +571,16 @@ VideoPanel::content_selection_changed () _right_crop->set_content (video_sel); _top_crop->set_content (video_sel); _bottom_crop->set_content (video_sel); - _scale->set_content (video_sel); film_content_changed (ContentProperty::VIDEO_FRAME_RATE); film_content_changed (VideoContentProperty::CROP); film_content_changed (VideoContentProperty::COLOUR_CONVERSION); film_content_changed (VideoContentProperty::FADE_IN); film_content_changed (VideoContentProperty::FADE_OUT); + film_content_changed (VideoContentProperty::RANGE); + film_content_changed (VideoContentProperty::USE); + film_content_changed (VideoContentProperty::CUSTOM_RATIO); + film_content_changed (VideoContentProperty::CUSTOM_SIZE); film_content_changed (FFmpegContentProperty::FILTERS); film_content_changed (DCPContentProperty::REFERENCE_VIDEO); @@ -504,10 +598,20 @@ VideoPanel::setup_sensitivity () } string why_not; - bool const can_reference = dcp && dcp->can_reference_video (why_not); - setup_refer_button (_reference, _reference_note, dcp, can_reference, why_not); + bool const can_reference = dcp && dcp->can_reference_video (_parent->film(), why_not); + wxString cannot; + if (why_not.empty()) { + cannot = _("Cannot reference this DCP's video."); + } else { + cannot = _("Cannot reference this DCP's video: ") + std_to_wx(why_not); + } + setup_refer_button (_reference, _reference_note, dcp, can_reference, cannot); + + bool const enable = !_reference->GetValue() && _use->GetValue(); + + _use->Enable (!_reference->GetValue()); - if (_reference->GetValue ()) { + if (!enable) { _frame_type->wrapped()->Enable (false); _left_crop->wrapped()->Enable (false); _right_crop->wrapped()->Enable (false); @@ -515,11 +619,14 @@ VideoPanel::setup_sensitivity () _bottom_crop->wrapped()->Enable (false); _fade_in->Enable (false); _fade_out->Enable (false); - _scale->wrapped()->Enable (false); + _scale_fit->Enable (false); + _scale_custom->Enable (false); + _scale_custom_edit->Enable (false); _description->Enable (false); _filters->Enable (false); _filters_button->Enable (false); _colour_conversion->Enable (false); + _range->Enable (false); } else { ContentList video_sel = _parent->selected_video (); FFmpegContentList ffmpeg_sel = _parent->selected_ffmpeg (); @@ -532,11 +639,14 @@ VideoPanel::setup_sensitivity () _bottom_crop->wrapped()->Enable (true); _fade_in->Enable (!video_sel.empty ()); _fade_out->Enable (!video_sel.empty ()); - _scale->wrapped()->Enable (true); + _scale_fit->Enable (true); + _scale_custom->Enable (true); + _scale_custom_edit->Enable (_scale_custom->GetValue()); _description->Enable (true); _filters->Enable (true); _filters_button->Enable (single && !ffmpeg_sel.empty ()); - _colour_conversion->Enable (single && !video_sel.empty ()); + _colour_conversion->Enable (!video_sel.empty()); + _range->Enable (single && !video_sel.empty()); } ContentList vc = _parent->selected_video (); @@ -556,8 +666,8 @@ void VideoPanel::fade_in_changed () { BOOST_FOREACH (shared_ptr i, _parent->selected_video ()) { - int const vfr = _parent->film()->video_frame_rate (); - i->video->set_fade_in (_fade_in->get (vfr).frames_round (vfr)); + double const vfr = i->active_video_frame_rate (_parent->film()); + i->video->set_fade_in (_fade_in->get(vfr).frames_round(vfr)); } } @@ -565,8 +675,16 @@ void VideoPanel::fade_out_changed () { BOOST_FOREACH (shared_ptr i, _parent->selected_video ()) { - int const vfr = _parent->film()->video_frame_rate (); - i->video->set_fade_out (_fade_out->get (vfr).frames_round (vfr)); + double const vfr = i->active_video_frame_rate (_parent->film()); + i->video->set_fade_out (_fade_out->get(vfr).frames_round(vfr)); + } +} + +void +VideoPanel::use_clicked () +{ + BOOST_FOREACH (shared_ptr i, _parent->selected_video()) { + i->video->set_use (_use->GetValue()); } } @@ -585,3 +703,39 @@ VideoPanel::reference_clicked () d->set_reference_video (_reference->GetValue ()); } + + +void +VideoPanel::scale_fit_clicked () +{ + BOOST_FOREACH (shared_ptr i, _parent->selected_video()) { + i->video->set_custom_ratio (optional()); + } +} + + +void +VideoPanel::scale_custom_clicked () +{ + if (!scale_custom_edit_clicked()) { + _scale_fit->SetValue (true); + } +} + + +bool +VideoPanel::scale_custom_edit_clicked () +{ + shared_ptr vc = _parent->selected_video().front()->video; + CustomScaleDialog* d = new CustomScaleDialog (this, vc->size(), _parent->film()->frame_size(), vc->custom_ratio(), vc->custom_size()); + int const r = d->ShowModal (); + if (r == wxID_OK) { + BOOST_FOREACH (shared_ptr i, _parent->selected_video()) { + i->video->set_custom_ratio (d->custom_ratio()); + i->video->set_custom_size (d->custom_size()); + } + } + d->Destroy (); + return r == wxID_OK; +} +