X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/ec376c8fd9ff813392030426acd2317c6fa6527e..20c81bed846981e90769826b94a91eebb91158f1:/src/gtk/filepicker.cpp diff --git a/src/gtk/filepicker.cpp b/src/gtk/filepicker.cpp index 4370de3f9b..c79eb51426 100644 --- a/src/gtk/filepicker.cpp +++ b/src/gtk/filepicker.cpp @@ -17,26 +17,17 @@ // For compilers that support precompilation, includes "wx.h". #include "wx/wxprec.h" -#include "wx/gtk/private.h" -#include "wx/filepicker.h" - +#if wxUSE_FILEPICKERCTRL && defined(__WXGTK26__) -#if wxUSE_TOOLTIPS +#include "wx/filepicker.h" #include "wx/tooltip.h" -#endif - -#include -#include - -#include // chdir +#include "wx/gtk/private.h" // ============================================================================ // implementation // ============================================================================ -#if wxUSE_FILEPICKERCTRL && defined(__WXGTK26__) - //----------------------------------------------------------------------------- // wxFileButton //----------------------------------------------------------------------------- @@ -52,10 +43,11 @@ bool wxFileButton::Create( wxWindow *parent, wxWindowID id, { if (!gtk_check_version(2,6,0)) { - // VERY IMPORTANT: this code is identic to relative code in wxFileButton; - // if you find a problem here, fix it also in wxFileButton ! + // VERY IMPORTANT: this code is identic to relative code in wxDirButton; + // if you find a problem here, fix it also in wxDirButton ! m_needParent = true; + m_acceptsFocus = true; if (!PreCreation( parent, pos, size ) || !wxControl::CreateBase(parent, id, pos, size, style & wxWINDOW_STYLE_MASK, @@ -66,9 +58,13 @@ bool wxFileButton::Create( wxWindow *parent, wxWindowID id, } // create the dialog associated with this button + // NB: unlike generic implementation, native GTK implementation needs to create + // the filedialog here as it needs to use gtk_file_chooser_button_new_with_dialog() SetWindowStyle(style); m_path = path; - if (!CreateDialog(message, wildcard)) + m_message = message; + m_wildcard = wildcard; + if ((m_dialog = CreateDialog()) == NULL) return false; // little trick used to avoid problems when there are other GTK windows 'grabbed': @@ -98,7 +94,7 @@ bool wxFileButton::Create( wxWindow *parent, wxWindowID id, m_parent->DoAddChild( this ); PostCreation(size); - SetBestSize(size); + SetInitialSize(size); } else return wxGenericFileButton::Create(parent, id, label, path, message, wildcard, @@ -122,7 +118,7 @@ void wxFileButton::OnDialogOK(wxCommandEvent& ev) if (ev.GetId() == wxID_OK) { // ...update our path - UpdatePathFromDialog(); + UpdatePathFromDialog(m_dialog); // ...and fire an event wxFileDirPickerEvent event(wxEVT_COMMAND_FILEPICKER_CHANGED, this, GetId(), m_path); @@ -130,6 +126,12 @@ void wxFileButton::OnDialogOK(wxCommandEvent& ev) } } +void wxFileButton::SetPath(const wxString &str) +{ + m_path = str; + UpdateDialogPath(m_dialog); +} + #endif // wxUSE_FILEPICKERCTRL && defined(__WXGTK26__) @@ -137,6 +139,8 @@ void wxFileButton::OnDialogOK(wxCommandEvent& ev) #if wxUSE_DIRPICKERCTRL && defined(__WXGTK26__) +#include // chdir + //----------------------------------------------------------------------------- // "current-folder-changed" //----------------------------------------------------------------------------- @@ -146,11 +150,17 @@ static void gtk_dirbutton_currentfolderchanged_callback(GtkFileChooserButton *wi wxDirButton *p) { // update the m_path member of the wxDirButtonGTK + // unless the path was changed by wxDirButton::SetPath() + if (p->m_bIgnoreNextChange) + { + p->m_bIgnoreNextChange=false; + return; + } wxASSERT(p); // NB: it's important to use gtk_file_chooser_get_filename instead of // gtk_file_chooser_get_current_folder (see GTK docs) ! - gchar* filename = gtk_file_chooser_get_filename(GTK_FILE_CHOOSER(widget)); + wxGtkString filename(gtk_file_chooser_get_filename(GTK_FILE_CHOOSER(widget))); p->UpdatePath(filename); // since GtkFileChooserButton when used to pick directories also uses a combobox, @@ -160,7 +170,6 @@ static void gtk_dirbutton_currentfolderchanged_callback(GtkFileChooserButton *wi // style was given. if (p->HasFlag(wxDIRP_CHANGE_DIR)) chdir(filename); - g_free(filename); // ...and fire an event wxFileDirPickerEvent event(wxEVT_COMMAND_DIRPICKER_CHANGED, p, p->GetId(), p->GetPath()); @@ -188,6 +197,7 @@ bool wxDirButton::Create( wxWindow *parent, wxWindowID id, // if you find a problem here, fix it also in wxFileButton ! m_needParent = true; + m_acceptsFocus = true; if (!PreCreation( parent, pos, size ) || !wxControl::CreateBase(parent, id, pos, size, style & wxWINDOW_STYLE_MASK, @@ -199,9 +209,11 @@ bool wxDirButton::Create( wxWindow *parent, wxWindowID id, // create the dialog associated with this button SetWindowStyle(style); - m_path = path; - if (!CreateDialog(message, wildcard)) + m_message = message; + m_wildcard = wildcard; + if ((m_dialog = CreateDialog()) == NULL) return false; + SetPath(path); // little trick used to avoid problems when there are other GTK windows 'grabbed': // GtkFileChooserDialog won't be responsive to user events if there is another @@ -229,7 +241,7 @@ bool wxDirButton::Create( wxWindow *parent, wxWindowID id, m_parent->DoAddChild( this ); PostCreation(size); - SetBestSize(size); + SetInitialSize(size); } else return wxGenericDirButton::Create(parent, id, label, path, message, wildcard, @@ -246,4 +258,18 @@ wxDirButton::~wxDirButton() m_dialog->m_widget = NULL; } +void wxDirButton::SetPath(const wxString &str) +{ + m_path = str; + + // wxDirButton uses the "current-folder-changed" signal which is triggered also + // when we set the path on the dialog associated with this button; thus we need + // to set the following flag to avoid sending a wxFileDirPickerEvent from this + // function (which would be inconsistent with wxFileButton's behaviour and in + // general with all wxWidgets control-manipulation functions which do not send events). + m_bIgnoreNextChange = true; + + UpdateDialogPath(m_dialog); +} + #endif // wxUSE_DIRPICKERCTRL && defined(__WXGTK26__)