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

Display Reaction Time #19

Closed
BlqzingIce opened this issue Apr 13, 2022 · 5 comments
Closed

Display Reaction Time #19

BlqzingIce opened this issue Apr 13, 2022 · 5 comments
Labels
enhancement New feature or request

Comments

@BlqzingIce
Copy link

An option to display a map's set reaction time instead of offset/jump distance in the level params.
Easily calculatable from jump distance: reactiontime = jumpdistance / (2 * njs) * 1000; (in ms)
Maybe change the existing jump distance setting to a dropdown menu to accommodate the extra option?

@kinsi55
Copy link
Owner

kinsi55 commented Apr 13, 2022

I considered that before but I feel like the only real purpose of the default offset is for mappers / giving mapping feedback tbh since most people nowadays override the offset anyways. I'll consider it for a future update tho

@kinsi55 kinsi55 added the enhancement New feature or request label Apr 13, 2022
@BlqzingIce BlqzingIce changed the title [enhancement] Display Reaction Time Display Reaction Time Apr 13, 2022
@BlqzingIce
Copy link
Author

Personally, I don't really get much from the offset (to be fair I am not a mapper), and I get some info from jump distance, but reaction time is just so much more intuitive to understand. I appreciate you considering it!

@kinsi55
Copy link
Owner

kinsi55 commented Apr 13, 2022

Yeah but like knowing the maps default RT doesnt really give you much if you are overriding it anyways

@BlqzingIce
Copy link
Author

I still think it would be useful information to have built into the song select menu, but that's true, for people who change everything anyways it's doesn't really matter.

@kinsi55
Copy link
Owner

kinsi55 commented Sep 7, 2023

Closing this as that info field isnt offset anymore anways but map age now

@kinsi55 kinsi55 closed this as completed Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants