Skip to content

Prefer per-engine high scores and show engine ring#1537

Open
Ape wants to merge 1 commit into
YARC-Official:devfrom
Ape:pr/per-engine-highscores
Open

Prefer per-engine high scores and show engine ring#1537
Ape wants to merge 1 commit into
YARC-Official:devfrom
Ape:pr/per-engine-highscores

Conversation

@Ape

@Ape Ape commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

With this commit we now prefer scores from the active engine preset when querying high scores, percentages, and stars.

This commit also adds a engine-colored ring around the music library difficulty badge.

In this screenshot we have custom engine, precision engine, and default engine:
presets

With this commit we now prefer scores from the active engine preset when
querying high scores, percentages, and stars. This commit also adds a
engine-colored ring around the music library difficulty badge.
@wyrdough

wyrdough commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Interesting idea! I'm not so sure about it from a UX and UI perspective, though. I think it could be unnecessarily confusing and not look great with the overall score screen design. People already sometimes get confused about what score is or is not showing. That said, I may very well be the only one with an objection, so if you want to bring it up on Discord to get wider feedback, you're more than welcome.

I do think at the very least this work could be useful on the upcoming more info screen where we intend to show local (and remote) leaderboard standings for each song.

@Ape

Ape commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

I'm not so sure about it from a UX and UI perspective, though.

Is that concerning the change about which score to show or the colored ring visualization?

@wyrdough

wyrdough commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Both. The score because people already get confused about what score is being shown and why and the colored ring because it doesn't seem to blend well with the overall vibe of the library menu. (This isn't to say we can't do this, just that I'd like some more discussion/feedback first)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants