Switch to GtkTooltip from deprecated GtkTooltips in wxGTK wxToolTip.

Don't use deprecated GtkTooltips in wxGTK code any more, use the new
GtkTooltip instead.

Closes #12034.

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@66431 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Vadim Zeitlin
2010-12-22 13:57:28 +00:00
parent 65776a1582
commit 853149575e
5 changed files with 99 additions and 37 deletions

View File

@@ -450,6 +450,10 @@ All (GUI):
- Fixed resizing columns in wxGrid when they were reordered. - Fixed resizing columns in wxGrid when they were reordered.
- Added wxImage::Rotate180() (Jeff Tupper). - Added wxImage::Rotate180() (Jeff Tupper).
GTK:
- Switch to GtkTooltip from deprecated GtkTooltips (Emilien Kia).
MSW: MSW:
- Native implementation of wxHyperlinkCtrl and wxProgressDialog under modern - Native implementation of wxHyperlinkCtrl and wxProgressDialog under modern

View File

@@ -848,8 +848,17 @@ int wxListBox::DoListHitTest(const wxPoint& point) const
#if wxUSE_TOOLTIPS #if wxUSE_TOOLTIPS
void wxListBox::GTKApplyToolTip( GtkTooltips *tips, const gchar *tip ) void wxListBox::GTKApplyToolTip( GtkTooltips *tips, const gchar *tip )
{ {
#if GTK_CHECK_VERSION(2, 12, 0)
if (!gtk_check_version(2, 12, 0))
{
gtk_widget_set_tooltip_text(GTK_WIDGET( m_treeview ), tip);
}
else
#endif
{
// RN: Is this needed anymore? // RN: Is this needed anymore?
gtk_tooltips_set_tip( tips, GTK_WIDGET( m_treeview ), tip, NULL ); gtk_tooltips_set_tip( tips, GTK_WIDGET( m_treeview ), tip, NULL );
}
} }
#endif // wxUSE_TOOLTIPS #endif // wxUSE_TOOLTIPS

View File

@@ -351,7 +351,7 @@ void wxToolBar::Init()
wxToolBar::~wxToolBar() wxToolBar::~wxToolBar()
{ {
if (m_tooltips) if (m_tooltips) // always NULL if GTK >= 2.12
{ {
gtk_object_destroy(GTK_OBJECT(m_tooltips)); gtk_object_destroy(GTK_OBJECT(m_tooltips));
g_object_unref(m_tooltips); g_object_unref(m_tooltips);
@@ -376,9 +376,12 @@ bool wxToolBar::Create( wxWindow *parent,
FixupStyle(); FixupStyle();
m_toolbar = GTK_TOOLBAR( gtk_toolbar_new() ); m_toolbar = GTK_TOOLBAR( gtk_toolbar_new() );
if (gtk_check_version(2, 12, 0))
{
m_tooltips = gtk_tooltips_new(); m_tooltips = gtk_tooltips_new();
g_object_ref(m_tooltips); g_object_ref(m_tooltips);
gtk_object_sink(GTK_OBJECT(m_tooltips)); gtk_object_sink(GTK_OBJECT(m_tooltips));
}
GtkSetStyle(); GtkSetStyle();
if (style & wxTB_DOCKABLE) if (style & wxTB_DOCKABLE)
@@ -528,9 +531,19 @@ bool wxToolBar::DoInsertTool(size_t pos, wxToolBarToolBase *toolBase)
} }
if (!HasFlag(wxTB_NO_TOOLTIPS) && !tool->GetShortHelp().empty()) if (!HasFlag(wxTB_NO_TOOLTIPS) && !tool->GetShortHelp().empty())
{ {
#if GTK_CHECK_VERSION(2, 12, 0)
if (!gtk_check_version(2, 12, 0))
{
gtk_tool_item_set_tooltip_text(tool->m_item,
wxGTK_CONV(tool->GetShortHelp()));
}
else
#endif
{
gtk_tool_item_set_tooltip(tool->m_item, gtk_tool_item_set_tooltip(tool->m_item,
m_tooltips, wxGTK_CONV(tool->GetShortHelp()), ""); m_tooltips, wxGTK_CONV(tool->GetShortHelp()), "");
} }
}
g_signal_connect(GTK_BIN(tool->m_item)->child, "button_press_event", g_signal_connect(GTK_BIN(tool->m_item)->child, "button_press_event",
G_CALLBACK(button_press_event), tool); G_CALLBACK(button_press_event), tool);
g_signal_connect(tool->m_item, "enter_notify_event", g_signal_connect(tool->m_item, "enter_notify_event",
@@ -695,10 +708,20 @@ void wxToolBar::SetToolShortHelp( int id, const wxString& helpString )
(void)tool->SetShortHelp(helpString); (void)tool->SetShortHelp(helpString);
if (tool->m_item) if (tool->m_item)
{ {
#if GTK_CHECK_VERSION(2, 12, 0)
if (!gtk_check_version(2, 12, 0))
{
gtk_tool_item_set_tooltip_text(tool->m_item,
wxGTK_CONV(helpString));
}
else
#endif
{
gtk_tool_item_set_tooltip(tool->m_item, gtk_tool_item_set_tooltip(tool->m_item,
m_tooltips, wxGTK_CONV(helpString), ""); m_tooltips, wxGTK_CONV(helpString), "");
} }
} }
}
} }
void wxToolBar::SetToolNormalBitmap( int id, const wxBitmap& bitmap ) void wxToolBar::SetToolNormalBitmap( int id, const wxBitmap& bitmap )

View File

@@ -63,14 +63,34 @@ void wxToolTip::GTKApply( wxWindow *win )
/* static */ /* static */
void wxToolTip::GTKApply(GtkWidget *w, const gchar *tip) void wxToolTip::GTKApply(GtkWidget *w, const gchar *tip)
{ {
#if GTK_CHECK_VERSION(2, 12, 0)
if (!gtk_check_version(2, 12, 0))
{
gtk_widget_set_tooltip_text(w, tip);
}
else
#endif
{
if ( !gs_tooltips ) if ( !gs_tooltips )
gs_tooltips = gtk_tooltips_new(); gs_tooltips = gtk_tooltips_new();
gtk_tooltips_set_tip(gs_tooltips, w, tip, NULL); gtk_tooltips_set_tip(gs_tooltips, w, tip, NULL);
}
} }
void wxToolTip::Enable( bool flag ) void wxToolTip::Enable( bool flag )
{ {
#if GTK_CHECK_VERSION(2, 12, 0)
if (!gtk_check_version(2, 12, 0))
{
GtkSettings* settings = gtk_settings_get_default();
if(!settings)
return;
gtk_settings_set_long_property(settings, "gtk-enable-tooltips", flag?TRUE:FALSE, NULL);
}
else
#endif
{
if (!gs_tooltips) if (!gs_tooltips)
return; return;
@@ -78,6 +98,7 @@ void wxToolTip::Enable( bool flag )
gtk_tooltips_enable( gs_tooltips ); gtk_tooltips_enable( gs_tooltips );
else else
gtk_tooltips_disable( gs_tooltips ); gtk_tooltips_disable( gs_tooltips );
}
} }
G_BEGIN_DECLS G_BEGIN_DECLS
@@ -87,12 +108,24 @@ G_END_DECLS
void wxToolTip::SetDelay( long msecs ) void wxToolTip::SetDelay( long msecs )
{ {
#if GTK_CHECK_VERSION(2, 12, 0)
if (!gtk_check_version(2, 12, 0))
{
GtkSettings* settings = gtk_settings_get_default();
if(!settings)
return;
gtk_settings_set_long_property(settings, "gtk-tooltip-timeout", msecs, NULL);
}
else
#endif
{
if (!gs_tooltips) if (!gs_tooltips)
return; return;
// FIXME: This is a deprecated function and might not even have an effect. // FIXME: This is a deprecated function and might not even have an effect.
// Try to not use it, after which remove the prototype above. // Try to not use it, after which remove the prototype above.
gtk_tooltips_set_delay( gs_tooltips, (int)msecs ); gtk_tooltips_set_delay( gs_tooltips, (int)msecs );
}
} }
void wxToolTip::SetAutoPop( long WXUNUSED(msecs) ) void wxToolTip::SetAutoPop( long WXUNUSED(msecs) )

View File

@@ -3782,30 +3782,23 @@ void wxWindowGTK::DoSetToolTip( wxToolTip *tip )
{ {
GtkWidget *w = GetConnectWidget(); GtkWidget *w = GetConnectWidget();
wxToolTip::GTKApply(w, NULL); wxToolTip::GTKApply(w, NULL);
#if GTK_CHECK_VERSION(2, 12, 0)
// Just applying NULL doesn't work on 2.12.0, so also use
// gtk_widget_set_has_tooltip. It is part of the new GtkTooltip API
// but seems also to work with the old GtkTooltips.
if (gtk_check_version(2, 12, 0) == NULL)
gtk_widget_set_has_tooltip(w, FALSE);
#endif
} }
} }
void wxWindowGTK::GTKApplyToolTip( GtkTooltips *tips, const gchar *tip ) void wxWindowGTK::GTKApplyToolTip( GtkTooltips *tips, const gchar *tip )
{ {
GtkWidget *w = GetConnectWidget(); GtkWidget *w = GetConnectWidget();
gtk_tooltips_set_tip(tips, w, tip, NULL);
#if GTK_CHECK_VERSION(2, 12, 0) #if GTK_CHECK_VERSION(2, 12, 0)
if ( !tip || tip[0] == '\0' ) if (!gtk_check_version(2, 12, 0))
{ {
// Just applying empty tool tip doesn't work on 2.12.0, so also use gtk_widget_set_tooltip_text (w, tip);
// gtk_widget_set_has_tooltip.
if (gtk_check_version(2, 12, 0) == NULL)
gtk_widget_set_has_tooltip(w, FALSE);
} }
else
#endif #endif
{
gtk_tooltips_set_tip(tips, w, tip, NULL);
}
} }
#endif // wxUSE_TOOLTIPS #endif // wxUSE_TOOLTIPS