X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Fexport_subtitles_dialog.cc;h=3fc5dcdb27ef22b2f66f4a2445d962bd664ee4c4;hb=6693132ed6fb863fdff496f42028d25f55eb8bc4;hp=bcc8adbe14e1fd9ca9c3bceb9a47a6b90ca05c88;hpb=9f294327cb59fce307c3a439f044d79b081376b0;p=dcpomatic.git

diff --git a/src/wx/export_subtitles_dialog.cc b/src/wx/export_subtitles_dialog.cc
index bcc8adbe1..3fc5dcdb2 100644
--- a/src/wx/export_subtitles_dialog.cc
+++ b/src/wx/export_subtitles_dialog.cc
@@ -27,52 +27,82 @@
 DCPOMATIC_DISABLE_WARNINGS
 #include <wx/filepicker.h>
 DCPOMATIC_ENABLE_WARNINGS
-#include <boost/bind.hpp>
+#include <boost/bind/bind.hpp>
 
 
 using std::string;
 using boost::bind;
 
 
-ExportSubtitlesDialog::ExportSubtitlesDialog (wxWindow* parent, string name, bool interop)
+ExportSubtitlesDialog::ExportSubtitlesDialog (wxWindow* parent, int reels, bool interop)
 	: TableDialog (parent, _("Export subtitles"), 2, 1, true)
-	, _initial_name (name)
+	, _interop (interop)
 	, _include_font (0)
 {
 	_split_reels = new CheckBox (this, _("Write reels into separate files"));
 	add (_split_reels, false);
 	add_spacer ();
-	if (interop) {
-		_include_font = new CheckBox (this, _("Define font in output and export font file"));
-		add (_include_font, false);
-		add_spacer ();
+
+	if (reels > 1) {
+		_split_reels->Enable (false);
+	}
+
+	_include_font = new CheckBox (this, _("Define font in output and export font file"));
+	add (_include_font, false);
+	add_spacer ();
+
+	if (!_interop) {
+		_include_font->Enable (false);
 	}
 
-	add (_("Output file"), true);
-	/* Don't warn overwrite here, because on Linux (at least) if we specify a filename like foo
-	   the wxFileDialog will check that foo exists, but we will add an extension so we actually
-	   need to check if foo.mov (or similar) exists.  I can't find a way to make wxWidgets do this,
-	   so disable its check and the caller will have to do it themselves.
-	*/
-	_file = new FilePickerCtrl (this, _("Select output file"), _("Subtitle files (.xml)|*.xml"), false, false);
-	_file->SetPath (_initial_name);
+	wxString const wildcard = _interop ? _("Subtitle files (.xml)|*.xml") : _("Subtitle files (.mxf)|*.mxf");
+
+	_file_label = add (_("Output file"), true);
+	_file = new FilePickerCtrl (this, _("Select output file"), wildcard, false, true);
 	add (_file);
 
-	_file->Bind (wxEVT_FILEPICKER_CHANGED, bind(&ExportSubtitlesDialog::file_changed, this));
+	_dir_label = add (_("Output folder"), true);
+	_dir = new DirPickerCtrl (this);
+	add (_dir);
+
+	_split_reels->Bind (wxEVT_CHECKBOX, bind(&ExportSubtitlesDialog::setup_sensitivity, this));
+	_include_font->Bind (wxEVT_CHECKBOX, bind(&ExportSubtitlesDialog::setup_sensitivity, this));
+	_file->Bind (wxEVT_FILEPICKER_CHANGED, bind(&ExportSubtitlesDialog::setup_sensitivity, this));
+	_dir->Bind (wxEVT_DIRPICKER_CHANGED, bind(&ExportSubtitlesDialog::setup_sensitivity, this));
 
 	layout ();
+	setup_sensitivity ();
+}
+
+
+void
+ExportSubtitlesDialog::setup_sensitivity ()
+{
+	bool const multi = split_reels() || (_interop && _include_font->GetValue());
+	_file_label->Enable (!multi);
+	_file->Enable (!multi);
+	_dir_label->Enable (multi);
+	_dir->Enable (multi);
 
-	wxButton* ok = dynamic_cast<wxButton *>(FindWindowById(wxID_OK, this));
-	ok->Enable (false);
+	wxButton* ok = dynamic_cast<wxButton *> (FindWindowById(wxID_OK, this));
+	DCPOMATIC_ASSERT (ok);
+	ok->Enable (path().is_absolute());
 }
 
 
+/** @return Either a full path to a file, if the output will be one file, or
+ *  a full path to a directory.
+ */
 boost::filesystem::path
 ExportSubtitlesDialog::path () const
 {
-	wxFileName fn (_file->GetPath());
-	fn.SetExt (".xml");
-	return wx_to_std (fn.GetFullPath());
+	if (_file->IsEnabled()) {
+		wxFileName fn (_file->GetPath());
+		fn.SetExt (_interop ? "xml" : "mxf");
+		return wx_to_std (fn.GetFullPath());
+	}
+
+	return wx_to_std (_dir->GetPath());
 }
 
 
@@ -86,14 +116,6 @@ ExportSubtitlesDialog::split_reels () const
 bool
 ExportSubtitlesDialog::include_font () const
 {
-	return _include_font ? _include_font->GetValue () : true;
+	return !_interop || _include_font->GetValue();
 }
 
-
-void
-ExportSubtitlesDialog::file_changed ()
-{
-	wxButton* ok = dynamic_cast<wxButton *> (FindWindowById(wxID_OK, this));
-	DCPOMATIC_ASSERT (ok);
-	ok->Enable (path().is_absolute());
-}