aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZeno- <kde.psych@gmail.com>2017-01-29 19:26:00 +1000
committerGitHub <noreply@github.com>2017-01-29 19:26:00 +1000
commit707e27b5c26b6c082abbd2f385a976c5bc8964fe (patch)
tree6d3e84e178221bda1c03fe402f9fa08b20a773a8
parent3eecc6ff4492ca21772544451fd8aa78871bac3b (diff)
downloadminetest-707e27b5c26b6c082abbd2f385a976c5bc8964fe.tar.gz
minetest-707e27b5c26b6c082abbd2f385a976c5bc8964fe.tar.bz2
minetest-707e27b5c26b6c082abbd2f385a976c5bc8964fe.zip
Rename height to scale for openConsole() (#5139)
For Game::openConsole() and GUIChatConsole::openConsole() the parameter name 'height' is misleading because it's actually a percentage of the screen/window height.
-rw-r--r--src/game.cpp8
-rw-r--r--src/guiChatConsole.cpp8
-rw-r--r--src/guiChatConsole.h2
3 files changed, 11 insertions, 7 deletions
diff --git a/src/game.cpp b/src/game.cpp
index 07d429c6b..4b4597a7a 100644
--- a/src/game.cpp
+++ b/src/game.cpp
@@ -1372,7 +1372,7 @@ protected:
void dropSelectedItem();
void openInventory();
- void openConsole(float height, const wchar_t *line=NULL);
+ void openConsole(float scale, const wchar_t *line=NULL);
void toggleFreeMove(float *statustext_time);
void toggleFreeMoveAlt(float *statustext_time, float *jump_timer);
void toggleFast(float *statustext_time);
@@ -2779,15 +2779,17 @@ void Game::openInventory()
}
-void Game::openConsole(float height, const wchar_t *line)
+void Game::openConsole(float scale, const wchar_t *line)
{
+ assert(scale > 0.0f && scale <= 1.0f);
+
#ifdef __ANDROID__
porting::showInputDialog(gettext("ok"), "", "", 2);
m_android_chat_open = true;
#else
if (gui_chat_console->isOpenInhibited())
return;
- gui_chat_console->openConsole(height);
+ gui_chat_console->openConsole(scale);
if (line) {
gui_chat_console->setCloseOnEnter(true);
gui_chat_console->replaceAndAddToHistory(line);
diff --git a/src/guiChatConsole.cpp b/src/guiChatConsole.cpp
index 867576175..bea5571f4 100644
--- a/src/guiChatConsole.cpp
+++ b/src/guiChatConsole.cpp
@@ -116,11 +116,13 @@ GUIChatConsole::~GUIChatConsole()
m_font->drop();
}
-void GUIChatConsole::openConsole(f32 height)
+void GUIChatConsole::openConsole(f32 scale)
{
+ assert(scale > 0.0f && scale <= 1.0f);
+
m_open = true;
- m_desired_height_fraction = height;
- m_desired_height = height * m_screensize.Y;
+ m_desired_height_fraction = scale;
+ m_desired_height = scale * m_screensize.Y;
reformatConsole();
m_animate_time_old = getTimeMs();
IGUIElement::setVisible(true);
diff --git a/src/guiChatConsole.h b/src/guiChatConsole.h
index 3013a1d31..4e3cae13f 100644
--- a/src/guiChatConsole.h
+++ b/src/guiChatConsole.h
@@ -41,7 +41,7 @@ public:
// Open the console (height = desired fraction of screen size)
// This doesn't open immediately but initiates an animation.
// You should call isOpenInhibited() before this.
- void openConsole(f32 height);
+ void openConsole(f32 scale);
bool isOpen() const;