From 65a4630f3176feb0f4dd4db6ce12cd6d32e2b369 Mon Sep 17 00:00:00 2001 From: sapier Date: Wed, 18 Jun 2014 22:38:29 +0200 Subject: [PATCH] Fix bounding rect for formspec elements label vertlabel and checkboxes --- builtin/mainmenu/tab_credits.lua | 2 +- builtin/mainmenu/tab_settings.lua | 4 ++-- src/guiFormSpecMenu.cpp | 26 ++++++++++++++++++++------ src/guiFormSpecMenu.h | 7 ++++--- 4 files changed, 27 insertions(+), 12 deletions(-) diff --git a/builtin/mainmenu/tab_credits.lua b/builtin/mainmenu/tab_credits.lua index f752d1728..8e5ce4d7b 100644 --- a/builtin/mainmenu/tab_credits.lua +++ b/builtin/mainmenu/tab_credits.lua @@ -22,7 +22,7 @@ tab_credits = { caption = fgettext("Credits"), cbf_formspec = function (tabview, name, tabdata) local logofile = defaulttexturedir .. "logo.png" - return "vertlabel[0,-0.5;CREDITS]" .. + return "vertlabel[0,-0.25;CREDITS]" .. "label[0.5,3;Minetest " .. core.get_version() .. "]" .. "label[0.5,3.3;http://minetest.net]" .. "image[0.5,1;" .. core.formspec_escape(logofile) .. "]" .. diff --git a/builtin/mainmenu/tab_settings.lua b/builtin/mainmenu/tab_settings.lua index 2d013a5ac..b6ffa86ed 100644 --- a/builtin/mainmenu/tab_settings.lua +++ b/builtin/mainmenu/tab_settings.lua @@ -79,7 +79,7 @@ end local function formspec(tabview, name, tabdata) local tab_string = - "vertlabel[0,0;" .. fgettext("SETTINGS") .. "]" .. + "vertlabel[0,-0.25;" .. fgettext("SETTINGS") .. "]" .. "box[0.75,0;3.25,4;#999999]" .. "checkbox[1,0;cb_fancy_trees;".. fgettext("Fancy Trees") .. ";" .. dump(core.setting_getbool("new_style_leaves")) .. "]".. @@ -108,7 +108,7 @@ local function formspec(tabview, name, tabdata) "checkbox[8,0;cb_shaders;".. fgettext("Shaders") .. ";" .. dump(core.setting_getbool("enable_shaders")) .. "]".. "button[1,4.5;2.25,0.5;btn_change_keys;".. fgettext("Change keys") .. "]" - + local android = false if android then tab_string = tab_string .. diff --git a/src/guiFormSpecMenu.cpp b/src/guiFormSpecMenu.cpp index c16600733..9e9e59fe4 100644 --- a/src/guiFormSpecMenu.cpp +++ b/src/guiFormSpecMenu.cpp @@ -62,6 +62,7 @@ with this program; if not, write to the Free Software Foundation, Inc., return; \ } +extern gui::IGUIEnvironment* guienv; /* GUIFormSpecMenu @@ -85,7 +86,8 @@ GUIFormSpecMenu::GUIFormSpecMenu(irr::IrrlichtDevice* dev, m_lock(false), m_form_src(fsrc), m_text_dst(tdst), - m_ext_ptr(ext_ptr) + m_ext_ptr(ext_ptr), + m_font(guienv->getSkin()->getFont()) { current_keys_pending.key_down = false; current_keys_pending.key_up = false; @@ -388,8 +390,6 @@ void GUIFormSpecMenu::parseCheckbox(parserData* data,std::string element) pos.X += stof(v_pos[0]) * (float) spacing.X; pos.Y += stof(v_pos[1]) * (float) spacing.Y; - core::rect rect = core::rect(pos.X, pos.Y+((imgsize.Y/2)-15), pos.X+300, pos.Y+((imgsize.Y/2)+15)); - bool fselected = false; if (selected == "true") @@ -397,6 +397,11 @@ void GUIFormSpecMenu::parseCheckbox(parserData* data,std::string element) std::wstring wlabel = narrow_to_wide(label.c_str()); + core::rect rect = core::rect( + pos.X, pos.Y + ((imgsize.Y/2) - 15), + pos.X + m_font->getDimension(wlabel.c_str()).Width + 25, // text size + size of checkbox + pos.Y + ((imgsize.Y/2) + 15)); + FieldSpec spec( narrow_to_wide(name.c_str()), wlabel, //Needed for displaying text on MSVC @@ -1083,8 +1088,6 @@ void GUIFormSpecMenu::parseLabel(parserData* data,std::string element) pos.X += stof(v_pos[0]) * (float)spacing.X; pos.Y += stof(v_pos[1]) * (float)spacing.Y; - core::rect rect = core::rect(pos.X, pos.Y+((imgsize.Y/2)-15), pos.X+300, pos.Y+((imgsize.Y/2)+15)); - if(data->bp_set != 2) errorstream<<"WARNING: invalid use of label without a size[] element"< rect = core::rect( + pos.X, pos.Y+((imgsize.Y/2)-15), + pos.X + m_font->getDimension(wlabel.c_str()).Width, + pos.Y+((imgsize.Y/2)+15)); + FieldSpec spec( L"", wlabel, @@ -1119,7 +1127,13 @@ void GUIFormSpecMenu::parseVertLabel(parserData* data,std::string element) pos.X += stof(v_pos[0]) * (float)spacing.X; pos.Y += stof(v_pos[1]) * (float)spacing.Y; - core::rect rect = core::rect(pos.X, pos.Y+((imgsize.Y/2)-15), pos.X+15, pos.Y+300); + core::rect rect = core::rect( + pos.X, pos.Y+((imgsize.Y/2)-15), + pos.X+15, pos.Y + + (m_font->getKerningHeight() + + m_font->getDimension(text.c_str()).Height) + * (text.length()+1)); + //actually text.length() would be correct but adding +1 avoids to break all mods if(data->bp_set != 2) errorstream<<"WARNING: invalid use of label without a size[] element"<