[BUG] pause menu and time of day system #7
Labels
No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
abgit/UGJ128#7
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
When you fish and play, since the "fish caught" window pauses the game, the time of day (not the hours:minutes time but the noon/night color overlay) gets out of sync and night (music and darkness) can start at around 13:00, thats bad because the rest of the afternoon is in the dark.
Also if you have the "fish caught" window open and you want to open your deck, 2 things :
@abgit wrote in #7 (comment):
all of this is fixed by
b8c8fd28efchangelog :
make navMenu (and deckContainer) z-index higher than fishingInterface
change type of variable "UserInterface.menuUp" from boolean to int, each menu that's not the pause menu (eg "fish caught" window, deck window) increments this variable when it's shown, and decreases it when you close it, game will behave as paused if UserInterface.isGamePaused == 1 or if UserInterface.menuUp > 0
implement this change in all functions and nodes that need pausing (fish movements, cloud movements, tweens for time of day color overlay, process function of pond.gd)
fix a bug caused by previous commit : I tried to fix the bug that would make menu music the music that plays outside of paused state, when switching fast back and forth between paused/unpaused states. I thought I fixed it, and for 100% volume it worked, but the volume change did not work for music anymore, this is now fixed, see
game_interface.gdfunctionswitchSongfix a bug (not really a bug tbh) that would give a line length of 128 instead of default of 96 after a gameover, the function
GameGlobals.restartGame()used old values, now it resets the deck and recalculates values based on an empty deck, this will be easier to maintain than whatever I wrote before (each variable one by one had to be added to the restartGame function)