Tidy up; rename add_label_to_grid_bag_sizer -> add_label_to_sizer.

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

diff --git a/src/wx/subtitle_panel.cc b/src/wx/subtitle_panel.cc
index f3083b194bb4a38c57c32b44f68ce588164f0441..e9de4fccd0bcca0e18c6761beea9974c3a21df84 100644 (file)
--- a/src/wx/subtitle_panel.cc
+++ b/src/wx/subtitle_panel.cc
@@ -36,6 +36,7 @@
 
 using std::vector;
 using std::string;
+using std::list;
 using boost::shared_ptr;
 using boost::lexical_cast;
 using boost::dynamic_pointer_cast;
@@ -45,72 +46,83 @@ SubtitlePanel::SubtitlePanel (ContentPanel* p)
        , _subtitle_view (0)
        , _fonts_dialog (0)
 {
-       wxFlexGridSizer* grid = new wxFlexGridSizer (2, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
+       wxGridBagSizer* grid = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
        _sizer->Add (grid, 0, wxALL, 8);
+       int r = 0;
 
        _reference = new wxCheckBox (this, wxID_ANY, _("Refer to existing DCP"));
-       grid->Add (_reference);
-       grid->AddSpacer (0);
+       grid->Add (_reference, wxGBPosition (r, 0), wxGBSpan (1, 2));
+       ++r;
 
        _use = new wxCheckBox (this, wxID_ANY, _("Use subtitles"));
-       grid->Add (_use);
-       grid->AddSpacer (0);
+       grid->Add (_use, wxGBPosition (r, 0), wxGBSpan (1, 2));
+       ++r;
 
        _burn = new wxCheckBox (this, wxID_ANY, _("Burn subtitles into image"));
-       grid->Add (_burn);
-       grid->AddSpacer (0);
+       grid->Add (_burn, wxGBPosition (r, 0), wxGBSpan (1, 2));
+       ++r;
 
        {
-               add_label_to_sizer (grid, this, _("X Offset"), true);
+               add_label_to_sizer (grid, this, _("X Offset"), true, wxGBPosition (r, 0));
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _x_offset = new wxSpinCtrl (this);
                s->Add (_x_offset);
                add_label_to_sizer (s, this, _("%"), false);
-               grid->Add (s);
+               grid->Add (s, wxGBPosition (r, 1));
+               ++r;
        }
 
        {
-               add_label_to_sizer (grid, this, _("Y Offset"), true);
+               add_label_to_sizer (grid, this, _("Y Offset"), true, wxGBPosition (r, 0));
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _y_offset = new wxSpinCtrl (this);
                s->Add (_y_offset);
                add_label_to_sizer (s, this, _("%"), false);
-               grid->Add (s);
+               grid->Add (s, wxGBPosition (r, 1));
+               ++r;
        }
 
        {
-               add_label_to_sizer (grid, this, _("X Scale"), true);
+               add_label_to_sizer (grid, this, _("X Scale"), true, wxGBPosition (r, 0));
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _x_scale = new wxSpinCtrl (this);
                s->Add (_x_scale);
                add_label_to_sizer (s, this, _("%"), false);
-               grid->Add (s);
+               grid->Add (s, wxGBPosition (r, 1));
+               ++r;
        }
 
        {
-               add_label_to_sizer (grid, this, _("Y Scale"), true);
+               add_label_to_sizer (grid, this, _("Y Scale"), true, wxGBPosition (r, 0));
                wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
                _y_scale = new wxSpinCtrl (this);
                s->Add (_y_scale);
                add_label_to_sizer (s, this, _("%"), false);
-               grid->Add (s);
+               grid->Add (s, wxGBPosition (r, 1));
+               ++r;
        }
 
-       add_label_to_sizer (grid, this, _("Language"), true);
+       add_label_to_sizer (grid, this, _("Language"), true, wxGBPosition (r, 0));
        _language = new wxTextCtrl (this, wxID_ANY);
-       grid->Add (_language, 1, wxEXPAND);
+       grid->Add (_language, wxGBPosition (r, 1));
+       ++r;
 
-       add_label_to_sizer (grid, this, _("Stream"), true);
+       add_label_to_sizer (grid, this, _("Stream"), true, wxGBPosition (r, 0));
        _stream = new wxChoice (this, wxID_ANY);
-       grid->Add (_stream, 1, wxEXPAND);
+       grid->Add (_stream, wxGBPosition (r, 1));
+       ++r;
 
-       _subtitle_view_button = new wxButton (this, wxID_ANY, _("View..."));
-       grid->Add (_subtitle_view_button);
-       grid->AddSpacer (0);
+       {
+               wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
+
+               _subtitle_view_button = new wxButton (this, wxID_ANY, _("View..."));
+               s->Add (_subtitle_view_button, 1, wxALL, DCPOMATIC_SIZER_GAP);
+               _fonts_dialog_button = new wxButton (this, wxID_ANY, _("Fonts..."));
+               s->Add (_fonts_dialog_button, 1, wxALL, DCPOMATIC_SIZER_GAP);
 
-       _fonts_dialog_button = new wxButton (this, wxID_ANY, _("Fonts..."));
-       grid->Add (_fonts_dialog_button);
-       grid->AddSpacer (0);
+               grid->Add (s, wxGBPosition (r, 0), wxGBSpan (1, 2));
+               ++r;
+       }
 
        _x_offset->SetRange (-100, 100);
        _y_offset->SetRange (-100, 100);
@@ -133,7 +145,7 @@ SubtitlePanel::SubtitlePanel (ContentPanel* p)
 void
 SubtitlePanel::film_changed (Film::Property property)
 {
-       if (property == Film::CONTENT) {
+       if (property == Film::CONTENT || property == Film::REEL_TYPE) {
                setup_sensitivity ();
        }
 }
@@ -219,7 +231,8 @@ SubtitlePanel::setup_sensitivity ()
        int ffmpeg_subs = 0;
        int subrip_or_dcp_subs = 0;
        int image_subs = 0;
-       BOOST_FOREACH (shared_ptr<SubtitleContent> i, _parent->selected_subtitle ()) {
+       SubtitleContentList sel = _parent->selected_subtitle ();
+       BOOST_FOREACH (shared_ptr<SubtitleContent> i, sel) {
                shared_ptr<const FFmpegContent> fc = boost::dynamic_pointer_cast<const FFmpegContent> (i);
                shared_ptr<const SubRipContent> sc = boost::dynamic_pointer_cast<const SubRipContent> (i);
                shared_ptr<const DCPSubtitleContent> dsc = boost::dynamic_pointer_cast<const DCPSubtitleContent> (i);
@@ -242,6 +255,24 @@ SubtitlePanel::setup_sensitivity ()
                }
        }
 
+       shared_ptr<DCPContent> dcp;
+       if (sel.size() == 1) {
+               dcp = dynamic_pointer_cast<DCPContent> (sel.front ());
+       }
+
+       list<string> why_not;
+       bool const can_reference = dcp && dcp->can_reference_subtitle (why_not);
+       _reference->Enable (can_reference);
+
+       wxString s;
+       if (!can_reference) {
+               s = _("Cannot reference this DCP.  ");
+               BOOST_FOREACH (string i, why_not) {
+                       s += std_to_wx(i) + wxT("  ");
+               }
+       }
+       _reference->SetToolTip (s);
+
        bool const reference = _reference->GetValue ();
 
        _use->Enable (!reference && any_subs > 0);
@@ -255,9 +286,6 @@ SubtitlePanel::setup_sensitivity ()
        _stream->Enable (!reference && ffmpeg_subs == 1);
        _subtitle_view_button->Enable (!reference && subrip_or_dcp_subs == 1);
        _fonts_dialog_button->Enable (!reference && subrip_or_dcp_subs == 1);
-
-       ContentList sel = _parent->selected ();
-       _reference->Enable (sel.size() == 1 && dynamic_pointer_cast<DCPContent> (sel.front ()));
 }
 
 void