how to make a menu open and close with the same button

Question

my code looks like this: void Update () {

     if (Input.GetKeyDown(KeyCode.Tab) && computerOn == false)
     {
         SuperStore.SetActive(true);
         computerOn = true;
     }

     if (Input.GetKeyDown(KeyCode.Tab) && computerOn == true)
     {
         SuperStore.SetActive(false);
         computerOn = false;
     }
 }

what i think currently happens is that when i press tab it opens and closes the menu really fast but i have no idea how to fix it please help.

btw SuperStore is the menu.

Answer 1

Best Answer

Answer by ShadyProductions · Jul 03, 2018 at 09:08 AM

What you need to do is make it an ELSE if

Because now you're excuting your first if, setting you bool to true, and then executing the other if thats not what you want.

  if (Input.GetKeyDown(KeyCode.Tab) && computerOn == false)
  {
      SuperStore.SetActive(true);
      computerOn = true;
  }
  else if (Input.GetKeyDown(KeyCode.Tab) && computerOn == true)
  {
      SuperStore.SetActive(false);
      computerOn = false;
  }

But you can write it much shorter.

 if (Input.GetKeyDown(KeyCode.Tab)) 
 {
     computerOn = !computerOn;
     SuperStore.SetActive(computerOn);
 }

Add comment · Show 1 · Share

HappyPants84 · Jul 03, 2018 at 09:11 AM 0

Share

Thanks, it worked perfectly

Answer 2

Answer by $$anonymous$$ · Jul 03, 2018 at 09:00 AM

Okay, so try this

 Bool MenuIsUp = false; 
 
 if (Input.GetKeyDown(KeyCode.Tab) )
      {
 
 
 if (MenuIsUp == false)
 
 {
          SuperStore.SetActive(true);
 MenuIsUp = true;
          
   } else {
 
 
 SuperStore.SetActive(false);
 MenuIsUp=false;
 
 }

I am unsure if this would work but it is the first thing that popped into my head when reading your issue

Add comment · Show 1 · Share

HappyPants84 · Jul 03, 2018 at 09:05 AM 0

Share

that is what the computerOn variable is for and yes I did set it to false in the start function

Answer 3

Answer by fghfghfg · Jul 03, 2018 at 09:50 AM

You could also do like diss if (Input.GetKeyDown(KeyCode.Tab)){ SuperStore.SetActive(!SuperStore.activeSelf); computerOn = SuperStore.activeSelf; }

much less code and way cleaner in my opinion.

Add comment · Share

how to make a menu open and close with the same button

3 Replies

Your answer

Follow this Question

Related Questions