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

Increase the default AudioStreamPlayer3D unit size to 10 #38224

Merged

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Apr 26, 2020

This makes it easier to hear sound while setting up the node.

Since this changes the default value, this may break existing projects slightly.

This also tweaks the Unit Size editor property hint for better usability.

See discussion in #25468 (comment).

@Calinou Calinou requested a review from a team as a code owner April 26, 2020 12:59
@Calinou Calinou added this to the 4.0 milestone Apr 26, 2020
@Calinou Calinou force-pushed the increase-audiostreamplayer3d-unit-size branch from df25d7a to fd8d1d6 Compare December 12, 2020 22:05
@Calinou Calinou changed the title Increase the default AudioStreamPlayer3D unit size to 5 Increase the default AudioStreamPlayer3D unit size to 10 Dec 12, 2020
This makes it easier to hear sound while setting up the node.

Since this changes the default value, this may break existing projects
slightly.

This also tweaks the Unit Size editor property hint for better usability.

See discussion in godotengine#25468.
@Calinou Calinou force-pushed the increase-audiostreamplayer3d-unit-size branch from fd8d1d6 to 87d107b Compare May 6, 2021 00:45
@Calinou Calinou requested a review from a team as a code owner May 6, 2021 00:45
@akien-mga
Copy link
Member

@reduz suggests that we should probably make a better editor for the attenuation curve, but that doesn't directly impact the changes here so this can be merged as is.

@akien-mga akien-mga merged commit f288a79 into godotengine:master Jun 3, 2021
@akien-mga
Copy link
Member

Thanks!

@Calinou Calinou deleted the increase-audiostreamplayer3d-unit-size branch April 25, 2022 17:52
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