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

Smooth volume transition #187

Merged
merged 2 commits into from
Jun 2, 2024

Conversation

LostLuma
Copy link
Collaborator

@LostLuma LostLuma commented Jun 1, 2024

Add a smooth transition when changing between different volume multipliers.

By default this feature will transition volume from 100% to 0% within 2 seconds, and reverse within 1 second.
The speed for both upward and downward transition can be adjusted in the config to be a bit shorter or longer.


This was originally added to Dynamic FPS Reforged by @N4TH4NOT before we had sound category support or a Forge version.
Since their idea still influenced this a lot (otherwise I wouldn't have had this idea in the first place) I've added them to our credits.

@LostLuma
Copy link
Collaborator Author

LostLuma commented Jun 1, 2024

As a side note, this might be the favorite feature I've ever made for this mod :) Makes it feel very polished.

@LostLuma LostLuma force-pushed the smooth-volume-transition branch from ae4a75b to 6e81217 Compare June 1, 2024 10:47
@Kichura
Copy link

Kichura commented Jun 1, 2024

Will this feature be backported to 1.20.1/4 or no?

@LostLuma
Copy link
Collaborator Author

LostLuma commented Jun 1, 2024

That's the plan, yeah. Generally I try to backport all the way to 1.16.5 (unless a feature doesn't work for older Minecraft versions).
I have a few other potential changes / features I wanted to look into doing so it'll probably still be two or three weeks until release.

@LostLuma LostLuma merged commit e90b332 into juliand665:main Jun 2, 2024
2 checks 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