Don't force the height of the action area, we don't scroll it anyway, and

2004-10-31  Matthias Clasen  <mclasen@redhat.com>

	* gtk/gtkentrycompletion.c (_gtk_entry_completion_resize_popup):
	Don't force the height of the action area, we don't scroll
	it anyway, and the current way of calculating the height can lead
	to broken display.  (#156573, Christian Persch)
This commit is contained in:
Matthias Clasen 2004-10-31 19:10:29 +00:00 committed by Matthias Clasen
parent d9f91432c9
commit 489d586f1f
5 changed files with 24 additions and 10 deletions

View File

@ -1,5 +1,10 @@
2004-10-31 Matthias Clasen <mclasen@redhat.com> 2004-10-31 Matthias Clasen <mclasen@redhat.com>
* gtk/gtkentrycompletion.c (_gtk_entry_completion_resize_popup):
Don't force the height of the action area, we don't scroll
it anyway, and the current way of calculating the height can lead
to broken display. (#156573, Christian Persch)
* tests/testentrycompletion.c (animation_timer): Don't try to * tests/testentrycompletion.c (animation_timer): Don't try to
remove stuff from an empty model. (#156575, Christian Persch) remove stuff from an empty model. (#156575, Christian Persch)

View File

@ -1,5 +1,10 @@
2004-10-31 Matthias Clasen <mclasen@redhat.com> 2004-10-31 Matthias Clasen <mclasen@redhat.com>
* gtk/gtkentrycompletion.c (_gtk_entry_completion_resize_popup):
Don't force the height of the action area, we don't scroll
it anyway, and the current way of calculating the height can lead
to broken display. (#156573, Christian Persch)
* tests/testentrycompletion.c (animation_timer): Don't try to * tests/testentrycompletion.c (animation_timer): Don't try to
remove stuff from an empty model. (#156575, Christian Persch) remove stuff from an empty model. (#156575, Christian Persch)

View File

@ -1,5 +1,10 @@
2004-10-31 Matthias Clasen <mclasen@redhat.com> 2004-10-31 Matthias Clasen <mclasen@redhat.com>
* gtk/gtkentrycompletion.c (_gtk_entry_completion_resize_popup):
Don't force the height of the action area, we don't scroll
it anyway, and the current way of calculating the height can lead
to broken display. (#156573, Christian Persch)
* tests/testentrycompletion.c (animation_timer): Don't try to * tests/testentrycompletion.c (animation_timer): Don't try to
remove stuff from an empty model. (#156575, Christian Persch) remove stuff from an empty model. (#156575, Christian Persch)

View File

@ -1,5 +1,10 @@
2004-10-31 Matthias Clasen <mclasen@redhat.com> 2004-10-31 Matthias Clasen <mclasen@redhat.com>
* gtk/gtkentrycompletion.c (_gtk_entry_completion_resize_popup):
Don't force the height of the action area, we don't scroll
it anyway, and the current way of calculating the height can lead
to broken display. (#156573, Christian Persch)
* tests/testentrycompletion.c (animation_timer): Don't try to * tests/testentrycompletion.c (animation_timer): Don't try to
remove stuff from an empty model. (#156575, Christian Persch) remove stuff from an empty model. (#156575, Christian Persch)

View File

@ -1284,6 +1284,7 @@ _gtk_entry_completion_resize_popup (GtkEntryCompletion *completion)
gtk_tree_view_column_cell_get_size (completion->priv->column, NULL, gtk_tree_view_column_cell_get_size (completion->priv->column, NULL,
NULL, NULL, NULL, &height); NULL, NULL, NULL, &height);
g_print ("item height %d\n", height);
if (items <= 0) if (items <= 0)
gtk_widget_hide (completion->priv->scrolled_window); gtk_widget_hide (completion->priv->scrolled_window);
else else
@ -1294,9 +1295,8 @@ _gtk_entry_completion_resize_popup (GtkEntryCompletion *completion)
GTK_WIDGET (completion->priv->entry)->window); GTK_WIDGET (completion->priv->entry)->window);
gdk_screen_get_monitor_geometry (screen, monitor_num, &monitor); gdk_screen_get_monitor_geometry (screen, monitor_num, &monitor);
width = MIN (completion->priv->entry->allocation.width, monitor.width); width = MIN (completion->priv->entry->allocation.width, monitor.width) - 2 * x_border;
gtk_widget_set_size_request (completion->priv->tree_view, gtk_widget_set_size_request (completion->priv->tree_view, width, items * height);
width - 2 * x_border, items * height);
/* default on no match */ /* default on no match */
completion->priv->current_selected = -1; completion->priv->current_selected = -1;
@ -1306,13 +1306,7 @@ _gtk_entry_completion_resize_popup (GtkEntryCompletion *completion)
if (items) if (items)
{ {
gtk_widget_show (completion->priv->action_view); gtk_widget_show (completion->priv->action_view);
gtk_widget_set_size_request (completion->priv->action_view, width, -1);
gtk_tree_view_column_cell_get_size (gtk_tree_view_get_column (GTK_TREE_VIEW (completion->priv->action_view), 0),
NULL, NULL, NULL, NULL,
&height);
gtk_widget_set_size_request (completion->priv->action_view,
width - 2 * x_border, items * height);
} }
else else
gtk_widget_hide (completion->priv->action_view); gtk_widget_hide (completion->priv->action_view);