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

Achievements: Fix the percentage counter when only unofficial achievements exist #10328

Merged

Conversation

CookiePLMonster
Copy link
Contributor

@CookiePLMonster CookiePLMonster commented Nov 27, 2023

Description of Changes

Fixes broken percentage display if unofficial achievements are enabled.

Rationale behind Changes

Sets with only unofficial achievements displayed wrong if unofficial test mode was enabled.

Before:
image

After:
image

Suggested Testing Steps

  1. Launch any game with an unofficial set.
  2. Enable "Unofficial Test Mode".
  3. Inspect the percentage unlock and text in the Achievements screen.

@CookiePLMonster CookiePLMonster force-pushed the fix-total-unofficial-achievements branch from cef84b1 to bc94487 Compare November 27, 2023 16:06
@CookiePLMonster CookiePLMonster marked this pull request as draft November 27, 2023 17:42
…ments exist

Now correctly shows "This game has no achievements" and no percentage bar,
as per the RetroAchievements integration guide.
@CookiePLMonster CookiePLMonster force-pushed the fix-total-unofficial-achievements branch from bc94487 to 014c052 Compare November 27, 2023 17:55
@CookiePLMonster CookiePLMonster changed the title Achievements: Count unofficial achievements in fullscreen UI Achievements: Fix the percentage counter when only unofficial achievements exist Nov 27, 2023
@CookiePLMonster CookiePLMonster marked this pull request as ready for review November 27, 2023 17:56
@stenzek stenzek merged commit 279c84f into PCSX2:master Dec 1, 2023
12 checks passed
@CookiePLMonster CookiePLMonster deleted the fix-total-unofficial-achievements branch December 1, 2023 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants