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

GM Sound Panel [WIP] #255

Merged
merged 8 commits into from
Jun 8, 2024
Merged

GM Sound Panel [WIP] #255

merged 8 commits into from
Jun 8, 2024

Conversation

Doubleumc
Copy link
Contributor

@Doubleumc Doubleumc commented May 5, 2024

About the pull request

Implements a GM panel that allows playing of ingame sounds.

image

Searchable list of all sound files in the codebase. Adjust sound category, volume, pitch, duration, and preview the sound at those settings. Can play sounds to only a single client (in their head, basically), as a localized sound, to everyone on a z-level, or to all humans/ghosts/xenos/globally on the server.

Currently a work-in-progress, can be test-merged but NOT ready for full merge. Needs to be tested and need feedback on the UX.

Explain why it's good for the game

More versatile tools for GMs to set the scene.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
add: GM panel with sound controls
/:cl:

@morrowwolf
Copy link
Member

Might be worth having client stored favorites

can set local sounds to play on click instead of needing a separate click, needed to move data storage to the datum as a consequence
filtering updated *after* the render, leaving the browser one render behind, now fixed
remove leftover comments and logging
@Doubleumc Doubleumc marked this pull request as ready for review June 8, 2024 23:35
@Doubleumc Doubleumc requested a review from fira as a code owner June 8, 2024 23:35
@Doubleumc Doubleumc removed do not merge needs tested testmerge this to see what breaks labels Jun 8, 2024
@morrowwolf morrowwolf enabled auto-merge (squash) June 8, 2024 23:38
@morrowwolf morrowwolf merged commit 93aa0e8 into cmss13-devs:master Jun 8, 2024
36 checks passed
@Doubleumc Doubleumc deleted the GM-sounds branch June 9, 2024 02:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants