Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/bugfix #72

Merged
merged 7 commits into from
Feb 24, 2022
Merged

Fix/bugfix #72

merged 7 commits into from
Feb 24, 2022

Conversation

ThomasJGabrielsen
Copy link
Collaborator

Fixed the following:
Pause menu does not pause music
Saving a new highscore populates the leaderboard with only that score

Created a custom comparator and restructured how highscores are saved so they can be easily sorted.
Added ability to pause individual sources as well as lower the general volume of the AudioListener.
Copy link
Collaborator

@Fueredoriku Fueredoriku left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There seems to be compilation errors tied to loading and adding highscores on the leaderboard.

Also, regarding the changes to PauseManager:
This array:
public AudioSource[] audioSources;
and this functionality:
// Pause every audiosource in array. foreach(AudioSource a in audioSources) { if (IsPaused) a.Pause(); else a.UnPause(); } // Reduce the listener volume level AudioListener.volume = IsPaused ? 0.3f : 1f;
should be implemented in and derived from the audioManager singleton.

Copy link
Member

@ddabble ddabble left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't tried running the code yet, but here's some feedback on the code itself 🤠

Assets/Scripts/GameState/GameOverScreen.cs Outdated Show resolved Hide resolved
Assets/Scripts/GameState/GameOverScreen.cs Outdated Show resolved Hide resolved
Assets/Scripts/GameState/LeaderboardData.cs Outdated Show resolved Hide resolved
Assets/Scripts/GameState/LeaderboardData.cs Outdated Show resolved Hide resolved
Assets/Scripts/GameState/LeaderboardData.cs Outdated Show resolved Hide resolved
Assets/Scripts/GameState/LeaderboardData.cs Outdated Show resolved Hide resolved
Assets/Scripts/GameState/LeaderboardData.cs Outdated Show resolved Hide resolved
Assets/Scripts/GameState/PauseManager.cs Outdated Show resolved Hide resolved
Assets/Scripts/GameState/PauseManager.cs Outdated Show resolved Hide resolved
Assets/Scripts/GameState/PauseManager.cs Outdated Show resolved Hide resolved
@ddabble ddabble mentioned this pull request Feb 19, 2022
Copy link
Member

@ddabble ddabble left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 🤩 (Still haven't tried the code yet, but since we're getting close to the deadline, any potential changes can be done later 🤠🤠)

@ddabble ddabble merged commit 69741df into develop Feb 24, 2022
@ddabble ddabble deleted the fix/bugfix branch February 24, 2022 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants