X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Fjob_view.cc;h=231176e4fd96c773d1f96a779de648647f0ddf3a;hb=e9ae050b0b15c91c3f591ad84938e60d271357b3;hp=b61da04f84da3ff1dd6dbba67f3c1e0187c1a7d5;hpb=e63de384db89a83937d42f10d4146c3adf14fdb2;p=dcpomatic.git

diff --git a/src/wx/job_view.cc b/src/wx/job_view.cc
index b61da04f8..231176e4f 100644
--- a/src/wx/job_view.cc
+++ b/src/wx/job_view.cc
@@ -18,6 +18,7 @@
 
 */
 
+
 #include "job_view.h"
 #include "wx_util.h"
 #include "message_dialog.h"
@@ -34,11 +35,13 @@
 #include <wx/wx.h>
 #include <boost/algorithm/string.hpp>
 
+
 using std::string;
 using std::min;
-using boost::shared_ptr;
+using std::shared_ptr;
 using boost::bind;
-using boost::dynamic_pointer_cast;
+using std::dynamic_pointer_cast;
+
 
 JobView::JobView (shared_ptr<Job> job, wxWindow* parent, wxWindow* container, wxFlexGridSizer* table)
 	: _job (job)
@@ -50,6 +53,7 @@ JobView::JobView (shared_ptr<Job> job, wxWindow* parent, wxWindow* container, wx
 
 }
 
+
 void
 JobView::setup ()
 {
@@ -61,7 +65,7 @@ JobView::setup ()
 	_gauge->SetMinSize (wxSize (0, -1));
 	_gauge_message->Add (_gauge, 0, wxEXPAND | wxLEFT | wxRIGHT);
 	_message = new StaticText (_container, wxT(" \n "), wxDefaultPosition, wxDefaultSize, wxST_ELLIPSIZE_MIDDLE);
-	_gauge_message->Add (_message, 1, wxEXPAND | wxALIGN_CENTER_VERTICAL | wxALL, 6);
+	_gauge_message->Add (_message, 1, wxEXPAND | wxALL, 6);
 	_table->Insert (n, _gauge_message, 1, wxEXPAND | wxLEFT | wxRIGHT);
 	++n;
 
@@ -69,12 +73,12 @@ JobView::setup ()
 
 	_cancel = new Button (_container, _("Cancel"));
 	_cancel->Bind (wxEVT_BUTTON, &JobView::cancel_clicked, this);
-	_buttons->Add (_cancel, 1, wxALIGN_CENTER_VERTICAL);
+	_buttons->Add (_cancel, 1, wxALIGN_CENTER_VERTICAL | wxRIGHT, DCPOMATIC_BUTTON_STACK_GAP);
 
 	_details = new Button (_container, _("Details..."));
 	_details->Bind (wxEVT_BUTTON, &JobView::details_clicked, this);
 	_details->Enable (false);
-	_buttons->Add (_details, 1, wxALIGN_CENTER_VERTICAL);
+	_buttons->Add (_details, 1, wxALIGN_CENTER_VERTICAL | wxRIGHT, DCPOMATIC_BUTTON_STACK_GAP);
 
 	finish_setup (_container, _buttons);
 
@@ -83,7 +87,7 @@ JobView::setup ()
 	_notify = new CheckBox (_container, _("Notify when complete"));
 	_notify->Bind (wxEVT_CHECKBOX, bind (&JobView::notify_clicked, this));
 	_notify->SetValue (Config::instance()->default_notify());
-	_controls->Add (_notify);
+	_controls->Add (_notify, 0, wxTOP, DCPOMATIC_BUTTON_STACK_GAP);
 
 	_table->Insert (n, _controls, 1, wxALIGN_CENTER_VERTICAL | wxALL, 3);
 
@@ -95,6 +99,7 @@ JobView::setup ()
 	_table->Layout ();
 }
 
+
 void
 JobView::maybe_pulse ()
 {
@@ -103,6 +108,7 @@ JobView::maybe_pulse ()
 	}
 }
 
+
 void
 JobView::progress ()
 {
@@ -110,7 +116,7 @@ JobView::progress ()
 	if (!_job->sub_name().empty ()) {
 		whole += _job->sub_name() + " ";
 	}
-	string s = _job->status ();
+	auto s = _job->status ();
 	/* Watch out for < > in the error string */
 	boost::algorithm::replace_all (s, "<", "&lt;");
 	boost::algorithm::replace_all (s, ">", "&gt;");
@@ -128,6 +134,7 @@ JobView::progress ()
 	}
 }
 
+
 void
 JobView::finished ()
 {
@@ -144,7 +151,7 @@ JobView::finished ()
 	}
 
 	if (_job->message()) {
-		MessageDialog* d = new MessageDialog (_parent, _job->name(), _job->message().get());
+		auto d = new MessageDialog (_parent, std_to_wx(_job->name()), std_to_wx(_job->message().get()));
 		d->ShowModal ();
 		d->Destroy ();
 	}
@@ -162,14 +169,16 @@ JobView::finished ()
 	}
 }
 
+
 void
 JobView::details_clicked (wxCommandEvent &)
 {
-	string s = _job->error_summary();
+	auto s = _job->error_summary();
 	s[0] = toupper (s[0]);
 	error_dialog (_parent, std_to_wx(s), std_to_wx(_job->error_details()));
 }
 
+
 void
 JobView::cancel_clicked (wxCommandEvent &)
 {
@@ -178,6 +187,7 @@ JobView::cancel_clicked (wxCommandEvent &)
 	}
 }
 
+
 void
 JobView::insert (int pos)
 {
@@ -186,6 +196,7 @@ JobView::insert (int pos)
 	_table->Layout ();
 }
 
+
 void
 JobView::detach ()
 {
@@ -193,6 +204,7 @@ JobView::detach ()
 	_table->Detach (_controls);
 }
 
+
 void
 JobView::notify_clicked ()
 {