Skip to content

Commit 8587b97

Browse files
jpgaribottijpgaribotti
committed
Simplify change
1 parent b6faf78 commit 8587b97

36 files changed

Lines changed: 12 additions & 106 deletions

‎include/TGUI/FormLoadOptions.hpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@
3636
namespace tgui
3737
{
3838
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
39-
/// @brief Options for loading widget forms (theme binding; extended in follow-up commits)
39+
/// @brief Options for loading widget forms
4040
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
4141
struct TGUI_API FormLoadOptions
4242
{

‎include/TGUI/Widget.hpp‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ namespace tgui
113113
virtual ~Widget();
114114

115115
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
116-
/// @brief Loads the widget from a form node (renderer map plus optional runtime themes / Theme.* fallbacks)
116+
/// @brief Loads the widget from a form node (with theme resources)
117117
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
118118
void load(const std::unique_ptr<DataIO::Node>& node, const WidgetLoadResources& resources);
119119

@@ -1166,7 +1166,7 @@ namespace tgui
11661166
using LoadingRenderersMap = std::map<String, std::shared_ptr<RendererData>>;
11671167

11681168
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
1169-
/// @brief Loads widget fields from a form node using the given resources (theme-aware path)
1169+
/// @brief Loads widget fields from a form node using the given resources
11701170
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
11711171
void loadUsingResources(const std::unique_ptr<DataIO::Node>& node, const WidgetLoadResources& resources);
11721172

@@ -1183,7 +1183,7 @@ namespace tgui
11831183
TGUI_NODISCARD virtual std::unique_ptr<DataIO::Node> save(SavingRenderersMap& renderers) const;
11841184

11851185
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
1186-
/// @brief Loads the widget from a tree of nodes (override for derived widgets; theme context from load(WidgetLoadResources))
1186+
/// @brief Loads the widget from a tree of nodes
11871187
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
11881188
virtual void load(const std::unique_ptr<DataIO::Node>& node, const LoadingRenderersMap& renderers);
11891189

‎include/TGUI/WidgetLoadResources.hpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ namespace tgui
4242

4343
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
4444
/// @internal
45-
/// @brief Renderers map passed while loading widgets from a form (extended for theme binding)
45+
/// @brief Resources used when loading widgets from a form file
4646
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
4747
struct TGUI_API WidgetLoadResources
4848
{

‎src/Widgets/ChatBox.cpp‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -567,9 +567,6 @@ namespace tgui
567567
return node;
568568
}
569569

570-
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
571-
572-
573570
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
574571

575572
void ChatBox::load(const std::unique_ptr<DataIO::Node>& node, const LoadingRenderersMap& renderers)

‎src/Widgets/ChildWindow.cpp‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1350,9 +1350,6 @@ namespace tgui
13501350
return node;
13511351
}
13521352

1353-
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
1354-
1355-
13561353
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
13571354

13581355
void ChildWindow::load(const std::unique_ptr<DataIO::Node>& node, const LoadingRenderersMap& renderers)

‎src/Widgets/ColorPicker.cpp‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -646,14 +646,13 @@ namespace tgui
646646
return ChildWindow::save(renderers);
647647
}
648648

649-
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
650-
651-
652649
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
653650

654651
void ColorPicker::load(const std::unique_ptr<DataIO::Node>& node, const LoadingRenderersMap& renderers)
655652
{
653+
// Remove the widgets that the ColorPicker constructor creates because they will be created when loading the child window
656654
removeAllWidgets();
655+
657656
ChildWindow::load(node, renderers);
658657

659658
if (!get<Slider>("#TGUI_INTERNAL$ColorPickerRed#") || !get<Slider>("#TGUI_INTERNAL$ColorPickerGreen#")

‎src/Widgets/ComboBox.cpp‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -896,9 +896,6 @@ namespace tgui
896896
return node;
897897
}
898898

899-
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
900-
901-
902899
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
903900

904901
void ComboBox::load(const std::unique_ptr<DataIO::Node>& node, const LoadingRenderersMap& renderers)

‎src/Widgets/ContextMenu.cpp‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -421,9 +421,6 @@ namespace tgui
421421
return node;
422422
}
423423

424-
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
425-
426-
427424
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
428425

429426
void ContextMenu::load(const std::unique_ptr<DataIO::Node>& node, const LoadingRenderersMap& renderers)

‎src/Widgets/EditBox.cpp‎

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -934,14 +934,11 @@ namespace tgui
934934
return node;
935935
}
936936

937-
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
938-
939-
940937
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
941938

942939
void EditBox::load(const std::unique_ptr<DataIO::Node>& node, const LoadingRenderersMap& renderers)
943940
{
944-
Widget::load(node, renderers);
941+
ClickableWidget::load(node, renderers);
945942

946943
if (node->propertyValuePairs[U"Text"])
947944
setText(Deserializer::deserialize(ObjectConverter::Type::String, node->propertyValuePairs[U"Text"]->value).getString());

‎src/Widgets/EditBoxSlider.cpp‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -296,9 +296,6 @@ namespace tgui
296296
return node;
297297
}
298298

299-
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
300-
301-
302299
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
303300

304301
void EditBoxSlider::load(const std::unique_ptr<DataIO::Node>& node, const LoadingRenderersMap& renderers)

0 commit comments

Comments
 (0)