[gedit] OpenDocumentSelector: ensure gtk_style_context_get*() with good state
- From: Sébastien Wilmet <swilmet src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gedit] OpenDocumentSelector: ensure gtk_style_context_get*() with good state
- Date: Sun, 27 Mar 2016 09:03:23 +0000 (UTC)
commit 4f08109d666f51937b2d3b2696f122f13941ecb7
Author: Sébastien Wilmet <swilmet gnome org>
Date: Sun Mar 27 10:56:57 2016 +0200
OpenDocumentSelector: ensure gtk_style_context_get*() with good state
The code was normally good, but call gtk_style_context_get_state()
inside the call to gtk_style_context_get*(), so we are 100% sure it's
the good state.
http://feaneron.com/2016/01/04/quick-guide-to-port-an-app-for-gtk-3-20/
gedit/gedit-open-document-selector.c | 32 +++++++++++++++++++++++---------
1 files changed, 23 insertions(+), 9 deletions(-)
---
diff --git a/gedit/gedit-open-document-selector.c b/gedit/gedit-open-document-selector.c
index 969dffa..9d22bfd 100644
--- a/gedit/gedit-open-document-selector.c
+++ b/gedit/gedit-open-document-selector.c
@@ -1008,7 +1008,6 @@ on_treeview_allocate (GtkWidget *widget G_GNUC_UNUSED,
{
GeditOpenDocumentSelectorStore *selector_store;
GtkStyleContext *context;
- GtkStateFlags state;
gint name_renderer_natural_size;
gint path_renderer_natural_size;
GtkBorder padding;
@@ -1022,7 +1021,9 @@ on_treeview_allocate (GtkWidget *widget G_GNUC_UNUSED,
selector_store = selector->selector_store;
context = gtk_widget_get_style_context (selector->treeview);
- state = gtk_style_context_get_state (context);
+ gtk_style_context_get_padding (context,
+ gtk_style_context_get_state (context),
+ &padding);
/* Treeview height computation */
gtk_cell_renderer_get_preferred_height (selector->name_renderer,
@@ -1035,7 +1036,6 @@ on_treeview_allocate (GtkWidget *widget G_GNUC_UNUSED,
NULL,
&path_renderer_natural_size);
- gtk_style_context_get_padding (context, state, &padding);
gtk_cell_renderer_get_padding (selector->name_renderer, NULL, &ypad);
gtk_widget_style_get (selector->treeview, "grid-line-width", &grid_line_width, NULL);
@@ -1062,23 +1062,37 @@ on_treeview_style_updated (GtkWidget *widget,
GeditOpenDocumentSelector *selector)
{
GtkStyleContext *context;
- GtkStateFlags state;
context = gtk_widget_get_style_context (widget);
- state = gtk_style_context_get_state (context);
/* Name label foreground and font size styling */
gtk_style_context_save (context);
gtk_style_context_add_class (context, "open-document-selector-name-label");
- gtk_style_context_get_color (context, state, &selector->name_label_color);
- gtk_style_context_get (context, state, "font-size", &selector->name_font_size, NULL);
+
+ gtk_style_context_get_color (context,
+ gtk_style_context_get_state (context),
+ &selector->name_label_color);
+
+ gtk_style_context_get (context,
+ gtk_style_context_get_state (context),
+ "font-size", &selector->name_font_size,
+ NULL);
+
gtk_style_context_restore (context);
/* Path label foreground and font size styling */
gtk_style_context_save (context);
gtk_style_context_add_class (context, "open-document-selector-path-label");
- gtk_style_context_get_color (context, state, &selector->path_label_color);
- gtk_style_context_get (context, state, "font-size", &selector->path_font_size, NULL);
+
+ gtk_style_context_get_color (context,
+ gtk_style_context_get_state (context),
+ &selector->path_label_color);
+
+ gtk_style_context_get (context,
+ gtk_style_context_get_state (context),
+ "font-size", &selector->path_font_size,
+ NULL);
+
gtk_style_context_restore (context);
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]