summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorHugues Ross <hugues.ross@gmail.com>2020-01-22 04:18:09 -0500
committerLoïc Blot <nerzhul@users.noreply.github.com>2020-01-22 10:18:09 +0100
commit3dfb6ecbb26516bcbcc1ef88860175e39af71381 (patch)
treee1d6bd74fec5658c6277529dea62645ea72e4964 /src
parent0877587cce1d96b1c8c009221684ab382e8f1929 (diff)
downloadminetest-3dfb6ecbb26516bcbcc1ef88860175e39af71381.tar.gz
minetest-3dfb6ecbb26516bcbcc1ef88860175e39af71381.tar.bz2
minetest-3dfb6ecbb26516bcbcc1ef88860175e39af71381.zip
Formspec: Don't start a button click when the pointer isn't on top (#9332)
Diffstat (limited to 'src')
-rw-r--r--src/gui/guiButton.cpp6
1 files changed, 5 insertions, 1 deletions
diff --git a/src/gui/guiButton.cpp b/src/gui/guiButton.cpp
index 0d69ff143..ed79999cf 100644
--- a/src/gui/guiButton.cpp
+++ b/src/gui/guiButton.cpp
@@ -203,8 +203,12 @@ bool GUIButton::OnEvent(const SEvent& event)
case EET_MOUSE_INPUT_EVENT:
if (event.MouseInput.Event == EMIE_LMOUSE_PRESSED_DOWN)
{
- if (!IsPushButton)
+ // Sometimes formspec elements can receive mouse events when the
+ // mouse is outside of the formspec. Thus, we test the position here.
+ if ( !IsPushButton && AbsoluteClippingRect.isPointInside(
+ core::position2d<s32>(event.MouseInput.X, event.MouseInput.Y ))) {
setPressed(true);
+ }
return true;
}