Clicking pause button makes character jump before pausing

Question

I have an issue with my Unity 5 game. When I click on the Pause button, my character jumps before the game is paused.

My script detects the click to trigger the jump jump before the pause button. I tried adding a tag check but that didn't work:

void Update () {
    if (Input.GetKeyDown (KeyCode.Space) && Time.timeScale == 1 || Input.GetMouseButtonDown (0) && Time.timeScale == 1 && gameObject.tag != "Pause") {
        GetComponent<Rigidbody2D> ().AddForce (Vector2.up * force);
    }

http://makeagif.com/i/bmCKQ3:

You can see in this gif, when I click on the pause button it jumps and pauses directly.

How can I click on the Pause button without making my player jump?

Answer 1

I had the same issue when I was doing a game. What basically happens is that your script detects if there is a mouse click, including when you click on UI components.

There is a solution to that fortunately:

using UnityEngine;
using System.Collections;
using UnityEngine.EventSystems;

public class PlayerControl : MonoBehaviour {

    void Update () {
        if (Input.GetMouseButtonDown(0) && !EventSystem.current.IsPointerOverGameObject(0))
        {
        MovePlayer();
        }
    }
}

So now every-time you tap on a screen unity checks if there is no UI element (buttons, panels etc.) where you tapped before moving your player.

Hope this helps! :)

Answer 2

Ok, the .gif makes it make more sense. I think this is what you're looking for:

void Update () {
    Vector2 buttonDimensions = new Vector2(50, 50);
    Rect pauseButtonPosition = new Rect(Screen.width - buttonDimensions.x, 0, buttonDimensions.x, buttonDimensions.y);

    bool mouseOnPauseButton = pauseButtonPosition.Contains(Input.mousePosition);

    if (!mouseOnPauseButton && Input.GetKeyDown(KeyCode.Space) && Time.timeScale == 1 || Input.GetMouseButtonDown(0) && Time.timeScale == 1 && gameObject.tag != "Pause") {
        GetComponent<Rigidbody2D>().AddForce(Vector2.up * force);
    }
}

So basically if the mouse is on the pause button when you try to jump, it wont.

Answer 3

I had trouble with Daniel's answer in my Phone, but I have found the answer for the problem it

if (Input.GetMouseButtonDown (0)) {    
if (UnityEngine.EventSystems.EventSystem.current.currentSelectedGameObject) return;
    GetComponent<Rigidbody2D> ().AddForce (Vector2.up * force);
}

Thanks to Georgedarocha, source: http://forum.unity3d.com/threads/ispointerovergameobject-ignoring-some-objects.277825/

Answer 4

For me at least, the below solution worked just fine.. Elmar had it almost right. Hope it helps the next person to find this!

using UnityEngine;
using System.Collections;
using UnityEngine.EventSystems;

public class PlayerControl : MonoBehaviour {

    void Update () {
        if (Input.GetMouseButtonDown(0) && EventSystem.current.IsPointerOverGameObject() == false)
        {
        MovePlayer();
        }
    }
}

Slight difference for smartphone games: https://docs.unity3d.com/2018.2/Documentation/ScriptReference/EventSystems.EventSystem.IsPointerOverGameObject.html

Answer 5

While the other answers “Hack” a solution for that individual button, it doesn’t fix the overall problem of a click propagating after it’s been detected by a script that uses it.

The input system can “consume” events. So that they are used up when required.and don’t get passed on to other objects.

For example Event.Use method.

There is a good discussion of this on the following thread:

https://forum.unity.com/threads/consuming-inputs-have-a-few-game-objects-handle-inputs-and-make-others-ignore-inputs-during-1-frame.348189/

You could also roll your own system where you can prioritize clicks to certain layers/ objects and if it gets “used” then it doesn’t get passed on to the next level of priority.