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

Speed up metronome in timing screen when pressing control key #31342

Merged
merged 4 commits into from
Dec 31, 2024

Conversation

minetoblend
Copy link
Contributor

@minetoblend minetoblend commented Dec 30, 2024

Changes the speed at which the metronome swings & the tick sample plays to 2x (or 3x when beat divisor is in triples mode) the speed when control is pressed, matching stable behavior.

Addresses one of the points made in #31263 (4. point in timing section)

Example on regular beat divisor

2024-12-30.03-36-53.mp4

Example on 1/3 beat divisor

2024-12-30.03-55-30.mp4

@bdach
Copy link
Collaborator

bdach commented Dec 30, 2024

looking at the videos attached, not sure how to feel about the metronome speeding up and the weight on the pendulum moving but the bpm number not changing at all... seems off

normal sped up
1735564694 1735564705

@peppy
Copy link
Member

peppy commented Dec 30, 2024

looking at the videos attached, not sure how to feel about the metronome speeding up and the weight on the pendulum moving but the bpm number not changing at all... seems off

normal sped up
1735564694 1735564705

Agree with the BPM also showing double. maybe 122 (244 audible) or something if we want to avoid any confusion, but I also think only showing the sped-up BPM is fine.

@minetoblend
Copy link
Contributor Author

minetoblend commented Dec 30, 2024

How about showing something like 122 (2x)? I feel like 122 (244 audible) could run out of space in that container quickly if the bpm is higher, as well as the change being not only audible but visual too given that the metronome swings faster.

@peppy
Copy link
Member

peppy commented Dec 30, 2024

Just show the actual audible number and call it a day. No suffix.

@bdach bdach merged commit 929173c into ppy:master Dec 31, 2024
8 of 10 checks passed
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.

4 participants