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

Update SFX for mod select overlay show/hide #18970

Merged
merged 6 commits into from
Jul 2, 2022

Conversation

nekodex
Copy link
Contributor

@nekodex nekodex commented Jul 1, 2022

@peppy
Copy link
Member

peppy commented Jul 2, 2022

Two things to note with this (non-critical but maybe worth considering?):

  • Because the initial display of the mod screen favours the columns being further to the right, the balance of the individual columns feels lob-sided towards the right. It's probably best to fake it and offset to allow them to feel balanced (ie. add a left bias equal to the visual offset of the first column).
  • The sound of the clicking of the "mods" button at song select is a bit overpowering. I don't think it really fits that well, and would probably work better as one of the softer button click sounds (especially since all three of the buttons there have their own sounds that play after click now).

@nekodex
Copy link
Contributor Author

nekodex commented Jul 2, 2022

  • Because the initial display of the mod screen favours the columns being further to the right, the balance of the individual columns feels lob-sided towards the right. It's probably best to fake it and offset to allow them to feel balanced (ie. add a left bias equal to the visual offset of the first column).

Yeah, that makes sense. I guess the only thing I'd check is whether it feels off when the screen aspect ratio is wide enough such that the columns appear completely on the left.

  • The sound of the clicking of the "mods" button at song select is a bit overpowering. I don't think it really fits that well, and would probably work better as one of the softer button click sounds (especially since all three of the buttons there have their own sounds that play after click now).

I still consider those mod button sounds as placeholders, so feel free to swap it for something else for the time being.

var channel = columnAppearSample?.GetChannel();
if (channel == null) return;

// don't play sample if column isn't visible, but try to play it at least 5 times (assuming there's at least 5 visible columns in `columnFlow`)
Copy link
Member

Choose a reason for hiding this comment

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

This conditional is a tad weird to read. To confirm, the goal is to play the sound even for off-screen columns up to a max of 5 to make the effect sound correct?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes

Copy link
Member

Choose a reason for hiding this comment

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

Hmm, the only discrepancy is that it will play 6 times with the code you have.

@peppy
Copy link
Member

peppy commented Jul 2, 2022

Actually after changing the button sound, the balance seems fine. It feels less bad without the button sound hiding the first few columns' sounds.

@peppy peppy enabled auto-merge July 2, 2022 03:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants