Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Don't recalculate statustext initial color everytime & review fixes
  • Loading branch information
nerzhul committed Jan 5, 2018
1 parent f40f414 commit 9146c6a
Show file tree
Hide file tree
Showing 6 changed files with 24 additions and 21 deletions.
22 changes: 12 additions & 10 deletions src/client/gameui.cpp
Expand Up @@ -41,6 +41,14 @@ inline static const char *yawToDirectionString(int yaw)
return direction[yaw];
}

GameUI::GameUI()
{
if (guienv && guienv->getSkin())
m_statustext_initial_color = guienv->getSkin()->getColor(gui::EGDC_BUTTON_TEXT);
else
m_statustext_initial_color = video::SColor(255, 0, 0, 0);

}
void GameUI::init()
{
// First line of debug text
Expand Down Expand Up @@ -169,16 +177,10 @@ void GameUI::update(const RunStats &stats, Client *client, MapDrawControl *draw_
status_y - status_height, status_x + status_width, status_y));

// Fade out
video::SColor initial_color(255, 0, 0, 0);

if (guienv->getSkin())
initial_color = guienv->getSkin()->getColor(gui::EGDC_BUTTON_TEXT);

video::SColor final_color = initial_color;
video::SColor final_color = m_statustext_initial_color;
final_color.setAlpha(0);
video::SColor fade_color = initial_color.getInterpolated_quadratic(
initial_color, final_color,
pow(m_statustext_time / statustext_time_max, 2.0f));
video::SColor fade_color = m_statustext_initial_color.getInterpolated_quadratic(
m_statustext_initial_color, final_color, m_statustext_time / statustext_time_max);
m_guitext_status->setOverrideColor(fade_color);
m_guitext_status->enableOverrideColor(true);
}
Expand Down Expand Up @@ -244,7 +246,7 @@ void GameUI::updateProfiler()
if (w < 400)
w = 400;

unsigned text_height = g_fontengine->getTextHeight();
u32 text_height = g_fontengine->getTextHeight();

core::position2di upper_left, lower_right;

Expand Down
7 changes: 4 additions & 3 deletions src/client/gameui.h
Expand Up @@ -46,7 +46,7 @@ class GameUI
friend class TestGameUI;

public:
GameUI() = default;
GameUI();
~GameUI() = default;

// Flags that can, or may, change during main game loop
Expand Down Expand Up @@ -100,10 +100,11 @@ class GameUI
gui::IGUIStaticText *m_guitext_status = nullptr;
std::wstring m_statustext;
float m_statustext_time = 0.0f;
video::SColor m_statustext_initial_color;

gui::IGUIStaticText *m_guitext_chat; // Chat text
gui::IGUIStaticText *m_guitext_chat = nullptr; // Chat text

gui::IGUIStaticText *m_guitext_profiler; // Profiler text
gui::IGUIStaticText *m_guitext_profiler = nullptr; // Profiler text
u8 m_profiler_current_page = 0;
const u8 m_profiler_max_page = 3;
};
9 changes: 4 additions & 5 deletions src/game.cpp
Expand Up @@ -1277,7 +1277,7 @@ class Game {
void updateChat(f32 dtime, const v2u32 &screensize);
static const ClientEventHandler clientEventHandler[CLIENTEVENT_MAX];

InputHandler *input;
InputHandler *input = nullptr;

Client *client = nullptr;
Server *server = nullptr;
Expand Down Expand Up @@ -2669,9 +2669,9 @@ void Game::toggleFog()
{
m_flags.force_fog_off = !m_flags.force_fog_off;
if (m_flags.force_fog_off)
m_game_ui->showTranslatedStatusText("Fog disabled");
m_game_ui->showTranslatedStatusText("Fog disabled");
else
m_game_ui->showTranslatedStatusText("Fog enabled");
m_game_ui->showTranslatedStatusText("Fog enabled");
}


Expand Down Expand Up @@ -2912,9 +2912,8 @@ inline void Game::step(f32 *dtime)
if (can_be_and_is_paused) { // This is for a singleplayer server
*dtime = 0; // No time passes
} else {
if (server) {
if (server)
server->step(*dtime);
}

client->step(*dtime);
}
Expand Down
2 changes: 1 addition & 1 deletion src/gui/guiFormSpecMenu.cpp
Expand Up @@ -2026,7 +2026,7 @@ void GUIFormSpecMenu::regenerateGui(v2u32 screensize)
assert(!m_tooltip_element);
// Note: parent != this so that the tooltip isn't clipped by the menu rectangle
m_tooltip_element = gui::StaticText::add(Environment, L"",
core::rect<s32>(0,0,110,18));
core::rect<s32>(0, 0, 110, 18));
m_tooltip_element->enableOverrideColor(true);
m_tooltip_element->setBackgroundColor(m_default_tooltip_bgcolor);
m_tooltip_element->setDrawBackground(true);
Expand Down
3 changes: 2 additions & 1 deletion src/irrlicht_changes/static_text.h
Expand Up @@ -92,7 +92,8 @@ namespace gui
s32 id = -1,
bool fillBackground = false)
{
return add(guienv, EnrichedString(text), rectangle, border, wordWrap, parent, id, fillBackground);
return add(guienv, EnrichedString(text), rectangle, border, wordWrap, parent,
id, fillBackground);
}

//! draws the element and its children
Expand Down
2 changes: 1 addition & 1 deletion util/updatepo.sh
Expand Up @@ -57,7 +57,7 @@ xgettext --package-name=minetest \
--keyword=fgettext_ne \
--keyword=strgettext \
--keyword=wstrgettext \
--keyword=showStatusTextSimple \
--keyword=showTranslatedStatusText \
--output $potfile \
--from-code=utf-8 \
`find src/ -name '*.cpp' -o -name '*.h'` \
Expand Down

0 comments on commit 9146c6a

Please sign in to comment.