Save window metrics in the config file (#1575).

Perhaps these should be in a separate GUI file, like how it's done for
Films, but I can't really see a big advantage (and there already
GUI-only details in there).

In this commit we also save the hints dialog size/position and
start it a bit larger (#2892).

6 files changed, 125 insertions, 38 deletions

diff --git a/src/wx/hints_dialog.cc b/src/wx/hints_dialog.cc
index e02c07518..47d96a70e 100644
--- a/src/wx/hints_dialog.cc
+++ b/src/wx/hints_dialog.cc
@@ -46,10 +46,11 @@ using namespace boost::placeholders;
 
 
 HintsDialog::HintsDialog(wxWindow* parent, std::weak_ptr<Film> film, bool ok)
-	: wxDialog(parent, wxID_ANY, _("Hints"))
+	: wxDialog(parent, wxID_ANY, _("Hints"), wxDefaultPosition, wxDefaultSize, wxDEFAULT_DIALOG_STYLE | wxRESIZE_BORDER)
 	, _film(film)
 	, _hints(0)
 	, _finished(false)
+	, _metrics(std::string{"hints"}, { 800, 600 }, this)
 {
 	auto sizer = new wxBoxSizer(wxVERTICAL);
 
@@ -58,7 +59,7 @@ HintsDialog::HintsDialog(wxWindow* parent, std::weak_ptr<Film> film, bool ok)
 	_gauge_message = new StaticText(this, {});
 	sizer->Add(_gauge_message, 0, wxALL | wxEXPAND, DCPOMATIC_SIZER_GAP);
 
-	_text = new wxRichTextCtrl(this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize(400, 300), wxRE_READONLY);
+	_text = new wxRichTextCtrl(this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxDefaultSize, wxRE_READONLY);
 	sizer->Add(_text, 1, wxEXPAND | wxALL, 6);
 
 	if (!ok) {
@@ -95,6 +96,8 @@ HintsDialog::HintsDialog(wxWindow* parent, std::weak_ptr<Film> film, bool ok)
 	}
 
 	film_change(ChangeType::DONE);
+
+	_metrics.bind();
 }
 
 
@@ -189,3 +192,10 @@ HintsDialog::progress(string m)
 {
 	_gauge_message->SetLabel(std_to_wx(m));
 }
+
+void
+HintsDialog::show()
+{
+	_metrics.show();
+}
+
diff --git a/src/wx/hints_dialog.h b/src/wx/hints_dialog.h
index 44420405a..e10e80717 100644
--- a/src/wx/hints_dialog.h
+++ b/src/wx/hints_dialog.h
@@ -19,6 +19,7 @@
 */
 
 
+#include "window_metrics.h"
 #include "lib/change_signaller.h"
 #include <dcp/warnings.h>
 LIBDCP_DISABLE_WARNINGS
@@ -36,6 +37,8 @@ class HintsDialog : public wxDialog
 public:
 	HintsDialog(wxWindow* parent, std::weak_ptr<Film>, bool ok);
 
+	void show();
+
 private:
 	void film_change(ChangeType);
 	void film_content_change(ChangeType type);
@@ -60,4 +63,6 @@ private:
 	boost::signals2::scoped_connection _hints_progress_connection;
 	boost::signals2::scoped_connection _hints_pulse_connection;
 	boost::signals2::scoped_connection _hints_finished_connection;
+
+	WindowMetrics _metrics;
 };
diff --git a/src/wx/text_view.cc b/src/wx/text_view.cc
index bde7b09e9..fd647f050 100644
--- a/src/wx/text_view.cc
+++ b/src/wx/text_view.cc
@@ -42,17 +42,15 @@ using namespace boost::placeholders;
 #endif
 
 
-WindowMetrics TextView::_metrics;
-
-
 TextView::TextView (
 	wxWindow* parent, shared_ptr<Film> film, shared_ptr<Content> content, shared_ptr<TextContent> text, shared_ptr<Decoder> decoder, FilmViewer& viewer
 	)
-	: wxDialog(parent, wxID_ANY, _("Captions"), _metrics.position, wxDefaultSize, wxDEFAULT_DIALOG_STYLE | wxRESIZE_BORDER)
+	: wxDialog(parent, wxID_ANY, _("Captions"), wxDefaultPosition, wxDefaultSize, wxDEFAULT_DIALOG_STYLE | wxRESIZE_BORDER)
 	, _content (content)
 	, _film_viewer (viewer)
+	, _metrics(std::string{"text"}, { 800, 600 }, this)
 {
-	_list = new wxListCtrl(this, wxID_ANY, wxDefaultPosition, _metrics.size, wxLC_REPORT | wxLC_SINGLE_SEL);
+	_list = new wxListCtrl(this, wxID_ANY, wxDefaultPosition, wxDefaultSize, wxLC_REPORT | wxLC_SINGLE_SEL);
 
 	{
 		wxListItem ip;
@@ -108,31 +106,14 @@ TextView::TextView (
 	while (!decoder->pass ()) {}
 	SetSizerAndFit (sizer);
 
-	_list->Bind(wxEVT_SIZE, boost::bind(&TextView::list_sized, this, _1));
-	Bind(wxEVT_MOVE, boost::bind(&TextView::moved, this, _1));
-}
-
-
-void
-TextView::list_sized(wxSizeEvent& ev)
-{
-	_metrics.size = ev.GetSize();
-	ev.Skip();
-}
-
-
-void
-TextView::moved(wxMoveEvent& ev)
-{
-	_metrics.position = ClientToScreen({0, 0});
-	ev.Skip();
+	_metrics.bind();
 }
 
 
 void
 TextView::show()
 {
-	_metrics.show(this);
+	_metrics.show();
 }
 
 void
diff --git a/src/wx/text_view.h b/src/wx/text_view.h
index ee7650673..dbb601fc4 100644
--- a/src/wx/text_view.h
+++ b/src/wx/text_view.h
@@ -55,8 +55,6 @@ private:
 	void data_start (ContentStringText cts);
 	void data_stop (dcpomatic::ContentTime time);
 	void subtitle_selected (wxListEvent &);
-	void moved(wxMoveEvent& ev);
-	void list_sized(wxSizeEvent& ev);
 
 	wxListCtrl* _list;
 	int _subs;
@@ -66,7 +64,7 @@ private:
 	std::weak_ptr<Content> _content;
 	FilmViewer& _film_viewer;
 
-	static WindowMetrics _metrics;
+	WindowMetrics _metrics;
 };
 
 
diff --git a/src/wx/window_metrics.cc b/src/wx/window_metrics.cc
index 4b0c6ce66..81ad7f2ea 100644
--- a/src/wx/window_metrics.cc
+++ b/src/wx/window_metrics.cc
@@ -20,26 +20,99 @@
 
 
 #include "window_metrics.h"
+#include "lib/config.h"
 #include <dcp/warnings.h>
 LIBDCP_DISABLE_WARNINGS
 #include <wx/app.h>
 #include <wx/wx.h>
 LIBDCP_ENABLE_WARNINGS
+#include <boost/bind/bind.hpp>
+
+
+#if BOOST_VERSION >= 106100
+using namespace boost::placeholders;
+#endif
+
+
+WindowMetrics::WindowMetrics(std::string name, wxSize default_size, wxWindow* window)
+	: _name(name)
+	, _window(window)
+{
+	if (auto metrics = Config::instance()->window_metrics(name)) {
+		_position = { metrics->first.x, metrics->first.y };
+		_size = { metrics->second.width, metrics->second.height };
+	} else {
+		_position = wxDefaultPosition;
+		_size = default_size;
+	}
+}
 
 
 void
-WindowMetrics::show(wxWindow* window) const
+WindowMetrics::bind()
+{
+	_window->Bind(wxEVT_SIZE, boost::bind(&WindowMetrics::sized, this, _1));
+	_window->Bind(wxEVT_MOVE, boost::bind(&WindowMetrics::moved, this, _1));
+}
+
+
+void
+WindowMetrics::show() const
 {
 #ifdef DCPOMATIC_LINUX
-	auto const position_before = position;
+	auto const position_before = _position;
+
+	_window->SetSize(_size);
+	_window->Show();
+
+	wxTheApp->CallAfter([this, position_before] {
+		_window->SetPosition(position_before);
+	});
+#else
+	_window->SetSize(_size);
+	_window->SetPosition(_position);
+	_window->Show();
 #endif
+}
+
+
+void
+WindowMetrics::write_to_config() const
+{
+	Config::instance()->set_window_metrics(_name, { _position.x, _position.y }, { _size.GetWidth(), _size.GetHeight() });
+}
+
+
+void
+WindowMetrics::sized(wxSizeEvent& ev)
+{
+	auto top_level_window = dynamic_cast<wxTopLevelWindow*>(_window);
+	if (top_level_window && top_level_window->IsMaximized()) {
+		/* On Windows the maximised state looks a little different to the
+		 * window just being as big as it can be; set wxDefaultSize here
+		 * so that we call Maximize in show().
+		 */
+		_size = wxDefaultSize;
+	} else {
+		_size = ev.GetSize();
+	}
+	write_to_config();
+	ev.Skip();
+}
 
-	window->Show();
 
+void
+WindowMetrics::moved(wxMoveEvent& ev)
+{
 #ifdef DCPOMATIC_LINUX
-	wxTheApp->CallAfter([window, position_before] {
-		window->SetPosition(position_before);
-	});
+	_position = _window->ClientToScreen({0, 0});
+#else
+	int x;
+	int y;
+	_window->GetScreenPosition(&x, &y);
+	_position = { x, y };
 #endif
+	write_to_config();
+	ev.Skip();
 }
 
diff --git a/src/wx/window_metrics.h b/src/wx/window_metrics.h
index 9a1e10a66..5d99dbe0a 100644
--- a/src/wx/window_metrics.h
+++ b/src/wx/window_metrics.h
@@ -29,16 +29,36 @@ LIBDCP_DISABLE_WARNINGS
 LIBDCP_ENABLE_WARNINGS
 
 
+class wxMoveEvent;
+class wxSizeEvent;
 class wxWindow;
 
 
 class WindowMetrics
 {
 public:
-	void show(wxWindow* window) const;
+	WindowMetrics(std::string name, wxSize default_size, wxWindow* window);
 
-	wxPoint position = wxDefaultPosition;
-	wxSize size = wxDefaultSize;
+	wxPoint position() const {
+		return _position;
+	}
+
+	wxSize size() const {
+		return _size;
+	}
+
+	void bind();
+	void show() const;
+
+private:
+	void moved(wxMoveEvent& event);
+	void sized(wxSizeEvent& event);
+	void write_to_config() const;
+
+	std::string _name;
+	wxWindow* _window;
+	wxPoint _position;
+	wxSize _size;
 };