X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Flanguage_tag_widget.cc;h=1a6f3fabc219dd799a137ed4873c154c9c7c91a0;hb=e8748f158249d7be906f6c6cf2411df45dd07a24;hp=f2f59f6888adcc5eef19ad22da9bf6888122d055;hpb=05d225e805607c8b413814ec7f6145cd454fadab;p=dcpomatic.git

diff --git a/src/wx/language_tag_widget.cc b/src/wx/language_tag_widget.cc
index f2f59f688..1a6f3fabc 100644
--- a/src/wx/language_tag_widget.cc
+++ b/src/wx/language_tag_widget.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2020 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2020-2021 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -22,17 +22,33 @@
 #include "dcpomatic_button.h"
 #include "language_tag_dialog.h"
 #include "language_tag_widget.h"
+#include "wx_ptr.h"
 #include "wx_util.h"
+#include "lib/scope_guard.h"
+#include <dcp/warnings.h>
+LIBDCP_DISABLE_WARNINGS
 #include <wx/wx.h>
+LIBDCP_ENABLE_WARNINGS
 
 
-LanguageTagWidget::LanguageTagWidget (wxWindow* parent, wxString tooltip, dcp::LanguageTag tag)
+using boost::optional;
+
+
+LanguageTagWidget::LanguageTagWidget (wxWindow* parent, wxString tooltip, optional<dcp::LanguageTag> tag, optional<wxString> size_to_fit)
 	: _parent (parent)
 	, _sizer (new wxBoxSizer(wxHORIZONTAL))
 {
-	_language = new wxStaticText (parent, wxID_ANY, wxT(""));
+	_language = new wxStaticText (parent, wxID_ANY, wxT(""), wxDefaultPosition, wxDefaultSize, wxST_ELLIPSIZE_END);
 	_language->SetToolTip (tooltip);
 	set (tag);
+
+	if (size_to_fit) {
+		int w;
+		int h;
+		_language->GetTextExtent (*size_to_fit, &w, &h);
+		_language->SetMinSize (wxSize(w, -1));
+	}
+
 	_sizer->Add (_language, 1, wxLEFT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_X_GAP);
 	_edit = new Button (parent, _("Edit..."));
 	_sizer->Add (_edit, 0, wxLEFT, DCPOMATIC_SIZER_GAP);
@@ -44,17 +60,29 @@ LanguageTagWidget::LanguageTagWidget (wxWindow* parent, wxString tooltip, dcp::L
 void
 LanguageTagWidget::edit ()
 {
-	LanguageTagDialog* d = new LanguageTagDialog(_parent, _tag);
-	d->ShowModal ();
-	set (d->get());
-	Changed (d->get());
-	d->Destroy ();
+	auto d = make_wx<LanguageTagDialog>(_parent, _tag.get_value_or(dcp::LanguageTag("en")));
+	if (d->ShowModal() == wxID_OK) {
+		set(d->get());
+		Changed(d->get());
+	}
 }
 
 
 void
-LanguageTagWidget::set (dcp::LanguageTag tag)
+LanguageTagWidget::set (optional<dcp::LanguageTag> tag)
 {
 	_tag = tag;
-	checked_set (_language, std_to_wx(tag.to_string()));
+	if (tag) {
+		checked_set (_language, std_to_wx(tag->to_string()));
+	} else {
+		checked_set (_language, wxT(""));
+	}
+}
+
+
+void
+LanguageTagWidget::enable (bool e)
+{
+	_language->Enable (e);
+	_edit->Enable (e);
 }