Fix assorted GTK3-related alignment problems.
[dcpomatic.git] / src / wx / kdm_timing_panel.cc
index 02b4a94261f4bd43ff159223589b303bbbe0d208..ae31e36c37066565a81c833fa58e3b99f1aeb5dd 100644 (file)
 /*
-    Copyright (C) 2015 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2015-2020 Carl Hetherington <cth@carlh.net>
 
-    This program is free software; you can redistribute it and/or modify
+    This file is part of DCP-o-matic.
+
+    DCP-o-matic is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
     the Free Software Foundation; either version 2 of the License, or
     (at your option) any later version.
 
-    This program is distributed in the hope that it will be useful,
+    DCP-o-matic is distributed in the hope that it will be useful,
     but WITHOUT ANY WARRANTY; without even the implied warranty of
     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
     GNU General Public License for more details.
 
     You should have received a copy of the GNU General Public License
-    along with this program; if not, write to the Free Software
-    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+    along with DCP-o-matic.  If not, see <http://www.gnu.org/licenses/>.
 
 */
 
 #include "kdm_timing_panel.h"
 #include "wx_util.h"
+#include "time_picker.h"
+#include "static_text.h"
+#include "lib/warnings.h"
+DCPOMATIC_DISABLE_WARNINGS
 #include <wx/datectrl.h>
