Untested use of Frame for video/audio content lengths.
[dcpomatic.git] / src / wx / timing_panel.cc
index 716a27b66d70f9474e40fd5f51695011e0b25068..74c1756a5205221edc15dbec4fd585a07d833961 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012-2013 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2015 Carl Hetherington <cth@carlh.net>
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
 
 #include "lib/content.h"
 #include "lib/image_content.h"
+#include "lib/raw_convert.h"
 #include "timing_panel.h"
 #include "wx_util.h"
 #include "timecode.h"
 #include "content_panel.h"
-#include <dcp/raw_convert.h>
 #include <set>
 
 using std::cout;
@@ -31,7 +31,6 @@ using std::string;
 using std::set;
 using boost::shared_ptr;
 using boost::dynamic_pointer_cast;
-using dcp::raw_convert;
 
 TimingPanel::TimingPanel (ContentPanel* p)
        /* horrid hack for apparent lack of context support with wxWidgets i18n code */
@@ -40,6 +39,40 @@ TimingPanel::TimingPanel (ContentPanel* p)
        wxFlexGridSizer* grid = new wxFlexGridSizer (2, 4, 4);
        _sizer->Add (grid, 0, wxALL, 8);
 
+       wxSize size = TimecodeBase::size (this);
+               
+       wxSizer* labels = new wxBoxSizer (wxHORIZONTAL);
+       //// TRANSLATORS: this is an abbreviation for "hours"
+       wxStaticText* t = new wxStaticText (this, wxID_ANY, _("h"), wxDefaultPosition, size, wxALIGN_CENTRE_HORIZONTAL);
+#ifdef DCPOMATIC_LINUX
+       /* Hack to work around failure to centre text on GTK */
+       gtk_label_set_line_wrap (GTK_LABEL (t->GetHandle()), FALSE);
+#endif         
+       labels->Add (t, 1, wxEXPAND);
+       add_label_to_sizer (labels, this, wxT (":"), false);
+       //// TRANSLATORS: this is an abbreviation for "minutes"
+       t = new wxStaticText (this, wxID_ANY, _("m"), wxDefaultPosition, size, wxALIGN_CENTRE_HORIZONTAL);
+#ifdef DCPOMATIC_LINUX
+       gtk_label_set_line_wrap (GTK_LABEL (t->GetHandle()), FALSE);
+#endif         
+       labels->Add (t, 1, wxEXPAND);
+       add_label_to_sizer (labels, this, wxT (":"), false);
+       //// TRANSLATORS: this is an abbreviation for "seconds"
+       t = new wxStaticText (this, wxID_ANY, _("s"), wxDefaultPosition, size, wxALIGN_CENTRE_HORIZONTAL);
+#ifdef DCPOMATIC_LINUX
+       gtk_label_set_line_wrap (GTK_LABEL (t->GetHandle()), FALSE);
+#endif         
+       labels->Add (t, 1, wxEXPAND);
+       add_label_to_sizer (labels, this, wxT (":"), false);
+       //// TRANSLATORS: this is an abbreviation for "frames"
+       t = new wxStaticText (this, wxID_ANY, _("f"), wxDefaultPosition, size, wxALIGN_CENTRE_HORIZONTAL);
+#ifdef DCPOMATIC_LINUX
+       gtk_label_set_line_wrap (GTK_LABEL (t->GetHandle()), FALSE);
+#endif         
+       labels->Add (t, 1, wxEXPAND);
+       grid->Add (new wxStaticText (this, wxID_ANY, wxT ("")));
+       grid->Add (labels);
+
        add_label_to_sizer (grid, this, _("Position"), true);
        _position = new Timecode<DCPTime> (this);
        grid->Add (_position);
@@ -67,6 +100,30 @@ TimingPanel::TimingPanel (ContentPanel* p)
                grid->Add (s, 1, wxEXPAND);
        }
 
+       grid->AddSpacer (0);
+
+       /* We can't use Wrap() here as it doesn't work with markup:
+        * http://trac.wxwidgets.org/ticket/13389
+        */
+
+       wxString in = _("<i>Only change this if it the content's frame rate has been read incorrectly.</i>");
+       wxString out;
+       int const width = 20;
+       int current = 0;
+       for (size_t i = 0; i < in.Length(); ++i) {
+               if (in[i] == ' ' && current >= width) {
+                       out += '\n';
+                       current = 0;
+               } else {
+                       out += in[i];
+                       ++current;
+               }
+       }
+       
+       t = new wxStaticText (this, wxID_ANY, wxT (""));
+       t->SetLabelMarkup (out);
+       grid->Add (t, 0, wxALIGN_CENTER_VERTICAL | wxLEFT | wxRIGHT, 6);
+
        _position->Changed.connect    (boost::bind (&TimingPanel::position_changed, this));
        _full_length->Changed.connect (boost::bind (&TimingPanel::full_length_changed, this));
        _trim_start->Changed.connect  (boost::bind (&TimingPanel::trim_start_changed, this));
