X-Git-Url: https://git.carlh.net/gitweb/?p=dcpomatic.git;a=blobdiff_plain;f=src%2Fwx%2Fscreen_dialog.cc;h=716f9e9eb3db91dbdb16546a19447e0084e0558f;hp=8da8f061e7c93d47871b49efc02a8a55fdca9401;hb=c103d8c1306e5fb3937b3a6c430a3fff32653fa3;hpb=689fa55d1529ad88449ca464e9107c4dcc54d1cb

diff --git a/src/wx/screen_dialog.cc b/src/wx/screen_dialog.cc
index 8da8f061e..716f9e9eb 100644
--- a/src/wx/screen_dialog.cc
+++ b/src/wx/screen_dialog.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012-2021 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2022 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -21,20 +21,19 @@
 
 #include "dcpomatic_button.h"
 #include "download_certificate_dialog.h"
-#include "file_dialog_wrapper.h"
 #include "screen_dialog.h"
 #include "static_text.h"
 #include "table_dialog.h"
 #include "wx_util.h"
 #include "lib/compose.hpp"
 #include "lib/util.h"
-#include "lib/warnings.h"
+#include <dcp/warnings.h>
 #include <dcp/exceptions.h>
 #include <dcp/certificate_chain.h>
-DCPOMATIC_DISABLE_WARNINGS
+LIBDCP_DISABLE_WARNINGS
 #include <wx/filepicker.h>
 #include <wx/validate.h>
-DCPOMATIC_ENABLE_WARNINGS
+LIBDCP_ENABLE_WARNINGS
 #include <iostream>
 
 
@@ -71,7 +70,7 @@ public:
 				_certificate = dcp::Certificate(dcp::file_to_string(wx_to_std(d->GetPath())));
 				_thumbprint->SetValue (std_to_wx(_certificate->thumbprint()));
 			} catch (dcp::MiscError& e) {
-				error_dialog (this, wxString::Format(_("Could not load certficate (%s)"), std_to_wx(e.what())));
+				error_dialog(this, wxString::Format(_("Could not load certificate (%s)"), std_to_wx(e.what())));
 			}
 		}
 	}
@@ -102,7 +101,13 @@ private:
 
 
 ScreenDialog::ScreenDialog (
-	wxWindow* parent, wxString title, string name, string notes, optional<dcp::Certificate> recipient, vector<TrustedDevice> trusted_devices
+	wxWindow* parent,
+	wxString title,
+	string name,
+	string notes,
+	optional<dcp::Certificate> recipient,
+	optional<string> recipient_file,
+	vector<TrustedDevice> trusted_devices
 	)
 	: wxDialog (parent, wxID_ANY, title)
 	, _recipient (recipient)
@@ -132,19 +137,35 @@ ScreenDialog::ScreenDialog (
         size.SetHeight (-1);
 
 	add_label_to_sizer (_sizer, this, _("Recipient certificate"), true, wxGBPosition(r, 0));
-	wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
+	auto s = new wxBoxSizer (wxHORIZONTAL);
 	_recipient_thumbprint = new StaticText (this, wxT (""), wxDefaultPosition, size);
 	_recipient_thumbprint->SetFont (font);
 	set_recipient (recipient);
+
 	_get_recipient_from_file = new Button (this, _("Get from file..."));
 	_download_recipient = new Button (this, _("Download..."));
-	s->Add (_recipient_thumbprint, 1, wxLEFT | wxRIGHT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_X_GAP);
+	s->Add (_recipient_thumbprint, 1, wxRIGHT | wxALIGN_CENTER_VERTICAL | wxALIGN_LEFT, DCPOMATIC_SIZER_X_GAP);
 	s->Add (_get_recipient_from_file, 0, wxLEFT | wxRIGHT | wxEXPAND, DCPOMATIC_SIZER_X_GAP);
 	s->Add (_download_recipient, 0, wxLEFT | wxRIGHT | wxEXPAND, DCPOMATIC_SIZER_X_GAP);
 	_sizer->Add (s, wxGBPosition (r, 1));
 	++r;
 
-	add_label_to_sizer (_sizer, this, _("Other trusted devices"), true, wxGBPosition (r, 0));
+	add_label_to_sizer (_sizer, this, _("Filename"), true, wxGBPosition(r, 0));
+	_recipient_file = new wxStaticText (this, wxID_ANY, wxT(""));
+	checked_set (_recipient_file, recipient_file.get_value_or(""));
+	_sizer->Add (_recipient_file, wxGBPosition(r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_Y_GAP);
+	++r;
+
+	{
+		int flags = wxALIGN_CENTER_VERTICAL | wxTOP;
+#ifdef __WXOSX__
+		flags |= wxALIGN_RIGHT;
+		auto m = new StaticText (this, _("Other trusted devices") + wxT(":"));
+#else
+		auto m = new StaticText (this, _("Other trusted devices"));
+#endif
+		_sizer->Add (m, wxGBPosition(r, 0), wxDefaultSpan, flags, DCPOMATIC_SIZER_Y_GAP);
+	}
 	++r;
 
 	vector<EditableListColumn> columns;
@@ -157,7 +178,8 @@ ScreenDialog::ScreenDialog (
 		[] (TrustedDevice const& d, int) {
 			return d.thumbprint();
 		},
-		false
+		EditableListTitle::INVISIBLE,
+		EditableListButton::NEW | EditableListButton::EDIT | EditableListButton::REMOVE
 		);
 
 	_sizer->Add (_trusted_device_list, wxGBPosition (r, 0), wxGBSpan (1, 3), wxEXPAND);
@@ -202,6 +224,17 @@ ScreenDialog::recipient () const
 }
 
 
+optional<string>
+ScreenDialog::recipient_file () const
+{
+	auto const f = wx_to_std(_recipient_file->GetLabel());
+	if (f.empty()) {
+		return {};
+	}
+	return f;
+}
+
+
 void
 ScreenDialog::load_recipient (boost::filesystem::path file)
 {
@@ -213,6 +246,7 @@ ScreenDialog::load_recipient (boost::filesystem::path file)
 			return;
 		}
 		set_recipient (c.leaf ());
+		checked_set (_recipient_file, file.string());
 	} catch (dcp::MiscError& e) {
 		error_dialog (this, _("Could not read certificate file."), std_to_wx(e.what()));
 	}
@@ -238,6 +272,7 @@ ScreenDialog::download_recipient ()
 	auto d = new DownloadCertificateDialog (this);
 	if (d->ShowModal() == wxID_OK) {
 		set_recipient (d->certificate());
+		checked_set (_recipient_file, d->url());
 	}
 	d->Destroy ();
 	setup_sensitivity ();