Skip to content

Fix Alembic quick-action sound effect's timing - #125

Merged
hex-agon merged 4 commits into
hex-agon:masterfrom
toxyy:fix-alembic-sound-effect
Nov 9, 2025
Merged

Fix Alembic quick-action sound effect's timing#125
hex-agon merged 4 commits into
hex-agon:masterfrom
toxyy:fix-alembic-sound-effect

Conversation

@toxyy

@toxyy toxyy commented Nov 7, 2025

Copy link
Copy Markdown
Contributor

In the current game, the sound effect for Alembic plays after the quick-action has gone away. This is unlike the quick-action sound effect for Agitator, which plays at the correct time, or at least you can click the game when the effect plays and you'll get the quick action.
This PR fixes Alembic's quick-action sound effect to play when the quick-action is available and it's highlighted green.
In other words, the sound effect plays at the correct time now.

toxyy added 2 commits November 7, 2025 06:57
Adds a manual found_gem (the sound effect that plays when a quick-action is available) sound effect to play at the correct time, when Alembic is highlighted green, not after.
@toxyy

toxyy commented Nov 7, 2025

Copy link
Copy Markdown
Contributor Author

One bug I've noticed is that the sound effect will play twice if you mess up Alembic and the game will play its effect at the same time or on the next tick. Not really motivated right now to fix that since the right sound is playing at the right time for me now, I can live with it.
Fixed it.

@toxyy

toxyy commented Nov 7, 2025

Copy link
Copy Markdown
Contributor Author

Simplified the logic of this PR and fixed the double sound effect bug. Should be good to go.

@hex-agon

hex-agon commented Nov 8, 2025

Copy link
Copy Markdown
Owner

Can you also add a check if the player is in the mixology lab? Just as a failsafe

@toxyy

toxyy commented Nov 8, 2025

Copy link
Copy Markdown
Contributor Author

Done.

@hex-agon
hex-agon merged commit 1cca0cd into hex-agon:master Nov 9, 2025
1 check passed
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.

2 participants