Conversation
|
Considering the amount of overlap, I think this would be better off as an option in the existing personal best counter, instead of a separate source, unless there's a particular reason you didn't do it that way. I definitely like the addition of the separator option. There are a few other things I'd like you to do before I merge this:
If you don't want to make those changes yourself, I can add the feature myself in a version or two, instead of merging this PR. Either way, thanks for the contribution! Also FYI there will be an API released with the mod for you to add new counters, sources, etc in a separate mod, all documented in the README. You can also use it before the release with |
…en don't apply -Merged PBGap and Personal Best Together -Added GetScoreRatio and GetBestScoreRatio functions into Utils -Added disabling settings that aren't applicable and graying them out (Only in PersonalBest and Score)
I noticed a lot of the counters where calculating the ratios so I added functions and applied them to Calculate the best score ratio and score ratio into utils. The score ratio takes in whether to include modifiers and which saber/s to get the score for (default is true and both). I also went and made it so when a setting isn't applicable it makes it so you can't change it and grays it out. I only added it to score and personal best and can be removed easily if need be. |
|
Looks good to me. The only thing I notice is that the |
+/-in front of the PB Gap