kinsi55 / BeatSaber_BetterSongList

Adds Various improvements to the Basegame song list like Filters, a persisted state and much more
MIT License
43 stars 11 forks source link

Display Reaction Time #19

Closed BlqzingIce closed 1 year ago

BlqzingIce commented 2 years ago

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 commented 2 years ago

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

BlqzingIce commented 2 years ago

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 commented 2 years ago

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

BlqzingIce commented 2 years ago

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 commented 1 year ago

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