Skip to content

Added sound menu + cleanup#106

Merged
ldominiq merged 2 commits into
devfrom
soundMenu
May 22, 2026
Merged

Added sound menu + cleanup#106
ldominiq merged 2 commits into
devfrom
soundMenu

Conversation

@ldominiq

Copy link
Copy Markdown
Owner

No description provided.

@ldominiq ldominiq self-assigned this May 22, 2026
Copilot AI review requested due to automatic review settings May 22, 2026 14:59

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces a dedicated Sounds settings screen alongside existing Settings/Graphics/Controls menus, wiring it into the main menu flow so players can adjust audio volumes at runtime.

Changes:

  • Added a new SoundsMenu UI with sliders (master/music/effects) and a Done button.
  • Extended SettingsMenu with a Sounds button and callback to open the new menu.
  • Integrated the new menu into App (creation, resize handling, escape/back navigation, cleanup) and adjusted the default musicVolume.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/client/menus/SoundsMenu.cpp New menu implementation for rendering and interacting with volume sliders.
include/client/menus/SoundsMenu.hpp New menu interface and slider APIs.
src/client/menus/SettingsMenu.cpp Adds Sounds button layout/render/hover/click handling.
include/client/menus/SettingsMenu.hpp Adds setSoundsCallback and stores callback/button state.
src/client/App.cpp Creates soundsMenu, adds volume sliders, routes Settings→Sounds and Escape/back behavior, handles resize/cleanup.
include/client/App.hpp Adds soundsMenu member and include.
src/client/AudioManager.cpp Removes several loadSfx(...) entries during asset preload.
include/client/AudioManager.hpp Changes default musicVolume from 0.0f to 0.8f.
Comments suppressed due to low confidence (1)

src/client/AudioManager.cpp:289

  • SoundId::Player_Jump is still played in updateFootsteps() (jump edge), but loadAllAssets() no longer loads any asset for that id. As a result, jump SFX will be silent at runtime. Please restore the loadSfx(SoundId::Player_Jump, ...) call (or remove the playback if the sound is being deprecated).

    loadSfx(SoundId::Player_Splash,      {
        "assets/sounds/liquids/splash.wav",
        "assets/sounds/liquids/splash2.wav",
        });

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/client/App.cpp Outdated
Comment thread src/client/App.cpp Outdated
Comment thread include/client/App.hpp Outdated
Comment thread src/client/AudioManager.cpp
@sonarqubecloud

Copy link
Copy Markdown

@ldominiq ldominiq merged commit e946f32 into dev May 22, 2026
2 checks passed
@ldominiq ldominiq deleted the soundMenu branch May 22, 2026 15:45
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.

3 participants