X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Fjob_view.cc;h=8e7040c3b855cb1d0f3ae87db05da65b7b7b5d6e;hb=2a17b002ab6a5fc52e0c40bea691cdb506ca59be;hp=83d3159b09dcd15b90d5c19a2cad1717e1e662d0;hpb=a5e041b46b675c1f42fc83920be45345662e0376;p=dcpomatic.git

diff --git a/src/wx/job_view.cc b/src/wx/job_view.cc
index 83d3159b0..8e7040c3b 100644
--- a/src/wx/job_view.cc
+++ b/src/wx/job_view.cc
@@ -28,45 +28,51 @@ using std::string;
 using std::min;
 using boost::shared_ptr;
 
-/** @param top Put this view at the top of table, otherwise put it at the bottom */
-JobView::JobView (shared_ptr<Job> job, wxWindow* parent, wxWindow* container, wxFlexGridSizer* table, bool top)
+JobView::JobView (shared_ptr<Job> job, wxWindow* parent, wxWindow* container, wxFlexGridSizer* table)
 	: _job (job)
+	, _table (table)
 	, _parent (parent)
+	, _container (container)
 {
-	int n = top ? 0 : (table->GetEffectiveRowsCount() * table->GetEffectiveColsCount());
+
+}
+
+void
+JobView::setup ()
+{
+	int n = insert_position ();
 
 	_gauge_message = new wxBoxSizer (wxVERTICAL);
-	_gauge = new wxGauge (container, wxID_ANY, 100);
+	_gauge = new wxGauge (_container, wxID_ANY, 100);
 	/* This seems to be required to allow the gauge to shrink under OS X */
 	_gauge->SetMinSize (wxSize (0, -1));
 	_gauge_message->Add (_gauge, 0, wxEXPAND | wxLEFT | wxRIGHT);
-	_message = new wxStaticText (container, wxID_ANY, wxT (" \n "));
+	_message = new wxStaticText (_container, wxID_ANY, wxT (" \n "), wxDefaultPosition, wxDefaultSize, wxST_ELLIPSIZE_MIDDLE);
 	_gauge_message->Add (_message, 1, wxEXPAND | wxALIGN_CENTER_VERTICAL | wxALL, 6);
-	table->Insert (n, _gauge_message, 1, wxEXPAND | wxLEFT | wxRIGHT);
+	_table->Insert (n, _gauge_message, 1, wxEXPAND | wxLEFT | wxRIGHT);
 	++n;
 
-	_cancel = new wxButton (container, wxID_ANY, _("Cancel"));
-	_cancel->Bind (wxEVT_COMMAND_BUTTON_CLICKED, &JobView::cancel_clicked, this);
-	table->Insert (n, _cancel, 1, wxALIGN_CENTER_VERTICAL | wxALL, 3);
-	++n;
+	wxBoxSizer* buttons = new wxBoxSizer (wxHORIZONTAL);
 
-	_pause = new wxButton (container, wxID_ANY, _("Pause"));
-	_pause->Bind (wxEVT_COMMAND_BUTTON_CLICKED, &JobView::pause_clicked, this);
-	table->Insert (n, _pause, 1, wxALIGN_CENTER_VERTICAL | wxALL, 3);
-	++n;
+	_cancel = new wxButton (_container, wxID_ANY, _("Cancel"));
+	_cancel->Bind (wxEVT_BUTTON, &JobView::cancel_clicked, this);
+	buttons->Add (_cancel, 1, wxALIGN_CENTER_VERTICAL);
 
-	_details = new wxButton (container, wxID_ANY, _("Details..."));
-	_details->Bind (wxEVT_COMMAND_BUTTON_CLICKED, &JobView::details_clicked, this);
+	_details = new wxButton (_container, wxID_ANY, _("Details..."));
+	_details->Bind (wxEVT_BUTTON, &JobView::details_clicked, this);
 	_details->Enable (false);
-	table->Insert (n, _details, 1, wxALIGN_CENTER_VERTICAL | wxALL, 3);
-	++n;
+	buttons->Add (_details, 1, wxALIGN_CENTER_VERTICAL);
 
-	_progress_connection = job->Progress.connect (boost::bind (&JobView::progress, this));
-	_finished_connection = job->Finished.connect (boost::bind (&JobView::finished, this));
+	finish_setup (_container, buttons);
+
+	_table->Insert (n, buttons, 1, wxALIGN_CENTER_VERTICAL | wxALL, 3);
+
+	_progress_connection = _job->Progress.connect (boost::bind (&JobView::progress, this));
+	_finished_connection = _job->Finished.connect (boost::bind (&JobView::finished, this));
 
 	progress ();
 
-	table->Layout ();
+	_table->Layout ();
 }
 
 void
@@ -87,6 +93,9 @@ JobView::progress ()
 	whole += _job->status ();
 	if (whole != _last_message) {
 		_message->SetLabelMarkup (std_to_wx (whole));
+		/* This hack fixes the size of _message on OS X */
+		_message->InvalidateBestSize ();
+		_message->SetSize (_message->GetBestSize ());
 		_gauge_message->Layout ();
 		_last_message = whole;
 	}
@@ -105,7 +114,6 @@ JobView::finished ()
 	}
 
 	_cancel->Enable (false);
-	_pause->Enable (false);
 	if (!_job->error_details().empty ()) {
 		_details->Enable (true);
 	}
@@ -126,15 +134,3 @@ JobView::cancel_clicked (wxCommandEvent &)
 		_job->cancel ();
 	}
 }
-
-void
-JobView::pause_clicked (wxCommandEvent &)
-{
-	if (_job->paused()) {
-		_job->resume ();
-		_pause->SetLabel (_("Pause"));
-	} else {
-		_job->pause ();
-		_pause->SetLabel (_("Resume"));
-	}
-}