Various JobView fixes.

4 files changed, 25 insertions, 7 deletions

diff --git a/src/wx/file_picker_ctrl.cc b/src/wx/file_picker_ctrl.cc
index 4e2f97618..8de1596a9 100644
--- a/src/wx/file_picker_ctrl.cc
+++ b/src/wx/file_picker_ctrl.cc
@@ -34,14 +34,16 @@ FilePickerCtrl::FilePickerCtrl (wxWindow* parent, wxString prompt, wxString wild
 {
 	_sizer = new wxBoxSizer (wxHORIZONTAL);
 
-	_file = new wxStaticText (this, wxID_ANY, wxT ("This is the length of the file label"));
+        wxClientDC dc (parent);
+        wxSize size = dc.GetTextExtent (wxT ("This is the length of the file label it should be quite long"));
+        size.SetHeight (-1);
+
+	_file = new wxButton (this, wxID_ANY, _("(None)"), wxDefaultPosition, size, wxBU_LEFT);
 	_sizer->Add (_file, 1, wxEXPAND | wxALL, 6);
-	_browse = new wxButton (this, wxID_ANY, _("Browse..."));
-	_sizer->Add (_browse, 0);
 
 	SetSizerAndFit (_sizer);
 
-	_browse->Bind (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&FilePickerCtrl::browse_clicked, this));
+	_file->Bind (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&FilePickerCtrl::browse_clicked, this));
 }
 
 void
@@ -49,7 +51,11 @@ FilePickerCtrl::SetPath (wxString p)
 {
 	_path = p;
 
-	_file->SetLabel (std_to_wx (filesystem::path (wx_to_std (_path)).leaf().string()));
+	if (!_path.IsEmpty ()) {
+		_file->SetLabel (std_to_wx (filesystem::path (wx_to_std (_path)).leaf().string()));
+	} else {
+		_file->SetLabel (_("(None)"));
+	}
 
 	wxCommandEvent ev (wxEVT_COMMAND_FILEPICKER_CHANGED, wxID_ANY);
 	GetEventHandler()->ProcessEvent (ev);
diff --git a/src/wx/file_picker_ctrl.h b/src/wx/file_picker_ctrl.h
index 4c721af48..51f11cf61 100644
--- a/src/wx/file_picker_ctrl.h
+++ b/src/wx/file_picker_ctrl.h
@@ -30,8 +30,7 @@ public:
 private:
 	void browse_clicked ();
 
-	wxStaticText* _file;
-	wxButton* _browse;
+	wxButton* _file;
 	wxString _path;
 	wxSizer* _sizer;
 	wxString _prompt;
diff --git a/src/wx/job_view_dialog.cc b/src/wx/job_view_dialog.cc
index a2e3cfe27..1a0c4d526 100644
--- a/src/wx/job_view_dialog.cc
+++ b/src/wx/job_view_dialog.cc
@@ -28,9 +28,19 @@ JobViewDialog::JobViewDialog (wxWindow* parent, wxString title, shared_ptr<Job>
 	_view = new JobView (job, this, this, _table);
 	layout ();
 	SetMinSize (wxSize (960, -1));
+
+	Bind (wxEVT_TIMER, boost::bind (&JobViewDialog::periodic, this));
+	_timer.reset (new wxTimer (this));
+	_timer->Start (1000);
 }
 
 JobViewDialog::~JobViewDialog ()
 {
 	delete _view;
 }
+
+void
+JobViewDialog::periodic ()
+{
+	_view->maybe_pulse ();
+}
diff --git a/src/wx/job_view_dialog.h b/src/wx/job_view_dialog.h
index 0282d9848..4ec00e896 100644
--- a/src/wx/job_view_dialog.h
+++ b/src/wx/job_view_dialog.h
@@ -30,5 +30,8 @@ public:
 	~JobViewDialog ();
 
 private:
+	void periodic ();
+
 	JobView* _view;
+	boost::shared_ptr<wxTimer> _timer;
 };