brl/gnome-shell
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

st/theme-context: Also remove theme nodes on stylesheet changes

Browse Source 
StThemeNodes are built around the assumption that they're "immutable",
that means they are created once with certain parameters (that resolve
to certain css properties) and then replaced with new ones in case those
parameters changed.

Changes to the internal information stored by theme nodes (i.e. the css
properties, font names or the cached scale-factor) are not all handled
the same though: For changes to the font or the scale-factor we remove
all theme nodes from the cache and let the widgets which are on stage
generate new theme nodes. For changes to the css properties/the
stylesheet, we invalidate the properties of all theme nodes but keep
them in the cache using `_st_theme_node_reset_for_stylesheet_change()`.

So be a bit more consistent and handle changes to the css-properties/the
stylesheet stored by StThemeNodes the same way as changes to the font or
scale-factor.

https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1218
This commit is contained in:
Jonas Dreßler 2020-04-25 13:17:13 +02:00 committed by Florian Müllner
parent 325ff73c5b
commit 4dfa39457e
3 changed files with 4 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
src/st/st-theme-context.c
View File

@ -290,22 +290,6 @@ on_icon_theme_changed (StTextureCache *cache,
g_source_set_name_by_id (id, "[gnome-shell] changed_idle");
}
static void
on_custom_stylesheets_changed (StTheme *theme,
StThemeContext *context)
{
GHashTableIter iter;
StThemeNode *node;
if (context->root_node)
_st_theme_node_reset_for_stylesheet_change (context->root_node);
g_hash_table_iter_init (&iter, context->nodes);
while (g_hash_table_iter_next (&iter, (gpointer *) &node, NULL))
_st_theme_node_reset_for_stylesheet_change (node);
}
/**
* st_theme_context_get_for_stage:
* @stage: a #ClutterStage
@ -358,9 +342,10 @@ st_theme_context_set_theme (StThemeContext *context,
if (context->theme)
{
context->stylesheets_changed_id =
g_signal_connect (context->theme, "custom-stylesheets-changed",
G_CALLBACK (on_custom_stylesheets_changed),
context);
g_signal_connect_swapped (context->theme,
"custom-stylesheets-changed",
G_CALLBACK (st_theme_context_changed),
context);
}
st_theme_context_changed (context);

1
src/st/st-theme-node-private.h
View File

@ -125,7 +125,6 @@ void _st_theme_node_ensure_background (StThemeNode *node);
void _st_theme_node_ensure_geometry (StThemeNode *node);
void _st_theme_node_apply_margins (StThemeNode *node,
ClutterActor *actor);
void _st_theme_node_reset_for_stylesheet_change (StThemeNode *node);
G_END_DECLS

7
src/st/st-theme-node.c
View File

@ -77,13 +77,6 @@ maybe_free_properties (StThemeNode *node)
}
}
void
_st_theme_node_reset_for_stylesheet_change (StThemeNode *node)
{
maybe_free_properties (node);
node->properties_computed = FALSE;
}
static void
st_theme_node_dispose (GObject *gobject)
{