/*
- Copyright (C) 2012 Carl Hetherington <cth@carlh.net>
+ Copyright (C) 2012-2015 Carl Hetherington <cth@carlh.net>
This program is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
using boost::shared_ptr;
using boost::weak_ptr;
-class JobRecord
+class JobRecord : public boost::noncopyable
{
public:
- JobRecord (shared_ptr<Job> job, wxScrolledWindow* window, wxPanel* panel, wxFlexGridSizer* table, bool pause)
+ JobRecord (shared_ptr<Job> job, wxScrolledWindow* window, wxPanel* panel, wxFlexGridSizer* table)
: _job (job)
, _window (window)
, _panel (panel)
- , _table (table)
- , _pause (0)
{
int n = 0;
-
- _name = new wxStaticText (panel, wxID_ANY, "");
- _name->SetLabelMarkup ("<b>" + _job->name () + "</b>");
- table->Insert (n, _name, 0, wxALIGN_CENTER_VERTICAL | wxALL, 6);
- ++n;
-
+
+ wxBoxSizer* gauge_message = new wxBoxSizer (wxVERTICAL);
_gauge = new wxGauge (panel, wxID_ANY, 100);
/* This seems to be required to allow the gauge to shrink under OS X */
_gauge->SetMinSize (wxSize (0, -1));
- table->Insert (n, _gauge, 1, wxEXPAND | wxLEFT | wxRIGHT);
- ++n;
-
- _message = new wxStaticText (panel, wxID_ANY, std_to_wx (""));
- table->Insert (n, _message, 1, wxALIGN_CENTER_VERTICAL | wxALL, 6);
+ gauge_message->Add (_gauge, 1, wxEXPAND | wxLEFT | wxRIGHT);
+ _message = new wxStaticText (panel, wxID_ANY, wxT (""), wxDefaultPosition, wxDefaultSize, wxST_ELLIPSIZE_END);
+ gauge_message->Add (_message, 1, wxEXPAND | wxALIGN_CENTER_VERTICAL | wxALL, 6);
+ table->Insert (n, gauge_message, 1, wxEXPAND | wxLEFT | wxRIGHT);
++n;
_cancel = new wxButton (panel, wxID_ANY, _("Cancel"));
_cancel->Bind (wxEVT_COMMAND_BUTTON_CLICKED, &JobRecord::cancel_clicked, this);
- table->Insert (n, _cancel, 1, wxALIGN_CENTER_VERTICAL | wxALL, 6);
+ table->Insert (n, _cancel, 1, wxALIGN_CENTER_VERTICAL | wxALL, 3);
++n;
- if (pause) {
- _pause = new wxButton (_panel, wxID_ANY, _("Pause"));
- _pause->Bind (wxEVT_COMMAND_BUTTON_CLICKED, &JobRecord::pause_clicked, this);
- table->Insert (n, _pause, 1, wxALIGN_CENTER_VERTICAL | wxALL, 6);
- ++n;
- }
+ _pause = new wxButton (_panel, wxID_ANY, _("Pause"));
+ _pause->Bind (wxEVT_COMMAND_BUTTON_CLICKED, &JobRecord::pause_clicked, this);
+ table->Insert (n, _pause, 1, wxALIGN_CENTER_VERTICAL | wxALL, 3);
+ ++n;
_details = new wxButton (_panel, wxID_ANY, _("Details..."));
_details->Bind (wxEVT_COMMAND_BUTTON_CLICKED, &JobRecord::details_clicked, this);
_details->Enable (false);
- table->Insert (n, _details, 1, wxALIGN_CENTER_VERTICAL | wxALL, 6);
+ table->Insert (n, _details, 1, wxALIGN_CENTER_VERTICAL | wxALL, 3);
++n;
- job->Progress.connect (boost::bind (&JobRecord::progress, this));
- job->Finished.connect (boost::bind (&JobRecord::finished, this));
+ _progress_connection = job->Progress.connect (boost::bind (&JobRecord::progress, this));
+ _finished_connection = job->Finished.connect (boost::bind (&JobRecord::finished, this));
table->Layout ();
- panel->FitInside ();
}
void maybe_pulse ()
{
- if (_job->running() && _job->progress_unknown ()) {
+ if (_job->running() && !_job->progress ()) {
_gauge->Pulse ();
}
}
private:
- void update_job_name ()
- {
- string n = "<b>" + _job->name () + "</b>";
- if (!_job->sub_name().empty ()) {
- n += "\n" + _job->sub_name ();
- }
-
- if (n != _last_name) {
- _name->SetLabelMarkup (std_to_wx (n));
- _last_name = n;
- }
- }
-
void progress ()
{
- float const p = _job->progress ();
- if (p >= 0) {
- checked_set (_message, _job->status ());
- update_job_name ();
- int const pp = min (100.0f, p * 100);
- _gauge->SetValue (pp);
+ string whole = "<b>" + _job->name () + "</b>: " + _job->sub_name() + " " + _job->status ();
+ if (whole != _last_message) {
+ _message->SetLabelMarkup (whole);
+ _last_message = whole;
+ }
+ if (_job->progress ()) {
+ _gauge->SetValue (min (100.0f, _job->progress().get() * 100));
}
-
- _table->Layout ();
- _window->FitInside ();
}
void finished ()
{
- checked_set (_message, _job->status ());
- update_job_name ();
+ progress ();
if (!_job->finished_cancelled ()) {
_gauge->SetValue (100);
}
_cancel->Enable (false);
- if (_pause) {
- _pause->Enable (false);
- }
+ _pause->Enable (false);
if (!_job->error_details().empty ()) {
_details->Enable (true);
}
-
- _table->Layout ();
- _window->FitInside ();
}
void details_clicked (wxCommandEvent &)
boost::shared_ptr<Job> _job;
wxScrolledWindow* _window;
wxPanel* _panel;
- wxFlexGridSizer* _table;
- wxStaticText* _name;
wxGauge* _gauge;
wxStaticText* _message;
wxButton* _cancel;
wxButton* _pause;
wxButton* _details;
- std::string _last_name;
+ std::string _last_message;
+
+ boost::signals2::scoped_connection _progress_connection;
+ boost::signals2::scoped_connection _finished_connection;
};
/** Must be called in the GUI thread */
-JobManagerView::JobManagerView (wxWindow* parent, Buttons buttons)
+JobManagerView::JobManagerView (wxWindow* parent)
: wxScrolledWindow (parent)
- , _buttons (buttons)
{
_panel = new wxPanel (this);
wxSizer* sizer = new wxBoxSizer (wxVERTICAL);
sizer->Add (_panel, 1, wxEXPAND);
SetSizer (sizer);
- int N = 5;
- if (buttons & PAUSE) {
- ++N;
- }
-
- _table = new wxFlexGridSizer (N, 6, 6);
- _table->AddGrowableCol (1, 1);
+ _table = new wxFlexGridSizer (4, 4, 6);
+ _table->AddGrowableCol (0, 1);
_panel->SetSizer (_table);
SetScrollRate (0, 32);
+ EnableScrolling (false, true);
Bind (wxEVT_TIMER, boost::bind (&JobManagerView::periodic, this));
_timer.reset (new wxTimer (this));
_timer->Start (1000);
-
+
JobManager::instance()->JobAdded.connect (bind (&JobManagerView::job_added, this, _1));
}
{
shared_ptr<Job> job = j.lock ();
if (job) {
- _job_records.push_back (shared_ptr<JobRecord> (new JobRecord (job, this, _panel, _table, _buttons & PAUSE)));
+ _job_records.push_back (shared_ptr<JobRecord> (new JobRecord (job, this, _panel, _table)));
}
}