Tweak layout.

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

diff --git a/src/wx/kdm_output_panel.cc b/src/wx/kdm_output_panel.cc
index 75971c14b5b6de795d8305d69c1d0c08bc3885e2..37c797192eafba8ac81ce0166e854775cb12e08a 100644 (file)
--- a/src/wx/kdm_output_panel.cc
+++ b/src/wx/kdm_output_panel.cc
@@ -89,7 +89,7 @@ KDMOutputPanel::KDMOutputPanel (wxWindow* parent)
        align->Add (format, 0, wxTOP, 2);
        table->Add (align, 0, wxALIGN_RIGHT | wxRIGHT, DCPOMATIC_SIZER_GAP - 2);
 #else
-       align->Add (format, 0, wxLEFT, DCPOMATIC_SIZER_GAP - 2);
+       align->Add (format, 0, wxLEFT, DCPOMATIC_SIZER_GAP);
        table->Add (align, 0, wxTOP | wxRIGHT | wxALIGN_TOP, DCPOMATIC_SIZER_GAP);
 #endif
        dcp::NameFormat::Map titles;
@@ -155,8 +155,8 @@ KDMOutputPanel::KDMOutputPanel (wxWindow* parent)
        _write_to->SetValue (Config::instance()->write_kdms_to_disk());
        _email->SetValue (Config::instance()->email_kdms());
 
-       _write_to->Bind     (wxEVT_CHECKBOX, boost::bind (&KDMOutputPanel::write_to_changed, this));
-       _email->Bind        (wxEVT_CHECKBOX, boost::bind (&KDMOutputPanel::email_changed, this));
+       _write_to->bind(&KDMOutputPanel::write_to_changed, this);
+       _email->bind(&KDMOutputPanel::email_changed, this);
        add_email_addresses->Bind (wxEVT_BUTTON, boost::bind(&KDMOutputPanel::add_email_addresses_clicked, this));
        _write_flat->Bind   (wxEVT_RADIOBUTTON, boost::bind (&KDMOutputPanel::kdm_write_type_changed, this));
        _write_folder->Bind (wxEVT_RADIOBUTTON, boost::bind (&KDMOutputPanel::kdm_write_type_changed, this));
@@ -172,6 +172,7 @@ KDMOutputPanel::write_to_changed ()
 {
        Config::instance()->set_write_kdms_to_disk(_write_to->GetValue());
        setup_sensitivity ();
+       MethodChanged();
 }
 
 
@@ -180,6 +181,7 @@ KDMOutputPanel::email_changed ()
 {
        Config::instance()->set_email_kdms(_email->GetValue());
        setup_sensitivity ();
+       MethodChanged();
 }
 
 
@@ -197,12 +199,11 @@ KDMOutputPanel::setup_sensitivity ()
 void
 KDMOutputPanel::advanced_clicked ()
 {
-       auto d = new KDMAdvancedDialog (this, _forensic_mark_video, _forensic_mark_audio, _forensic_mark_audio_up_to);
-       d->ShowModal ();
-       _forensic_mark_video = d->forensic_mark_video ();
-       _forensic_mark_audio = d->forensic_mark_audio ();
-       _forensic_mark_audio_up_to = d->forensic_mark_audio_up_to ();
-       d->Destroy ();
+       KDMAdvancedDialog dialog(this, _forensic_mark_video, _forensic_mark_audio, _forensic_mark_audio_up_to);
+       dialog.ShowModal();
+       _forensic_mark_video = dialog.forensic_mark_video();
+       _forensic_mark_audio = dialog.forensic_mark_audio();
+       _forensic_mark_audio_up_to = dialog.forensic_mark_audio_up_to();
 }
 
 
@@ -343,10 +344,16 @@ KDMOutputPanel::directory () const
 void
 KDMOutputPanel::add_email_addresses_clicked ()
 {
-       auto dialog = new ExtraKDMEmailDialog (this, _extra_addresses);
-       if (dialog->ShowModal() == wxID_OK) {
-               _extra_addresses = dialog->get();
+       ExtraKDMEmailDialog dialog(this, _extra_addresses);
+       if (dialog.ShowModal() == wxID_OK) {
+               _extra_addresses = dialog.get();
        }
-       dialog->Destroy();
+}
+
+
+bool
+KDMOutputPanel::method_selected() const
+{
+       return _write_to->GetValue() || _email->GetValue();
 }