Add a close button to the AudioDialog on Linux in case it doesn't get a close button...
authorCarl Hetherington <cth@carlh.net>
Fri, 12 Jun 2015 22:55:51 +0000 (23:55 +0100)
committerCarl Hetherington <cth@carlh.net>
Fri, 12 Jun 2015 22:55:51 +0000 (23:55 +0100)
ChangeLog
src/wx/audio_dialog.cc

index b75cf7cb492ef4f7bb38fb69334f955af627867a..f029fb104a44cdf5cc7aea2fe5ac62456400c59f 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,7 @@
 2015-06-12  Carl Hetherington  <cth@carlh.net>
 
+       * Add a close button to the audio plot on Linux (#602).
+
        * Version 2.0.51 released.
 
 2015-06-12  Carl Hetherington  <cth@carlh.net>
index 0034e793d020934b7e3c9e5e31f9c88a01625fe3..7a13a24a08b54e51d8ae19f0b4a43e02383c41f8 100644 (file)
@@ -39,7 +39,8 @@ AudioDialog::AudioDialog (wxWindow* parent, shared_ptr<Film> film)
        wxFont subheading_font (*wxNORMAL_FONT);
        subheading_font.SetWeight (wxFONTWEIGHT_BOLD);
 
-       wxBoxSizer* sizer = new wxBoxSizer (wxHORIZONTAL);
+       wxBoxSizer* overall_sizer = new wxBoxSizer (wxVERTICAL);
+       wxBoxSizer* lr_sizer = new wxBoxSizer (wxHORIZONTAL);
        
        wxBoxSizer* left = new wxBoxSizer (wxVERTICAL);
 
@@ -48,7 +49,7 @@ AudioDialog::AudioDialog (wxWindow* parent, shared_ptr<Film> film)
        _peak_time = new wxStaticText (this, wxID_ANY, wxT (""));
        left->Add (_peak_time, 0, wxALL, 12);
 
-       sizer->Add (left, 1, wxALL, 12);
+       lr_sizer->Add (left, 1, wxALL, 12);
 
        wxBoxSizer* right = new wxBoxSizer (wxVERTICAL);
 
@@ -91,11 +92,20 @@ AudioDialog::AudioDialog (wxWindow* parent, shared_ptr<Film> film)
        _smoothing->Bind (wxEVT_SCROLL_THUMBTRACK, boost::bind (&AudioDialog::smoothing_changed, this));
        right->Add (_smoothing, 0, wxEXPAND);
 
-       sizer->Add (right, 0, wxALL, 12);
+       lr_sizer->Add (right, 0, wxALL, 12);
 
-       SetSizer (sizer);
-       sizer->Layout ();
-       sizer->SetSizeHints (this);
+       overall_sizer->Add (lr_sizer);
+
+#ifdef DCPOMATIC_LINUX 
+       wxSizer* buttons = CreateSeparatedButtonSizer (wxCLOSE);
+       if (buttons) {
+               overall_sizer->Add (buttons, wxSizerFlags().Expand().DoubleBorder());
+       }
+#endif 
+
+       SetSizer (overall_sizer);
+       overall_sizer->Layout ();
+       overall_sizer->SetSizeHints (this);
 }
 
 void