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

[dcpomatic.git] / src / wx / timecode.cc

diff --git a/src/wx/timecode.cc b/src/wx/timecode.cc
index 0602358a9788083ac2b7fcfa58b160bb2b379037..0230d665d10cc8bc29cf917ab280522255f75311 100644 (file)
--- a/src/wx/timecode.cc
+++ b/src/wx/timecode.cc
@@ -31,7 +31,7 @@ TimecodeBase::TimecodeBase (wxWindow* parent, bool set_button)
        : wxPanel (parent)
        , _set_button (0)
 {
-       wxSize const s = TimecodeBase::size (parent);
+       auto const s = TimecodeBase::size (parent);
 
        wxTextValidator validator (wxFILTER_INCLUDE_CHAR_LIST);
        wxArrayString list;
@@ -46,7 +46,7 @@ TimecodeBase::TimecodeBase (wxWindow* parent, bool set_button)
        _sizer = new wxBoxSizer (wxHORIZONTAL);
 
        _editable = new wxPanel (this);
-       wxSizer* editable_sizer = new wxBoxSizer (wxHORIZONTAL);
+       auto editable_sizer = new wxBoxSizer (wxHORIZONTAL);
        _hours = new wxTextCtrl (_editable, wxID_ANY, wxT(""), wxDefaultPosition, s, 0, validator);
        _hours->SetMaxLength (2);
        editable_sizer->Add (_hours);
@@ -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
@@ -131,7 +146,11 @@ wxSize
 TimecodeBase::size (wxWindow* parent)
 {
        wxClientDC dc (parent);
-       wxSize size = dc.GetTextExtent (wxT ("9999"));
+#ifdef DCPOMATIC_OSX
+       auto size = dc.GetTextExtent(wxT("999"));
+#else
+       auto size = dc.GetTextExtent(wxT("99999"));
+#endif
        size.SetHeight (-1);
        return size;
 }