-#include <wx/timectrl.h>
+#include <wx/dateevt.h>
+DCPOMATIC_ENABLE_WARNINGS
+
+using std::cout;
+using boost::bind;
 
 KDMTimingPanel::KDMTimingPanel (wxWindow* parent)
        : wxPanel (parent, wxID_ANY)
 {
-       wxFlexGridSizer* table = new wxFlexGridSizer (3, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
-       add_label_to_sizer (table, this, _("From"), true);
+       wxBoxSizer* overall_sizer = new wxBoxSizer (wxVERTICAL);
+
+#ifdef __WXGTK3__
+       /* wxDatePickerCtrl is too small with the GTK3 backend so we need to make it bigger with some fudge factors */
+       wxClientDC dc (parent);
+       wxSize size = dc.GetTextExtent (wxT("99/99/9999"));
+       size.SetWidth (size.GetWidth() * 1.75);
+       size.SetHeight (-1);
+#else
+       wxSize size = wxDefaultSize;
+#endif
+
+       wxSizer* table = new wxBoxSizer (wxHORIZONTAL);
+       add_label_to_sizer (table, this, _("From"), true, 0, wxLEFT | wxRIGHT | wxALIGN_CENTRE_VERTICAL);
        wxDateTime from;
        from.SetToCurrent ();
-       _from_date = new wxDatePickerCtrl (this, wxID_ANY, from);
-       table->Add (_from_date, 1, wxEXPAND);
-       _from_time = new wxTimePickerCtrl (this, wxID_ANY, from);
-       table->Add (_from_time, 1, wxEXPAND);
+       _from_date = new wxDatePickerCtrl (this, wxID_ANY, from, wxDefaultPosition, size);
+       table->Add (_from_date, 0, wxALIGN_CENTER_VERTICAL);
+
+#ifdef __WXGTK3__
+       _from_time = new TimePickerText (this, from);
+#else
+       _from_time = new TimePickerSpin (this, from);
+#endif
 
-       add_label_to_sizer (table, this, _("Until"), true);
+#ifdef DCPOMATIC_OSX
+       /* Hack to tweak alignment, which I can't get right by "proper" means for some reason */
+       table->Add (_from_time, 0, wxALIGN_CENTER_VERTICAL | wxTOP, 4);
+#else
+       table->Add (_from_time, 0, wxALIGN_CENTER_VERTICAL);
+#endif
+
+       add_label_to_sizer (table, this, _("until"), true, 0, wxLEFT | wxRIGHT | wxALIGN_CENTRE_VERTICAL);
        wxDateTime to = from;
        /* 1 week from now */
        to.Add (wxDateSpan (0, 0, 1, 0));
-       _until_date = new wxDatePickerCtrl (this, wxID_ANY, to);
-       table->Add (_until_date, 1, wxEXPAND);
-       _until_time = new wxTimePickerCtrl (this, wxID_ANY, to);
-       table->Add (_until_time, 1, wxEXPAND);
+       _until_date = new wxDatePickerCtrl (this, wxID_ANY, to, wxDefaultPosition, size);
+       table->Add (_until_date, 0, wxALIGN_CENTER_VERTICAL);
+
+#ifdef __WXGTK3__
+       _until_time = new TimePickerText (this, to);
+#else
+       _until_time = new TimePickerSpin (this, to);
+#endif
+
+#ifdef DCPOMATIC_OSX
+       table->Add (_until_time, 0, wxALIGN_CENTER_VERTICAL | wxTOP, 4);
+#else
+       table->Add (_until_time, 0, wxALIGN_CENTER_VERTICAL);
+#endif
+
+       overall_sizer->Add (table);
+
+       _warning = new StaticText (this, wxT(""));
+       overall_sizer->Add (_warning, 0, wxTOP, DCPOMATIC_SIZER_GAP);
+       wxFont font = _warning->GetFont();
+       font.SetStyle(wxFONTSTYLE_ITALIC);
+       font.SetPointSize(font.GetPointSize() - 1);
+       _warning->SetForegroundColour (wxColour (255, 0, 0));
+       _warning->SetFont(font);
 
-       SetSizer (table);
+       /* I said I've been to the year 3000.  Not much has changed but they live underwater.  And your In-in-in-interop DCP
+           is pretty fine.
+        */
+       _from_date->SetRange(wxDateTime(1, wxDateTime::Jan, 1900, 0, 0, 0, 0), wxDateTime(31, wxDateTime::Dec, 3000, 0, 0, 0, 0));
+       _until_date->SetRange(wxDateTime(1, wxDateTime::Jan, 1900, 0, 0, 0, 0), wxDateTime(31, wxDateTime::Dec, 3000, 0, 0, 0, 0));
+
+       _from_date->Bind (wxEVT_DATE_CHANGED, bind (&KDMTimingPanel::changed, this));
+       _until_date->Bind (wxEVT_DATE_CHANGED, bind (&KDMTimingPanel::changed, this));
+       _from_time->Changed.connect (bind (&KDMTimingPanel::changed, this));
+       _until_time->Changed.connect (bind (&KDMTimingPanel::changed, this));
+
+       SetSizer (overall_sizer);
 }
 
 boost::posix_time::ptime
@@ -53,13 +116,12 @@ KDMTimingPanel::from () const
 }
 
 boost::posix_time::ptime
-KDMTimingPanel::posix_time (wxDatePickerCtrl* date_picker, wxTimePickerCtrl* time_picker)
+KDMTimingPanel::posix_time (wxDatePickerCtrl* date_picker, TimePicker* time_picker)
 {
        wxDateTime const date = date_picker->GetValue ();
-       wxDateTime const time = time_picker->GetValue ();
        return boost::posix_time::ptime (
                boost::gregorian::date (date.GetYear(), date.GetMonth() + 1, date.GetDay()),
-               boost::posix_time::time_duration (time.GetHour(), time.GetMinute(), time.GetSecond())
+               boost::posix_time::time_duration (time_picker->hours(), time_picker->minutes(), 0)
                );
 }
 
@@ -68,3 +130,21 @@ KDMTimingPanel::until () const
 {
        return posix_time (_until_date, _until_time);
 }
+
+bool
+KDMTimingPanel::valid () const
+{
+       return until() > from();
+}
+
+void
+KDMTimingPanel::changed () const
+{
+       if (valid ()) {
+               _warning->SetLabel (wxT (""));
+       } else {
+               _warning->SetLabel (_("The 'until' time must be after the 'from' time."));
+       }
+
+       TimingChanged ();
+}