From cd45a70f2d9f2fa98b46c90842fa739e9542f285 Mon Sep 17 00:00:00 2001 From: Yuri Sizov Date: Mon, 4 Oct 2021 17:09:32 +0300 Subject: [PATCH 1/3] Reorganize Theme resource code for better maintainability --- scene/resources/theme.cpp | 668 +++++++++++++++++++------------------- scene/resources/theme.h | 6 +- 2 files changed, 347 insertions(+), 327 deletions(-) diff --git a/scene/resources/theme.cpp b/scene/resources/theme.cpp index 9ec47673bb..58d305a2f7 100644 --- a/scene/resources/theme.cpp +++ b/scene/resources/theme.cpp @@ -29,221 +29,18 @@ /*************************************************************************/ #include "theme.h" -#include "core/os/file_access.h" #include "core/print_string.h" -void Theme::_emit_theme_changed() { - if (no_change_propagation) { - return; - } +// Universal Theme resources used when no other theme has the item. +Ref Theme::default_theme; +Ref Theme::project_default_theme; - _change_notify(); - emit_changed(); -} - -PoolVector Theme::_get_icon_list(const String &p_node_type) const { - PoolVector ilret; - List il; - - get_icon_list(p_node_type, &il); - ilret.resize(il.size()); - - int i = 0; - PoolVector::Write w = ilret.write(); - for (List::Element *E = il.front(); E; E = E->next(), i++) { - w[i] = E->get(); - } - return ilret; -} - -PoolVector Theme::_get_icon_types() const { - PoolVector ilret; - List il; - - get_icon_types(&il); - ilret.resize(il.size()); - - int i = 0; - PoolVector::Write w = ilret.write(); - for (List::Element *E = il.front(); E; E = E->next(), i++) { - w[i] = E->get(); - } - return ilret; -} - -PoolVector Theme::_get_stylebox_list(const String &p_node_type) const { - PoolVector ilret; - List il; - - get_stylebox_list(p_node_type, &il); - ilret.resize(il.size()); - - int i = 0; - PoolVector::Write w = ilret.write(); - for (List::Element *E = il.front(); E; E = E->next(), i++) { - w[i] = E->get(); - } - return ilret; -} - -PoolVector Theme::_get_stylebox_types() const { - PoolVector ilret; - List il; - - get_stylebox_types(&il); - ilret.resize(il.size()); - - int i = 0; - PoolVector::Write w = ilret.write(); - for (List::Element *E = il.front(); E; E = E->next(), i++) { - w[i] = E->get(); - } - return ilret; -} - -PoolVector Theme::_get_font_list(const String &p_node_type) const { - PoolVector ilret; - List il; - - get_font_list(p_node_type, &il); - ilret.resize(il.size()); - - int i = 0; - PoolVector::Write w = ilret.write(); - for (List::Element *E = il.front(); E; E = E->next(), i++) { - w[i] = E->get(); - } - return ilret; -} - -PoolVector Theme::_get_font_types() const { - PoolVector ilret; - List il; - - get_font_types(&il); - ilret.resize(il.size()); - - int i = 0; - PoolVector::Write w = ilret.write(); - for (List::Element *E = il.front(); E; E = E->next(), i++) { - w[i] = E->get(); - } - return ilret; -} - -PoolVector Theme::_get_color_list(const String &p_node_type) const { - PoolVector ilret; - List il; - - get_color_list(p_node_type, &il); - ilret.resize(il.size()); - - int i = 0; - PoolVector::Write w = ilret.write(); - for (List::Element *E = il.front(); E; E = E->next(), i++) { - w[i] = E->get(); - } - return ilret; -} - -PoolVector Theme::_get_color_types() const { - PoolVector ilret; - List il; - - get_color_types(&il); - ilret.resize(il.size()); - - int i = 0; - PoolVector::Write w = ilret.write(); - for (List::Element *E = il.front(); E; E = E->next(), i++) { - w[i] = E->get(); - } - return ilret; -} - -PoolVector Theme::_get_constant_list(const String &p_node_type) const { - PoolVector ilret; - List il; - - get_constant_list(p_node_type, &il); - ilret.resize(il.size()); - - int i = 0; - PoolVector::Write w = ilret.write(); - for (List::Element *E = il.front(); E; E = E->next(), i++) { - w[i] = E->get(); - } - return ilret; -} - -PoolVector Theme::_get_constant_types() const { - PoolVector ilret; - List il; - - get_constant_types(&il); - ilret.resize(il.size()); - - int i = 0; - PoolVector::Write w = ilret.write(); - for (List::Element *E = il.front(); E; E = E->next(), i++) { - w[i] = E->get(); - } - return ilret; -} - -PoolVector Theme::_get_theme_item_list(DataType p_data_type, const String &p_node_type) const { - switch (p_data_type) { - case DATA_TYPE_COLOR: - return _get_color_list(p_node_type); - case DATA_TYPE_CONSTANT: - return _get_constant_list(p_node_type); - case DATA_TYPE_FONT: - return _get_font_list(p_node_type); - case DATA_TYPE_ICON: - return _get_icon_list(p_node_type); - case DATA_TYPE_STYLEBOX: - return _get_stylebox_list(p_node_type); - case DATA_TYPE_MAX: - break; // Can't happen, but silences warning. - } - - return PoolVector(); -} - -PoolVector Theme::_get_theme_item_types(DataType p_data_type) const { - switch (p_data_type) { - case DATA_TYPE_COLOR: - return _get_color_types(); - case DATA_TYPE_CONSTANT: - return _get_constant_types(); - case DATA_TYPE_FONT: - return _get_font_types(); - case DATA_TYPE_ICON: - return _get_icon_types(); - case DATA_TYPE_STYLEBOX: - return _get_stylebox_types(); - case DATA_TYPE_MAX: - break; // Can't happen, but silences warning. - } - - return PoolVector(); -} - -PoolVector Theme::_get_type_list(const String &p_node_type) const { - PoolVector ilret; - List il; - - get_type_list(&il); - ilret.resize(il.size()); - - int i = 0; - PoolVector::Write w = ilret.write(); - for (List::Element *E = il.front(); E; E = E->next(), i++) { - w[i] = E->get(); - } - return ilret; -} +// Universal default values, final fallback for every theme. +Ref Theme::default_icon; +Ref Theme::default_style; +Ref Theme::default_font; +// Dynamic properties. bool Theme::_set(const StringName &p_name, const Variant &p_value) { String sname = p_name; @@ -371,6 +168,37 @@ void Theme::_get_property_list(List *p_list) const { } } +// Universal fallback Theme resources. +Ref Theme::get_default() { + return default_theme; +} + +void Theme::set_default(const Ref &p_default) { + default_theme = p_default; +} + +Ref Theme::get_project_default() { + return project_default_theme; +} + +void Theme::set_project_default(const Ref &p_project_default) { + project_default_theme = p_project_default; +} + +// Universal fallback values for theme item types. +void Theme::set_default_icon(const Ref &p_icon) { + default_icon = p_icon; +} + +void Theme::set_default_style(const Ref &p_style) { + default_style = p_style; +} + +void Theme::set_default_font(const Ref &p_font) { + default_font = p_font; +} + +// Fallback values for theme item types, configurable per theme. void Theme::set_default_theme_font(const Ref &p_default_font) { if (default_theme_font == p_default_font) { return; @@ -393,40 +221,7 @@ Ref Theme::get_default_theme_font() const { return default_theme_font; } -Ref Theme::project_default_theme; -Ref Theme::default_theme; -Ref Theme::default_icon; -Ref Theme::default_style; -Ref Theme::default_font; - -Ref Theme::get_default() { - return default_theme; -} - -void Theme::set_default(const Ref &p_default) { - default_theme = p_default; -} - -Ref Theme::get_project_default() { - return project_default_theme; -} - -void Theme::set_project_default(const Ref &p_project_default) { - project_default_theme = p_project_default; -} - -void Theme::set_default_icon(const Ref &p_icon) { - default_icon = p_icon; -} - -void Theme::set_default_style(const Ref &p_style) { - default_style = p_style; -} - -void Theme::set_default_font(const Ref &p_font) { - default_font = p_font; -} - +// Icons. void Theme::set_icon(const StringName &p_name, const StringName &p_node_type, const Ref &p_icon) { if (icon_map[p_node_type].has(p_name) && icon_map[p_node_type][p_name].is_valid()) { icon_map[p_node_type][p_name]->disconnect("changed", this, "_emit_theme_changed"); @@ -511,6 +306,7 @@ void Theme::get_icon_types(List *p_list) const { } } +// Shaders. void Theme::set_shader(const StringName &p_name, const StringName &p_node_type, const Ref &p_shader) { shader_map[p_node_type][p_name] = p_shader; @@ -552,6 +348,7 @@ void Theme::get_shader_list(const StringName &p_node_type, List *p_l } } +// Styleboxes. void Theme::set_stylebox(const StringName &p_name, const StringName &p_node_type, const Ref &p_style) { if (style_map[p_node_type].has(p_name) && style_map[p_node_type][p_name].is_valid()) { style_map[p_node_type][p_name]->disconnect("changed", this, "_emit_theme_changed"); @@ -636,6 +433,7 @@ void Theme::get_stylebox_types(List *p_list) const { } } +// Fonts. void Theme::set_font(const StringName &p_name, const StringName &p_node_type, const Ref &p_font) { if (font_map[p_node_type][p_name].is_valid()) { font_map[p_node_type][p_name]->disconnect("changed", this, "_emit_theme_changed"); @@ -722,6 +520,7 @@ void Theme::get_font_types(List *p_list) const { } } +// Colors. void Theme::set_color(const StringName &p_name, const StringName &p_node_type, const Color &p_color) { color_map[p_node_type][p_name] = p_color; @@ -794,6 +593,7 @@ void Theme::get_color_types(List *p_list) const { } } +// Theme constants. void Theme::set_constant(const StringName &p_name, const StringName &p_node_type, int p_constant) { constant_map[p_node_type][p_name] = p_constant; @@ -866,6 +666,7 @@ void Theme::get_constant_types(List *p_list) const { } } +// Generic methods for managing theme items. void Theme::set_theme_item(DataType p_data_type, const StringName &p_name, const StringName &p_node_type, const Variant &p_value) { switch (p_data_type) { case DATA_TYPE_COLOR: { @@ -1070,6 +871,260 @@ void Theme::get_theme_item_types(DataType p_data_type, List *p_list) } } +// Theme types. +void Theme::get_type_list(List *p_list) const { + ERR_FAIL_NULL(p_list); + + Set types; + const StringName *key = nullptr; + + while ((key = icon_map.next(key))) { + types.insert(*key); + } + + key = nullptr; + + while ((key = style_map.next(key))) { + types.insert(*key); + } + + key = nullptr; + + while ((key = font_map.next(key))) { + types.insert(*key); + } + + key = nullptr; + + while ((key = color_map.next(key))) { + types.insert(*key); + } + + key = nullptr; + + while ((key = constant_map.next(key))) { + types.insert(*key); + } + + for (Set::Element *E = types.front(); E; E = E->next()) { + p_list->push_back(E->get()); + } +} + +// Internal methods for getting lists as a Vector of String (compatible with public API). +PoolVector Theme::_get_icon_list(const String &p_node_type) const { + PoolVector ilret; + List il; + + get_icon_list(p_node_type, &il); + ilret.resize(il.size()); + + int i = 0; + PoolVector::Write w = ilret.write(); + for (List::Element *E = il.front(); E; E = E->next(), i++) { + w[i] = E->get(); + } + return ilret; +} + +PoolVector Theme::_get_icon_types() const { + PoolVector ilret; + List il; + + get_icon_types(&il); + ilret.resize(il.size()); + + int i = 0; + PoolVector::Write w = ilret.write(); + for (List::Element *E = il.front(); E; E = E->next(), i++) { + w[i] = E->get(); + } + return ilret; +} + +PoolVector Theme::_get_stylebox_list(const String &p_node_type) const { + PoolVector ilret; + List il; + + get_stylebox_list(p_node_type, &il); + ilret.resize(il.size()); + + int i = 0; + PoolVector::Write w = ilret.write(); + for (List::Element *E = il.front(); E; E = E->next(), i++) { + w[i] = E->get(); + } + return ilret; +} + +PoolVector Theme::_get_stylebox_types() const { + PoolVector ilret; + List il; + + get_stylebox_types(&il); + ilret.resize(il.size()); + + int i = 0; + PoolVector::Write w = ilret.write(); + for (List::Element *E = il.front(); E; E = E->next(), i++) { + w[i] = E->get(); + } + return ilret; +} + +PoolVector Theme::_get_font_list(const String &p_node_type) const { + PoolVector ilret; + List il; + + get_font_list(p_node_type, &il); + ilret.resize(il.size()); + + int i = 0; + PoolVector::Write w = ilret.write(); + for (List::Element *E = il.front(); E; E = E->next(), i++) { + w[i] = E->get(); + } + return ilret; +} + +PoolVector Theme::_get_font_types() const { + PoolVector ilret; + List il; + + get_font_types(&il); + ilret.resize(il.size()); + + int i = 0; + PoolVector::Write w = ilret.write(); + for (List::Element *E = il.front(); E; E = E->next(), i++) { + w[i] = E->get(); + } + return ilret; +} + +PoolVector Theme::_get_color_list(const String &p_node_type) const { + PoolVector ilret; + List il; + + get_color_list(p_node_type, &il); + ilret.resize(il.size()); + + int i = 0; + PoolVector::Write w = ilret.write(); + for (List::Element *E = il.front(); E; E = E->next(), i++) { + w[i] = E->get(); + } + return ilret; +} + +PoolVector Theme::_get_color_types() const { + PoolVector ilret; + List il; + + get_color_types(&il); + ilret.resize(il.size()); + + int i = 0; + PoolVector::Write w = ilret.write(); + for (List::Element *E = il.front(); E; E = E->next(), i++) { + w[i] = E->get(); + } + return ilret; +} + +PoolVector Theme::_get_constant_list(const String &p_node_type) const { + PoolVector ilret; + List il; + + get_constant_list(p_node_type, &il); + ilret.resize(il.size()); + + int i = 0; + PoolVector::Write w = ilret.write(); + for (List::Element *E = il.front(); E; E = E->next(), i++) { + w[i] = E->get(); + } + return ilret; +} + +PoolVector Theme::_get_constant_types() const { + PoolVector ilret; + List il; + + get_constant_types(&il); + ilret.resize(il.size()); + + int i = 0; + PoolVector::Write w = ilret.write(); + for (List::Element *E = il.front(); E; E = E->next(), i++) { + w[i] = E->get(); + } + return ilret; +} + +PoolVector Theme::_get_theme_item_list(DataType p_data_type, const String &p_node_type) const { + switch (p_data_type) { + case DATA_TYPE_COLOR: + return _get_color_list(p_node_type); + case DATA_TYPE_CONSTANT: + return _get_constant_list(p_node_type); + case DATA_TYPE_FONT: + return _get_font_list(p_node_type); + case DATA_TYPE_ICON: + return _get_icon_list(p_node_type); + case DATA_TYPE_STYLEBOX: + return _get_stylebox_list(p_node_type); + case DATA_TYPE_MAX: + break; // Can't happen, but silences warning. + } + + return PoolVector(); +} + +PoolVector Theme::_get_theme_item_types(DataType p_data_type) const { + switch (p_data_type) { + case DATA_TYPE_COLOR: + return _get_color_types(); + case DATA_TYPE_CONSTANT: + return _get_constant_types(); + case DATA_TYPE_FONT: + return _get_font_types(); + case DATA_TYPE_ICON: + return _get_icon_types(); + case DATA_TYPE_STYLEBOX: + return _get_stylebox_types(); + case DATA_TYPE_MAX: + break; // Can't happen, but silences warning. + } + + return PoolVector(); +} + +PoolVector Theme::_get_type_list(const String &p_node_type) const { + PoolVector ilret; + List il; + + get_type_list(&il); + ilret.resize(il.size()); + + int i = 0; + PoolVector::Write w = ilret.write(); + for (List::Element *E = il.front(); E; E = E->next(), i++) { + w[i] = E->get(); + } + return ilret; +} + +// Theme bulk manipulations. +void Theme::_emit_theme_changed() { + if (no_change_propagation) { + return; + } + + _change_notify(); + emit_changed(); +} + void Theme::_freeze_change_propagation() { no_change_propagation = true; } @@ -1079,57 +1134,6 @@ void Theme::_unfreeze_and_propagate_changes() { _emit_theme_changed(); } -void Theme::clear() { - //these need disconnecting - { - const StringName *K = nullptr; - while ((K = icon_map.next(K))) { - const StringName *L = nullptr; - while ((L = icon_map[*K].next(L))) { - Ref icon = icon_map[*K][*L]; - if (icon.is_valid()) { - icon->disconnect("changed", this, "_emit_theme_changed"); - } - } - } - } - - { - const StringName *K = nullptr; - while ((K = style_map.next(K))) { - const StringName *L = nullptr; - while ((L = style_map[*K].next(L))) { - Ref style = style_map[*K][*L]; - if (style.is_valid()) { - style->disconnect("changed", this, "_emit_theme_changed"); - } - } - } - } - - { - const StringName *K = nullptr; - while ((K = font_map.next(K))) { - const StringName *L = nullptr; - while ((L = font_map[*K].next(L))) { - Ref font = font_map[*K][*L]; - if (font.is_valid()) { - font->disconnect("changed", this, "_emit_theme_changed"); - } - } - } - } - - icon_map.clear(); - style_map.clear(); - font_map.clear(); - shader_map.clear(); - color_map.clear(); - constant_map.clear(); - - _emit_theme_changed(); -} - void Theme::copy_default_theme() { Ref default_theme2 = get_default(); copy_theme(default_theme2); @@ -1258,43 +1262,55 @@ void Theme::merge_with(const Ref &p_other) { _unfreeze_and_propagate_changes(); } -void Theme::get_type_list(List *p_list) const { - ERR_FAIL_NULL(p_list); - - Set types; - const StringName *key = nullptr; - - while ((key = icon_map.next(key))) { - types.insert(*key); +void Theme::clear() { + //these need disconnecting + { + const StringName *K = nullptr; + while ((K = icon_map.next(K))) { + const StringName *L = nullptr; + while ((L = icon_map[*K].next(L))) { + Ref icon = icon_map[*K][*L]; + if (icon.is_valid()) { + icon->disconnect("changed", this, "_emit_theme_changed"); + } + } + } } - key = nullptr; - - while ((key = style_map.next(key))) { - types.insert(*key); + { + const StringName *K = nullptr; + while ((K = style_map.next(K))) { + const StringName *L = nullptr; + while ((L = style_map[*K].next(L))) { + Ref style = style_map[*K][*L]; + if (style.is_valid()) { + style->disconnect("changed", this, "_emit_theme_changed"); + } + } + } } - key = nullptr; - - while ((key = font_map.next(key))) { - types.insert(*key); + { + const StringName *K = nullptr; + while ((K = font_map.next(K))) { + const StringName *L = nullptr; + while ((L = font_map[*K].next(L))) { + Ref font = font_map[*K][*L]; + if (font.is_valid()) { + font->disconnect("changed", this, "_emit_theme_changed"); + } + } + } } - key = nullptr; + icon_map.clear(); + style_map.clear(); + font_map.clear(); + shader_map.clear(); + color_map.clear(); + constant_map.clear(); - while ((key = color_map.next(key))) { - types.insert(*key); - } - - key = nullptr; - - while ((key = constant_map.next(key))) { - types.insert(*key); - } - - for (Set::Element *E = types.front(); E; E = E->next()) { - p_list->push_back(E->get()); - } + _emit_theme_changed(); } void Theme::_bind_methods() { diff --git a/scene/resources/theme.h b/scene/resources/theme.h index 12cbcb660b..7e24d1e0b5 100644 --- a/scene/resources/theme.h +++ b/scene/resources/theme.h @@ -90,12 +90,16 @@ protected: bool _get(const StringName &p_name, Variant &r_ret) const; void _get_property_list(List *p_list) const; - static Ref project_default_theme; + // Universal Theme resources used when no other theme has the item. static Ref default_theme; + static Ref project_default_theme; + + // Universal default values, final fallback for every theme. static Ref default_icon; static Ref default_style; static Ref default_font; + // Default values configurable for each individual theme. Ref default_theme_font; static void _bind_methods(); From 98cf645258c4ee3b9d13f2b1016b15ea212724c6 Mon Sep 17 00:00:00 2001 From: Yuri Sizov Date: Mon, 4 Oct 2021 17:44:53 +0300 Subject: [PATCH 2/3] Expose API to retrieve Theme's default font --- doc/classes/Control.xml | 7 +++++++ doc/classes/Theme.xml | 9 ++++++++- scene/gui/control.cpp | 34 ++++++++++++++++++++++++++++++++++ scene/gui/control.h | 2 ++ scene/resources/theme.cpp | 10 +++++++--- scene/resources/theme.h | 1 + 6 files changed, 59 insertions(+), 4 deletions(-) diff --git a/doc/classes/Control.xml b/doc/classes/Control.xml index 72993e2db1..18c2db33bd 100644 --- a/doc/classes/Control.xml +++ b/doc/classes/Control.xml @@ -362,6 +362,13 @@ See [method get_color] for details. + + + + Returns the default font from the first matching [Theme] in the tree if that [Theme] has a valid [member Theme.default_font] value. + See [method get_color] for details. + + diff --git a/doc/classes/Theme.xml b/doc/classes/Theme.xml index ca1ff60c68..74b0f187a6 100644 --- a/doc/classes/Theme.xml +++ b/doc/classes/Theme.xml @@ -238,6 +238,12 @@ Returns [code]false[/code] if the theme does not have [code]node_type[/code]. + + + + Returns [code]true[/code] if this theme has a valid [member default_font] value. + + @@ -403,7 +409,8 @@ - The theme's default font. + The default font of this [Theme] resource. Used as a fallback value for font items defined in this theme, but having invalid values. If this value is also invalid, the global default value is used. + Use [method has_default_font] to check if this value is valid. diff --git a/scene/gui/control.cpp b/scene/gui/control.cpp index 50bf20970b..84f0180c36 100644 --- a/scene/gui/control.cpp +++ b/scene/gui/control.cpp @@ -1299,6 +1299,37 @@ bool Control::has_constant(const StringName &p_name, const StringName &p_theme_t return Theme::get_default()->has_constant(p_name, type); } +Ref Control::get_theme_default_font() const { + // First, look through each control or window node in the branch, until no valid parent can be found. + // Only nodes with a theme resource attached are considered. + // For each theme resource see if their assigned theme has the default value defined and valid. + Control *theme_owner = data.theme_owner; + + while (theme_owner) { + if (theme_owner && theme_owner->data.theme->has_default_theme_font()) { + return theme_owner->data.theme->get_default_theme_font(); + } + + Node *parent = theme_owner->get_parent(); + Control *parent_c = Object::cast_to(parent); + if (parent_c) { + theme_owner = parent_c->data.theme_owner; + } else { + theme_owner = nullptr; + } + } + + // Secondly, check the project-defined Theme resource. + if (Theme::get_project_default().is_valid()) { + if (Theme::get_project_default()->has_default_theme_font()) { + return Theme::get_project_default()->get_default_theme_font(); + } + } + + // Lastly, fall back on the default Theme. + return Theme::get_default()->get_default_theme_font(); +} + Rect2 Control::get_parent_anchorable_rect() const { if (!is_inside_tree()) { return Rect2(); @@ -2793,6 +2824,8 @@ void Control::_bind_methods() { ClassDB::bind_method(D_METHOD("has_color", "name", "theme_type"), &Control::has_color, DEFVAL("")); ClassDB::bind_method(D_METHOD("has_constant", "name", "theme_type"), &Control::has_constant, DEFVAL("")); + ClassDB::bind_method(D_METHOD("get_theme_default_font"), &Control::get_theme_default_font); + ClassDB::bind_method(D_METHOD("get_parent_control"), &Control::get_parent_control); ClassDB::bind_method(D_METHOD("set_h_grow_direction", "direction"), &Control::set_h_grow_direction); @@ -2991,6 +3024,7 @@ void Control::_bind_methods() { BIND_VMETHOD(MethodInfo(Variant::BOOL, "has_point", PropertyInfo(Variant::VECTOR2, "point"))); } + Control::Control() { data.parent = nullptr; diff --git a/scene/gui/control.h b/scene/gui/control.h index 745897e878..51ef20f3fe 100644 --- a/scene/gui/control.h +++ b/scene/gui/control.h @@ -446,6 +446,8 @@ public: bool has_color(const StringName &p_name, const StringName &p_theme_type = StringName()) const; bool has_constant(const StringName &p_name, const StringName &p_theme_type = StringName()) const; + Ref get_theme_default_font() const; + /* TOOLTIP */ void set_tooltip(const String &p_tooltip); diff --git a/scene/resources/theme.cpp b/scene/resources/theme.cpp index 58d305a2f7..c1add622da 100644 --- a/scene/resources/theme.cpp +++ b/scene/resources/theme.cpp @@ -221,6 +221,10 @@ Ref Theme::get_default_theme_font() const { return default_theme_font; } +bool Theme::has_default_theme_font() const { + return default_theme_font.is_valid(); +} + // Icons. void Theme::set_icon(const StringName &p_name, const StringName &p_node_type, const Ref &p_icon) { if (icon_map[p_node_type].has(p_name) && icon_map[p_node_type][p_name].is_valid()) { @@ -451,7 +455,7 @@ void Theme::set_font(const StringName &p_name, const StringName &p_node_type, co Ref Theme::get_font(const StringName &p_name, const StringName &p_node_type) const { if (font_map.has(p_node_type) && font_map[p_node_type].has(p_name) && font_map[p_node_type][p_name].is_valid()) { return font_map[p_node_type][p_name]; - } else if (default_theme_font.is_valid()) { + } else if (has_default_theme_font()) { return default_theme_font; } else { return default_font; @@ -1354,10 +1358,9 @@ void Theme::_bind_methods() { ClassDB::bind_method(D_METHOD("get_constant_list", "node_type"), &Theme::_get_constant_list); ClassDB::bind_method(D_METHOD("get_constant_types"), &Theme::_get_constant_types); - ClassDB::bind_method(D_METHOD("clear"), &Theme::clear); - ClassDB::bind_method(D_METHOD("set_default_font", "font"), &Theme::set_default_theme_font); ClassDB::bind_method(D_METHOD("get_default_font"), &Theme::get_default_theme_font); + ClassDB::bind_method(D_METHOD("has_default_font"), &Theme::has_default_theme_font); ClassDB::bind_method(D_METHOD("set_theme_item", "data_type", "name", "node_type", "value"), &Theme::set_theme_item); ClassDB::bind_method(D_METHOD("get_theme_item", "data_type", "name", "node_type"), &Theme::get_theme_item); @@ -1374,6 +1377,7 @@ void Theme::_bind_methods() { ClassDB::bind_method("copy_default_theme", &Theme::copy_default_theme); ClassDB::bind_method(D_METHOD("copy_theme", "other"), &Theme::copy_theme); ClassDB::bind_method(D_METHOD("merge_with", "other"), &Theme::merge_with); + ClassDB::bind_method(D_METHOD("clear"), &Theme::clear); ADD_PROPERTY(PropertyInfo(Variant::OBJECT, "default_font", PROPERTY_HINT_RESOURCE_TYPE, "Font"), "set_default_font", "get_default_font"); diff --git a/scene/resources/theme.h b/scene/resources/theme.h index 7e24d1e0b5..69c582ee6e 100644 --- a/scene/resources/theme.h +++ b/scene/resources/theme.h @@ -120,6 +120,7 @@ public: void set_default_theme_font(const Ref &p_default_font); Ref get_default_theme_font() const; + bool has_default_theme_font() const; void set_icon(const StringName &p_name, const StringName &p_node_type, const Ref &p_icon); Ref get_icon(const StringName &p_name, const StringName &p_node_type) const; From 457d48a1f110a0d1d66d15becf0c435692aded64 Mon Sep 17 00:00:00 2001 From: Yuri Sizov Date: Mon, 4 Oct 2021 18:05:23 +0300 Subject: [PATCH 3/3] Make Theme report property list changes less often --- scene/resources/theme.cpp | 55 +++++++++++++++++++++++---------------- scene/resources/theme.h | 2 +- 2 files changed, 34 insertions(+), 23 deletions(-) diff --git a/scene/resources/theme.cpp b/scene/resources/theme.cpp index c1add622da..f30df692fd 100644 --- a/scene/resources/theme.cpp +++ b/scene/resources/theme.cpp @@ -227,7 +227,9 @@ bool Theme::has_default_theme_font() const { // Icons. void Theme::set_icon(const StringName &p_name, const StringName &p_node_type, const Ref &p_icon) { + bool existing = false; if (icon_map[p_node_type].has(p_name) && icon_map[p_node_type][p_name].is_valid()) { + existing = true; icon_map[p_node_type][p_name]->disconnect("changed", this, "_emit_theme_changed"); } @@ -237,7 +239,7 @@ void Theme::set_icon(const StringName &p_name, const StringName &p_node_type, co icon_map[p_node_type][p_name]->connect("changed", this, "_emit_theme_changed", varray(), CONNECT_REFERENCE_COUNTED); } - _emit_theme_changed(); + _emit_theme_changed(!existing); } Ref Theme::get_icon(const StringName &p_name, const StringName &p_node_type) const { @@ -264,7 +266,7 @@ void Theme::rename_icon(const StringName &p_old_name, const StringName &p_name, icon_map[p_node_type][p_name] = icon_map[p_node_type][p_old_name]; icon_map[p_node_type].erase(p_old_name); - _emit_theme_changed(); + _emit_theme_changed(true); } void Theme::clear_icon(const StringName &p_name, const StringName &p_node_type) { @@ -277,7 +279,7 @@ void Theme::clear_icon(const StringName &p_name, const StringName &p_node_type) icon_map[p_node_type].erase(p_name); - _emit_theme_changed(); + _emit_theme_changed(true); } void Theme::get_icon_list(StringName p_node_type, List *p_list) const { @@ -312,9 +314,10 @@ void Theme::get_icon_types(List *p_list) const { // Shaders. void Theme::set_shader(const StringName &p_name, const StringName &p_node_type, const Ref &p_shader) { + bool existing = (shader_map.has(p_node_type) && shader_map[p_node_type].has(p_name)); shader_map[p_node_type][p_name] = p_shader; - _emit_theme_changed(); + _emit_theme_changed(!existing); } Ref Theme::get_shader(const StringName &p_name, const StringName &p_node_type) const { @@ -335,7 +338,7 @@ void Theme::clear_shader(const StringName &p_name, const StringName &p_node_type shader_map[p_node_type].erase(p_name); - _emit_theme_changed(); + _emit_theme_changed(true); } void Theme::get_shader_list(const StringName &p_node_type, List *p_list) const { @@ -354,7 +357,9 @@ void Theme::get_shader_list(const StringName &p_node_type, List *p_l // Styleboxes. void Theme::set_stylebox(const StringName &p_name, const StringName &p_node_type, const Ref &p_style) { + bool existing = false; if (style_map[p_node_type].has(p_name) && style_map[p_node_type][p_name].is_valid()) { + existing = true; style_map[p_node_type][p_name]->disconnect("changed", this, "_emit_theme_changed"); } @@ -364,7 +369,7 @@ void Theme::set_stylebox(const StringName &p_name, const StringName &p_node_type style_map[p_node_type][p_name]->connect("changed", this, "_emit_theme_changed", varray(), CONNECT_REFERENCE_COUNTED); } - _emit_theme_changed(); + _emit_theme_changed(!existing); } Ref Theme::get_stylebox(const StringName &p_name, const StringName &p_node_type) const { @@ -391,7 +396,7 @@ void Theme::rename_stylebox(const StringName &p_old_name, const StringName &p_na style_map[p_node_type][p_name] = style_map[p_node_type][p_old_name]; style_map[p_node_type].erase(p_old_name); - _emit_theme_changed(); + _emit_theme_changed(true); } void Theme::clear_stylebox(const StringName &p_name, const StringName &p_node_type) { @@ -404,7 +409,7 @@ void Theme::clear_stylebox(const StringName &p_name, const StringName &p_node_ty style_map[p_node_type].erase(p_name); - _emit_theme_changed(); + _emit_theme_changed(true); } void Theme::get_stylebox_list(StringName p_node_type, List *p_list) const { @@ -439,7 +444,9 @@ void Theme::get_stylebox_types(List *p_list) const { // Fonts. void Theme::set_font(const StringName &p_name, const StringName &p_node_type, const Ref &p_font) { + bool existing = false; if (font_map[p_node_type][p_name].is_valid()) { + existing = true; font_map[p_node_type][p_name]->disconnect("changed", this, "_emit_theme_changed"); } @@ -449,7 +456,7 @@ void Theme::set_font(const StringName &p_name, const StringName &p_node_type, co font_map[p_node_type][p_name]->connect("changed", this, "_emit_theme_changed", varray(), CONNECT_REFERENCE_COUNTED); } - _emit_theme_changed(); + _emit_theme_changed(!existing); } Ref Theme::get_font(const StringName &p_name, const StringName &p_node_type) const { @@ -478,7 +485,7 @@ void Theme::rename_font(const StringName &p_old_name, const StringName &p_name, font_map[p_node_type][p_name] = font_map[p_node_type][p_old_name]; font_map[p_node_type].erase(p_old_name); - _emit_theme_changed(); + _emit_theme_changed(true); } void Theme::clear_font(const StringName &p_name, const StringName &p_node_type) { @@ -491,7 +498,7 @@ void Theme::clear_font(const StringName &p_name, const StringName &p_node_type) font_map[p_node_type].erase(p_name); - _emit_theme_changed(); + _emit_theme_changed(true); } void Theme::get_font_list(StringName p_node_type, List *p_list) const { @@ -526,9 +533,10 @@ void Theme::get_font_types(List *p_list) const { // Colors. void Theme::set_color(const StringName &p_name, const StringName &p_node_type, const Color &p_color) { + bool existing = has_color_nocheck(p_name, p_node_type); color_map[p_node_type][p_name] = p_color; - _emit_theme_changed(); + _emit_theme_changed(!existing); } Color Theme::get_color(const StringName &p_name, const StringName &p_node_type) const { @@ -555,7 +563,7 @@ void Theme::rename_color(const StringName &p_old_name, const StringName &p_name, color_map[p_node_type][p_name] = color_map[p_node_type][p_old_name]; color_map[p_node_type].erase(p_old_name); - _emit_theme_changed(); + _emit_theme_changed(true); } void Theme::clear_color(const StringName &p_name, const StringName &p_node_type) { @@ -564,7 +572,7 @@ void Theme::clear_color(const StringName &p_name, const StringName &p_node_type) color_map[p_node_type].erase(p_name); - _emit_theme_changed(); + _emit_theme_changed(true); } void Theme::get_color_list(StringName p_node_type, List *p_list) const { @@ -599,9 +607,10 @@ void Theme::get_color_types(List *p_list) const { // Theme constants. void Theme::set_constant(const StringName &p_name, const StringName &p_node_type, int p_constant) { + bool existing = has_constant_nocheck(p_name, p_node_type); constant_map[p_node_type][p_name] = p_constant; - _emit_theme_changed(); + _emit_theme_changed(!existing); } int Theme::get_constant(const StringName &p_name, const StringName &p_node_type) const { @@ -628,7 +637,7 @@ void Theme::rename_constant(const StringName &p_old_name, const StringName &p_na constant_map[p_node_type][p_name] = constant_map[p_node_type][p_old_name]; constant_map[p_node_type].erase(p_old_name); - _emit_theme_changed(); + _emit_theme_changed(true); } void Theme::clear_constant(const StringName &p_name, const StringName &p_node_type) { @@ -637,7 +646,7 @@ void Theme::clear_constant(const StringName &p_name, const StringName &p_node_ty constant_map[p_node_type].erase(p_name); - _emit_theme_changed(); + _emit_theme_changed(true); } void Theme::get_constant_list(StringName p_node_type, List *p_list) const { @@ -1120,12 +1129,14 @@ PoolVector Theme::_get_type_list(const String &p_node_type) const { } // Theme bulk manipulations. -void Theme::_emit_theme_changed() { +void Theme::_emit_theme_changed(bool p_notify_list_changed) { if (no_change_propagation) { return; } - _change_notify(); + if (p_notify_list_changed) { + _change_notify(); + } emit_changed(); } @@ -1135,7 +1146,7 @@ void Theme::_freeze_change_propagation() { void Theme::_unfreeze_and_propagate_changes() { no_change_propagation = false; - _emit_theme_changed(); + _emit_theme_changed(true); } void Theme::copy_default_theme() { @@ -1314,7 +1325,7 @@ void Theme::clear() { color_map.clear(); constant_map.clear(); - _emit_theme_changed(); + _emit_theme_changed(true); } void Theme::_bind_methods() { @@ -1372,7 +1383,7 @@ void Theme::_bind_methods() { ClassDB::bind_method(D_METHOD("get_type_list", "node_type"), &Theme::_get_type_list); - ClassDB::bind_method(D_METHOD("_emit_theme_changed"), &Theme::_emit_theme_changed); + ClassDB::bind_method(D_METHOD("_emit_theme_changed", "notify_list_changed"), &Theme::_emit_theme_changed, DEFVAL(false)); ClassDB::bind_method("copy_default_theme", &Theme::copy_default_theme); ClassDB::bind_method(D_METHOD("copy_theme", "other"), &Theme::copy_theme); diff --git a/scene/resources/theme.h b/scene/resources/theme.h index 69c582ee6e..31de21f809 100644 --- a/scene/resources/theme.h +++ b/scene/resources/theme.h @@ -61,7 +61,7 @@ public: private: bool no_change_propagation = false; - void _emit_theme_changed(); + void _emit_theme_changed(bool p_notify_list_changed = false); HashMap>> icon_map; HashMap>> style_map;