From: Carl Hetherington <cth@carlh.net>
Date: Mon, 22 Nov 2021 19:47:14 +0000 (+0100)
Subject: Fill in zeros if a part of a timecode is entered.
X-Git-Tag: v2.15.177
X-Git-Url: https://main.carlh.net/gitweb/?p=dcpomatic.git;a=commitdiff_plain;h=121988b23c485ccb5ac8220c1776d10cb33e0db7

Fill in zeros if a part of a timecode is entered.

It looks better if zeros are added elsewhere when you, for example,
put a number into the seconds box and click "set".
---

diff --git a/src/wx/timecode.cc b/src/wx/timecode.cc
index 0e798bb3a..0230d665d 100644
--- a/src/wx/timecode.cc
+++ b/src/wx/timecode.cc
@@ -105,7 +105,7 @@ TimecodeBase::clear ()
 void
 TimecodeBase::changed ()
 {
-	if (_set_button) {
+	if (_set_button && !_ignore_changed) {
 		_set_button->Enable (true);
 	}
 }
@@ -117,6 +117,21 @@ TimecodeBase::set_clicked ()
 	if (_set_button) {
 		_set_button->Enable (false);
 	}
+
+	_ignore_changed = true;
+	if (_hours->GetValue().IsEmpty()) {
+		_hours->SetValue(wxT("0"));
+	}
+	if (_minutes->GetValue().IsEmpty()) {
+		_minutes->SetValue(wxT("0"));
+	}
+	if (_seconds->GetValue().IsEmpty()) {
+		_seconds->SetValue(wxT("0"));
+	}
+	if (_frames->GetValue().IsEmpty()) {
+		_frames->SetValue(wxT("0"));
+	}
+	_ignore_changed = false;
 }
 
 void
diff --git a/src/wx/timecode.h b/src/wx/timecode.h
index ccab0ecfc..c31a6740c 100644
--- a/src/wx/timecode.h
+++ b/src/wx/timecode.h
@@ -54,6 +54,8 @@ protected:
 	wxTextCtrl* _frames;
 	wxButton* _set_button;
 	wxStaticText* _fixed;
+
+	bool _ignore_changed = false;
 };
 
 template <class T>