Skip to content

Add compareJewelsOfSameType option for ItemsTab#9466

Open
roychl666 wants to merge 1 commit intoPathOfBuildingCommunity:devfrom
roychl666:dev
Open

Add compareJewelsOfSameType option for ItemsTab#9466
roychl666 wants to merge 1 commit intoPathOfBuildingCommunity:devfrom
roychl666:dev

Conversation

@roychl666
Copy link

Compare only jewels of the same type when the option is set to true. Maintain existing behavior by defaulting the option to false.
This PR picks up the work started by @michelrtm in #6314. I've updated it to work with the tip of tree.

Fixes #6275.

Description of the problem being solved:

Currently, jewels (e.g., Abyss, Crimson, Cobalt, Viridian) are compared against all occupied jewel slots, including incompatible types like Cluster Jewels, which creates irrelevant data. This change introduces a toggle to ensure jewels are only compared against slots containing the same jewel type.

Steps taken to verify a working solution:

  • Hover on existing jewels with the option enabled/disabled
  • Hover on unused jewels with the option enabled/disabled

Link to a build that showcases this PR:

https://pobb.in/6z027yW7pPix

Before screenshot:

With option default disabled.

  • Hover on existing abyss jewel
Screenshot 2026-02-14 112622
  • Hover on unused abyss jewel
Screenshot 2026-02-14 112642
  • Hover on existing crimson jewel
Screenshot 2026-02-14 112707
  • Hover on unused viridian jewel
Screenshot 2026-02-14 112730

After screenshot:

Enable the option.
Screenshot 2026-02-14 112825

  • Hover on existing abyss jewel
Screenshot 2026-02-14 112949
  • Hover on unused abyss jewel
Screenshot 2026-02-14 113008
  • Hover on existing crimson jewel
Screenshot 2026-02-14 113021
  • Hover on unused viridian jewel
Screenshot 2026-02-14 113030

Compare only jewels of the same type when the option is set to true.
Maintain existing behavior by defaulting the option to false.
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.

An option for "DO NOT COMPARE WITH CLUSTER JEWELS"

1 participant