Use wx_ptr more.

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

diff --git a/src/wx/dir_picker_ctrl.cc b/src/wx/dir_picker_ctrl.cc
index 2330e584a5ea90748ac682a8367cdb1041bdd055..34e54532e10f7bc60e3904603661abce093014aa 100644 (file)
--- a/src/wx/dir_picker_ctrl.cc
+++ b/src/wx/dir_picker_ctrl.cc
@@ -18,24 +18,36 @@
 
 */
 
-#include <wx/wx.h>
-#include <wx/stdpaths.h>
-#include <wx/filepicker.h>
-#include <boost/filesystem.hpp>
+
+#include "dcpomatic_button.h"
 #include "dir_picker_ctrl.h"
+#include "static_text.h"
+#include "wx_ptr.h"
 #include "wx_util.h"
+#include <dcp/warnings.h>
+LIBDCP_DISABLE_WARNINGS
+#include <wx/filepicker.h>
+#include <wx/stdpaths.h>
+#include <wx/wx.h>
+LIBDCP_ENABLE_WARNINGS
+#include <boost/filesystem.hpp>
+
 
 using namespace std;
 using namespace boost;
 
+
 DirPickerCtrl::DirPickerCtrl (wxWindow* parent)
        : wxPanel (parent)
 {
        _sizer = new wxBoxSizer (wxHORIZONTAL);
 
-       _folder = new wxStaticText (this, wxID_ANY, wxT (""));
-       _sizer->Add (_folder, 1, wxEXPAND | wxALL, 6);
-       _browse = new wxButton (this, wxID_ANY, _("Browse..."));
+       _folder = new StaticText (this, wxT(""));
+       wxFont font = _folder->GetFont ();
+       font.SetStyle (wxFONTSTYLE_ITALIC);
+       _folder->SetFont (font);
+       _sizer->Add (_folder, 1, wxEXPAND | wxALL, DCPOMATIC_SIZER_GAP);
+       _browse = new Button (this, _("Browse..."));
        _sizer->Add (_browse, 0);
 
        SetSizer (_sizer);
@@ -51,7 +63,7 @@ DirPickerCtrl::SetPath (wxString p)
        if (_path == wxStandardPaths::Get().GetDocumentsDir()) {
                _folder->SetLabel (_("My Documents"));
        } else {
-               _folder->SetLabel (std_to_wx (filesystem::path (wx_to_std (_path)).leaf().string()));
+               _folder->SetLabel (_path);
        }
 
        wxCommandEvent ev (wxEVT_DIRPICKER_CHANGED, wxID_ANY);
@@ -59,6 +71,8 @@ DirPickerCtrl::SetPath (wxString p)
 
        _sizer->Layout ();
        SetMinSize (wxSize (max (400, _sizer->GetSize().GetWidth()), -1));
+
+       Changed ();
 }
 
 wxString
@@ -70,9 +84,8 @@ DirPickerCtrl::GetPath () const
 void
 DirPickerCtrl::browse_clicked ()
 {
-       wxDirDialog* d = new wxDirDialog (this);
+       auto d = make_wx<wxDirDialog>(this);
        if (d->ShowModal () == wxID_OK) {
                SetPath (d->GetPath ());
        }
-       d->Destroy ();
 }