X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/a11a0ead4aeaf485a7d0ecf4928ee81bd9f9715b..3ec704b4221ee4d66ebb1cb98f1dc1017f68bec3:/src/gtk/aboutdlg.cpp?ds=sidebyside diff --git a/src/gtk/aboutdlg.cpp b/src/gtk/aboutdlg.cpp index e801284c92..853d9a2101 100644 --- a/src/gtk/aboutdlg.cpp +++ b/src/gtk/aboutdlg.cpp @@ -22,6 +22,7 @@ #if wxUSE_ABOUTDLG && defined(__WXGTK26__) #ifndef WX_PRECOMP + #include "wx/utils.h" // for wxLaunchDefaultBrowser() #endif //WX_PRECOMP #include "wx/aboutdlg.h" @@ -33,15 +34,32 @@ // GtkArray: temporary array of GTK strings // ---------------------------------------------------------------------------- +namespace +{ + class GtkArray { public: + // Create empty GtkArray + GtkArray() : m_strings(0), m_count(0) + { + } + + // Create GtkArray from wxArrayString. Note that the created object is + // only valid as long as 'a' is! GtkArray(const wxArrayString& a) { m_count = a.size(); m_strings = new const gchar *[m_count + 1]; + for ( size_t n = 0; n < m_count; n++ ) + { +#if wxUSE_UNICODE_UTF8 + m_strings[n] = a[n].utf8_str(); +#else m_strings[n] = wxGTK_CONV_SYS(a[n]).release(); +#endif + } // array must be NULL-terminated m_strings[m_count] = NULL; @@ -51,8 +69,10 @@ public: ~GtkArray() { +#if !wxUSE_UNICODE_UTF8 for ( size_t n = 0; n < m_count; n++ ) free(wx_const_cast(gchar *, m_strings[n])); +#endif delete [] m_strings; } @@ -64,24 +84,57 @@ private: DECLARE_NO_COPY_CLASS(GtkArray) }; +} // anonymous namespace + // ============================================================================ // implementation // ============================================================================ +// GTK+ about dialog is modeless, keep track of it in this variable +static GtkAboutDialog *gs_aboutDialog = NULL; + +extern "C" void +wxGtkAboutDialogOnClose(GtkAboutDialog *about) +{ + gtk_widget_destroy(GTK_WIDGET(about)); + if ( about == gs_aboutDialog ) + gs_aboutDialog = NULL; +} + +extern "C" void +wxGtkAboutDialogOnLink(GtkAboutDialog * WXUNUSED(about), + const gchar *link, + gpointer WXUNUSED(data)) +{ + wxLaunchDefaultBrowser(wxGTK_CONV_BACK_SYS(link)); +} + void wxAboutBox(const wxAboutDialogInfo& info) { if ( !gtk_check_version(2,6,0) ) { - GtkAboutDialog * const dlg = GTK_ABOUT_DIALOG(gtk_about_dialog_new()); - gtk_about_dialog_set_name(dlg, info.GetName()); + // don't create another dialog if one is already present + if ( !gs_aboutDialog ) + gs_aboutDialog = GTK_ABOUT_DIALOG(gtk_about_dialog_new()); + + GtkAboutDialog * const dlg = gs_aboutDialog; + gtk_about_dialog_set_name(dlg, wxGTK_CONV_SYS(info.GetName())); if ( info.HasVersion() ) - gtk_about_dialog_set_version(dlg, info.GetVersion()); + gtk_about_dialog_set_version(dlg, wxGTK_CONV_SYS(info.GetVersion())); + else + gtk_about_dialog_set_version(dlg, NULL); if ( info.HasCopyright() ) - gtk_about_dialog_set_copyright(dlg, info.GetCopyright()); + gtk_about_dialog_set_copyright(dlg, wxGTK_CONV_SYS(info.GetCopyright())); + else + gtk_about_dialog_set_copyright(dlg, NULL); if ( info.HasDescription() ) - gtk_about_dialog_set_comments(dlg, info.GetDescription()); + gtk_about_dialog_set_comments(dlg, wxGTK_CONV_SYS(info.GetDescription())); + else + gtk_about_dialog_set_comments(dlg, NULL); if ( info.HasLicence() ) - gtk_about_dialog_set_license(dlg, info.GetLicence()); + gtk_about_dialog_set_license(dlg, wxGTK_CONV_SYS(info.GetLicence())); + else + gtk_about_dialog_set_license(dlg, NULL); wxIcon icon = info.GetIcon(); if ( icon.Ok() ) @@ -89,20 +142,72 @@ void wxAboutBox(const wxAboutDialogInfo& info) if ( info.HasWebSite() ) { - gtk_about_dialog_set_website(dlg, info.GetWebSiteURL()); - gtk_about_dialog_set_website_label(dlg, info.GetWebSiteDescription()); + // NB: must be called before gtk_about_dialog_set_website() as + // otherwise it has no effect (although GTK+ docs don't mention + // this...) + gtk_about_dialog_set_url_hook(wxGtkAboutDialogOnLink, NULL, NULL); + + gtk_about_dialog_set_website(dlg, wxGTK_CONV_SYS(info.GetWebSiteURL())); + gtk_about_dialog_set_website_label + ( + dlg, + wxGTK_CONV_SYS(info.GetWebSiteDescription()) + ); + } + else + { + gtk_about_dialog_set_website(dlg, NULL); + gtk_about_dialog_set_website_label(dlg, NULL); + gtk_about_dialog_set_url_hook(NULL, NULL, NULL); } if ( info.HasDevelopers() ) gtk_about_dialog_set_authors(dlg, GtkArray(info.GetDevelopers())); + else + gtk_about_dialog_set_authors(dlg, GtkArray()); if ( info.HasDocWriters() ) gtk_about_dialog_set_documenters(dlg, GtkArray(info.GetDocWriters())); + else + gtk_about_dialog_set_documenters(dlg, GtkArray()); if ( info.HasArtists() ) gtk_about_dialog_set_artists(dlg, GtkArray(info.GetArtists())); + else + gtk_about_dialog_set_artists(dlg, GtkArray()); + + wxString transCredits; if ( info.HasTranslators() ) - gtk_about_dialog_set_translator_credits(dlg, _("translator-credits")); + { + const wxArrayString& translators = info.GetTranslators(); + const size_t count = translators.size(); + for ( size_t n = 0; n < count; n++ ) + { + transCredits << translators[n] << _T('\n'); + } + } + else // no translators explicitely specified + { + // maybe we have translator credits in the message catalog? + wxString translator = _("translator-credits"); + + // gtk_about_dialog_set_translator_credits() is smart enough to + // detect if "translator-credits" is untranslated and hide the + // translators tab in that case, however it will still show the + // "credits" button, (at least GTK 2.10.6) even if there are no + // credits informations at all, so we still need to do the check + // ourselves + if ( translator != wxT("translator-credits") ) // untranslated! + transCredits = translator; + } + + if ( !transCredits.empty() ) + gtk_about_dialog_set_translator_credits(dlg, wxGTK_CONV_SYS(transCredits)); + else + gtk_about_dialog_set_translator_credits(dlg, NULL); + + g_signal_connect(dlg, "response", + G_CALLBACK(wxGtkAboutDialogOnClose), NULL); - gtk_widget_show(GTK_WIDGET(dlg)); + gtk_window_present(GTK_WINDOW(dlg)); return; }