diff options
author | Benjamin Otte <otte@redhat.com> | 2015-07-06 04:29:09 +0200 |
---|---|---|
committer | Benjamin Otte <otte@redhat.com> | 2015-07-06 04:29:09 +0200 |
commit | 3c2535e80804d8cfda2cafcae28dd29178e83fa9 (patch) | |
tree | 0de7331588f3ce413330680ee6fe95a027df7e6d | |
parent | 5188c84df16eac75fc0d402ad9ccc95f6b4e67eb (diff) | |
download | gtk+-3c2535e80804d8cfda2cafcae28dd29178e83fa9.tar.gz |
cssnode: Do finer-grained position invalidation
As a side effect, this should fix this bug:
https://bugzilla.gnome.org/show_bug.cgi?id=751910
-rw-r--r-- | gtk/gtkcssnode.c | 31 |
1 files changed, 26 insertions, 5 deletions
diff --git a/gtk/gtkcssnode.c b/gtk/gtkcssnode.c index 50c38915af..29301eba7e 100644 --- a/gtk/gtkcssnode.c +++ b/gtk/gtkcssnode.c @@ -643,15 +643,22 @@ gtk_css_node_reposition (GtkCssNode *node, /* Take a reference here so the whole function has a reference */ g_object_ref (node); - if (node->next_sibling) + if (node->visible) { - if (node->visible) - gtk_css_node_invalidate (node->next_sibling, GTK_CSS_CHANGE_POSITION | GTK_CSS_CHANGE_ANY_SIBLING); + if (node->next_sibling) + gtk_css_node_invalidate (node->next_sibling, + GTK_CSS_CHANGE_ANY_SIBLING + | GTK_CSS_CHANGE_NTH_CHILD + | (node->previous_sibling ? 0 : GTK_CSS_CHANGE_FIRST_CHILD)); + else if (node->previous_sibling) + gtk_css_node_invalidate (node->previous_sibling, GTK_CSS_CHANGE_LAST_CHILD); } if (old_parent != NULL) { g_signal_emit (old_parent, cssnode_signals[NODE_REMOVED], 0, node, node->previous_sibling); + if (old_parent->first_child) + gtk_css_node_invalidate (old_parent->first_child, GTK_CSS_CHANGE_NTH_LAST_CHILD); } if (old_parent != new_parent) @@ -687,12 +694,26 @@ gtk_css_node_reposition (GtkCssNode *node, if (new_parent) { g_signal_emit (new_parent, cssnode_signals[NODE_ADDED], 0, node, previous); + gtk_css_node_invalidate (new_parent->first_child, GTK_CSS_CHANGE_NTH_LAST_CHILD); } if (node->next_sibling) - gtk_css_node_invalidate_style (node->next_sibling); + { + if (node->previous_sibling == NULL) + gtk_css_node_invalidate (node->next_sibling, GTK_CSS_CHANGE_FIRST_CHILD); + else + gtk_css_node_invalidate_style (node->next_sibling); + } + else if (node->previous_sibling) + { + gtk_css_node_invalidate (node->previous_sibling, GTK_CSS_CHANGE_LAST_CHILD); + } - gtk_css_node_invalidate (node, GTK_CSS_CHANGE_ANY_PARENT | GTK_CSS_CHANGE_POSITION | GTK_CSS_CHANGE_ANY_SIBLING); + gtk_css_node_invalidate (node, GTK_CSS_CHANGE_ANY_PARENT + | GTK_CSS_CHANGE_ANY_SIBLING + | GTK_CSS_CHANGE_NTH_CHILD + | (node->previous_sibling ? 0 : GTK_CSS_CHANGE_FIRST_CHILD) + | (node->next_sibling ? 0 : GTK_CSS_CHANGE_LAST_CHILD)); g_object_unref (node); } |