Hide gui

[Wunka]

Fixes: #2343
fixes #2448

[IntegratedQuantum]

Should this behavior maybe apply to all menu keys (inventory, creative inventory, chat, ...)? In that case I think it should be added to the toggleGameMenu function.

[Wunka]

Should this behavior maybe apply to all menu keys (inventory, creative inventory, chat, ...)? In that case I think it should be added to the toggleGameMenu function.

I don't quite understand what you mean.. But while I looked at this I found that the guis are only hidden but you can still write in the chat. I will open an Issue for that

[IntegratedQuantum]

I don't quite understand what you mean..

If you press t to open chat, or e to open inventory, then maybe we should also unhide the GUI, since you likely want to go to the inventory or chat, but you cannot see it if it's hidden.

[Wunka]

Would this then not go against why we wanted to only ESC have this function? Reference: #2343 (comment)

[IntegratedQuantum]

Well, for taking the screenshot you could still hide the gui again with F1 when you are in the inventory/menu.

[Wunka]

done

[Wunka]

there is some wierd bug for me now. So please wait

[Wunka]

removed the return. Now it unhides and opens inventory chat etc at the same time, but else there would be many cases where I would need to put the if(hideGui). Example for when opening the chat. Now #2448 is fixed but you can still hide the gui while writing in the chat