From e241b3d295fe4158239170f17391e08473e159c5 Mon Sep 17 00:00:00 2001
From: Carl Hetherington <cth@carlh.net>
Date: Tue, 30 Jul 2013 20:14:47 +0100
Subject: [PATCH] Make length editable only for still images.

---
 src/wx/timecode.cc     | 51 +++++++++++++++++++++++++++++-------------
 src/wx/timecode.h      |  6 +++++
 src/wx/timing_panel.cc |  3 +++
 3 files changed, 44 insertions(+), 16 deletions(-)

diff --git a/src/wx/timecode.cc b/src/wx/timecode.cc
index f826dd93f..033bd2bd0 100644
--- a/src/wx/timecode.cc
+++ b/src/wx/timecode.cc
@@ -42,26 +42,33 @@ Timecode::Timecode (wxWindow* parent)
 	}
 
 	validator.SetIncludes (list);
+
+	_sizer = new wxBoxSizer (wxHORIZONTAL);
 	
-	wxBoxSizer* sizer = new wxBoxSizer (wxHORIZONTAL);
-	_hours = new wxTextCtrl (this, wxID_ANY, wxT(""), wxDefaultPosition, size, 0, validator);
+	_editable = new wxPanel (this);
+	wxSizer* editable_sizer = new wxBoxSizer (wxHORIZONTAL);
+	_hours = new wxTextCtrl (_editable, wxID_ANY, wxT(""), wxDefaultPosition, size, 0, validator);
 	_hours->SetMaxLength (2);
-	sizer->Add (_hours);
-	add_label_to_sizer (sizer, this, wxT (":"), false);
-	_minutes = new wxTextCtrl (this, wxID_ANY, wxT(""), wxDefaultPosition, size, 0, validator);
+	editable_sizer->Add (_hours);
+	add_label_to_sizer (editable_sizer, _editable, wxT (":"), false);
+	_minutes = new wxTextCtrl (_editable, wxID_ANY, wxT(""), wxDefaultPosition, size, 0, validator);
 	_minutes->SetMaxLength (2);
-	sizer->Add (_minutes);
-	add_label_to_sizer (sizer, this, wxT (":"), false);
-	_seconds = new wxTextCtrl (this, wxID_ANY, wxT(""), wxDefaultPosition, size, 0, validator);
+	editable_sizer->Add (_minutes);
+	add_label_to_sizer (editable_sizer, _editable, wxT (":"), false);
+	_seconds = new wxTextCtrl (_editable, wxID_ANY, wxT(""), wxDefaultPosition, size, 0, validator);
 	_seconds->SetMaxLength (2);
-	sizer->Add (_seconds);
-	add_label_to_sizer (sizer, this, wxT ("."), false);
-	_frames = new wxTextCtrl (this, wxID_ANY, wxT(""), wxDefaultPosition, size, 0, validator);
+	editable_sizer->Add (_seconds);
+	add_label_to_sizer (editable_sizer, _editable, wxT ("."), false);
+	_frames = new wxTextCtrl (_editable, wxID_ANY, wxT(""), wxDefaultPosition, size, 0, validator);
 	_frames->SetMaxLength (2);
-	sizer->Add (_frames);
-	_set_button = new wxButton (this, wxID_ANY, _("Set"));
-	sizer->Add (_set_button, 0, wxLEFT | wxRIGHT, 8);
+	editable_sizer->Add (_frames);
+	_set_button = new wxButton (_editable, wxID_ANY, _("Set"));
+	editable_sizer->Add (_set_button, 0, wxLEFT | wxRIGHT, 8);
+	_editable->SetSizerAndFit (editable_sizer);
+	_sizer->Add (_editable);
 
+	_fixed = add_label_to_sizer (_sizer, this, wxT ("42"), false);
+	
 	_hours->Bind	  (wxEVT_COMMAND_TEXT_UPDATED,   boost::bind (&Timecode::changed, this));
 	_minutes->Bind	  (wxEVT_COMMAND_TEXT_UPDATED,   boost::bind (&Timecode::changed, this));
 	_seconds->Bind	  (wxEVT_COMMAND_TEXT_UPDATED,   boost::bind (&Timecode::changed, this));
@@ -69,8 +76,10 @@ Timecode::Timecode (wxWindow* parent)
 	_set_button->Bind (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&Timecode::set_clicked, this));
 
 	_set_button->Enable (false);
-	
-	SetSizerAndFit (sizer);
+
+	set_editable (true);
+
+	SetSizerAndFit (_sizer);
 }
 
 void
@@ -88,6 +97,8 @@ Timecode::set (Time t, int fps)
 	checked_set (_minutes, lexical_cast<string> (m));
 	checked_set (_seconds, lexical_cast<string> (s));
 	checked_set (_frames, lexical_cast<string> (f));
+
+	_fixed->SetLabel (wxString::Format ("%02d:%02d:%02d.%02d", h, m, s, f));
 }
 
 Time
@@ -118,3 +129,11 @@ Timecode::set_clicked ()
 	Changed ();
 	_set_button->Enable (false);
 }
+
+void
+Timecode::set_editable (bool e)
+{
+	_editable->Show (e);
+	_fixed->Show (!e);
+	_sizer->Layout ();
+}
diff --git a/src/wx/timecode.h b/src/wx/timecode.h
index bebfa3f5f..5b094e39f 100644
--- a/src/wx/timecode.h
+++ b/src/wx/timecode.h
@@ -29,16 +29,22 @@ public:
 	void set (Time, int);
 	Time get (int) const;
 
+	void set_editable (bool);
+
 	boost::signals2::signal<void ()> Changed;
 
 private:
 	void changed ();
 	void set_clicked ();
 	
+	wxSizer* _sizer;
+	wxPanel* _editable;
 	wxTextCtrl* _hours;
+	wxStaticText* _hours_label;
 	wxTextCtrl* _minutes;
 	wxTextCtrl* _seconds;
 	wxTextCtrl* _frames;
 	wxButton* _set_button;
+	wxStaticText* _fixed;
 };
 
diff --git a/src/wx/timing_panel.cc b/src/wx/timing_panel.cc
index ee62797e3..313543d2d 100644
--- a/src/wx/timing_panel.cc
+++ b/src/wx/timing_panel.cc
@@ -24,6 +24,7 @@
 #include "timecode.h"
 #include "film_editor.h"
 
+using std::cout;
 using boost::shared_ptr;
 using boost::dynamic_pointer_cast;
 
@@ -60,6 +61,8 @@ TimingPanel::film_content_changed (shared_ptr<Content> content, int property)
 			_length->set (0, 24);
 		}
 	}
+
+	_length->set_editable (dynamic_pointer_cast<StillImageContent> (content));
 }
 
 void
-- 
2.30.2