/*
- Copyright (C) 2012-2018 Carl Hetherington <cth@carlh.net>
+ Copyright (C) 2012-2019 Carl Hetherington <cth@carlh.net>
This file is part of DCP-o-matic.
#include "text_view.h"
#include "content_panel.h"
#include "fonts_dialog.h"
+#include "dcp_text_track_dialog.h"
#include "subtitle_appearance_dialog.h"
+#include "static_text.h"
+#include "check_box.h"
+#include "dcpomatic_button.h"
#include "lib/ffmpeg_content.h"
#include "lib/string_text_file_content.h"
#include "lib/ffmpeg_subtitle_stream.h"
using std::list;
using std::cout;
using boost::shared_ptr;
+using boost::optional;
using boost::dynamic_pointer_cast;
+/** @param t Original text type of the content, if known */
TextPanel::TextPanel (ContentPanel* p, TextType t)
: ContentSubPanel (p, std_to_wx(text_type_to_name(t)))
+ , _dcp_track_label (0)
+ , _dcp_track (0)
+ , _language_label (0)
+ , _language (0)
, _text_view (0)
, _fonts_dialog (0)
, _original_type (t)
{
- wxBoxSizer* reference_sizer = new wxBoxSizer (wxVERTICAL);
-
- _reference = new wxCheckBox (this, wxID_ANY, _("Use this DCP's subtitle as OV and make VF"));
- reference_sizer->Add (_reference, 0, wxLEFT | wxRIGHT | wxTOP, DCPOMATIC_SIZER_GAP);
+ wxString refer = _("Use this DCP's subtitle as OV and make VF");
+ if (t == TEXT_CLOSED_CAPTION) {
+ refer = _("Use this DCP's closed caption as OV and make VF");
+ }
- _reference_note = new wxStaticText (this, wxID_ANY, _(""));
+ _reference = new CheckBox (this, refer);
+ _reference_note = new StaticText (this, wxT(""));
_reference_note->Wrap (200);
- reference_sizer->Add (_reference_note, 0, wxLEFT | wxRIGHT, DCPOMATIC_SIZER_GAP);
wxFont font = _reference_note->GetFont();
font.SetStyle(wxFONTSTYLE_ITALIC);
font.SetPointSize(font.GetPointSize() - 1);
_reference_note->SetFont(font);
- _sizer->Add (reference_sizer);
-
- wxGridBagSizer* grid = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
- _sizer->Add (grid, 0, wxALL, 8);
- int r = 0;
-
- wxBoxSizer* use = new wxBoxSizer (wxHORIZONTAL);
- _use = new wxCheckBox (this, wxID_ANY, _("Use as"));
- use->Add (_use, 0, wxEXPAND | wxRIGHT, DCPOMATIC_SIZER_GAP);
+ _use = new CheckBox (this, _("Use as"));
_type = new wxChoice (this, wxID_ANY);
_type->Append (_("open subtitles"));
_type->Append (_("closed captions"));
- use->Add (_type, 1, wxEXPAND, 0);
- grid->Add (use, wxGBPosition (r, 0), wxGBSpan (1, 2));
- ++r;
- _burn = new wxCheckBox (this, wxID_ANY, _("Burn subtitles into image"));
- grid->Add (_burn, wxGBPosition (r, 0), wxGBSpan (1, 2));
- ++r;
+ _burn = new CheckBox (this, _("Burn subtitles into image"));
+
+ _offset_label = create_label (this, _("Offset"), true);
+ _x_offset_label = create_label (this, _("X"), true);
+ _x_offset = new wxSpinCtrl (this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize(64, -1));
+ _x_offset_pc_label = new StaticText (this, _("%"));
+ _y_offset_label = create_label (this, _("Y"), true);
+ _y_offset = new wxSpinCtrl (this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize(64, -1));
+ _y_offset_pc_label = new StaticText (this, _("%"));
+
+ _scale_label = create_label (this, _("Scale"), true);
+ _x_scale_label = create_label (this, _("X"), true);
+ _x_scale = new wxSpinCtrl (this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize(64, -1));
+ _x_scale_pc_label = new StaticText (this, _("%"));
+ _y_scale_label = create_label (this, S_("Coord|Y"), true);
+ _y_scale = new wxSpinCtrl (this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize(64, -1));
+ _y_scale_pc_label = new StaticText (this, _("%"));
+
+ _line_spacing_label = create_label (this, _("Line spacing"), true);
+ _line_spacing = new wxSpinCtrl (this);
+ _line_spacing_pc_label = new StaticText (this, _("%"));
+
+ _stream_label = create_label (this, _("Stream"), true);
+ _stream = new wxChoice (this, wxID_ANY);
- {
- 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, wxGBPosition (r, 1));
- ++r;
- }
+ _text_view_button = new Button (this, _("View..."));
+ _fonts_dialog_button = new Button (this, _("Fonts..."));
+ _appearance_dialog_button = new Button (this, _("Appearance..."));
- {
- 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, wxGBPosition (r, 1));
- ++r;
- }
+ _x_offset->SetRange (-100, 100);
+ _y_offset->SetRange (-100, 100);
+ _x_scale->SetRange (10, 1000);
+ _y_scale->SetRange (10, 1000);
+ _line_spacing->SetRange (10, 1000);
- {
- 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, wxGBPosition (r, 1));
- ++r;
+ _reference->Bind (wxEVT_CHECKBOX, boost::bind (&TextPanel::reference_clicked, this));
+ _use->Bind (wxEVT_CHECKBOX, boost::bind (&TextPanel::use_toggled, this));
+ _type->Bind (wxEVT_CHOICE, boost::bind (&TextPanel::type_changed, this));
+ _burn->Bind (wxEVT_CHECKBOX, boost::bind (&TextPanel::burn_toggled, this));
+ _x_offset->Bind (wxEVT_SPINCTRL, boost::bind (&TextPanel::x_offset_changed, this));
+ _y_offset->Bind (wxEVT_SPINCTRL, boost::bind (&TextPanel::y_offset_changed, this));
+ _x_scale->Bind (wxEVT_SPINCTRL, boost::bind (&TextPanel::x_scale_changed, this));
+ _y_scale->Bind (wxEVT_SPINCTRL, boost::bind (&TextPanel::y_scale_changed, this));
+ _line_spacing->Bind (wxEVT_SPINCTRL, boost::bind (&TextPanel::line_spacing_changed, this));
+ _stream->Bind (wxEVT_CHOICE, boost::bind (&TextPanel::stream_changed, this));
+ _text_view_button->Bind (wxEVT_BUTTON, boost::bind (&TextPanel::text_view_clicked, this));
+ _fonts_dialog_button->Bind (wxEVT_BUTTON, boost::bind (&TextPanel::fonts_dialog_clicked, this));
+ _appearance_dialog_button->Bind (wxEVT_BUTTON, boost::bind (&TextPanel::appearance_dialog_clicked, this));
+
+ add_to_grid();
+ content_selection_changed ();
+}
+
+void
+TextPanel::setup_visibility ()
+{
+ switch (current_type()) {
+ case TEXT_OPEN_SUBTITLE:
+ if (_dcp_track_label) {
+ _dcp_track_label->Destroy ();
+ _dcp_track_label = 0;
+ }
+ if (_dcp_track) {
+ _dcp_track->Destroy ();
+ _dcp_track = 0;
+ }
+ if (!_language_label) {
+ _language_label = create_label (this, _("Language"), true);
+ add_label_to_sizer (_grid, _language_label, true, wxGBPosition(_language_row, 0));
+ }
+ if (!_language) {
+ _language = new wxTextCtrl (this, wxID_ANY);
+ _language->Bind (wxEVT_TEXT, boost::bind(&TextPanel::language_changed, this));
+ _grid->Add (_language, wxGBPosition(_language_row, 1), wxDefaultSpan, wxEXPAND);
+ film_content_changed (TextContentProperty::LANGUAGE);
+ }
+ break;
+ case TEXT_CLOSED_CAPTION:
+ if (_language_label) {
+ _language_label->Destroy ();
+ _language_label = 0;
+ }
+ if (_language) {
+ _language->Destroy ();
+ _language = 0;
+ }
+ if (!_dcp_track_label) {
+ _dcp_track_label = create_label (this, _("CCAP track"), true);
+ add_label_to_sizer (_grid, _dcp_track_label, true, wxGBPosition(_language_row, 0));
+ }
+ if (!_dcp_track) {
+ _dcp_track = new wxChoice (this, wxID_ANY);
+ _dcp_track->Bind (wxEVT_CHOICE, boost::bind(&TextPanel::dcp_track_changed, this));
+ _grid->Add (_dcp_track, wxGBPosition(_language_row, 1), wxDefaultSpan, wxEXPAND);
+ update_dcp_tracks ();
+ film_content_changed (TextContentProperty::DCP_TRACK);
+ }
+ break;
+ default:
+ break;
}
- {
- 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, wxGBPosition (r, 1));
+ _grid->Layout ();
+}
+
+void
+TextPanel::add_to_grid ()
+{
+ Config::Interface const interface = Config::instance()->interface_complexity();
+
+ int r = 0;
+
+ _reference->Show (interface == Config::INTERFACE_FULL);
+ _reference_note->Show (interface == Config::INTERFACE_FULL);
+
+ if (interface == Config::INTERFACE_FULL) {
+ wxBoxSizer* reference_sizer = new wxBoxSizer (wxVERTICAL);
+ reference_sizer->Add (_reference, 0);
+ reference_sizer->Add (_reference_note, 0);
+ _grid->Add (reference_sizer, wxGBPosition(r, 0), wxGBSpan(1, 4));
++r;
}
+ wxBoxSizer* use = new wxBoxSizer (wxHORIZONTAL);
+ use->Add (_use, 0, wxEXPAND | wxRIGHT, DCPOMATIC_SIZER_GAP);
+ use->Add (_type, 1, wxEXPAND, 0);
+ _grid->Add (use, wxGBPosition (r, 0), wxGBSpan (1, 2));
+ ++r;
+
+ _grid->Add (_burn, wxGBPosition (r, 0), wxGBSpan (1, 2));
+ ++r;
+
+ add_label_to_sizer (_grid, _offset_label, true, wxGBPosition (r, 0));
+ wxBoxSizer* offset = new wxBoxSizer (wxHORIZONTAL);
+ add_label_to_sizer (offset, _x_offset_label, true);
+ offset->Add (_x_offset, 0, wxRIGHT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_X_GAP);
+ offset->Add (_x_offset_pc_label, 0, wxRIGHT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_X_GAP * 2);
+ add_label_to_sizer (offset, _y_offset_label, true);
+ offset->Add (_y_offset, 0);
+ add_label_to_sizer (offset, _y_offset_pc_label, false);
+ _grid->Add (offset, wxGBPosition (r, 1));
+ ++r;
+
+ add_label_to_sizer (_grid, _scale_label, true, wxGBPosition (r, 0));
+ wxBoxSizer* scale = new wxBoxSizer (wxHORIZONTAL);
+ add_label_to_sizer (scale, _x_scale_label, true);
+ scale->Add (_x_scale, 0, wxRIGHT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_X_GAP);
+ scale->Add (_x_scale_pc_label, 0, wxRIGHT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_X_GAP * 2);
+ add_label_to_sizer (scale, _y_scale_label, true);
+ scale->Add (_y_scale, 0);
+ add_label_to_sizer (scale, _y_scale_pc_label, false);
+ _grid->Add (scale, wxGBPosition (r, 1));
+ ++r;
+
{
- add_label_to_sizer (grid, this, _("Line spacing"), true, wxGBPosition (r, 0));
+ add_label_to_sizer (_grid, _line_spacing_label, true, wxGBPosition (r, 0));
wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
- _line_spacing = new wxSpinCtrl (this);
s->Add (_line_spacing);
- add_label_to_sizer (s, this, _("%"), false);
- grid->Add (s, wxGBPosition (r, 1));
+ add_label_to_sizer (s, _line_spacing_pc_label, false);
+ _grid->Add (s, wxGBPosition (r, 1));
++r;
}
- add_label_to_sizer (grid, this, _("Language"), true, wxGBPosition (r, 0));
- _language = new wxTextCtrl (this, wxID_ANY);
- grid->Add (_language, wxGBPosition (r, 1));
+ _language_row = r;
+ setup_visibility ();
++r;
- add_label_to_sizer (grid, this, _("Stream"), true, wxGBPosition (r, 0));
- _stream = new wxChoice (this, wxID_ANY);
- grid->Add (_stream, wxGBPosition (r, 1));
+ add_label_to_sizer (_grid, _stream_label, true, wxGBPosition (r, 0));
+ _grid->Add (_stream, wxGBPosition (r, 1));
++r;
{
wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
- _text_view_button = new wxButton (this, wxID_ANY, _("View..."));
s->Add (_text_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);
- _appearance_dialog_button = new wxButton (this, wxID_ANY, _("Appearance..."));
s->Add (_appearance_dialog_button, 1, wxALL, DCPOMATIC_SIZER_GAP);
- grid->Add (s, wxGBPosition (r, 0), wxGBSpan (1, 2));
+ _grid->Add (s, wxGBPosition (r, 0), wxGBSpan (1, 2));
++r;
}
+}
- _x_offset->SetRange (-100, 100);
- _y_offset->SetRange (-100, 100);
- _x_scale->SetRange (10, 1000);
- _y_scale->SetRange (10, 1000);
- _line_spacing->SetRange (10, 1000);
+void
+TextPanel::update_dcp_track_selection ()
+{
+ DCPOMATIC_ASSERT (_dcp_track);
- _reference->Bind (wxEVT_CHECKBOX, boost::bind (&TextPanel::reference_clicked, this));
- _use->Bind (wxEVT_CHECKBOX, boost::bind (&TextPanel::use_toggled, this));
- _type->Bind (wxEVT_CHOICE, boost::bind (&TextPanel::type_changed, this));
- _burn->Bind (wxEVT_CHECKBOX, boost::bind (&TextPanel::burn_toggled, this));
- _x_offset->Bind (wxEVT_SPINCTRL, boost::bind (&TextPanel::x_offset_changed, this));
- _y_offset->Bind (wxEVT_SPINCTRL, boost::bind (&TextPanel::y_offset_changed, this));
- _x_scale->Bind (wxEVT_SPINCTRL, boost::bind (&TextPanel::x_scale_changed, this));
- _y_scale->Bind (wxEVT_SPINCTRL, boost::bind (&TextPanel::y_scale_changed, this));
- _line_spacing->Bind (wxEVT_SPINCTRL, boost::bind (&TextPanel::line_spacing_changed, this));
- _language->Bind (wxEVT_TEXT, boost::bind (&TextPanel::language_changed, this));
- _stream->Bind (wxEVT_CHOICE, boost::bind (&TextPanel::stream_changed, this));
- _text_view_button->Bind (wxEVT_BUTTON, boost::bind (&TextPanel::text_view_clicked, this));
- _fonts_dialog_button->Bind (wxEVT_BUTTON, boost::bind (&TextPanel::fonts_dialog_clicked, this));
- _appearance_dialog_button->Bind (wxEVT_BUTTON, boost::bind (&TextPanel::appearance_dialog_clicked, this));
+ optional<DCPTextTrack> selected;
+ bool many = false;
+ BOOST_FOREACH (shared_ptr<Content> i, _parent->selected_text()) {
+ shared_ptr<TextContent> t = i->text_of_original_type(_original_type);
+ if (t) {
+ optional<DCPTextTrack> dt = t->dcp_track();
+ if (dt && selected && *dt != *selected) {
+ many = true;
+ } else if (!selected) {
+ selected = dt;
+ }
+ }
+ }
+
+ int n = 0;
+ BOOST_FOREACH (DCPTextTrack i, _parent->film()->closed_caption_tracks()) {
+ if (!many && selected && *selected == i) {
+ _dcp_track->SetSelection (n);
+ }
+ ++n;
+ }
+
+ if (!selected || many) {
+ _dcp_track->SetSelection (wxNOT_FOUND);
+ }
+}
+
+void
+TextPanel::update_dcp_tracks ()
+{
+ DCPOMATIC_ASSERT (_dcp_track);
+
+ _dcp_track->Clear ();
+ BOOST_FOREACH (DCPTextTrack i, _parent->film()->closed_caption_tracks()) {
+ /* XXX: don't display the "magic" track which has empty name and language;
+ this is a nasty hack (see also Film::closed_caption_tracks)
+ */
+ if (!i.name.empty() || !i.language.empty()) {
+ _dcp_track->Append (std_to_wx(i.summary()));
+ }
+ }
+
+ if (_parent->film()->closed_caption_tracks().size() < 6) {
+ _dcp_track->Append (_("Add new..."));
+ }
+
+ update_dcp_track_selection ();
+}
+
+void
+TextPanel::dcp_track_changed ()
+{
+ optional<DCPTextTrack> track;
+
+ if (_dcp_track->GetSelection() == int(_dcp_track->GetCount()) - 1) {
+ DCPTextTrackDialog* d = new DCPTextTrackDialog (this);
+ if (d->ShowModal() == wxID_OK) {
+ track = d->get();
+ }
+ d->Destroy ();
+ } else {
+ /* Find the DCPTextTrack that was selected */
+ BOOST_FOREACH (DCPTextTrack i, _parent->film()->closed_caption_tracks()) {
+ if (i.summary() == wx_to_std(_dcp_track->GetStringSelection())) {
+ track = i;
+ }
+ }
+ }
+
+ if (track) {
+ BOOST_FOREACH (shared_ptr<Content> i, _parent->selected_text()) {
+ shared_ptr<TextContent> t = i->text_of_original_type(_original_type);
+ if (t && t->type() == TEXT_CLOSED_CAPTION) {
+ t->set_dcp_track(*track);
+ }
+ }
+ }
+
+ update_dcp_tracks ();
}
void
_type->SetSelection (0);
}
setup_sensitivity ();
+ setup_visibility ();
} else if (property == TextContentProperty::BURN) {
checked_set (_burn, text ? text->burn() : false);
} else if (property == TextContentProperty::X_OFFSET) {
} else if (property == TextContentProperty::LINE_SPACING) {
checked_set (_line_spacing, text ? lrint (text->line_spacing() * 100) : 100);
} else if (property == TextContentProperty::LANGUAGE) {
- checked_set (_language, text ? text->language() : "");
+ if (_language) {
+ checked_set (_language, text ? text->language() : "");
+ }
+ } else if (property == TextContentProperty::DCP_TRACK) {
+ if (_dcp_track) {
+ update_dcp_track_selection ();
+ }
} else if (property == DCPContentProperty::REFERENCE_TEXT) {
if (scs) {
shared_ptr<DCPContent> dcp = dynamic_pointer_cast<DCPContent> (scs);
}
}
+/** @return the text type that is currently selected in the drop-down */
+TextType
+TextPanel::current_type () const
+{
+ switch (_type->GetSelection()) {
+ case 0:
+ return TEXT_OPEN_SUBTITLE;
+ case 1:
+ return TEXT_CLOSED_CAPTION;
+ }
+
+ return TEXT_UNKNOWN;
+}
+
void
TextPanel::type_changed ()
{
BOOST_FOREACH (shared_ptr<Content> i, _parent->selected_text()) {
- switch (_type->GetSelection()) {
- case 0:
- i->text_of_original_type(_original_type)->set_type (TEXT_OPEN_SUBTITLE);
- break;
- case 1:
- i->text_of_original_type(_original_type)->set_type (TEXT_CLOSED_CAPTION);
- break;
- }
+ i->text_of_original_type(_original_type)->set_type (current_type ());
}
+
+ setup_visibility ();
}
void
TextPanel::setup_sensitivity ()
{
int any_subs = 0;
+ /* We currently assume that FFmpeg subtitles are bitmapped */
int ffmpeg_subs = 0;
+ /* DCP subs can't have their line spacing changed */
+ int dcp_subs = 0;
ContentList sel = _parent->selected_text ();
BOOST_FOREACH (shared_ptr<Content> i, sel) {
/* These are the content types that could include subtitles */
++ffmpeg_subs;
++any_subs;
}
- } else if (sc || dc || dsc) {
+ } else if (dc || dsc) {
+ ++dcp_subs;
+ ++any_subs;
+ } else if (sc) {
/* XXX: in the future there could be bitmap subs from DCPs */
++any_subs;
}
}
string why_not;
- bool const can_reference = dcp && dcp->can_reference_text (_original_type, why_not);
+ bool const can_reference = dcp && dcp->can_reference_text (_parent->film(), _original_type, why_not);
setup_refer_button (_reference, _reference_note, dcp, can_reference, why_not);
bool const reference = _reference->GetValue ();
+ TextType const type = current_type ();
+
+ /* Set up _type */
+ _type->Clear ();
+ _type->Append (_("open subtitles"));
+ if (ffmpeg_subs == 0) {
+ _type->Append (_("closed captions"));
+ }
+
+ switch (type) {
+ case TEXT_OPEN_SUBTITLE:
+ _type->SetSelection (0);
+ break;
+ case TEXT_CLOSED_CAPTION:
+ if (_type->GetCount() > 1) {
+ _type->SetSelection (1);
+ }
+ break;
+ default:
+ break;
+ }
+
/* Set up sensitivity */
_use->Enable (!reference && any_subs > 0);
bool const use = _use->GetValue ();
_type->Enable (!reference && any_subs > 0 && use);
- _burn->Enable (!reference && any_subs > 0 && use && _type->GetSelection() == 0);
- _x_offset->Enable (!reference && any_subs > 0 && use);
- _y_offset->Enable (!reference && any_subs > 0 && use);
- _x_scale->Enable (!reference && any_subs > 0 && use);
- _y_scale->Enable (!reference && any_subs > 0 && use);
- _line_spacing->Enable (!reference && use);
- _language->Enable (!reference && any_subs > 0 && use);
+ _burn->Enable (!reference && any_subs > 0 && use && type == TEXT_OPEN_SUBTITLE);
+ _x_offset->Enable (!reference && any_subs > 0 && use && type == TEXT_OPEN_SUBTITLE);
+ _y_offset->Enable (!reference && any_subs > 0 && use && type == TEXT_OPEN_SUBTITLE);
+ _x_scale->Enable (!reference && any_subs > 0 && use && type == TEXT_OPEN_SUBTITLE);
+ _y_scale->Enable (!reference && any_subs > 0 && use && type == TEXT_OPEN_SUBTITLE);
+ _line_spacing->Enable (!reference && use && type == TEXT_OPEN_SUBTITLE && dcp_subs < any_subs);
_stream->Enable (!reference && ffmpeg_subs == 1);
- _text_view_button->Enable (!reference);
- _fonts_dialog_button->Enable (!reference);
- _appearance_dialog_button->Enable (!reference && any_subs > 0 && use);
+ /* Ideally we would check here to see if the FFmpeg content has "string" subs (i.e. not bitmaps) */
+ _text_view_button->Enable (!reference && any_subs > 0 && ffmpeg_subs == 0);
+ _fonts_dialog_button->Enable (!reference && any_subs > 0 && ffmpeg_subs == 0 && type == TEXT_OPEN_SUBTITLE);
+ _appearance_dialog_button->Enable (!reference && any_subs > 0 && use && type == TEXT_OPEN_SUBTITLE);
}
void
void
TextPanel::language_changed ()
{
+ DCPOMATIC_ASSERT (_language);
BOOST_FOREACH (shared_ptr<Content> i, _parent->selected_text ()) {
i->text_of_original_type(_original_type)->set_language (wx_to_std (_language->GetValue()));
}
film_content_changed (TextContentProperty::LANGUAGE);
film_content_changed (TextContentProperty::FONTS);
film_content_changed (TextContentProperty::TYPE);
+ film_content_changed (TextContentProperty::DCP_TRACK);
film_content_changed (DCPContentProperty::REFERENCE_TEXT);
}
ContentList c = _parent->selected_text ();
DCPOMATIC_ASSERT (c.size() == 1);
- shared_ptr<Decoder> decoder = decoder_factory (c.front(), _parent->film()->log(), false);
+ shared_ptr<Decoder> decoder = decoder_factory (_parent->film(), c.front(), shared_ptr<Decoder>());
if (decoder) {
_text_view = new TextView (this, _parent->film(), c.front(), c.front()->text_of_original_type(_original_type), decoder, _parent->film_viewer());
ContentList c = _parent->selected_text ();
DCPOMATIC_ASSERT (c.size() == 1);
- SubtitleAppearanceDialog* d = new SubtitleAppearanceDialog (this, c.front(), c.front()->text_of_original_type(_original_type));
+ SubtitleAppearanceDialog* d = new SubtitleAppearanceDialog (this, _parent->film(), c.front(), c.front()->text_of_original_type(_original_type));
if (d->ShowModal () == wxID_OK) {
d->apply ();
}