UQdeco2800 / 2022-studio-2

MIT License
5 stars 1 forks source link

<Collaboration> Restrict menu to maximum one can be open at a time #292

Closed PeterOu8 closed 2 years ago

PeterOu8 commented 2 years ago

Description

Task: Restrict menu to maximum one can be open at a time. Feature: Implement functionality to enhance UX

Currently, the game allows the player to open up more than 1 type of menu. To avoid this, we will implement a feature that will only allow one menu at a time to be opened.

Documentation

*Completion Date: October 14

Members

PeterOu8 commented 2 years ago

@lyphng @jgapusan Will you guys be able to work on this feature with me?

lyphng commented 2 years ago

that sounds great peter sure thing!

jgapusan commented 2 years ago

hey @PeterOu8 what did you have in mind with this particular task?

PeterOu8 commented 2 years ago

I am thinking of setting a flag to determine what menu is currently opened and handle it from there

jgapusan commented 2 years ago

that seems like the best way to go about it, are you alright to implement that yourself ?

PeterOu8 commented 2 years ago

that seems like the best way to go about it, are you alright to implement that yourself ?

yeah sure

LachlanBenson commented 2 years ago

@PeterOu8 just flagging an incomplete part of this task

currently if you press "Q" while away from a crafting table, it then prevents the user from opening their inventory using "I"

if you press Q again, then you can use the inventory as usual

seems like a toggle flag/switch is being toggled when it shouldn't be.

Maybe need to add a condition for it only toggling the flag if the craftingtable is actually activated?

PeterOu8 commented 2 years ago

Thanks for the notification, I will put the flag handling in the openingCraftingMenu function

LachlanBenson commented 2 years ago

@PeterOu8 confirmed issue is resolved