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

Show player divisions in scoreboard #6160

Open
wants to merge 4 commits into
base: deploy/fafdevelop
Choose a base branch
from

Conversation

BlackYps
Copy link
Contributor

@BlackYps BlackYps commented May 7, 2024

Description of the proposed changes

Show the division of players in the scoreboard instead of rating if available (i.e. in matchmaker games)
Closes #4258

Testing done on the proposed changes

Tested them on a matchmaker replay, as well as a normal game to make sure that the rating still shows up when no divisions are available.

The missing space between division and subdivision is because the added space in the game options is not present in current replays. This is just an artefact of the testing process and will be fixed once the release happened.

grafik

Checklist

  • Changes are annotated, including comments where useful
  • Changes are documented in the changelog for the next game version

@BlackYps BlackYps added area: ui Anything to do with the User Interface of the Game ui: scoreboard related to scoreboard ui labels May 7, 2024
@BlackYps BlackYps requested a review from Garanas May 7, 2024 23:06
@Garanas
Copy link
Member

Garanas commented May 8, 2024

🤔 , I thought we also merged the images of the badges at some point in the past. We could use those instead of the text

@BlackYps
Copy link
Contributor Author

BlackYps commented May 8, 2024

According to this #4263 using the images causes problems with the layout. I have no experience with the ui elements of the game, so I don't feel competent to possibly address these problems.
And I would rather have a text implementation than none at all. Seeing that the linked PR is over a year old I don't think someone will show up and implement the images anytime soon.

But even if they do, it doesn't hurt to have the text solution in the meantime.

@MrRowey
Copy link
Member

MrRowey commented May 9, 2024

What will this show in custom games tho? and how will it affect Scoreboard UI Mods ?

@BlackYps
Copy link
Contributor Author

BlackYps commented May 9, 2024

Custom games still show the rating. UI mods are unaffected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ui Anything to do with the User Interface of the Game ui: scoreboard related to scoreboard ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add division to score board for matchmaker games
3 participants