Uff, you're right, there were some missing parts from previous patch. Some remarks:
>moz_gtk_entry_paint:
>+ //gtk_render_background(style, cr, rect->x + x, rect->y + y, rect->width - 2*x, rect->height - 2*y);
>+ //gtk_render_frame(style, cr, rect->x + x, rect->y + y, rect->width - 2*x, rect->height - 2*y);
> Remove this if it is not used or add a TODO comment explaining why it is kept.
> I can't make much sense of what is happening with the focus_width and the
> rectangles in this function. The old logic looked closer to what we are going
> to need, so I'd prefer to stay with that until this is sorted out.
Removed.
In moz_gtk_treeview_paint:
>- gtk_widget_modify_bg(gTreeViewWidget, state_type,
>- &gTreeViewWidget->style->base[state_type]);
>+ gtk_style_context_get_background_color(style, state_type, &base_col);
>+ gtk_widget_override_background_color(gScrolledWindowWidget, state_type, &base_col);
>
>Is this necessary?
>If so, it looks like it should be applied to the TreeView widget.
Not necessary, removed.
>+ gtk_render_frame(style_tree, cr,
>+ rect->x + xthickness, rect->y + ythickness,
>+ rect->width - 2 * xthickness,
>+ rect->height - 2 * ythickness);
>
>Is this necessary?
>The only gtk_render_frame I see in GtkScrolledWindow is for the rubber band.
Not necessary, removed.
moz_gtk_tab_paint:
> Are you sure about GTK_STATE_FLAG_INSENSITIVE for unselected tabs? Where does
> that come from? I would have expected _NORMAL.
It doesn't matter, changed to _NORMAL.
> moz_gtk_menu_separator_paint:
> + gtk_style_context_add_class(style, GTK_STYLE_CLASS_DEFAULT);
> What is this for?
Created attachment 568338
gkt3drawing, v.4
Uff, you're right, there were some missing parts from previous patch. Some remarks:
>moz_gtk_ entry_paint: background( style, cr, rect->x + x, rect->y + y, rect->width - 2*x, rect->height - 2*y); frame(style, cr, rect->x + x, rect->y + y, rect->width - 2*x, rect->height - 2*y);
>+ //gtk_render_
>+ //gtk_render_
> Remove this if it is not used or add a TODO comment explaining why it is kept.
> I can't make much sense of what is happening with the focus_width and the
> rectangles in this function. The old logic looked closer to what we are going
> to need, so I'd prefer to stay with that until this is sorted out.
Removed.
In moz_gtk_ treeview_ paint:
>- gtk_widget_ modify_ bg(gTreeViewWid get, state_type, t->style- >base[state_ type]); context_ get_background_ color(style, state_type, &base_col); override_ background_ color(gScrolled WindowWidget, state_type, &base_col);
>- &gTreeViewWidge
>+ gtk_style_
>+ gtk_widget_
>
>Is this necessary?
>If so, it looks like it should be applied to the TreeView widget.
Not necessary, removed.
>+ gtk_render_ frame(style_ tree, cr,
>+ rect->x + xthickness, rect->y + ythickness,
>+ rect->width - 2 * xthickness,
>+ rect->height - 2 * ythickness);
>
>Is this necessary?
>The only gtk_render_frame I see in GtkScrolledWindow is for the rubber band.
Not necessary, removed.
moz_gtk_tab_paint:
> Are you sure about GTK_STATE_ FLAG_INSENSITIV E for unselected tabs? Where does
> that come from? I would have expected _NORMAL.
It doesn't matter, changed to _NORMAL.
> moz_gtk_ menu_separator_ paint: context_ add_class( style, GTK_STYLE_ CLASS_DEFAULT) ;
> + gtk_style_
> What is this for?
Some leftover I guess, removed.