Skip to content

Commit 9d46f8b

Browse files
committed
Add text panel colors colors to skin params to make it easy to change by client application.
1 parent a464792 commit 9d46f8b

4 files changed

Lines changed: 26 additions & 21 deletions

File tree

Sources/BuiltInNodes/BI_UINodePanels.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -184,12 +184,12 @@ void NodeUIMultiLineTextPanel::Draw (NUIE::NodeUIDrawingEnvironment& env, const
184184

185185
NUIE::Color NodeUIMultiLineTextPanel::GetTextColor (NUIE::NodeUIDrawingEnvironment& env) const
186186
{
187-
return env.GetSkinParams ().GetNodeContentTextColor ();
187+
return env.GetSkinParams ().GetTextPanelTextColor ();
188188
}
189189

190190
NUIE::Color NodeUIMultiLineTextPanel::GetBackgroundColor (NUIE::NodeUIDrawingEnvironment& env) const
191191
{
192-
return env.GetSkinParams ().GetNodeContentBackgroundColor ();
192+
return env.GetSkinParams ().GetTextPanelBackgroundColor ();
193193
}
194194

195195
NodeUISlotPanel::NodeUISlotPanel (const NUIE::UINode& node, NUIE::NodeUIDrawingEnvironment& env) :

Sources/BuiltInNodes/BI_ViewerUINodes.cpp

Lines changed: 1 addition & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -107,22 +107,6 @@ bool MultiLineViewerNode::IsForceCalculated () const
107107

108108
void MultiLineViewerNode::UpdateNodeDrawingImage (NUIE::NodeUIDrawingEnvironment& env, NUIE::NodeDrawingImage& drawingImage) const
109109
{
110-
class NodeUIMultiLineTextViewerPanel : public NodeUIMultiLineTextPanel
111-
{
112-
public:
113-
NodeUIMultiLineTextViewerPanel (const std::vector<std::wstring>& nodeTexts, NUIE::NodeUIDrawingEnvironment& env, size_t textsPerPage) :
114-
NodeUIMultiLineTextPanel (nodeTexts, env, textsPerPage)
115-
{
116-
117-
}
118-
119-
private:
120-
virtual NUIE::Color GetBackgroundColor (NUIE::NodeUIDrawingEnvironment&) const override
121-
{
122-
return NUIE::Color (255, 255, 100);
123-
}
124-
};
125-
126110
std::vector<std::wstring> nodeTexts;
127111
if (HasCalculatedValue ()) {
128112
NE::ValueConstPtr nodeValue = GetCalculatedValue ();
@@ -154,7 +138,7 @@ void MultiLineViewerNode::UpdateNodeDrawingImage (NUIE::NodeUIDrawingEnvironment
154138
NUIE::NodePanelDrawer drawer;
155139
drawer.AddPanel (NUIE::NodeUIPanelPtr (new NodeUIHeaderPanel (GetNodeName ())));
156140
drawer.AddPanel (NUIE::NodeUIPanelPtr (new NodeUISlotPanel (*this, env)));
157-
drawer.AddPanel (NUIE::NodeUIPanelPtr (new NodeUIMultiLineTextViewerPanel (nodeTextsToShow, env, textsPerPage)));
141+
drawer.AddPanel (NUIE::NodeUIPanelPtr (new NodeUIMultiLineTextPanel (nodeTextsToShow, env, textsPerPage)));
158142
if (textCount > textsPerPage) {
159143
drawer.AddPanel (NUIE::NodeUIPanelPtr (new NodeUILeftRightButtonsPanel ("minus", L"<", "plus", L">", std::to_wstring (currentPage) + L" / " + std::to_wstring (pageCount) + L" (" + std::to_wstring (textCount) + L")", env)));
160144
}

Sources/NodeUIEngine/NUIE_SkinParams.cpp

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,11 +27,13 @@ DefaultSkinParams::DefaultSkinParams () :
2727
slotTextBackgroundColor (225, 225, 225),
2828
selectionBlendColor (0, 255, 0),
2929
selectionRectPen (Color (0, 138, 184), 1.0),
30-
hasValueStatusColor (Color (0, 255, 0)),
31-
hasNoValueStatusColor (Color (255, 0, 0)),
30+
hasValueStatusColor (0, 255, 0),
31+
hasNoValueStatusColor (255, 0, 0),
3232
buttonBorderPen (Color (50, 75, 100), 1.0),
3333
buttonBackgroundColor (150, 175, 200),
3434
selectedButtonBackgroundColor (190, 215, 240),
35+
textPanelTextColor (0, 0, 0),
36+
textPanelBackgroundColor (255, 255, 100),
3537
groupNameFont (L"Arial", 18.0),
3638
groupNameColor (Color (0, 0, 0)),
3739
groupBackgroundColor (Color (160, 200, 240)),
@@ -135,6 +137,16 @@ const Color& DefaultSkinParams::GetSelectedButtonBackgroundColor () const
135137
return selectedButtonBackgroundColor;
136138
}
137139

140+
const Color& DefaultSkinParams::GetTextPanelTextColor () const
141+
{
142+
return textPanelTextColor;
143+
}
144+
145+
const Color& DefaultSkinParams::GetTextPanelBackgroundColor () const
146+
{
147+
return textPanelBackgroundColor;
148+
}
149+
138150
const Font& DefaultSkinParams::GetGroupNameFont () const
139151
{
140152
return groupNameFont;

Sources/NodeUIEngine/NUIE_SkinParams.hpp

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,9 @@ class SkinParams
3737
virtual const Color& GetButtonBackgroundColor () const = 0;
3838
virtual const Color& GetSelectedButtonBackgroundColor () const = 0;
3939

40+
virtual const Color& GetTextPanelTextColor () const = 0;
41+
virtual const Color& GetTextPanelBackgroundColor () const = 0;
42+
4043
virtual const Font& GetGroupNameFont () const = 0;
4144
virtual const Color& GetGroupNameColor () const = 0;
4245
virtual const Color& GetGroupBackgroundColor () const = 0;
@@ -73,6 +76,9 @@ class DefaultSkinParams : public SkinParams
7376
virtual const Color& GetButtonBackgroundColor () const override;
7477
virtual const Color& GetSelectedButtonBackgroundColor () const override;
7578

79+
virtual const Color& GetTextPanelTextColor () const override;
80+
virtual const Color& GetTextPanelBackgroundColor () const override;
81+
7682
virtual const Font& GetGroupNameFont () const override;
7783
virtual const Color& GetGroupNameColor () const override;
7884
virtual const Color& GetGroupBackgroundColor () const override;
@@ -103,6 +109,9 @@ class DefaultSkinParams : public SkinParams
103109
Color buttonBackgroundColor;
104110
Color selectedButtonBackgroundColor;
105111

112+
Color textPanelTextColor;
113+
Color textPanelBackgroundColor;
114+
106115
Font groupNameFont;
107116
Color groupNameColor;
108117
Color groupBackgroundColor;

0 commit comments

Comments
 (0)