Click on gameObject with disabled renderer?

I'm trying to make my 2D object toggle visibility when I click on it. To make it invisible, I turn off the renderer. The problem is that when the renderer is disabled, It doesn't detect my clicks so it cannot turn back to visible. Here's the script for now:

Code (CSharp):

1. using UnityEngine;
2. using System.Collections;
3. using System;
4.  
5. public class Drumnote : MonoBehaviour {
6.  
7.     void Awake ()
8.     {
9.         gameObject.GetComponent<Renderer>().enabled = false;
10.     }
11.    
12.     void Update ()
13.     {
14.  
15.     }
16.     void OnMouseDown()
17.     {
18.         if (gameObject.GetComponent<Renderer>().enabled == false)
19.         {
20.             gameObject.GetComponent<Renderer>().enabled = true;
21.         }
22.         if (gameObject.GetComponent<Renderer>().enabled == true)
23.         {
24.             gameObject.GetComponent<Renderer>().enabled = false;
25.         }
26.     }
27. }
28.  

Any idea?

 

your code working:

Code (CSharp):

1. using UnityEngine;
2. using System.Collections;
3.  
4. public class ToggleVisible : MonoBehaviour {
5.  
6.     Renderer renderer;
7.     bool isVisible;
8.  
9.     void Awake ()
10.     {
11.         renderer = GetComponent<Renderer>();
12.     }
13.  
14.     void OnMouseDown()
15.     {
16.         isVisible = !isVisible;
17.         renderer.enabled = isVisible;
18.     }
19. }
20.  

but you say 2D objects so maybe better:

Code (CSharp):

1. using UnityEngine;
2. using System.Collections;
3.  
4. [RequireComponent (typeof(Collider2D))]
5. public class ToggleVisible : MonoBehaviour {
6.  
7.     SpriteRenderer renderer;
8.     bool isVisible;
9.  
10.     void Awake ()
11.     {
12.         renderer = GetComponent<SpriteRenderer>();
13.     }
14.  
15.     void OnMouseDown()
16.     {
17.         isVisible = !isVisible;
18.         renderer.enabled = isVisible;
19.     }
20. }
21.  

 

THANK YOU! It worked. Just had to turn off the renderer on awake on your script and everything works fine! Thank you again!

 

Renderer is the base class for SpriteRenderer and MeshRenderer, so the original code works for both situation. Also, your 'isVisible' bool is redundant:

Code (CSharp):

1. [RequireComponent (typeof(Collider2D))]
2. public class ToggleVisible : MonoBehaviour {
3.  
4.     SpriteRenderer renderer;
5.  
6.     void Awake ()
7.     {
8.         renderer = GetComponent<SpriteRenderer>();
9.     }
10.  
11.     void OnMouseDown()
12.     {
13.         renderer.enabled = !renderer.enabled;
14.     }
15. }

 

Its probably worth knowing why too.

OnMouseDown has nothing to do with the renders, it works on colliders.

The original script was checking if the renderer was enabled after you had just enabled it. Meaning the second if would always run. Making it an else if would also have worked.