Fix layout of KDM dialog with lots of cinemas.
authorCarl Hetherington <cth@carlh.net>
Sun, 10 Jan 2016 01:37:01 +0000 (01:37 +0000)
committerCarl Hetherington <cth@carlh.net>
Sun, 10 Jan 2016 01:37:01 +0000 (01:37 +0000)
src/wx/kdm_dialog.cc

index 7bd256d038a6527b25b4740b76186e6bce988308..0ee502d49c46b889080f309e370525062e145734 100644 (file)
@@ -77,16 +77,17 @@ KDMDialog::KDMDialog (wxWindow* parent, boost::shared_ptr<const Film> film)
        _output = new KDMOutputPanel (this, film->interop ());
        vertical->Add (_output, 0, wxEXPAND | wxTOP, DCPOMATIC_SIZER_GAP);
 
-       /* Make an overall sizer to get a nice border, and put some buttons in */
-
-       wxBoxSizer* overall_sizer = new wxBoxSizer (wxVERTICAL);
-       overall_sizer->Add (vertical, 0, wxEXPAND | wxTOP | wxLEFT | wxRIGHT, DCPOMATIC_DIALOG_BORDER);
 
        wxSizer* buttons = CreateSeparatedButtonSizer (wxOK | wxCANCEL);
        if (buttons) {
-               overall_sizer->Add (buttons, 0, wxEXPAND | wxALL, DCPOMATIC_SIZER_Y_GAP);
+               vertical->Add (buttons, 0, wxEXPAND | wxALL, DCPOMATIC_SIZER_Y_GAP);
        }
 
+       /* Make an overall sizer to get a nice border, and put some buttons in */
+
+       wxBoxSizer* overall_sizer = new wxBoxSizer (wxVERTICAL);
+       overall_sizer->Add (vertical, 0, wxEXPAND | wxTOP | wxLEFT | wxRIGHT, DCPOMATIC_DIALOG_BORDER);
+
        /* Bind */
 
        _screens->ScreensChanged.connect (boost::bind (&KDMDialog::setup_sensitivity, this));