X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/0cf3e587a2ec542ba1eb6e03a84c54edefae1881..19c7ac3d3bf1ac2113505a805a79c40ce3db1f5b:/src/gtk/filedlg.cpp diff --git a/src/gtk/filedlg.cpp b/src/gtk/filedlg.cpp index ce722561ac..d7a2d8a57e 100644 --- a/src/gtk/filedlg.cpp +++ b/src/gtk/filedlg.cpp @@ -10,7 +10,7 @@ // For compilers that support precompilation, includes "wx.h". #include "wx/wxprec.h" -#if wxUSE_FILEDLG && defined(__WXGTK24__) +#if wxUSE_FILEDLG #include "wx/filedlg.h" @@ -40,22 +40,24 @@ static void gtk_filedialog_ok_callback(GtkWidget *widget, wxFileDialog *dialog) // gtk version numbers must be identical with the one in ctor (that calls set_do_overwrite_confirmation) #if GTK_CHECK_VERSION(2,7,3) - if(gtk_check_version(2,7,3) != NULL) + if (gtk_check_version(2, 7, 3) != NULL) #endif - if ((style & wxFD_SAVE) && (style & wxFD_OVERWRITE_PROMPT)) { - if ( g_file_test(filename, G_FILE_TEST_EXISTS) ) + if ((style & wxFD_SAVE) && (style & wxFD_OVERWRITE_PROMPT)) { - wxString msg; - - msg.Printf( - _("File '%s' already exists, do you really want to overwrite it?"), - wxString(filename, *wxConvFileName)); - - wxMessageDialog dlg(dialog, msg, _("Confirm"), - wxYES_NO | wxICON_QUESTION); - if (dlg.ShowModal() != wxID_YES) - return; + if ( g_file_test(filename, G_FILE_TEST_EXISTS) ) + { + wxString msg; + + msg.Printf( + _("File '%s' already exists, do you really want to overwrite it?"), + wxString(filename, *wxConvFileName)); + + wxMessageDialog dlg(dialog, msg, _("Confirm"), + wxYES_NO | wxICON_QUESTION); + if (dlg.ShowModal() != wxID_YES) + return; + } } } @@ -63,13 +65,13 @@ static void gtk_filedialog_ok_callback(GtkWidget *widget, wxFileDialog *dialog) { if ( !g_file_test(filename, G_FILE_TEST_EXISTS) ) { - wxMessageDialog dlg( dialog, _("Please choose an existing file."), + wxMessageDialog dlg( dialog, _("Please choose an existing file."), _("Error"), wxOK| wxICON_ERROR); dlg.ShowModal(); return; } } - + // change to the directory where the user went if asked if (style & wxFD_CHANGE_DIR) { @@ -80,7 +82,7 @@ static void gtk_filedialog_ok_callback(GtkWidget *widget, wxFileDialog *dialog) wxCommandEvent event(wxEVT_COMMAND_BUTTON_CLICKED, wxID_OK); event.SetEventObject(dialog); - dialog->GetEventHandler()->ProcessEvent(event); + dialog->HandleWindowEvent(event); } } @@ -91,11 +93,12 @@ static void gtk_filedialog_ok_callback(GtkWidget *widget, wxFileDialog *dialog) extern "C" { -static void gtk_filedialog_cancel_callback(GtkWidget *w, wxFileDialog *dialog) +static void +gtk_filedialog_cancel_callback(GtkWidget * WXUNUSED(w), wxFileDialog *dialog) { wxCommandEvent event(wxEVT_COMMAND_BUTTON_CLICKED, wxID_CANCEL); event.SetEventObject(dialog); - dialog->GetEventHandler()->ProcessEvent(event); + dialog->HandleWindowEvent(event); } static void gtk_filedialog_response_callback(GtkWidget *w, @@ -111,7 +114,6 @@ static void gtk_filedialog_response_callback(GtkWidget *w, static void gtk_filedialog_update_preview_callback(GtkFileChooser *chooser, gpointer user_data) { -#if GTK_CHECK_VERSION(2,4,0) GtkWidget *preview = GTK_WIDGET(user_data); wxGtkString filename(gtk_file_chooser_get_preview_filename(chooser)); @@ -127,23 +129,39 @@ static void gtk_filedialog_update_preview_callback(GtkFileChooser *chooser, g_object_unref (pixbuf); gtk_file_chooser_set_preview_widget_active(chooser, have_preview); -#else - wxUnusedVar(chooser); - wxUnusedVar(user_data); -#endif // GTK+ 2.4+ } } // extern "C" +//----------------------------------------------------------------------------- +// "size_request" from m_extraControl +//----------------------------------------------------------------------------- + +extern "C" { +static void extra_widget_size_request(GtkWidget*, GtkRequisition* req, wxWindow* win) +{ + // allow dialog to be resized smaller horizontally + req->width = win->GetMinWidth(); +} +} + +void wxFileDialog::AddChildGTK(wxWindowGTK* child) +{ + g_signal_connect_after(child->m_widget, "size_request", + G_CALLBACK(extra_widget_size_request), child); + gtk_file_chooser_set_extra_widget( + GTK_FILE_CHOOSER(m_widget), child->m_widget); +} //----------------------------------------------------------------------------- // wxFileDialog //----------------------------------------------------------------------------- -IMPLEMENT_DYNAMIC_CLASS(wxFileDialog,wxGenericFileDialog) +IMPLEMENT_DYNAMIC_CLASS(wxFileDialog,wxFileDialogBase) -BEGIN_EVENT_TABLE(wxFileDialog,wxGenericFileDialog) +BEGIN_EVENT_TABLE(wxFileDialog,wxFileDialogBase) EVT_BUTTON(wxID_OK, wxFileDialog::OnFakeOk) + EVT_SIZE(wxFileDialog::OnSize) END_EVENT_TABLE() wxFileDialog::wxFileDialog(wxWindow *parent, const wxString& message, @@ -153,18 +171,16 @@ wxFileDialog::wxFileDialog(wxWindow *parent, const wxString& message, long style, const wxPoint& pos, const wxSize& sz, const wxString& name) - : wxGenericFileDialog(parent, message, defaultDir, defaultFileName, - wildCard, style, pos, sz, name, true ) + : wxFileDialogBase() { - if (gtk_check_version(2,4,0)) + parent = GetParentForModalDialog(parent); + + if (!wxFileDialogBase::Create(parent, message, defaultDir, defaultFileName, + wildCard, style, pos, sz, name)) { - wxGenericFileDialog::Create( parent, message, defaultDir, - defaultFileName, wildCard, style, pos ); return; } - parent = GetParentForModalDialog(parent); - if (!PreCreation(parent, pos, wxDefaultSize) || !CreateBase(parent, wxID_ANY, pos, wxDefaultSize, style, wxDefaultValidator, wxT("filedialog"))) @@ -197,22 +213,24 @@ wxFileDialog::wxFileDialog(wxWindow *parent, const wxString& message, GTK_STOCK_CANCEL, GTK_RESPONSE_CANCEL, ok_btn_stock, GTK_RESPONSE_ACCEPT, NULL); + g_object_ref(m_widget); + GtkFileChooser* file_chooser = GTK_FILE_CHOOSER(m_widget); - m_fc.SetWidget( GTK_FILE_CHOOSER(m_widget) ); + m_fc.SetWidget(file_chooser); gtk_dialog_set_default_response(GTK_DIALOG(m_widget), GTK_RESPONSE_ACCEPT); if ( style & wxFD_MULTIPLE ) - gtk_file_chooser_set_select_multiple(GTK_FILE_CHOOSER(m_widget), true); + gtk_file_chooser_set_select_multiple(file_chooser, true); // gtk_widget_hide_on_delete is used here to avoid that Gtk automatically // destroys the dialog when the user press ESC on the dialog: in that case // a second call to ShowModal() would result in a bunch of Gtk-CRITICAL // errors... - g_signal_connect (G_OBJECT(m_widget), + g_signal_connect(m_widget, "delete_event", G_CALLBACK (gtk_widget_hide_on_delete), - (gpointer)this); + this); // local-only property could be set to false to allow non-local files to be // loaded. In that case get/set_uri(s) should be used instead of @@ -243,8 +261,7 @@ wxFileDialog::wxFileDialog(wxWindow *parent, const wxString& message, const wxString dir = fn.GetPath(); if ( !dir.empty() ) { - gtk_file_chooser_set_current_folder(GTK_FILE_CHOOSER(m_widget), - dir.fn_str()); + gtk_file_chooser_set_current_folder(file_chooser, dir.fn_str()); } const wxString fname = fn.GetFullName(); @@ -252,191 +269,146 @@ wxFileDialog::wxFileDialog(wxWindow *parent, const wxString& message, { if ( !fname.empty() ) { - gtk_file_chooser_set_current_name(GTK_FILE_CHOOSER(m_widget), - fname.fn_str()); + gtk_file_chooser_set_current_name(file_chooser, fname.fn_str()); } #if GTK_CHECK_VERSION(2,7,3) if ((style & wxFD_OVERWRITE_PROMPT) && !gtk_check_version(2,7,3)) - gtk_file_chooser_set_do_overwrite_confirmation(GTK_FILE_CHOOSER(m_widget), TRUE); + gtk_file_chooser_set_do_overwrite_confirmation(file_chooser, true); #endif } else // wxFD_OPEN { if ( !fname.empty() ) { - gtk_file_chooser_set_filename(GTK_FILE_CHOOSER(m_widget), + gtk_file_chooser_set_filename(file_chooser, fn.GetFullPath().fn_str()); } } -#if GTK_CHECK_VERSION(2,4,0) if ( style & wxFD_PREVIEW ) { GtkWidget *previewImage = gtk_image_new(); - gtk_file_chooser_set_preview_widget(GTK_FILE_CHOOSER(m_widget), - previewImage); + gtk_file_chooser_set_preview_widget(file_chooser, previewImage); g_signal_connect(m_widget, "update-preview", G_CALLBACK(gtk_filedialog_update_preview_callback), previewImage); } -#endif // GTK+ 2.4+ } -void wxFileDialog::OnFakeOk( wxCommandEvent &event ) +wxFileDialog::~wxFileDialog() { - if (!gtk_check_version(2,4,0)) - EndDialog(wxID_OK); - else - wxGenericFileDialog::OnOk( event ); + if (m_extraControl) + { + // get chooser to drop its reference right now, allowing wxWindow dtor + // to verify that ref count drops to zero + gtk_file_chooser_set_extra_widget( + GTK_FILE_CHOOSER(m_widget), NULL); + } +} + +void wxFileDialog::OnFakeOk(wxCommandEvent& WXUNUSED(event)) +{ + EndDialog(wxID_OK); } int wxFileDialog::ShowModal() { - if (!gtk_check_version(2,4,0)) - return wxDialog::ShowModal(); - else - return wxGenericFileDialog::ShowModal(); + CreateExtraControl(); + + return wxDialog::ShowModal(); } -bool wxFileDialog::Show( bool show ) +void wxFileDialog::DoSetSize(int WXUNUSED(x), int WXUNUSED(y), + int WXUNUSED(width), int WXUNUSED(height), + int WXUNUSED(sizeFlags)) { - if (!gtk_check_version(2,4,0)) - return wxDialog::Show( show ); - else - return wxGenericFileDialog::Show( show ); } -void wxFileDialog::DoSetSize(int x, int y, int width, int height, int sizeFlags ) +void wxFileDialog::OnSize(wxSizeEvent&) { - if (!m_wxwindow) - return; - else - wxGenericFileDialog::DoSetSize( x, y, width, height, sizeFlags ); + // avoid calling DoLayout(), which will set the (wrong) size of + // m_extraControl, its size is managed by GtkFileChooser } wxString wxFileDialog::GetPath() const { - if (!gtk_check_version(2,4,0)) + wxFileName fn = m_fc.GetPath(); + + if (HasFdFlag(wxFD_SAVE)) { - return m_fc.GetPath(); + // add extension + if (!fn.HasExt()) + { + wxFileName wildcard( "/dummy", m_fc.GetCurrentWildCard() ); + wxString ext = wildcard.GetExt(); + if (!ext.empty() && (ext.Find('?') == wxNOT_FOUND) && (ext.Find('*') == wxNOT_FOUND)) + fn.SetExt( ext ); + } } - return wxGenericFileDialog::GetPath(); + return fn.GetFullPath(); } void wxFileDialog::GetFilenames(wxArrayString& files) const { - if (!gtk_check_version(2,4,0)) - { - m_fc.GetFilenames( files ); - } - else - wxGenericFileDialog::GetFilenames( files ); + m_fc.GetFilenames( files ); } void wxFileDialog::GetPaths(wxArrayString& paths) const { - if (!gtk_check_version(2,4,0)) - { - m_fc.GetPaths( paths ); - } - else - wxGenericFileDialog::GetPaths( paths ); + m_fc.GetPaths( paths ); } void wxFileDialog::SetMessage(const wxString& message) { - if (!gtk_check_version(2,4,0)) - { - m_message = message; - SetTitle(message); - } - else - wxGenericFileDialog::SetMessage( message ); + m_message = message; + SetTitle(message); } void wxFileDialog::SetPath(const wxString& path) { - if (!gtk_check_version(2,4,0)) - { - m_fc.SetPath( path ); - } - else - wxGenericFileDialog::SetPath( path ); + m_fc.SetPath( path ); } void wxFileDialog::SetDirectory(const wxString& dir) { - if (!gtk_check_version(2,4,0)) - { - m_fc.SetDirectory( dir ); - } - else - wxGenericFileDialog::SetDirectory( dir ); + m_fc.SetDirectory( dir ); } wxString wxFileDialog::GetDirectory() const { - if (!gtk_check_version(2,4,0)) - { - m_fc.GetDirectory(); - } - - return wxGenericFileDialog::GetDirectory(); + return m_fc.GetDirectory(); } void wxFileDialog::SetFilename(const wxString& name) { - if (!gtk_check_version(2,4,0)) - { - if (HasFdFlag(wxFD_SAVE)) - gtk_file_chooser_set_current_name(GTK_FILE_CHOOSER(m_widget), wxGTK_CONV(name)); - else - SetPath(wxFileName(GetDirectory(), name).GetFullPath()); - } + if (HasFdFlag(wxFD_SAVE)) + gtk_file_chooser_set_current_name(GTK_FILE_CHOOSER(m_widget), wxGTK_CONV(name)); else - wxGenericFileDialog::SetFilename( name ); + SetPath(wxFileName(GetDirectory(), name).GetFullPath()); } wxString wxFileDialog::GetFilename() const { - if (!gtk_check_version(2,4,0)) - return m_fc.GetFilename(); - else - return wxGenericFileDialog::GetFilename(); + return m_fc.GetFilename(); } void wxFileDialog::SetWildcard(const wxString& wildCard) { - if (!gtk_check_version(2,4,0)) - { - m_fc.SetWildcard( wildCard ); - } - else - wxGenericFileDialog::SetWildcard( wildCard ); + m_fc.SetWildcard( wildCard ); } void wxFileDialog::SetFilterIndex(int filterIndex) { - if (!gtk_check_version(2,4,0)) - { - m_fc.SetFilterIndex( filterIndex); - } - else - wxGenericFileDialog::SetFilterIndex( filterIndex ); + m_fc.SetFilterIndex( filterIndex); } int wxFileDialog::GetFilterIndex() const { - if (!gtk_check_version(2,4,0)) - { - return m_fc.GetFilterIndex(); - } - else - return wxGenericFileDialog::GetFilterIndex(); + return m_fc.GetFilterIndex(); } -#endif // wxUSE_FILEDLG && __WXGTK24__ +#endif // wxUSE_FILEDLG