From be32d0b93234df78d89a3208936167e98dc47ccc Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Mon, 25 May 2015 14:48:57 +0100 Subject: b37e348604b75f346c8b423dd6b67a4663102871 from master; tweak job view. --- src/wx/job_manager_view.cc | 80 +++++++++++----------------------------------- src/wx/job_manager_view.h | 3 +- 2 files changed, 20 insertions(+), 63 deletions(-) (limited to 'src') diff --git a/src/wx/job_manager_view.cc b/src/wx/job_manager_view.cc index ec58607af..72efae593 100644 --- a/src/wx/job_manager_view.cc +++ b/src/wx/job_manager_view.cc @@ -46,44 +46,37 @@ public: , _table (table) { int n = 0; - - _name = new wxStaticText (panel, wxID_ANY, ""); - string const jn = "" + _job->name () + ""; - _name->SetLabelMarkup (std_to_wx (jn)); - 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; _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); + 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; _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 () @@ -95,43 +88,21 @@ public: private: - void update_job_name () - { - string n = "" + _job->name () + ""; - if (!_job->sub_name().empty ()) { - n += "\n" + _job->sub_name (); - } - - if (n != _last_name) { - _name->SetLabelMarkup (std_to_wx (n)); - _last_name = n; - } - } - - void update_status () - { - string s = _job->status (); - if (s.length() > 25) { - s = s.substr (0, 25) + "..."; - } - checked_set (_message, s); - } - void progress () { - update_status (); - update_job_name (); + string whole = "" + _job->name () + ": " + _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 () { - update_status (); - update_job_name (); + progress (); if (!_job->finished_cancelled ()) { _gauge->SetValue (100); @@ -142,9 +113,6 @@ private: if (!_job->error_details().empty ()) { _details->Enable (true); } - - _table->Layout (); - _window->FitInside (); } void details_clicked (wxCommandEvent &) @@ -174,13 +142,12 @@ private: 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; @@ -195,8 +162,8 @@ JobManagerView::JobManagerView (wxWindow* parent) sizer->Add (_panel, 1, wxEXPAND); SetSizer (sizer); - _table = new wxFlexGridSizer (6, 6, 6); - _table->AddGrowableCol (1, 1); + _table = new wxFlexGridSizer (4, 4, 6); + _table->AddGrowableCol (0, 1); _panel->SetSizer (_table); SetScrollRate (0, 32); @@ -206,18 +173,9 @@ JobManagerView::JobManagerView (wxWindow* parent) _timer.reset (new wxTimer (this)); _timer->Start (1000); - Bind (wxEVT_SIZE, boost::bind (&JobManagerView::sized, this, _1)); JobManager::instance()->JobAdded.connect (bind (&JobManagerView::job_added, this, _1)); } -void -JobManagerView::sized (wxSizeEvent& ev) -{ - _table->FitInside (_panel); - _table->Layout (); - ev.Skip (); -} - void JobManagerView::job_added (weak_ptr j) { diff --git a/src/wx/job_manager_view.h b/src/wx/job_manager_view.h index fde9199ff..244299a40 100644 --- a/src/wx/job_manager_view.h +++ b/src/wx/job_manager_view.h @@ -1,5 +1,5 @@ /* - Copyright (C) 2012 Carl Hetherington + Copyright (C) 2012-2015 Carl Hetherington 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 @@ -39,7 +39,6 @@ public: private: void job_added (boost::weak_ptr); void periodic (); - void sized (wxSizeEvent &); wxPanel* _panel; wxFlexGridSizer* _table; -- cgit v1.2.3