Skip to content

Commit

Permalink
Improve on feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
Schweini07 committed Mar 31, 2024
1 parent 7d52b34 commit 3e00a47
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 46 deletions.
40 changes: 11 additions & 29 deletions gui-builder/include/WidgetProperties/EditBoxSliderProperties.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -89,17 +89,17 @@ struct EditBoxSliderProperties : WidgetProperties
pair.second["EditBox.DefaultTextStyle"] = {"TextStyle", tgui::Serializer::serialize(editBoxRenderer->getDefaultTextStyle())};

const auto sliderRenderer = editBoxSlider->getSliderSharedRenderer();
pair.second["Borders"] = {"Outline", tgui::Serializer::serialize(sliderRenderer->getBorders())};
pair.second["TrackColor"] = {"Color", tgui::Serializer::serialize(sliderRenderer->getTrackColor())};
pair.second["TrackColorHover"] = {"Color", tgui::Serializer::serialize(sliderRenderer->getTrackColorHover())};
pair.second["ThumbColor"] = {"Color", tgui::Serializer::serialize(sliderRenderer->getThumbColor())};
pair.second["ThumbColorHover"] = {"Color", tgui::Serializer::serialize(sliderRenderer->getThumbColorHover())};
pair.second["BorderColor"] = {"Color", tgui::Serializer::serialize(sliderRenderer->getBorderColor())};
pair.second["BorderColorHover"] = {"Color", tgui::Serializer::serialize(sliderRenderer->getBorderColorHover())};
pair.second["TextureTrack"] = {"Texture", tgui::Serializer::serialize(sliderRenderer->getTextureTrack())};
pair.second["TextureTrackHover"] = {"Texture", tgui::Serializer::serialize(sliderRenderer->getTextureTrackHover())};
pair.second["TextureThumb"] = {"Texture", tgui::Serializer::serialize(sliderRenderer->getTextureThumb())};
pair.second["TextureThumbHover"] = {"Texture", tgui::Serializer::serialize(sliderRenderer->getTextureThumbHover())};
pair.second["Slider.Borders"] = {"Outline", tgui::Serializer::serialize(sliderRenderer->getBorders())};
pair.second["Slider.TrackColor"] = {"Color", tgui::Serializer::serialize(sliderRenderer->getTrackColor())};
pair.second["Slider.TrackColorHover"] = {"Color", tgui::Serializer::serialize(sliderRenderer->getTrackColorHover())};
pair.second["Slider.ThumbColor"] = {"Color", tgui::Serializer::serialize(sliderRenderer->getThumbColor())};
pair.second["Slider.ThumbColorHover"] = {"Color", tgui::Serializer::serialize(sliderRenderer->getThumbColorHover())};
pair.second["Slider.BorderColor"] = {"Color", tgui::Serializer::serialize(sliderRenderer->getBorderColor())};
pair.second["Slider.BorderColorHover"] = {"Color", tgui::Serializer::serialize(sliderRenderer->getBorderColorHover())};
pair.second["Slider.TextureTrack"] = {"Texture", tgui::Serializer::serialize(sliderRenderer->getTextureTrack())};
pair.second["Slider.TextureTrackHover"] = {"Texture", tgui::Serializer::serialize(sliderRenderer->getTextureTrackHover())};
pair.second["Slider.TextureThumb"] = {"Texture", tgui::Serializer::serialize(sliderRenderer->getTextureThumb())};
pair.second["Slider.TextureThumbHover"] = {"Texture", tgui::Serializer::serialize(sliderRenderer->getTextureThumbHover())};
return pair;
}

Expand All @@ -125,24 +125,6 @@ struct EditBoxSliderProperties : WidgetProperties
else
return "Left";
}

private:

const std::set<tgui::String> m_editBoxProperties = {
"Padding", "CaretWidth", "TextColor", "TextColorDisabled", "TextColorFocused", "SelectedTextColor",
"SelectedTextBackgroundColor", "DefaultTextColor", "BackgroundColor", "BackgroundColorHover",
"BackgroundColorDisabled", "BackgroundColorFocused", "CaretColor", "CaretColorHover", "CaretColorFocused",
"BorderColor", "BorderColorHover", "BorderColorDisabled", "BorderColorFocused", "Texture", "TextureHover",
"TextureDisabled", "TextureFocused", "TextStyle", "DefaultTextStyle", "Borders"
};

const std::set<tgui::String> m_SliderProperties = {
"Borders", "TrackColor", "TrackColorHover", "ThumbColor", "ThumbColorHover",
"BorderColor", "BorderColorHover", "TextureTrack", "TextureTrackHover", "TextureThumb",
"TextureThumbHover"
};


};

#endif // TGUI_GUI_BUILDER_EDIT_BOX_SLIDER_PROPERTIES_HPP
16 changes: 0 additions & 16 deletions gui-builder/include/WidgetProperties/SpinControlProperties.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -102,22 +102,6 @@ struct SpinControlProperties : WidgetProperties
pair.second["SpinText.DefaultTextStyle"] = {"TextStyle", tgui::Serializer::serialize(textRenderer->getDefaultTextStyle())};
return pair;
}

private:

const std::set<tgui::String> m_buttonProperties = {
"ButtonsBorders", "BorderBetweenArrows", "ButtonsBackgroundColor", "ButtonsBackgroundColorHover", "ArrowColor",
"ArrowColorHover", "ButtonsBorderColor", "TextureArrowUp", "TextureArrowUpHover", "TextureArrowDown",
"TextureArrowDownHover"
};

const std::set<tgui::String> m_textProperties = {
"Padding", "CaretWidth", "TextColor", "TextColorDisabled", "TextColorFocused", "SelectedTextColor",
"SelectedTextBackgroundColor", "DefaultTextColor", "BackgroundColor", "BackgroundColorHover",
"BackgroundColorDisabled", "BackgroundColorFocused", "CaretColor", "CaretColorHover", "CaretColorFocused",
"BorderColor", "BorderColorHover", "BorderColorDisabled", "BorderColorFocused", "Texture", "TextureHover",
"TextureDisabled", "TextureFocused", "TextStyle", "DefaultTextStyle", "Borders"
};
};

#endif // TGUI_GUI_BUILDER_SPIN_CONTROL_PROPERTIES_HPP
4 changes: 3 additions & 1 deletion src/Widgets/EditBoxSlider.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////

#include <TGUI/Widgets/EditBoxSlider.hpp>
#include <iostream>
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////

namespace tgui
Expand Down Expand Up @@ -214,6 +215,7 @@ namespace tgui
{
if (m_slider->getValue() != value && inRange(value))
{
std::cout << value << "\n";
m_slider->setValue(value);
setString(String::fromNumberRounded(value, m_decimalPlaces));
return true;
Expand Down Expand Up @@ -313,7 +315,7 @@ namespace tgui

m_editBox->onTextChange.disconnectAll();
m_editBox->onTextChange([this](const String &text) {
m_slider->setValue(text.toFloat());
setValue(text.toFloat());
});

m_editBox->onReturnOrUnfocus.disconnectAll();
Expand Down

0 comments on commit 3e00a47

Please sign in to comment.