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

[Bug][UI/UX] Shiny button can cycle variants even if non-shiny form is uncaught #5457

Merged
merged 4 commits into from
Mar 1, 2025

Conversation

Wlowscha
Copy link
Contributor

@Wlowscha Wlowscha commented Mar 1, 2025

Why am I making these changes?

When the variant button was removed by #5213, a minor bug was introduced which prevents the player from switching between variants if the non-shiny form is not caught. This has been reported on discord e.g. here. See also these screenshots (from a save where all three variants of Miraidon are unlocked, but the non-shiny form is not): the "cycle shiny" button does not appear.
mira_dex
mira_ss

What are the changes from a developer perspective?

canCycleShiny is now true if at least two variants are caught. The logic for cycling shiny has been reworked to loop correctly ig the non-shiny form is not caught.

Screenshots/Videos

Shiny Miraidon cycling correctly:

Pokerogue.Mozilla.Firefox.2025-03-01.17-42-09.mp4

Proof that the button is still working properly when non-shiny forms are also caught:

Pokerogue.Mozilla.Firefox.2025-03-01.17-45-08.mp4

Checklist

  • I'm using beta as my base branch
  • There is no overlap with another PR?
  • The PR is self-contained and cannot be split into smaller PRs?
  • Have I provided a clear explanation of the changes?
  • Have I tested the changes manually?
  • Are all unit tests still passing? (npm run test)
    • Have I created new automated tests (npm run create-test) or updated existing tests related to the PR's changes?
  • Have I provided screenshots/videos of the changes (if applicable)?
    • Have I made sure that any UI change works for both UI themes (default and legacy)?

@Wlowscha Wlowscha requested a review from a team as a code owner March 1, 2025 16:53
@Madmadness65 Madmadness65 added UI/UX User interface/-experience related P2 Bug Minor. Non crashing Incorrect move/ability/interaction labels Mar 1, 2025
@damocleas damocleas enabled auto-merge (squash) March 1, 2025 20:54
@damocleas damocleas merged commit 04ac5c3 into pagefaultgames:beta Mar 1, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 Bug Minor. Non crashing Incorrect move/ability/interaction UI/UX User interface/-experience related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants