Tweak vertical spacing of wxChoice on macOS (#2747).
authorCarl Hetherington <cth@carlh.net>
Fri, 2 Feb 2024 23:22:22 +0000 (00:22 +0100)
committerCarl Hetherington <cth@carlh.net>
Sun, 4 Feb 2024 17:43:00 +0000 (18:43 +0100)
src/tools/dcpomatic_disk.cc
src/wx/kdm_cpl_panel.cc
src/wx/wx_util.h

index 74ab30f8e402c2010078689114caa14101131d97..c907a73b00b6f0dc47ac7ac9d4d69095648210f7 100644 (file)
@@ -150,9 +150,9 @@ public:
                add_label_to_sizer (grid, overall_panel, _("Drive"), true, wxGBPosition(r, 0));
                auto drive_sizer = new wxBoxSizer (wxHORIZONTAL);
                _drive = new wxChoice (overall_panel, wxID_ANY);
-               drive_sizer->Add (_drive, 1, wxALIGN_CENTER_VERTICAL | wxRIGHT, DCPOMATIC_SIZER_X_GAP);
+               drive_sizer->Add(_drive, 1, wxTOP, 2);
                _drive_refresh = new wxButton (overall_panel, wxID_ANY, _("Refresh"));
-               drive_sizer->Add (_drive_refresh, 0);
+               drive_sizer->Add(_drive_refresh, 0, wxLEFT, DCPOMATIC_SIZER_X_GAP);
                grid->Add (drive_sizer, wxGBPosition(r, 1), wxDefaultSpan, wxEXPAND);
                ++r;
 
index 4699582aac538a4302a2472ae7307e336a0410da..4e1eb8f3404a62003f1c3a08a5dbffa1e08a7dae 100644 (file)
@@ -41,11 +41,11 @@ KDMCPLPanel::KDMCPLPanel (wxWindow* parent, vector<CPLSummary> cpls)
 
        /* CPL choice */
        auto s = new wxBoxSizer (wxHORIZONTAL);
-       add_label_to_sizer (s, this, _("CPL"), true, 0, wxLEFT | wxRIGHT | wxALIGN_CENTER_VERTICAL);
+       add_label_to_sizer (s, this, _("CPL"), true, 0, wxLEFT | wxRIGHT | wxALIGN_CENTRE_VERTICAL);
        _cpl = new wxChoice (this, wxID_ANY);
-       s->Add (_cpl, 1, wxEXPAND);
+       s->Add (_cpl, 1, wxTOP | wxEXPAND, DCPOMATIC_CHOICE_TOP_PAD);
        _cpl_browse = new Button (this, _("Browse..."));
-       s->Add (_cpl_browse, 0, wxLEFT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_X_GAP);
+       s->Add (_cpl_browse, 0, wxLEFT, DCPOMATIC_SIZER_X_GAP);
        vertical->Add (s, 0, wxEXPAND | wxTOP, DCPOMATIC_SIZER_GAP + 2);
 
        /* CPL details */
index 66b01640c5b1175583fee8898c5f7bc11335a88a..dcf5bbc1871d9cf1f16883821ab3a0dda7f88c34 100644 (file)
@@ -74,7 +74,7 @@ class PasswordEntry;
 
 /** Amount by which you need to top-pad a choice to make it line up, in some cases */
 #ifdef DCPOMATIC_OSX
-#define DCPOMATIC_CHOICE_TOP_PAD 1
+#define DCPOMATIC_CHOICE_TOP_PAD 2
 #else
 #define DCPOMATIC_CHOICE_TOP_PAD 0
 #endif