callisto-debian/gtk3
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

treeview: don't use gtk_render_focus() for dnd indicator

Browse Source 
There's no reason this should be a focus ring rather than an actual
frame. In the past this was probably used to get a dashed effect, but
now we even support that natively for borders.
This commit is contained in:
Cosimo Cecchi
2012-02-13 22:43:07 -05:00
parent a3b097639f
commit 713c532940
1 changed files with 4 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
gtk/gtktreeview.c
View File

@ -4693,7 +4693,6 @@ gtk_tree_view_bin_draw (GtkWidget *widget,
GList *first_column, *last_column; GList *first_column, *last_column;
gint vertical_separator; gint vertical_separator;
gint horizontal_separator; gint horizontal_separator;
gint focus_line_width;
gboolean allow_rules; gboolean allow_rules;
gboolean has_can_focus_cell; gboolean has_can_focus_cell;
gboolean rtl; gboolean rtl;
@ -4710,7 +4709,6 @@ gtk_tree_view_bin_draw (GtkWidget *widget,
"horizontal-separator", &horizontal_separator, "horizontal-separator", &horizontal_separator,
"vertical-separator", &vertical_separator, "vertical-separator", &vertical_separator,
"allow-rules", &allow_rules, "allow-rules", &allow_rules,
"focus-line-width", &focus_line_width,
NULL); NULL);
if (tree_view->priv->tree == NULL) if (tree_view->priv->tree == NULL)
@ -5198,12 +5196,10 @@ gtk_tree_view_bin_draw (GtkWidget *widget,
if (tree == NULL) if (tree == NULL)
break; break;
gtk_render_focus (context, cr, gtk_render_frame (context, cr,
0, gtk_tree_view_get_row_y_offset (tree_view, tree, node) 0, gtk_tree_view_get_row_y_offset (tree_view, tree, node),
- focus_line_width / 2,
gdk_window_get_width (tree_view->priv->bin_window), gdk_window_get_width (tree_view->priv->bin_window),
gtk_tree_view_get_row_height (tree_view, node) gtk_tree_view_get_row_height (tree_view, node));
- focus_line_width + 1);
break; break;
} }