diff --git a/builtin/common/settings/components.lua b/builtin/common/settings/components.lua index 99fb0cd76..4052ee01c 100644 --- a/builtin/common/settings/components.lua +++ b/builtin/common/settings/components.lua @@ -438,11 +438,41 @@ function make.key(setting) if value == "" then return height end + + local critical_keys = { + keymap_chat = true, + keymap_drop = true, + keymap_inventory = true, + keymap_cmd = true, + keymap_cmd_local = true, + keymap_dig = true, + keymap_place = true, + } + for _, o in ipairs(core.full_settingtypes) do - if o.type == "key" and o.name ~= setting.name and core.are_keycodes_equal(core.settings:get(o.name), value) then - table.insert(fs, ("label[0,%f;%s]"):format(height + 0.3, - core.colorize(mt_color_orange, fgettext([[Conflicts with "$1"]], fgettext(o.readable_name))))) - height = height + 0.6 + if o.type == "key" and o.name ~= setting.name and + core.are_keycodes_equal(core.settings:get(o.name), value) then + + local is_current_close_world = setting.name == "keymap_close_world" + local is_other_close_world = o.name == "keymap_close_world" + + -- Current means current key you've messed with, other means the other key it conflicts with. + local is_current_critical = critical_keys[setting.name] + local is_other_critical = critical_keys[o.name] + + local show_warning_current = false + + if (is_current_close_world and is_other_critical) + or (is_other_close_world and is_current_critical) + or (not is_current_close_world and not is_other_close_world) then + show_warning_current = true + end + + if show_warning_current then + table.insert(fs, ("label[0,%f;%s]"):format(height + 0.3, + core.colorize(mt_color_orange, fgettext([[Conflicts with "$1"]], fgettext(o.readable_name))))) + height = height + 0.6 + end end end return height @@ -454,12 +484,12 @@ function make.key(setting) get_formspec = function(self, avail_w) self.resettable = core.settings:has(setting.name) - local btn_bind_width = math.max(2.5, avail_w/2) + local btn_bind_width = math.max(2.5, avail_w / 2) local value = core.settings:get(setting.name) local fs = { ("label[0,0.4;%s]"):format(get_label(setting)), ("button_key[%f,0;%f,0.8;%s;%s]"):format( - btn_bind_width, btn_bind_width-0.8, + btn_bind_width, btn_bind_width - 0.8, btn_bind, core.formspec_escape(value)), ("image_button[%f,0;0.8,0.8;%s;%s;]"):format(avail_w - 0.8, core.formspec_escape(defaulttexturedir .. "clear.png"), diff --git a/builtin/settingtypes.txt b/builtin/settingtypes.txt index 47cbe38f1..75b60d61e 100644 --- a/builtin/settingtypes.txt +++ b/builtin/settingtypes.txt @@ -337,6 +337,10 @@ keymap_slot31 (Hotbar slot 31) key keymap_slot32 (Hotbar slot 32) key +# Modifier key bind for closing your world. +# Requires ESC + the selected key to work. +keymap_close_world (Return to Main Menu) key SYSTEM_SCANCODE_225 + [*Touchscreen] # Enables the touchscreen controls, allowing you to play the game with a touchscreen. diff --git a/src/client/inputhandler.cpp b/src/client/inputhandler.cpp index d79bee4d6..56bdb184e 100644 --- a/src/client/inputhandler.cpp +++ b/src/client/inputhandler.cpp @@ -137,6 +137,7 @@ bool MyEventReceiver::OnEvent(const SEvent &event) // This is separate from other keyboard handling so that it also works in menus. if (event.EventType == EET_KEY_INPUT_EVENT) { KeyPress keyCode(event.KeyInput); + if (keyCode == getKeySetting("keymap_fullscreen")) { if (event.KeyInput.PressedDown && !fullscreen_is_down) { IrrlichtDevice *device = RenderingEngine::get_raw_device(); @@ -150,8 +151,15 @@ bool MyEventReceiver::OnEvent(const SEvent &event) } fullscreen_is_down = event.KeyInput.PressedDown; return true; - } else if (keyCode == EscapeKey && - event.KeyInput.PressedDown && event.KeyInput.Shift) { + + } else if (keyCode == getKeySetting("keymap_close_world")) { + close_world_down = event.KeyInput.PressedDown; + + } else if (keyCode == EscapeKey) { + esc_down = event.KeyInput.PressedDown; + } + + if (esc_down && close_world_down) { g_gamecallback->disconnect(); return true; } diff --git a/src/client/inputhandler.h b/src/client/inputhandler.h index 85da30ff8..ce2c41a63 100644 --- a/src/client/inputhandler.h +++ b/src/client/inputhandler.h @@ -128,6 +128,9 @@ private: // Intentionally not reset by clearInput/releaseAllKeys. bool fullscreen_is_down = false; + bool close_world_down = false; + bool esc_down = false; + PointerType last_pointer_type = PointerType::Mouse; }; diff --git a/src/defaultsettings.cpp b/src/defaultsettings.cpp index 916ce9e51..f970da060 100644 --- a/src/defaultsettings.cpp +++ b/src/defaultsettings.cpp @@ -219,6 +219,7 @@ void set_default_settings() settings->setDefault("keymap_slot30", ""); settings->setDefault("keymap_slot31", ""); settings->setDefault("keymap_slot32", ""); + USEKEY2("keymap_close_world", "SYSTEM_SCANCODE_225", "KEY_LSHIFT"); #ifndef NDEBUG // Default keybinds for quicktune in debug builds