X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Fvideo_panel.cc;h=b27db2547baa72e5e2d1f7b9720184dd1997685d;hb=3ffd0163026be24e5373e0674c3301ed37546e44;hp=b9c09cc69672007c6d76c7117a01ad817ee8d466;hpb=1d2a51bdc8315fa7283be329669860e435a1513f;p=dcpomatic.git

diff --git a/src/wx/video_panel.cc b/src/wx/video_panel.cc
index b9c09cc69..b27db2547 100644
--- a/src/wx/video_panel.cc
+++ b/src/wx/video_panel.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012-2020 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2021 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -18,32 +18,38 @@
 
 */
 
-#include "filter_dialog.h"
-#include "video_panel.h"
-#include "wx_util.h"
+
+#include "check_box.h"
 #include "content_colour_conversion_dialog.h"
-#include "content_widget.h"
 #include "content_panel.h"
-#include "static_text.h"
-#include "check_box.h"
+#include "content_widget.h"
 #include "custom_scale_dialog.h"
 #include "dcpomatic_button.h"
-#include "lib/filter.h"
-#include "lib/ffmpeg_content.h"
+#include "filter_dialog.h"
+#include "static_text.h"
+#include "video_panel.h"
+#include "wx_util.h"
 #include "lib/colour_conversion.h"
 #include "lib/config.h"
-#include "lib/util.h"
-#include "lib/ratio.h"
-#include "lib/frame_rate_change.h"
 #include "lib/dcp_content.h"
+#include "lib/ffmpeg_content.h"
+#include "lib/film.h"
+#include "lib/filter.h"
+#include "lib/frame_rate_change.h"
+#include "lib/ratio.h"
+#include "lib/util.h"
 #include "lib/video_content.h"
+#include <dcp/warnings.h>
+LIBDCP_DISABLE_WARNINGS
 #include <wx/spinctrl.h>
 #include <wx/tglbtn.h>
-#include <boost/unordered_set.hpp>
+LIBDCP_ENABLE_WARNINGS
 #include <boost/functional/hash.hpp>
+#include <boost/unordered_set.hpp>
 #include <set>
 #include <iostream>
 
+
 using std::vector;
 using std::string;
 using std::pair;
@@ -62,11 +68,18 @@ using namespace boost::placeholders;
 
 VideoPanel::VideoPanel (ContentPanel* p)
 	: ContentSubPanel (p, _("Video"))
+{
+
+}
+
+
+void
+VideoPanel::create ()
 {
 	_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();
+	auto font = _reference_note->GetFont();
 	font.SetStyle(wxFONTSTYLE_ITALIC);
 	font.SetPointSize(font.GetPointSize() - 1);
 	_reference_note->SetFont(font);
@@ -89,9 +102,13 @@ VideoPanel::VideoPanel (ContentPanel* p)
 	int const crop_width = 128;
 	int const link_width = 32;
 	int const link_height = 64;
+#elif defined(__WXGTK20__)
+	int const crop_width = 56;
+	int const link_width = 24;
+	int const link_height = 32;
 #elif defined(DCPOMATIC_OSX)
 	int const crop_width = 56;
-	int const link_width = 19;
+	int const link_width = 23;
 	int const link_height = 28;
 #else
 	int const crop_width = 56;
@@ -105,13 +122,13 @@ VideoPanel::VideoPanel (ContentPanel* p)
 		new wxSpinCtrl (this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize(crop_width, -1)),
 		VideoContentProperty::CROP,
 		&Content::video,
-		boost::mem_fn (&VideoContent::left_crop),
+		boost::mem_fn (&VideoContent::requested_left_crop),
 		boost::mem_fn (&VideoContent::set_left_crop),
 		boost::bind (&VideoPanel::left_crop_changed, this)
 		);
 
 	_left_right_link = new wxToggleButton (this, wxID_ANY, wxT(""), wxDefaultPosition, wxSize(link_width, link_height));
-	_left_right_link->SetBitmap (wxBitmap(bitmap_path("link"), wxBITMAP_TYPE_PNG));
+	_left_right_link->SetBitmap (wxBitmap(bitmap_path("link.png"), wxBITMAP_TYPE_PNG));
 
 	_right_crop_label = create_label (this, _("Right"), true);
 	_right_crop = new ContentSpinCtrl<VideoContent> (
@@ -119,7 +136,7 @@ VideoPanel::VideoPanel (ContentPanel* p)
 		new wxSpinCtrl (this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize(crop_width, -1)),
 		VideoContentProperty::CROP,
 		&Content::video,
-		boost::mem_fn (&VideoContent::right_crop),
+		boost::mem_fn (&VideoContent::requested_right_crop),
 		boost::mem_fn (&VideoContent::set_right_crop),
 		boost::bind (&VideoPanel::right_crop_changed, this)
 		);
@@ -130,13 +147,13 @@ VideoPanel::VideoPanel (ContentPanel* p)
 		new wxSpinCtrl (this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize(crop_width, -1)),
 		VideoContentProperty::CROP,
 		&Content::video,
-		boost::mem_fn (&VideoContent::top_crop),
+		boost::mem_fn (&VideoContent::requested_top_crop),
 		boost::mem_fn (&VideoContent::set_top_crop),
 		boost::bind (&VideoPanel::top_crop_changed, this)
 		);
 
 	_top_bottom_link = new wxToggleButton (this, wxID_ANY, wxT(""), wxDefaultPosition, wxSize(link_width, link_height));
-	_top_bottom_link->SetBitmap (wxBitmap(bitmap_path("link"), wxBITMAP_TYPE_PNG));
+	_top_bottom_link->SetBitmap (wxBitmap(bitmap_path("link.png"), wxBITMAP_TYPE_PNG));
 
 	_bottom_crop_label = create_label (this, _("Bottom"), true);
 	_bottom_crop = new ContentSpinCtrl<VideoContent> (
@@ -144,7 +161,7 @@ VideoPanel::VideoPanel (ContentPanel* p)
 		new wxSpinCtrl (this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize(crop_width, -1)),
 		VideoContentProperty::CROP,
 		&Content::video,
-		boost::mem_fn (&VideoContent::bottom_crop),
+		boost::mem_fn (&VideoContent::requested_bottom_crop),
 		boost::mem_fn (&VideoContent::set_bottom_crop),
 		boost::bind (&VideoPanel::bottom_crop_changed, this)
 		);
@@ -156,7 +173,7 @@ VideoPanel::VideoPanel (ContentPanel* p)
 	_fade_out = new Timecode<ContentTime> (this);
 
 	wxClientDC dc (this);
-	wxSize size = dc.GetTextExtent (wxT ("A quite long name"));
+	auto size = dc.GetTextExtent (wxT ("A quite long name"));
 #ifdef __WXGTK3__
 	size.SetWidth (size.GetWidth() + 64);
 #endif
@@ -167,7 +184,7 @@ VideoPanel::VideoPanel (ContentPanel* p)
 	_scale_custom = new wxRadioButton (this, wxID_ANY, _("custom"));
 	_scale_custom_edit = new Button (this, _("Edit..."), wxDefaultPosition, small_button_size(this, _("Edit...")));
 
-	_colour_conversion_label = create_label (this, _("Colour conversion"), true);
+	_colour_conversion_label = create_label (this, _("Colour"), true);
 	_colour_conversion = new wxChoice (this, wxID_ANY, wxDefaultPosition, size);
 	_colour_conversion->Append (_("None"));
 	for (auto const& i: PresetColourConversion::all()) {
@@ -204,7 +221,7 @@ 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));
 
-	_reference->Bind                     (wxEVT_CHECKBOX, boost::bind (&VideoPanel::reference_clicked, this));
+	_reference->bind(&VideoPanel::reference_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));
@@ -215,14 +232,17 @@ VideoPanel::VideoPanel (ContentPanel* p)
 	_top_bottom_link->Bind               (wxEVT_TOGGLEBUTTON, boost::bind(&VideoPanel::top_bottom_link_clicked, this));
 
 	add_to_grid ();
+
+	_sizer->Layout ();
 }
 
