Simply-Love-SM5 icon indicating copy to clipboard operation
Simply-Love-SM5 copied to clipboard

Respect "Show EX Score" in score summary

Open SugoiFactory opened this issue 10 months ago • 4 comments

(Changes for set summary screen only)

  • Show bigger EX score over smaller ITG score if "Show EX Score" was chosen for the song.
  • Show blue/white judgment counts if "Show EX Score" was chosen for the song, regardless of FA+ window being enabled or not.
  • (minor fix) Properly hide W0 number if blue/white split was not being used in any way.

SugoiFactory avatar Apr 09 '24 07:04 SugoiFactory

I haven't tested this PR, but thanks for beating me to this bugfix :)

quietly-turning avatar Apr 15 '24 18:04 quietly-turning

Regarding point 2,

Show blue/white judgment counts if "Show EX Score" was chosen for the song, regardless of FA+ window being enabled or not.

I don't agree with this because I think it makes sense to display the judgments that the player was seeing in game, which is independent of the EX Score display.

teejusb avatar Apr 15 '24 18:04 teejusb

My reasoning was that without the blue/white judgments, the EX score felt meaningless (I also sometimes play with FA+ window off to reduce visual feedback during gameplay but am interested in the EX score and final FA+ counts for ITL purposes). If there's a more valid use case for wanting to show EX score but also only wanting the regular counts visible then I can modify the logic for setting the flag a bit.

SugoiFactory avatar Apr 16 '24 05:04 SugoiFactory

I think that makes sense but I think the condition can then be showExScore OR FA+ was enabled instead of just showExScore. That covers the EX score not shown, but windows were displayed in gameplay case.

teejusb avatar Apr 17 '24 05:04 teejusb