Skip to content

differentiable reset sounds #1401

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

sctanf
Copy link
Member

@sctanf sctanf commented Apr 21, 2025

original sounds between full reset and mounting could be differentiated by speed, so instead play a unique pattern
I am not sure this is needed, but maybe it is a good idea.

@github-actions github-actions bot added the Area: GUI Related to the GUI label Apr 21, 2025
@sctanf
Copy link
Member Author

sctanf commented May 1, 2025

needs feedback on the reset start tones

@sctanf sctanf marked this pull request as ready for review May 1, 2025 06:28
@sctanf sctanf requested review from ImUrX and loucass003 as code owners May 1, 2025 06:28
@ImUrX ImUrX requested a review from Polymoria June 1, 2025 21:50
@ImUrX ImUrX added the Type: Enhancement Adds or improves a feature label Jun 1, 2025
@ImUrX
Copy link
Member

ImUrX commented Jun 1, 2025

if using F# I would make the reset sound ends end in G4, will make it feel like it ended correctly

@sctanf
Copy link
Member Author

sctanf commented Jun 1, 2025

that sounds good and you can commit it; since reset sounds might be replaced again anyway i dont know if this should be worked on

@ImUrX
Copy link
Member

ImUrX commented Jun 1, 2025

I mean until it gets changed why not

Please feel free to reject if you like it less ^^ that is oki
@Polymoria
Copy link
Contributor

I wanted to change only what @ImUrX asked, but I think I played too much. Please feel free to reject if you don't like.

@sctanf
Copy link
Member Author

sctanf commented Jun 3, 2025

if you think it is good, that is more than likely fine for this pr

Copy link
Contributor

@Polymoria Polymoria left a comment

Choose a reason for hiding this comment

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

<3

@Polymoria Polymoria self-requested a review June 3, 2025 05:43
Copy link
Contributor

@Polymoria Polymoria left a comment

Choose a reason for hiding this comment

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

<33

Copy link
Member

@ImUrX ImUrX left a comment

Choose a reason for hiding this comment

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

<333

@ImUrX ImUrX force-pushed the differentiable-reset-sounds branch from 3206156 to aa27526 Compare June 3, 2025 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: GUI Related to the GUI Type: Enhancement Adds or improves a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants