{
DCPOMATIC_ASSERT (_film->audio_language());
auto d = new LanguageTagDialog (_panel, *_film->audio_language());
- d->ShowModal ();
- _film->set_audio_language(d->get());
+ if (d->ShowModal() == wxID_OK) {
+ _film->set_audio_language(d->get());
+ }
d->Destroy ();
}
overall_sizer->Add (_list, 0, wxALL, DCPOMATIC_SIZER_GAP);
overall_sizer->Add (add, 0, wxALL, DCPOMATIC_SIZER_GAP);
- auto buttons = CreateSeparatedButtonSizer (wxOK);
+ auto buttons = CreateSeparatedButtonSizer(wxOK | wxCANCEL);
if (buttons) {
overall_sizer->Add (buttons, wxSizerFlags().Expand().DoubleBorder());
}
LanguageTagWidget::edit ()
{
auto d = new LanguageTagDialog(_parent, _tag.get_value_or(dcp::LanguageTag("en")));
- d->ShowModal ();
- set (d->get());
- Changed (d->get());
+ if (d->ShowModal() == wxID_OK) {
+ set(d->get());
+ Changed(d->get());
+ }
d->Destroy ();
}