@@ -76,6 +133,40 @@ TimingPanel::TimingPanel (ContentPanel* p)
        _set_video_frame_rate->Bind   (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&TimingPanel::set_video_frame_rate, this));
 }
 
+void
+TimingPanel::update_full_length ()
+{
+       ContentList cl = _parent->selected ();
+
+       set<DCPTime> check;
+       for (ContentList::const_iterator i = cl.begin (); i != cl.end(); ++i) {
+               check.insert ((*i)->full_length ());
+       }
+       
+       if (check.size() == 1) {
+               _full_length->set (cl.front()->full_length (), _parent->film()->video_frame_rate ());
+       } else {
+               _full_length->clear ();
+       }
+}
+
+void
+TimingPanel::update_play_length ()
+{
+       ContentList cl = _parent->selected ();
+
+       set<DCPTime> check;
+       for (ContentList::const_iterator i = cl.begin (); i != cl.end(); ++i) {
+               check.insert ((*i)->length_after_trim ());
+       }
+       
+       if (check.size() == 1) {
+               _play_length->set (cl.front()->length_after_trim (), _parent->film()->video_frame_rate ());
+       } else {
+               _play_length->clear ();
+       }
+}
+
 void
 TimingPanel::film_content_changed (int property)
 {
@@ -105,16 +196,7 @@ TimingPanel::film_content_changed (int property)
                property == VideoContentProperty::VIDEO_FRAME_TYPE
                ) {
 
-               set<DCPTime> check;
-               for (ContentList::const_iterator i = cl.begin (); i != cl.end(); ++i) {
-                       check.insert ((*i)->full_length ());
-               }
-               
-               if (check.size() == 1) {
-                       _full_length->set (cl.front()->full_length (), film_video_frame_rate);
-               } else {
-                       _full_length->clear ();
-               }
+               update_full_length ();
 
        } else if (property == ContentProperty::TRIM_START) {
 
@@ -151,16 +233,7 @@ TimingPanel::film_content_changed (int property)
                property == VideoContentProperty::VIDEO_FRAME_TYPE
                ) {
 
-               set<DCPTime> check;
-               for (ContentList::const_iterator i = cl.begin (); i != cl.end(); ++i) {
-                       check.insert ((*i)->length_after_trim ());
-               }
-               
-               if (check.size() == 1) {
-                       _play_length->set (cl.front()->length_after_trim (), film_video_frame_rate);
-               } else {
-                       _play_length->clear ();
-               }
+               update_play_length ();
        }
 
        if (property == VideoContentProperty::VIDEO_FRAME_RATE) {
@@ -174,10 +247,10 @@ TimingPanel::film_content_changed (int property)
                        }
                }
                if (check.size() == 1) {
-                       _video_frame_rate->SetValue (std_to_wx (raw_convert<string> (vc->video_frame_rate (), 5)));
+                       checked_set (_video_frame_rate, raw_convert<string> (vc->video_frame_rate (), 5));
                        _video_frame_rate->Enable (true);
                } else {
-                       _video_frame_rate->SetValue ("");
+                       checked_set (_video_frame_rate, wxT (""));
                        _video_frame_rate->Enable (false);
                }
        }
@@ -211,8 +284,8 @@ TimingPanel::full_length_changed ()
        for (ContentList::iterator i = c.begin(); i != c.end(); ++i) {
                shared_ptr<ImageContent> ic = dynamic_pointer_cast<ImageContent> (*i);
                if (ic && ic->still ()) {
-                       /* XXX: No effective FRC here... is this right? */
-                       ic->set_video_length (ContentTime (_full_length->get (_parent->film()->video_frame_rate()), FrameRateChange (1, 1)));
+                       int const vfr = _parent->film()->video_frame_rate ();
+                       ic->set_video_length (_full_length->get (vfr).frames (vfr));
                }
        }
 }
@@ -282,3 +355,12 @@ TimingPanel::content_selection_changed ()
        film_content_changed (ContentProperty::TRIM_END);
        film_content_changed (VideoContentProperty::VIDEO_FRAME_RATE);
 }
+
+void
+TimingPanel::film_changed (Film::Property p)
+{
+       if (p == Film::VIDEO_FRAME_RATE) {
+               update_full_length ();
+               update_play_length ();
+       }
+}