+
 void
 VideoPanel::add_to_grid ()
 {
 	int r = 0;
 
-	wxBoxSizer* reference_sizer = new wxBoxSizer (wxVERTICAL);
+	auto reference_sizer = new wxBoxSizer (wxVERTICAL);
 	reference_sizer->Add (_reference, 0);
 	reference_sizer->Add (_reference_note, 0);
 	_grid->Add (reference_sizer, wxGBPosition(r, 0), wxGBSpan(1, 3));
@@ -233,7 +253,7 @@ VideoPanel::add_to_grid ()
 	++r;
 
 	int cr = 0;
-	wxGridBagSizer* crop = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
+	auto crop = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
 
 	add_label_to_sizer (crop, _left_crop_label, true, wxGBPosition (cr, 0));
 	_left_crop->add (crop, wxGBPosition(cr, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
@@ -274,9 +294,9 @@ VideoPanel::add_to_grid ()
 
 	add_label_to_sizer (_grid, _scale_label, true, wxGBPosition (r, 0));
 	{
-		wxSizer* v = new wxBoxSizer (wxVERTICAL);
+		auto v = new wxBoxSizer (wxVERTICAL);
 		v->Add (_scale_fit, 0, wxBOTTOM, 4);
-		wxSizer* h = new wxBoxSizer (wxHORIZONTAL);
+		auto h = new wxBoxSizer (wxHORIZONTAL);
 		h->Add (_scale_custom, 1, wxRIGHT | wxALIGN_CENTER_VERTICAL, 6);
 		h->Add (_scale_custom_edit, 0, wxALIGN_CENTER_VERTICAL);
 		v->Add (h, 0);
@@ -286,7 +306,7 @@ VideoPanel::add_to_grid ()
 
 	add_label_to_sizer (_grid, _colour_conversion_label, true, wxGBPosition(r, 0));
 	{
-		wxSizer* s = new wxBoxSizer (wxHORIZONTAL);
+		auto s = new wxBoxSizer (wxHORIZONTAL);
 		s->Add (_colour_conversion, 1, wxALIGN_CENTER_VERTICAL | wxTOP | wxBOTTOM | wxRIGHT, 6);
 		s->Add (_edit_colour_conversion_button, 0, wxALIGN_CENTER_VERTICAL);
 		_grid->Add (s, wxGBPosition (r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
@@ -301,10 +321,11 @@ VideoPanel::add_to_grid ()
 	++r;
 }
 
+
 void
 VideoPanel::range_changed ()
 {
-	ContentList vc = _parent->selected_video ();
+	auto vc = _parent->selected_video ();
 	if (vc.size() != 1) {
 		return;
 	}
@@ -323,17 +344,17 @@ VideoPanel::range_changed ()
 
 
 void
-VideoPanel::film_changed (Film::Property property)
+VideoPanel::film_changed(FilmProperty property)
 {
 	switch (property) {
-	case Film::VIDEO_FRAME_RATE:
-	case Film::CONTAINER:
-	case Film::RESOLUTION:
+	case FilmProperty::VIDEO_FRAME_RATE:
+	case FilmProperty::CONTAINER:
+	case FilmProperty::RESOLUTION:
 		setup_description ();
 		setup_sensitivity ();
 		break;
-	case Film::REEL_TYPE:
-	case Film::INTEROP:
+	case FilmProperty::REEL_TYPE:
+	case FilmProperty::INTEROP:
 		setup_sensitivity ();
 		break;
 	default:
@@ -341,6 +362,7 @@ VideoPanel::film_changed (Film::Property property)
 	}
 }
 
+
 std::size_t
 hash_value (boost::optional<ColourConversion> const & c)
 {
@@ -355,10 +377,10 @@ hash_value (boost::optional<ColourConversion> const & c)
 void
 VideoPanel::film_content_changed (int property)
 {
-	ContentList vc = _parent->selected_video ();
+	auto vc = _parent->selected_video ();
 	shared_ptr<Content> vcs;
 	shared_ptr<FFmpegContent> fcs;
-	if (!vc.empty ()) {
+	if (!vc.empty()) {
 		vcs = vc.front ();
 		fcs = dynamic_pointer_cast<FFmpegContent> (vcs);
 	}
@@ -366,10 +388,11 @@ VideoPanel::film_content_changed (int property)
 	if (property == ContentProperty::VIDEO_FRAME_RATE ||
 	    property == VideoContentProperty::FRAME_TYPE ||
 	    property == VideoContentProperty::CROP ||
-	    property == VideoContentProperty::SCALE) {
+	    property == VideoContentProperty::CUSTOM_RATIO ||
+	    property == VideoContentProperty::CUSTOM_SIZE) {
 		setup_description ();
 	} else if (property == VideoContentProperty::COLOUR_CONVERSION) {
-		boost::unordered_set<optional<ColourConversion> > check;
+		boost::unordered_set<optional<ColourConversion>> check;
 		for (auto i: vc) {
 			check.insert (i->video->colour_conversion());
 		}
@@ -377,14 +400,14 @@ VideoPanel::film_content_changed (int property)
 		/* 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<PresetColourConversion> cc = PresetColourConversion::all ();
+		auto 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<size_t> preset = vcs->video->colour_conversion().get().preset ();
+				auto preset = vcs->video->colour_conversion().get().preset();
 				if (preset) {
 					checked_set (_colour_conversion, preset.get() + 1);
 				} else {
@@ -471,7 +494,7 @@ VideoPanel::film_content_changed (int property)
 void
 VideoPanel::setup_description ()
 {
-	ContentList vc = _parent->selected_video ();
+	auto vc = _parent->selected_video ();
 	if (vc.empty ()) {
 		checked_set (_description, wxT (""));
 		return;
@@ -480,7 +503,7 @@ VideoPanel::setup_description ()
 		return;
 	}
 
-	string d = vc.front()->video->processing_description (_parent->film());
+	auto d = vc.front()->video->processing_description(_parent->film());
 	size_t lines = count (d.begin(), d.end(), '\n');
 
 	for (int i = lines; i < 6; ++i) {
@@ -488,16 +511,17 @@ VideoPanel::setup_description ()
 	}
 
 	checked_set (_description, d);
-	_sizer->Layout ();
+	layout ();
 }
 
+
 void
 VideoPanel::colour_conversion_changed ()
 {
-	ContentList vc = _parent->selected_video ();
+	auto vc = _parent->selected_video ();
 
 	int const s = _colour_conversion->GetSelection ();
-	vector<PresetColourConversion> all = PresetColourConversion::all ();
+	auto all = PresetColourConversion::all ();
 
 	if (s == int(all.size() + 1)) {
 		edit_colour_conversion_clicked ();
@@ -512,28 +536,29 @@ VideoPanel::colour_conversion_changed ()
 	}
 }
 
+
 void
 VideoPanel::edit_colour_conversion_clicked ()
 {
-	ContentList vc = _parent->selected_video ();
+	auto vc = _parent->selected_video ();
 
-	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) {
+	ContentColourConversionDialog dialog(this, vc.front()->video->yuv());
+	dialog.set(vc.front()->video->colour_conversion().get_value_or(PresetColourConversion::all().front().conversion));
+	if (dialog.ShowModal() == wxID_OK) {
 		for (auto i: vc) {
-			i->video->set_colour_conversion (d->get ());
+			i->video->set_colour_conversion(dialog.get());
 		}
 	} else {
 		/* Reset the colour conversion choice */
 		film_content_changed (VideoContentProperty::COLOUR_CONVERSION);
 	}
-	d->Destroy ();
 }
 
+
 void
 VideoPanel::content_selection_changed ()
 {
-	ContentList video_sel = _parent->selected_video ();
+	auto video_sel = _parent->selected_video ();
 
 	_frame_type->set_content (video_sel);
 	_left_crop->set_content (video_sel);
@@ -556,10 +581,11 @@ VideoPanel::content_selection_changed ()
 	setup_sensitivity ();
 }
 
+
 void
 VideoPanel::setup_sensitivity ()
 {
-	ContentList sel = _parent->selected ();
+	auto sel = _parent->selected ();
 
 	shared_ptr<DCPContent> dcp;
 	if (sel.size() == 1) {
@@ -600,8 +626,8 @@ VideoPanel::setup_sensitivity ()
 		_colour_conversion->Enable (false);
 		_range->Enable (false);
 	} else {
-		ContentList video_sel = _parent->selected_video ();
-		FFmpegContentList ffmpeg_sel = _parent->selected_ffmpeg ();
+		auto video_sel = _parent->selected_video ();
+		auto ffmpeg_sel = _parent->selected_ffmpeg ();
 		bool const single = video_sel.size() == 1;
 
 		_frame_type->wrapped()->Enable (true);
@@ -619,7 +645,7 @@ VideoPanel::setup_sensitivity ()
 		_range->Enable (single && !video_sel.empty() && !dcp);
 	}
 
-	ContentList vc = _parent->selected_video ();
+	auto vc = _parent->selected_video ();
 	shared_ptr<Content> vcs;
 	if (!vc.empty ()) {
 		vcs = vc.front ();
@@ -632,21 +658,25 @@ VideoPanel::setup_sensitivity ()
 	}
 }
 
+
 void
 VideoPanel::fade_in_changed ()
 {
-	for (auto i: _parent->selected_video ()) {
-		double const vfr = i->active_video_frame_rate (_parent->film());
-		i->video->set_fade_in (_fade_in->get(vfr).frames_round(vfr));
+	auto const hmsf = _fade_in->get();
+	for (auto i: _parent->selected_video()) {
+		auto const vfr = i->active_video_frame_rate(_parent->film());
+		i->video->set_fade_in (dcpomatic::ContentTime(hmsf, vfr).frames_round(vfr));
 	}
 }
 
+
 void
 VideoPanel::fade_out_changed ()
 {
-	for (auto i: _parent->selected_video ()) {
-		double const vfr = i->active_video_frame_rate (_parent->film());
-		i->video->set_fade_out (_fade_out->get(vfr).frames_round(vfr));
+	auto const hmsf = _fade_out->get();
+	for (auto i: _parent->selected_video()) {
+		auto const vfr = i->active_video_frame_rate (_parent->film());
+		i->video->set_fade_out (dcpomatic::ContentTime(hmsf, vfr).frames_round(vfr));
 	}
 }
 
@@ -654,12 +684,12 @@ VideoPanel::fade_out_changed ()
 void
 VideoPanel::reference_clicked ()
 {
-	ContentList c = _parent->selected ();
+	auto c = _parent->selected ();
 	if (c.size() != 1) {
 		return;
 	}
 
-	shared_ptr<DCPContent> d = dynamic_pointer_cast<DCPContent> (c.front ());
+	auto d = dynamic_pointer_cast<DCPContent> (c.front ());
 	if (!d) {
 		return;
 	}
@@ -673,7 +703,10 @@ VideoPanel::scale_fit_clicked ()
 {
 	for (auto i: _parent->selected_video()) {
 		i->video->set_custom_ratio (optional<float>());
+		i->video->set_custom_size (optional<dcp::Size>());
 	}
+
+	setup_sensitivity ();
 }
 
 
@@ -683,46 +716,61 @@ VideoPanel::scale_custom_clicked ()
 	if (!scale_custom_edit_clicked()) {
 		_scale_fit->SetValue (true);
 	}
+
+	setup_sensitivity ();
 }
 
 
 bool
 VideoPanel::scale_custom_edit_clicked ()
 {
-	shared_ptr<const VideoContent> 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) {
-		for (auto i: _parent->selected_video()) {
-			i->video->set_custom_ratio (d->custom_ratio());
-			i->video->set_custom_size (d->custom_size());
-		}
+	auto vc = _parent->selected_video().front()->video;
+	auto size = vc->size();
+	DCPOMATIC_ASSERT(size);
+
+	CustomScaleDialog dialog(this, *size, _parent->film()->frame_size(), vc->custom_ratio(), vc->custom_size());
+	if (dialog.ShowModal() != wxID_OK) {
+		return false;
+	}
+
+	for (auto i: _parent->selected_video()) {
+		i->video->set_custom_ratio(dialog.custom_ratio());
+		i->video->set_custom_size(dialog.custom_size());
 	}
-	d->Destroy ();
-	return r == wxID_OK;
+
+	return true;
 }
 
 
 void
 VideoPanel::left_right_link_clicked ()
 {
-	right_crop_changed ();
+	if (_left_changed_last) {
+		left_crop_changed ();
+	} else {
+		right_crop_changed ();
+	}
 }
 
 
 void
 VideoPanel::top_bottom_link_clicked ()
 {
-	bottom_crop_changed ();
+	if (_top_changed_last) {
+		top_crop_changed ();
+	} else {
+		bottom_crop_changed ();
+	}
 }
 
 
 void
 VideoPanel::left_crop_changed ()
 {
+	_left_changed_last = true;
 	if (_left_right_link->GetValue()) {
-		for (auto i: _parent->selected_video()) {
-			i->video->set_right_crop (i->video->left_crop());
+		for (auto const& i: _parent->selected_video()) {
+			i->video->set_right_crop (i->video->requested_left_crop());
 		}
 	}
 }
@@ -731,9 +779,10 @@ VideoPanel::left_crop_changed ()
 void
 VideoPanel::right_crop_changed ()
 {
+	_left_changed_last = false;
 	if (_left_right_link->GetValue()) {
-		for (auto i: _parent->selected_video()) {
-			i->video->set_left_crop (i->video->right_crop());
+		for (auto const& i: _parent->selected_video()) {
+			i->video->set_left_crop (i->video->requested_right_crop());
 		}
 	}
 }
@@ -742,9 +791,10 @@ VideoPanel::right_crop_changed ()
 void
 VideoPanel::top_crop_changed ()
 {
+	_top_changed_last = true;
 	if (_top_bottom_link->GetValue()) {
 		for (auto i: _parent->selected_video()) {
-			i->video->set_bottom_crop (i->video->top_crop());
+			i->video->set_bottom_crop (i->video->requested_top_crop());
 		}
 	}
 }
@@ -753,12 +803,12 @@ VideoPanel::top_crop_changed ()
 void
 VideoPanel::bottom_crop_changed ()
 {
+	_top_changed_last = false;
 	if (_top_bottom_link->GetValue()) {
 		for (auto i: _parent->selected_video()) {
-			i->video->set_top_crop (i->video->bottom_crop());
+			i->video->set_top_crop (i->video->requested_bottom_crop());
 		}
 	}
 }
 
 
-